X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Flmv%2Flmv_fld.c;h=db45c4bd358c739e93c429e275b23324c8b352ec;hb=b2368774a01eb89981e2ceb92be9673e4b403d62;hp=b5ec306dcc224d9d73d1424227e66ac59d72dc22;hpb=72057a3af19ee02d9a686bd7e7d074917e381310;p=fs%2Flustre-release.git

diff --git a/lustre/lmv/lmv_fld.c b/lustre/lmv/lmv_fld.c
index b5ec306..db45c4b 100644
--- a/lustre/lmv/lmv_fld.c
+++ b/lustre/lmv/lmv_fld.c
@@ -40,7 +40,6 @@
 #include <linux/seq_file.h>
 
 #include <obd_support.h>
-#include <lustre/lustre_idl.h>
 #include <lustre_fid.h>
 #include <lustre_lib.h>
 #include <lustre_net.h>
@@ -53,32 +52,37 @@ int lmv_fld_lookup(struct lmv_obd *lmv, const struct lu_fid *fid, u32 *mds)
 {
 	struct obd_device *obd = lmv2obd_dev(lmv);
 	int rc;
+
 	ENTRY;
 
-	/* FIXME: Currently ZFS still use local seq for ROOT unfortunately, and
-	 * this fid_is_local check should be removed once LU-2240 is fixed */
+	/*
+	 * FIXME: Currently ZFS still use local seq for ROOT unfortunately, and
+	 * this fid_is_local check should be removed once LU-2240 is fixed
+	 */
 	if (!fid_is_sane(fid) || !(fid_seq_in_fldb(fid_seq(fid)) ||
 				   fid_seq_is_local_file(fid_seq(fid)))) {
-		CERROR("%s: invalid FID "DFID"\n", obd->obd_name, PFID(fid));
-		RETURN(-EINVAL);
+		rc = -EINVAL;
+		CERROR("%s: invalid FID "DFID": rc = %d\n", obd->obd_name,
+		       PFID(fid), rc);
+		RETURN(rc);
 	}
 
 	rc = fld_client_lookup(&lmv->lmv_fld, fid_seq(fid), mds,
-                               LU_SEQ_RANGE_MDT, NULL);
-        if (rc) {
-		CERROR("Error while looking for mds number. Seq %#llx"
-                       ", err = %d\n", fid_seq(fid), rc);
-                RETURN(rc);
-        }
+			       LU_SEQ_RANGE_MDT, NULL);
+	if (rc) {
+		CERROR("%s: Error while looking for mds number. Seq %#llx: rc = %d\n",
+		       obd->obd_name, fid_seq(fid), rc);
+		RETURN(rc);
+	}
 
-        CDEBUG(D_INODE, "FLD lookup got mds #%x for fid="DFID"\n",
-               *mds, PFID(fid));
+	CDEBUG(D_INODE, "FLD lookup got mds #%x for fid="DFID"\n",
+	       *mds, PFID(fid));
 
-        if (*mds >= lmv->desc.ld_tgt_count) {
-                CERROR("FLD lookup got invalid mds #%x (max: %x) "
-                       "for fid="DFID"\n", *mds, lmv->desc.ld_tgt_count,
-                       PFID(fid));
-                rc = -EINVAL;
-        }
-        RETURN(rc);
+	if (*mds >= lmv->desc.ld_tgt_count) {
+		rc = -EINVAL;
+		CERROR("%s: FLD lookup got invalid mds #%x (max: %x) for fid="DFID": rc = %d\n",
+		       obd->obd_name, *mds, lmv->desc.ld_tgt_count, PFID(fid),
+		       rc);
+	}
+	RETURN(rc);
 }