From b937be892a3dc68dd2fe3f248608937a8a79d424 Mon Sep 17 00:00:00 2001 From: Bobi Jam Date: Thu, 17 Sep 2015 10:45:33 +0800 Subject: [PATCH] LU-6813 llite: omit to update wire data In ll_setattr_raw(), after op_data->op_attr has been copied, the attr is updated and op_data->op_attr does not get updated afterward. Signed-off-by: Bobi Jam Change-Id: I85b94a8ddc62184bfbcb128bd90f88ac03837e46 Reviewed-on: http://review.whamcloud.com/16462 Reviewed-by: Jinshan Xiong Tested-by: Jenkins Tested-by: Maloo Reviewed-by: Niu Yawei Reviewed-by: Oleg Drokin --- lustre/llite/llite_lib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lustre/llite/llite_lib.c b/lustre/llite/llite_lib.c index afb7b55..78a3495 100644 --- a/lustre/llite/llite_lib.c +++ b/lustre/llite/llite_lib.c @@ -1619,8 +1619,6 @@ int ll_setattr_raw(struct dentry *dentry, struct iattr *attr, bool hsm_import) if (op_data == NULL) GOTO(out, rc = -ENOMEM); - op_data->op_attr = *attr; - if (!hsm_import && attr->ia_valid & ATTR_SIZE) { /* If we are changing file size, file content is * modified, flag it. */ @@ -1629,6 +1627,8 @@ int ll_setattr_raw(struct dentry *dentry, struct iattr *attr, bool hsm_import) ll_file_clear_flag(lli, LLIF_DATA_MODIFIED); } + op_data->op_attr = *attr; + rc = ll_md_setattr(dentry, op_data); if (rc) GOTO(out, rc); -- 1.8.3.1