]> 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 16:58:08 +0000 (18:58 +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 08886c3a28c61cf267e7e308b821a1391802dc27..8a6f3e230fce6bfc75228e2dad87246e6d411d9e 100644 (file)
@@ -4207,7 +4207,7 @@ static void bnxt_alloc_one_rx_ring_page(struct bnxt *bp,
        for (i = 0; i < bp->rx_agg_ring_size; i++) {
                if (bnxt_alloc_rx_page(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);