]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
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)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 9 Sep 2025 17:02:22 +0000 (19:02 +0200)
[ Upstream commit 7000f4fa9b24ae2511b07babd0d49e888db5d265 ]

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>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/ethernet/broadcom/bnxt/bnxt.c

index cb76ab78904fc195fd8ea7c307bc39ebac476fdb..d47c1d81c49b82fa04c8960d5ec6d11a09efb32a 100644 (file)
@@ -4390,7 +4390,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);