Commit 588de43a authored by Oleg Drokin's avatar Oleg Drokin Committed by Greg Kroah-Hartman
Browse files

lustre/llite: Do not send parent dir fid in getattr by fid 



Sending getattr by fid in this case is pointless, as the parent
might havelong changed and we have no control over it, but it's
irrelevant anyway, since we already have the child fid.

Signed-off-by: default avatarOleg Drokin <oleg.drokin@intel.com>
Reviewed-on: http://review.whamcloud.com/7910
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-3240


Reviewed-by: default avatarDmitry Eremin <dmitry.eremin@intel.com>
Reviewed-by: default avatarwangdi <di.wang@intel.com>
Reviewed-by: default avatarAndreas Dilger <andreas.dilger@intel.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent bf08ee0d

drivers/staging/lustre/lustre/llite/dir.c

+1 −1
Original line number Diff line number Diff line
@@ -362,7 +362,7 @@ struct page *ll_get_dir_page(struct inode *dir, __u64 hash, 
		struct ptlrpc_request *request;

		struct md_op_data *op_data;



		op_data = ll_prep_md_op_data(NULL, dir, NULL, NULL, 0, 0,

		op_data = ll_prep_md_op_data(NULL, dir, dir, NULL, 0, 0,

		LUSTRE_OPC_ANY, NULL);

		if (IS_ERR(op_data))

			return (void *)op_data;

drivers/staging/lustre/lustre/llite/file.c

+1 −1
Original line number Diff line number Diff line
@@ -2891,7 +2891,7 @@ int __ll_inode_revalidate_it(struct dentry *dentry, struct lookup_intent *it, 
			oit.it_op = IT_LOOKUP;



		/* Call getattr by fid, so do not provide name at all. */

		op_data = ll_prep_md_op_data(NULL, dentry->d_parent->d_inode,

		op_data = ll_prep_md_op_data(NULL, dentry->d_inode,

					     dentry->d_inode, NULL, 0, 0,

					     LUSTRE_OPC_ANY, NULL);

		if (IS_ERR(op_data))

CRA Git | Maintained and supported by SUSTech CRA and CCSE