From 339d351cb45d49f4a2e5199f24423ce68052e10a Mon Sep 17 00:00:00 2001 From: Amir Shehata Date: Thu, 18 Apr 2019 17:12:49 -0700 Subject: [PATCH] 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 Change-Id: I4f320274576790e3332f66f30aad5c2b3450b955 Reviewed-by: Olaf Weber Reviewed-by: Sebastien Buisson Reviewed-by: Chris Horn Signed-off-by: Minh Diep Reviewed-on: https://review.whamcloud.com/36033 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Oleg Drokin --- lnet/lnet/lib-msg.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) 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); } -- 1.8.3.1