From: Arshad Hussain Date: Tue, 16 May 2023 03:00:49 +0000 (+0530) Subject: LU-16796 libcfs: Remove reference to LASSERT_ATOMIC_ZERO X-Git-Tag: 2.15.57~7 X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=d48ddf585cd2dd93092b3d67f1d38d5125505c04;p=fs%2Flustre-release.git LU-16796 libcfs: Remove reference to LASSERT_ATOMIC_ZERO This patch removes all reference to LASSERT_ATOMIC_ZERO macro. Signed-off-by: Arshad Hussain Change-Id: I73259599d1dee6277fadf66181699f1282274a80 Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/51004 Tested-by: jenkins Tested-by: Maloo Reviewed-by: James Simmons Reviewed-by: Timothy Day Reviewed-by: Neil Brown Reviewed-by: Oleg Drokin --- diff --git a/libcfs/include/libcfs/libcfs_private.h b/libcfs/include/libcfs/libcfs_private.h index 53ff128..9b80be4 100644 --- a/libcfs/include/libcfs/libcfs_private.h +++ b/libcfs/include/libcfs/libcfs_private.h @@ -246,22 +246,6 @@ int libcfs_debug_cleanup(void); int libcfs_debug_clear_buffer(void); int libcfs_debug_mark_buffer(const char *text); -#define LASSERT_ATOMIC_ENABLED (1) - -#if LASSERT_ATOMIC_ENABLED - -/** assert value of @a is equal to @v */ -#define LASSERT_ATOMIC_EQ(a, v) \ - LASSERTF(atomic_read(a) == v, "value: %d\n", atomic_read((a))); - -#else /* !LASSERT_ATOMIC_ENABLED */ - -#define LASSERT_ATOMIC_EQ(a, v) do {} while (0) - -#endif /* LASSERT_ATOMIC_ENABLED */ - -#define LASSERT_ATOMIC_ZERO(a) LASSERT_ATOMIC_EQ(a, 0) - #define CFS_ALLOC_PTR(ptr) LIBCFS_ALLOC(ptr, sizeof(*(ptr))); #define CFS_ALLOC_PTR_ARRAY(ptr, count) \ LIBCFS_ALLOC(ptr, (count) * sizeof(*(ptr))) diff --git a/lustre/ldlm/ldlm_lib.c b/lustre/ldlm/ldlm_lib.c index 38e3632..ee248fe 100644 --- a/lustre/ldlm/ldlm_lib.c +++ b/lustre/ldlm/ldlm_lib.c @@ -1675,10 +1675,10 @@ void target_destroy_export(struct obd_export *exp) if (imp != NULL) client_destroy_import(imp); - LASSERT_ATOMIC_ZERO(&exp->exp_locks_count); - LASSERT_ATOMIC_ZERO(&exp->exp_rpc_count); - LASSERT_ATOMIC_ZERO(&exp->exp_cb_count); - LASSERT_ATOMIC_ZERO(&exp->exp_replay_count); + LASSERT(atomic_read(&exp->exp_locks_count) == 0); + LASSERT(atomic_read(&exp->exp_rpc_count) == 0); + LASSERT(atomic_read(&exp->exp_cb_count) == 0); + LASSERT(atomic_read(&exp->exp_replay_count) == 0); } EXPORT_SYMBOL(target_destroy_export); diff --git a/lustre/ptlrpc/sec.c b/lustre/ptlrpc/sec.c index 101a472..ab747f8 100644 --- a/lustre/ptlrpc/sec.c +++ b/lustre/ptlrpc/sec.c @@ -1289,8 +1289,8 @@ static void sec_cop_destroy_sec(struct ptlrpc_sec *sec) { struct ptlrpc_sec_policy *policy = sec->ps_policy; - LASSERT_ATOMIC_ZERO(&sec->ps_refcount); - LASSERT_ATOMIC_ZERO(&sec->ps_nctx); + LASSERT(atomic_read(&sec->ps_refcount) == 0); + LASSERT(atomic_read(&sec->ps_nctx) == 0); LASSERT(policy->sp_cops->destroy_sec); CDEBUG(D_SEC, "%s@%p: being destroyed\n", sec->ps_policy->sp_name, sec);