Commit 7000f4fa authored by Alok Tiwari's avatar Alok Tiwari Committed by Jakub Kicinski
Browse files

bnxt_en: fix incorrect page count in RX aggr ring log 



The warning in bnxt_alloc_one_rx_ring_netmem() reports the number
of pages allocated for the RX aggregation ring. However, it
mistakenly used bp->rx_ring_size instead of bp->rx_agg_ring_size,
leading to confusing or misleading log output.

Use the correct bp->rx_agg_ring_size value to fix this.

Fixes: c0c050c5 ("bnxt_en: New Broadcom ethernet driver.")
Signed-off-by: default avatarAlok Tiwari <alok.a.tiwari@oracle.com>
Reviewed-by: default avatarJacob Keller <jacob.e.keller@intel.com>
Reviewed-by: default avatarMichael Chan <michael.chan@broadcom.com>
Reviewed-by: default avatarSomnath Kotur <somnath.kotur@broadcom.com>
Link: https://patch.msgid.link/20250830062331.783783-1-alok.a.tiwari@oracle.com


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

drivers/net/ethernet/broadcom/bnxt/bnxt.c

+1 −1
Original line number Diff line number Diff line
@@ -4397,7 +4397,7 @@ static void bnxt_alloc_one_rx_ring_netmem(struct bnxt *bp, 
	for (i = 0; i < bp->rx_agg_ring_size; i++) {

		if (bnxt_alloc_rx_netmem(bp, rxr, prod, GFP_KERNEL)) {

			netdev_warn(bp->dev, "init'ed rx ring %d with %d/%d pages only\n",

				    ring_nr, i, bp->rx_ring_size);

				    ring_nr, i, bp->rx_agg_ring_size);

			break;

		}

		prod = NEXT_RX_AGG(prod);