Whamcloud - gitweb
LU-12201 lnet: detach response tracker 70/34770/8
authorAmir Shehata <ashehata@whamcloud.com>
Fri, 19 Apr 2019 00:12:49 +0000 (17:12 -0700)
committerAmir Shehata <ashehata@whamcloud.com>
Fri, 7 Jun 2019 18:07:07 +0000 (18:07 +0000)
We need to unlink the response tracker from MDs even if the
corresponding message failed to send.

Test-Parameters: forbuildonly
Signed-off-by: Amir Shehata <ashehata@whamcloud.com>
Change-Id: I4f320274576790e3332f66f30aad5c2b3450b955
Reviewed-on: https://review.whamcloud.com/34770
Reviewed-by: Olaf Weber <olaf.weber@hpe.com>
Reviewed-by: Sebastien Buisson <sbuisson@ddn.com>
Reviewed-by: Chris Horn <hornc@cray.com>
Tested-by: Jenkins
lnet/lnet/lib-msg.c

index be2f94a..74491a2 100644 (file)
@@ -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);
        }