RE: [PATCH linux-next] net:stmmac: Fix the unsigned expression compared with zero

-----Original Message-----
From: menglong8.dong@gmail.com <redacted>
Sent: 2021年7月15日 15:46
To: davem@davemloft.net
Cc: peppe.cavallaro@st.com; alexandre.torgue@foss.st.com;
joabreu@synopsys.com; kuba@kernel.org; mcoquelin.stm32@gmail.com;
netdev@vger.kernel.org; linux-stm32@st-md-mailman.stormreply.com;
linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org; Zhang
Yunkai [off-list ref]; Zeal Robot [off-list ref]
Subject: [PATCH linux-next] net:stmmac: Fix the unsigned expression compared
with zero

From: Zhang Yunkai <redacted>

WARNING:  Unsigned expression "queue" compared with zero.
Reported-by: Zeal Robot <redacted>
Signed-off-by: Zhang Yunkai <redacted>
---
 drivers/net/ethernet/stmicro/stmmac/stmmac_main.c | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c

b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
index 7b8404a21544..a4cf2c640531 100644

--- a/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c
+++ b/drivers/net/ethernet/stmicro/stmmac/stmmac_main.c

@@ -1699,7 +1699,7 @@ static int init_dma_rx_desc_rings(struct net_device

*dev, gfp_t flags)
 	return 0;

 err_init_rx_buffers:
-	while (queue >= 0) {
+	do {
 		struct stmmac_rx_queue *rx_q = &priv->rx_queue[queue];

 		if (rx_q->xsk_pool)

@@ -1710,11 +1710,7 @@ static int init_dma_rx_desc_rings(struct

net_device *dev, gfp_t flags)
 		rx_q->buf_alloc_num = 0;
 		rx_q->xsk_pool = NULL;

-		if (queue == 0)
-			break;
-
-		queue--;
-	}
+	} while (queue--);

 	return ret;
 }