From: wang di <di.wang@intel.com>
Date: Mon, 3 Feb 2014 02:06:14 +0000 (-0800)
Subject: LU-3186 lmv: do not need allocate FID for open by FID
X-Git-Tag: 2.3.65~87
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=3f774e441856542ad7c465597c76afd3a015be61;p=fs%2Flustre-release.git

LU-3186 lmv: do not need allocate FID for open by FID

We do not need allocate FID or set op_fid2 if it is for
Open by FID, otherwise, it will cause the MDT to open
the file with new FID.

Signed-off-by: wang di <di.wang@intel.com>
Change-Id: I67e5a52c643228a6d8bd0190ca1a78b047fd1e7a
Reviewed-on: http://review.whamcloud.com/6099
Reviewed-by: Fan Yong <fan.yong@intel.com>
Tested-by: Hudson
Tested-by: Maloo <whamcloud.maloo@gmail.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
---

diff --git a/lustre/lmv/lmv_intent.c b/lustre/lmv/lmv_intent.c
index 14f49d3..47722b4 100644
--- a/lustre/lmv/lmv_intent.c
+++ b/lustre/lmv/lmv_intent.c
@@ -179,7 +179,10 @@ int lmv_intent_open(struct obd_export *exp, struct md_op_data *op_data,
 	if (IS_ERR(tgt))
 		RETURN(PTR_ERR(tgt));
 
-	if (it->it_op & IT_CREAT) {
+	/* If it is ready to open the file by FID, do not need
+	 * allocate FID at all, otherwise it will confuse MDT */
+	if ((it->it_op & IT_CREAT) &&
+	    !(it->it_flags & MDS_OPEN_BY_FID)) {
 		/*
 		 * For open with IT_CREATE and for IT_CREATE cases allocate new
 		 * fid and setup FLD for it.