Commit 2764ab51 authored by Jason Xing's avatar Jason Xing Committed by Jakub Kicinski
Browse files

stmmac: xsk: fix negative overflow of budget in zerocopy mode 



A negative overflow can happen when the budget number of descs are
consumed. as long as the budget is decreased to zero, it will again go
into while (budget-- > 0) statement and get decreased by one, so the
overflow issue can happen. It will lead to returning true whereas the
expected value should be false.

In this case where all the budget is used up, it means zc function
should return false to let the poll run again because normally we
might have more data to process. Without this patch, zc function would
return true instead.

Fixes: 132c32ee ("net: stmmac: Add TX via XDP zero-copy socket")
Signed-off-by: default avatarJason Xing <kernelxing@tencent.com>
Reviewed-by: default avatarAleksandr Loktionov <aleksandr.loktionov@intel.com>
Link: https://patch.msgid.link/20250723142327.85187-2-kerneljasonxing@gmail.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent eccf7a34

drivers/net/ethernet/stmicro/stmmac/stmmac_main.c

+1 −1
Original line number Diff line number Diff line
@@ -2596,7 +2596,7 @@ static bool stmmac_xdp_xmit_zc(struct stmmac_priv *priv, u32 queue, u32 budget) 


	budget = min(budget, stmmac_tx_avail(priv, queue));



	while (budget-- > 0) {

	for (; budget > 0; budget--) {

		struct stmmac_metadata_request meta_req;

		struct xsk_tx_metadata *meta = NULL;

		dma_addr_t dma_addr;