From 8103cda2c83d3344fd08c816cab53ab5b9cb380f Mon Sep 17 00:00:00 2001 From: Timothy Day Date: Fri, 29 Mar 2024 00:48:07 -0700 Subject: [PATCH] LU-8191 lustre: convert lmv,lod,lov functions to static Static analysis shows that a number of functions could be made static. This patch declares several functions in lmv, lod, and lov static. Also, remove one unused function: lov_lsm_entry() Another function is intentionally unused for debugging purposes. It was detected by static analysis, but it has been left untouched. Lustre-change: https://review.whamcloud.com/51479 Lustre-commit: 78605b74a56a5338aa93bbf394fe35d0f7c17c5b Test-Parameters: trivial Signed-off-by: Timothy Day Change-Id: If409226ea201587c7f95d4a65ffaef72671b5ac2 Reviewed-on: https://review.whamcloud.com/c/ex/lustre-release/+/54627 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Andreas Dilger --- lustre/lod/lod_lov.c | 13 +++++++------ lustre/lod/lod_object.c | 4 ++-- lustre/lod/lod_qos.c | 8 +++++--- lustre/lov/lov_cl_internal.h | 5 ----- lustre/lov/lov_ea.c | 17 ----------------- lustre/lov/lov_io.c | 5 +++-- lustre/lov/lov_obd.c | 4 ++-- 7 files changed, 19 insertions(+), 37 deletions(-) diff --git a/lustre/lod/lod_lov.c b/lustre/lod/lod_lov.c index 7ab37de..e9adc4d 100644 --- a/lustre/lod/lod_lov.c +++ b/lustre/lod/lod_lov.c @@ -1890,8 +1890,8 @@ struct lov_comp_md_entry_v1 *comp_entry_v1(struct lov_comp_md_v1 *comp, int i) le16_to_cpu(comp->lcm_entry_count) - 1); \ entry++) -int lod_erase_dom_stripe(struct lov_comp_md_v1 *comp_v1, - struct lov_comp_md_entry_v1 *dom_ent) +static int lod_erase_dom_stripe(struct lov_comp_md_v1 *comp_v1, + struct lov_comp_md_entry_v1 *dom_ent) { struct lov_comp_md_entry_v1 *ent; __u16 entries; @@ -2015,10 +2015,11 @@ out: return d->lod_dom_stripesize_cur_kb << 10; } -int lod_dom_stripesize_choose(const struct lu_env *env, struct lod_device *d, - struct lov_comp_md_v1 *comp_v1, - struct lov_comp_md_entry_v1 *dom_ent, - __u32 stripe_size) +static int lod_dom_stripesize_choose(const struct lu_env *env, + struct lod_device *d, + struct lov_comp_md_v1 *comp_v1, + struct lov_comp_md_entry_v1 *dom_ent, + __u32 stripe_size) { struct lov_comp_md_entry_v1 *ent; struct lu_extent *dom_ext, *ext; diff --git a/lustre/lod/lod_object.c b/lustre/lod/lod_object.c index f5ad8ff..34e8ce7 100644 --- a/lustre/lod/lod_object.c +++ b/lustre/lod/lod_object.c @@ -2347,7 +2347,7 @@ void lod_free_foreign_lov(struct lod_object *lo) * \retval 0 on success * \retval negative if failed */ -int lod_alloc_foreign_lmv(struct lod_object *lo, size_t size) +static int lod_alloc_foreign_lmv(struct lod_object *lo, size_t size) { OBD_ALLOC_LARGE(lo->ldo_foreign_lmv, size); if (lo->ldo_foreign_lmv == NULL) @@ -2364,7 +2364,7 @@ int lod_alloc_foreign_lmv(struct lod_object *lo, size_t size) * * \param[in] lo object */ -void lod_free_foreign_lmv(struct lod_object *lo) +static void lod_free_foreign_lmv(struct lod_object *lo) { if (lo->ldo_foreign_lmv != NULL) OBD_FREE_LARGE(lo->ldo_foreign_lmv, lo->ldo_foreign_lmv_size); diff --git a/lustre/lod/lod_qos.c b/lustre/lod/lod_qos.c index a202441..e07c8ad 100644 --- a/lustre/lod/lod_qos.c +++ b/lustre/lod/lod_qos.c @@ -2567,7 +2567,8 @@ free_comp: /** * prepare enough OST avoidance bitmap space */ -int lod_prepare_avoidance(const struct lu_env *env, struct lod_object *lo) +static int lod_prepare_avoidance(const struct lu_env *env, + struct lod_object *lo) { struct lod_device *lod = lu2lod_dev(lo->ldo_obj.do_lu.lo_dev); struct lod_avoid_guide *lag = &lod_env_info(env)->lti_avoid; @@ -2629,8 +2630,9 @@ int lod_prepare_avoidance(const struct lu_env *env, struct lod_object *lo) * Collect information of used OSTs and OSSs in the overlapped components * of other mirrors */ -void lod_collect_avoidance(struct lod_object *lo, struct lod_avoid_guide *lag, - int comp_idx) +static void lod_collect_avoidance(struct lod_object *lo, + struct lod_avoid_guide *lag, + int comp_idx) { struct lod_device *lod = lu2lod_dev(lo->ldo_obj.do_lu.lo_dev); struct lod_layout_component *lod_comp = &lo->ldo_comp_entries[comp_idx]; diff --git a/lustre/lov/lov_cl_internal.h b/lustre/lov/lov_cl_internal.h index 4a2d64a..dcb3b14 100644 --- a/lustre/lov/lov_cl_internal.h +++ b/lustre/lov/lov_cl_internal.h @@ -313,10 +313,6 @@ struct lov_object { * Preferred mirror is initialized by the preferred * bit of lsme. It can be changed when the preferred * is inaccessible. - * In order to make lov_lsm_entry() return the same - * mirror in the same IO context, it's only possible - * to change the preferred mirror when the - * lo_active_ios reaches zero. */ int lo_preferred_mirror; /** @@ -650,7 +646,6 @@ struct lu_object *lovsub_object_alloc(const struct lu_env *env, const struct lu_object_header *hdr, struct lu_device *dev); -int lov_lsm_entry(const struct lov_stripe_md *lsm, __u64 offset); int lov_io_layout_at(struct lov_io *lio, __u64 offset); #define lov_foreach_target(lov, var) \ diff --git a/lustre/lov/lov_ea.c b/lustre/lov/lov_ea.c index af51276..c864221 100644 --- a/lustre/lov/lov_ea.c +++ b/lustre/lov/lov_ea.c @@ -793,23 +793,6 @@ void dump_lsm(unsigned int level, const struct lov_stripe_md *lsm) } } -int lov_lsm_entry(const struct lov_stripe_md *lsm, __u64 offset) -{ - int i; - - for (i = 0; i < lsm->lsm_entry_count; i++) { - struct lov_stripe_md_entry *lse = lsm->lsm_entries[i]; - - if ((offset >= lse->lsme_extent.e_start && - offset < lse->lsme_extent.e_end) || - (offset == OBD_OBJECT_EOF && - lse->lsme_extent.e_end == OBD_OBJECT_EOF)) - return i; - } - - return -1; -} - /** * lmm_layout_gen overlaps stripe_offset field, it needs to be reset back when * sending to MDT for passing striping checks diff --git a/lustre/lov/lov_io.c b/lustre/lov/lov_io.c index 605ed4b..97437f1 100644 --- a/lustre/lov/lov_io.c +++ b/lustre/lov/lov_io.c @@ -1436,8 +1436,9 @@ static int lov_io_read_ahead(const struct lu_env *env, RETURN(0); } -int lov_io_lru_reserve(const struct lu_env *env, - const struct cl_io_slice *ios, loff_t pos, size_t bytes) +static int lov_io_lru_reserve(const struct lu_env *env, + const struct cl_io_slice *ios, loff_t pos, + size_t bytes) { struct lov_io *lio = cl2lov_io(env, ios); struct lov_stripe_md *lsm = lio->lis_object->lo_lsm; diff --git a/lustre/lov/lov_obd.c b/lustre/lov/lov_obd.c index 209e081..4cc892e 100644 --- a/lustre/lov/lov_obd.c +++ b/lustre/lov/lov_obd.c @@ -112,8 +112,8 @@ static int lov_set_osc_active(struct obd_device *obd, struct obd_uuid *uuid, static int lov_notify(struct obd_device *obd, struct obd_device *watched, enum obd_notify_event ev); -int lov_connect_osc(struct obd_device *obd, u32 index, int activate, - struct obd_connect_data *data) +static int lov_connect_osc(struct obd_device *obd, u32 index, int activate, + struct obd_connect_data *data) { struct lov_obd *lov = &obd->u.lov; struct obd_uuid *tgt_uuid; -- 1.8.3.1