From: alex <alex>
Date: Thu, 12 May 2005 20:31:51 +0000 (+0000)
Subject: b=6284
X-Git-Tag: 1.4.10~1166
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=17aba1a77740e3aaa2e8a8ddabe68256f3f5db74;p=fs%2Flustre-release.git

b=6284

 - for unknown yet reason, ptlrpc_client_replied() returns true while
   rq_repmsg isn't allocated yet. this causes oops. the patch checks
   the both and outputs debug info
---

diff --git a/lustre/ptlrpc/pinger.c b/lustre/ptlrpc/pinger.c
index 756cd63..ce1c682 100644
--- a/lustre/ptlrpc/pinger.c
+++ b/lustre/ptlrpc/pinger.c
@@ -55,7 +55,10 @@ static int ptlrpc_ping_interpret(struct ptlrpc_request *req,
 
         /* if ping reply is an error, don't drop "replied" flag
          * on import, so pinger will invalidate it */
-        if (ptlrpc_client_replied(req) && req->rq_repmsg->type == PTL_RPC_MSG_ERR)
+        if (ptlrpc_client_replied(req) && req->rq_repmsg == NULL)
+                CWARN("replied (%d) w/o rep buffer?\n", req->rq_replied);
+        if (ptlrpc_client_replied(req) && req->rq_repmsg &&
+                        req->rq_repmsg->type == PTL_RPC_MSG_ERR)
                 return 0;
         
         imp->imp_waiting_ping_reply = 0;