From 0831670f2bb195dea61b79345702c7c657452411 Mon Sep 17 00:00:00 2001 From: Timothy Day Date: Mon, 24 Feb 2025 01:12:08 -0500 Subject: [PATCH] LU-16518 lnet: fix unused-but-set-variable warnings Remove unused primary_nid variable in lnet_peer_merge_data() to silence a Clang compiler warning. Test-Parameters: trivial Signed-off-by: Timothy Day Change-Id: Ib66fd31c7acc08fa66578cd7ab571f278f98afe1 Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/58181 Reviewed-by: Andreas Dilger Reviewed-by: Max Wang Reviewed-by: James Simmons Reviewed-by: Serguei Smirnov Reviewed-by: Oleg Drokin Tested-by: jenkins Tested-by: Maloo --- lnet/lnet/peer.c | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/lnet/lnet/peer.c b/lnet/lnet/peer.c index da51655..e44ead4 100644 --- a/lnet/lnet/peer.c +++ b/lnet/lnet/peer.c @@ -3251,7 +3251,6 @@ static int lnet_peer_merge_data(struct lnet_peer *lp, struct lnet_ping_iter pi; struct lnet_nid nid; u32 *stp; - struct lnet_nid primary = {}; bool want_large_primary; unsigned int flags; int ncurnis; @@ -3301,8 +3300,6 @@ static int lnet_peer_merge_data(struct lnet_peer *lp, */ ping_iter_first(&pi, pbuf, NULL); stp = ping_iter_next(&pi, &nid); - if (stp) - primary = nid; want_large_primary = (pbuf->pb_info.pi_features & LNET_PING_FEAT_PRIMARY_LARGE); for (; stp; stp = ping_iter_next(&pi, &nid)) { @@ -3314,10 +3311,8 @@ static int lnet_peer_merge_data(struct lnet_peer *lp, addnis[naddnis].ns_status = *stp; naddnis += 1; } - if (want_large_primary && nid.nid_size) { - primary = nid; + if (want_large_primary && nid.nid_size) want_large_primary = false; - } } /* * Check for NIDs in curnis[] not present in pbuf. -- 1.8.3.1