From f94aecf16859e2e0dcd999eb8862227e50671ee8 Mon Sep 17 00:00:00 2001 From: Oleg Drokin Date: Wed, 22 May 2019 01:22:49 -0400 Subject: [PATCH] LU-12324 mdd: Do not record xattr size get in changelogs It looks like if the xattr itself was not fetched there's no need to create a changelog entry for it. The real get will come and we'd do it there Lustre-change: https://review.whamcloud.com/34936 Lustre-commit: 8f2599c599011a50549cdc79f0b3379239cf4f0c Change-Id: I5b19f9309f65da0a4c58cb79a95787dab862eb94 Signed-off-by: Oleg Drokin Signed-off-by: Mikhail Pershin Reviewed-by: Andreas Dilger Reviewed-by: Sebastien Buisson Signed-off-by: Minh Diep Reviewed-on: https://review.whamcloud.com/35010 Tested-by: Jenkins Tested-by: Maloo --- lustre/mdd/mdd_object.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lustre/mdd/mdd_object.c b/lustre/mdd/mdd_object.c index 669be61..312c5f8 100644 --- a/lustre/mdd/mdd_object.c +++ b/lustre/mdd/mdd_object.c @@ -428,7 +428,7 @@ static int mdd_xattr_get(const struct lu_env *env, mdd = mdo2mdd(obj); /* record only getting user xattrs and acls */ - if (rc >= 0 && + if (rc >= 0 && buf->lb_buf && mdd_changelog_enabled(env, mdd, CL_GETXATTR) && (has_prefix(name, XATTR_USER_PREFIX) || has_prefix(name, XATTR_NAME_POSIX_ACL_ACCESS) || -- 1.8.3.1