From 8f2599c599011a50549cdc79f0b3379239cf4f0c 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 Change-Id: I5b19f9309f65da0a4c58cb79a95787dab862eb94 Signed-off-by: Oleg Drokin Signed-off-by: Mikhail Pershin Reviewed-on: https://review.whamcloud.com/34936 Tested-by: Jenkins Reviewed-by: Andreas Dilger Tested-by: Maloo Reviewed-by: Sebastien Buisson --- 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 ff99634..e94118e 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