Whamcloud - gitweb
LU-5259 ptlrpc: request gets stuck in UNREGISTERING phase 46/10846/3
authorAndriy Skulysh <Andriy_Skulysh@xyratex.com>
Thu, 26 Jun 2014 10:37:36 +0000 (13:37 +0300)
committerOleg Drokin <oleg.drokin@intel.com>
Wed, 2 Jul 2014 19:17:30 +0000 (19:17 +0000)
Exit condition from UNREGISTERING phase is releasing of
both reply and bulk buffers.

Call ptlrpc_unregister_bulk() if ptlrpc_unregister_reply()
wasn't completed in async mode before switching to
UNREGISTERING phase.

Change-Id: Ia0420aecf0391e1a45063add2aa75bb4b3caa5a6
Xyratex-bug-id: MRP-1960
Signed-off-by: Andriy Skulysh <Andriy_Skulysh@xyratex.com>
Reviewed-on: http://review.whamcloud.com/10846
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Jenkins
Reviewed-by: Ann Koehler <amk@cray.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Liang Zhen <liang.zhen@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/ptlrpc/client.c

index 8ec94a3..679d05f 100644 (file)
@@ -1665,8 +1665,10 @@ int ptlrpc_check_set(const struct lu_env *env, struct ptlrpc_request_set *set)
                             req->rq_waiting || req->rq_wait_ctx) {
                                 int status;
 
-                                if (!ptlrpc_unregister_reply(req, 1))
-                                        continue;
+                               if (!ptlrpc_unregister_reply(req, 1)) {
+                                       ptlrpc_unregister_bulk(req, 1);
+                                       continue;
+                               }
 
                                spin_lock(&imp->imp_lock);
                                if (ptlrpc_import_delay_req(imp, req, &status)){