bnxt_en: fix incorrect page count in RX aggr ring log
authorAlok Tiwari <alok.a.tiwari@oracle.com>
Sat, 30 Aug 2025 06:23:27 +0000 (23:23 -0700)
committerJakub Kicinski <kuba@kernel.org>
Mon, 1 Sep 2025 19:51:05 +0000 (12:51 -0700)
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: c0c050c58d84 ("bnxt_en: New Broadcom ethernet driver.")
Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Reviewed-by: Michael Chan <michael.chan@broadcom.com>
Reviewed-by: Somnath Kotur <somnath.kotur@broadcom.com>
Link: https://patch.msgid.link/20250830062331.783783-1-alok.a.tiwari@oracle.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/broadcom/bnxt/bnxt.c

index 31e3d825b4bc07750fc26a6c8c6931613179d1ad..0daa08cecaf288a8f02d80d5eb81854fd20ec1f9 100644 (file)
@@ -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);