LU-12201 lnet: detach response tracker

We need to unlink the response tracker from MDs even if the
corresponding message failed to send.

Lustre-change: https://review.whamcloud.com/34770
Lustre-commit: 1bb91b966d15345b4c89245d51f6cb631b052779

Signed-off-by: Amir Shehata <ashehata@whamcloud.com>
Change-Id: I4f320274576790e3332f66f30aad5c2b3450b955
Reviewed-by: Olaf Weber <olaf.weber@hpe.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Chris Horn <hornc@cray.com>
Signed-off-by: Minh Diep <mdiep@whamcloud.com>
Reviewed-on: https://review.whamcloud.com/36033
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>

diff --git a/lnet/lnet/lib-msg.c b/lnet/lnet/lib-msg.c
index af5295b..959e995 100644 (file)
--- a/lnet/lnet/lib-msg.c
+++ b/lnet/lnet/lib-msg.c
@@ -774,13 +774,7 @@ lnet_msg_detach_md(struct lnet_msg *msg, int cpt, int status)
        }
 
        if (unlink) {
-               /*
-                * if this is an ACK or a REPLY then make sure to remove the
-                * response tracker.
-                */
-               if (msg->msg_ev.type == LNET_EVENT_REPLY ||
-                   msg->msg_ev.type == LNET_EVENT_ACK)
-                       lnet_detach_rsp_tracker(msg->msg_md, cpt);
+               lnet_detach_rsp_tracker(md, cpt);
                lnet_md_unlink(md);
        }