From: Jinshan Xiong <jinshan.xiong@intel.com>
Date: Wed, 3 Jul 2013 09:25:04 +0000 (-0700)
Subject: LU-3546 mdt: reply error to client after version check
X-Git-Tag: 2.4.52~55
X-Git-Url: https://git.whamcloud.com/?p=fs%2Flustre-release.git;a=commitdiff_plain;h=e42b042bbe043895e2eda073d7d67cee77f4cc72

LU-3546 mdt: reply error to client after version check

This is missing from MDT implementation which will cause the client
to wait for reply forever.

Signed-off-by: Jinshan Xiong <jinshan.xiong@intel.com>
Change-Id: I9e46d83f8db6b6ded57a284914a2ede88dd987f4
Reviewed-on: http://review.whamcloud.com/6864
Tested-by: Hudson
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Reviewed-by: Mike Pershin <mike.pershin@intel.com>
Tested-by: Maloo <whamcloud.maloo@gmail.com>
---

diff --git a/lustre/mdt/mdt_handler.c b/lustre/mdt/mdt_handler.c
index c4ce385..832aad6 100644
--- a/lustre/mdt/mdt_handler.c
+++ b/lustre/mdt/mdt_handler.c
@@ -3355,16 +3355,21 @@ static int mdt_handle0(struct ptlrpc_request *req,
                         if (likely(h != NULL)) {
                                 rc = mdt_req_handle(info, h, req);
                         } else {
-                                CERROR("The unsupported opc: 0x%x\n",
-                                       lustre_msg_get_opc(msg) );
+				CERROR("%s: opc unsupported: 0x%x\n",
+					mdt_obd_name(info->mti_mdt),
+					lustre_msg_get_opc(msg));
                                 req->rq_status = -ENOTSUPP;
                                 rc = ptlrpc_error(req);
                                 RETURN(rc);
                         }
                 }
-        } else
-                CERROR(LUSTRE_MDT_NAME" drops mal-formed request\n");
-        RETURN(rc);
+	} else {
+		CDEBUG(D_INFO, "%s: drops mal-formed request: rc = %d\n",
+			mdt_obd_name(info->mti_mdt), rc);
+		req->rq_status = rc;
+		rc = ptlrpc_error(req);
+	}
+	RETURN(rc);
 }
 
 /*