Commit 5ec35d45 authored by John L. Hammond's avatar John L. Hammond Committed by Greg Kroah-Hartman

staging: lustre: simplify inline functions in lustre_fid.h

Several inline functions return a structure that was passed in.
Their is no need for this so just make these function void.
Signed-off-by: default avatarJohn L. Hammond <john.hammond@intel.com>
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-2675
Reviewed-on: http://review.whamcloud.com/8641Reviewed-by: default avatarwangdi <di.wang@intel.com>
Reviewed-by: default avatarFan Yong <fan.yong@intel.com>
Reviewed-by: default avatarOleg Drokin <oleg.drokin@intel.com>
Signed-off-by: default avatarJames Simmons <jsimmons@infradead.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent d8f183b3
...@@ -393,14 +393,12 @@ struct ldlm_namespace; ...@@ -393,14 +393,12 @@ struct ldlm_namespace;
* but was moved into name[1] along with the OID to avoid consuming the * but was moved into name[1] along with the OID to avoid consuming the
* renaming name[2,3] fields that need to be used for the quota identifier. * renaming name[2,3] fields that need to be used for the quota identifier.
*/ */
static inline struct ldlm_res_id * static inline void
fid_build_reg_res_name(const struct lu_fid *fid, struct ldlm_res_id *res) fid_build_reg_res_name(const struct lu_fid *fid, struct ldlm_res_id *res)
{ {
memset(res, 0, sizeof(*res)); memset(res, 0, sizeof(*res));
res->name[LUSTRE_RES_ID_SEQ_OFF] = fid_seq(fid); res->name[LUSTRE_RES_ID_SEQ_OFF] = fid_seq(fid);
res->name[LUSTRE_RES_ID_VER_OID_OFF] = fid_ver_oid(fid); res->name[LUSTRE_RES_ID_VER_OID_OFF] = fid_ver_oid(fid);
return res;
} }
/* /*
...@@ -416,29 +414,25 @@ static inline bool fid_res_name_eq(const struct lu_fid *fid, ...@@ -416,29 +414,25 @@ static inline bool fid_res_name_eq(const struct lu_fid *fid,
/* /*
* Extract FID from LDLM resource. Reverse of fid_build_reg_res_name(). * Extract FID from LDLM resource. Reverse of fid_build_reg_res_name().
*/ */
static inline struct lu_fid * static inline void
fid_extract_from_res_name(struct lu_fid *fid, const struct ldlm_res_id *res) fid_extract_from_res_name(struct lu_fid *fid, const struct ldlm_res_id *res)
{ {
fid->f_seq = res->name[LUSTRE_RES_ID_SEQ_OFF]; fid->f_seq = res->name[LUSTRE_RES_ID_SEQ_OFF];
fid->f_oid = (__u32)(res->name[LUSTRE_RES_ID_VER_OID_OFF]); fid->f_oid = (__u32)(res->name[LUSTRE_RES_ID_VER_OID_OFF]);
fid->f_ver = (__u32)(res->name[LUSTRE_RES_ID_VER_OID_OFF] >> 32); fid->f_ver = (__u32)(res->name[LUSTRE_RES_ID_VER_OID_OFF] >> 32);
LASSERT(fid_res_name_eq(fid, res)); LASSERT(fid_res_name_eq(fid, res));
return fid;
} }
/* /*
* Build (DLM) resource identifier from global quota FID and quota ID. * Build (DLM) resource identifier from global quota FID and quota ID.
*/ */
static inline struct ldlm_res_id * static inline void
fid_build_quota_res_name(const struct lu_fid *glb_fid, union lquota_id *qid, fid_build_quota_res_name(const struct lu_fid *glb_fid, union lquota_id *qid,
struct ldlm_res_id *res) struct ldlm_res_id *res)
{ {
fid_build_reg_res_name(glb_fid, res); fid_build_reg_res_name(glb_fid, res);
res->name[LUSTRE_RES_ID_QUOTA_SEQ_OFF] = fid_seq(&qid->qid_fid); res->name[LUSTRE_RES_ID_QUOTA_SEQ_OFF] = fid_seq(&qid->qid_fid);
res->name[LUSTRE_RES_ID_QUOTA_VER_OID_OFF] = fid_ver_oid(&qid->qid_fid); res->name[LUSTRE_RES_ID_QUOTA_VER_OID_OFF] = fid_ver_oid(&qid->qid_fid);
return res;
} }
/* /*
...@@ -455,14 +449,12 @@ static inline void fid_extract_from_quota_res(struct lu_fid *glb_fid, ...@@ -455,14 +449,12 @@ static inline void fid_extract_from_quota_res(struct lu_fid *glb_fid,
(__u32)(res->name[LUSTRE_RES_ID_QUOTA_VER_OID_OFF] >> 32); (__u32)(res->name[LUSTRE_RES_ID_QUOTA_VER_OID_OFF] >> 32);
} }
static inline struct ldlm_res_id * static inline void
fid_build_pdo_res_name(const struct lu_fid *fid, unsigned int hash, fid_build_pdo_res_name(const struct lu_fid *fid, unsigned int hash,
struct ldlm_res_id *res) struct ldlm_res_id *res)
{ {
fid_build_reg_res_name(fid, res); fid_build_reg_res_name(fid, res);
res->name[LUSTRE_RES_ID_HSH_OFF] = hash; res->name[LUSTRE_RES_ID_HSH_OFF] = hash;
return res;
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment