Received-SPF: pass (google.com: domain of linux-nfs+bounces-91-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
From: jsq <thfeathers@sina.cn>
To: chuck.lever@oracle.com,
	jlayton@kernel.org
Cc: neilb@suse.de,
	kolga@netapp.com,
	Dai.Ngo@oracle.com,
	tom@talpey.com,
	linux-nfs@vger.kernel.org,
	jsq <thfeathers@sina.cn>
Subject: [PATCH] SUNRPC: _xprt_switch_find_current_entry return xprt with condition find_active
Date: Mon, 27 Nov 2023 23:39:59 +0800
Message-ID: <20231127153959.2067-1-thfeathers@sina.cn>
Precedence: bulk
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit

current function always return a active xprt or NULL no matter what find_active
---
 net/sunrpc/xprtmultipath.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/net/sunrpc/xprtmultipath.c b/net/sunrpc/xprtmultipath.c
index 701250b305db..94f3b5f444a1 100644
--- a/net/sunrpc/xprtmultipath.c
+++ b/net/sunrpc/xprtmultipath.c
@@ -283,8 +283,7 @@ struct rpc_xprt *_xprt_switch_find_current_entry(struct list_head *head,
 	list_for_each_entry_rcu(pos, head, xprt_switch) {
 		if (cur == pos)
 			found = true;
-		if (found && ((find_active && xprt_is_active(pos)) ||
-			      (!find_active && xprt_is_active(pos))))
+		if (found && (find_active == xprt_is_active(pos)))
 			return pos;
 	}
 	return NULL;
-- 
2.43.0.windows.1