From: Amir Shehata <ashehata@whamcloud.com>
Date: Fri, 19 Apr 2019 00:12:49 +0000 (-0700)
Subject: LU-12201 lnet: detach response tracker
X-Git-Tag: 2.12.3-RC1~52
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=339d351cb45d49f4a2e5199f24423ce68052e10a;p=fs%2Flustre-release.git

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
--- 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);
 	}