From dfa318a29b8fe708468989d67ac6928a42bec72d Mon Sep 17 00:00:00 2001 From: Sebastien Buisson Date: Thu, 1 Mar 2018 01:18:32 +0900 Subject: [PATCH] LU-9727 mdd: properly call recording_changelog() recording_changelog() must be called everywhere in the code instead of directly checking (mdd->mdd_cl.mc_flags & CLM_ON). Signed-off-by: Sebastien Buisson Change-Id: I9ed5aac4871573e6aea94cfd4dc46b95d5df1e4a Reviewed-on: https://review.whamcloud.com/31456 Tested-by: Jenkins Reviewed-by: John L. Hammond Tested-by: Maloo Reviewed-by: Olaf Weber Reviewed-by: James Simmons Reviewed-by: Oleg Drokin --- lustre/mdd/mdd_object.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lustre/mdd/mdd_object.c b/lustre/mdd/mdd_object.c index 40760f0..7eb9950 100644 --- a/lustre/mdd/mdd_object.c +++ b/lustre/mdd/mdd_object.c @@ -434,7 +434,7 @@ static int mdd_xattr_get(const struct lu_env *env, int rc2; /* Not recording */ - if (!(mdd->mdd_cl.mc_flags & CLM_ON)) + if (!recording_changelog(env, mdd)) RETURN(rc); if (!(mdd->mdd_cl.mc_mask & (1 << CL_GETXATTR))) RETURN(rc); @@ -960,7 +960,7 @@ static int mdd_changelog_data_store_xattr(const struct lu_env *env, LASSERT(handle != NULL); /* Not recording */ - if (!(mdd->mdd_cl.mc_flags & CLM_ON)) + if (!recording_changelog(env, mdd)) RETURN(0); if ((mdd->mdd_cl.mc_mask & (1 << type)) == 0) RETURN(0); @@ -2900,7 +2900,7 @@ static int mdd_open(const struct lu_env *env, struct md_object *obj, mdd_obj->mod_count++; /* Not recording */ - if (!(mdd->mdd_cl.mc_flags & CLM_ON)) + if (!recording_changelog(env, mdd)) GOTO(out, rc); if (!(mdd->mdd_cl.mc_mask & (1 << type))) GOTO(out, rc); -- 1.8.3.1