git: 31cbd8f27d68 - main - bnxt_en: dcb: stop zeroing ETS and PFC config in bnxt_dcb_init()

From: Sumit Saxena <ssaxena_at_FreeBSD.org>
Date: Fri, 09 Oct 2026 18:57:27 UTC
The branch main has been updated by ssaxena:

URL: https://cgit.FreeBSD.org/src/commit/?id=31cbd8f27d687ce9b10a22c858296532be400bb4

commit 31cbd8f27d687ce9b10a22c858296532be400bb4
Author:     Sumit Saxena <sumit.saxena@broadcom.com>
AuthorDate: 2026-10-09 10:41:09 +0000
Commit:     Sumit Saxena <ssaxena@FreeBSD.org>
CommitDate: 2026-10-09 18:55:56 +0000

    bnxt_en: dcb: stop zeroing ETS and PFC config in bnxt_dcb_init()
    
    bnxt_dcb_init() currently pushes all-zero ETS and PFC settings to
    the firmware at every attach. This reserves bandwidth allocation
    for RoCE traffic, which prevents L2 traffic from reaching line rate.
    
    These settings should only be updated with valid values when the
    RoCE driver is loaded. Therefore, we should stop programming them
    during bnxt_dcb_init().
    
    Signed-off-by:  Andy Gospodarek <gospo@broadcom.com>
    Reviewed by:    chandrakanth.patil_broadcom.com, gallatin
    MFC after:      2 weeks
    Sponsored by:   Broadcom Inc.
    Differential Revision:  https://reviews.freebsd.org/D60516
---
 sys/dev/bnxt/bnxt_en/bnxt_dcb.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/sys/dev/bnxt/bnxt_en/bnxt_dcb.c b/sys/dev/bnxt/bnxt_en/bnxt_dcb.c
index 9841878785f7..9efd3ba8653d 100644
--- a/sys/dev/bnxt/bnxt_en/bnxt_dcb.c
+++ b/sys/dev/bnxt/bnxt_en/bnxt_dcb.c
@@ -865,9 +865,6 @@ bnxt_dcb_setdcbx(struct bnxt_softc *softc, uint8_t mode)
 void
 bnxt_dcb_init(struct bnxt_softc *softc)
 {
-	struct bnxt_ieee_ets ets = {0};
-	struct bnxt_ieee_pfc pfc = {0};
-
 	softc->dcbx_cap = 0;
 
 	if (softc->hwrm_spec_code < 0x10501)
@@ -888,9 +885,6 @@ bnxt_dcb_init(struct bnxt_softc *softc)
 	else if (softc->fw_cap & BNXT_FW_CAP_DCBX_AGENT)
 		softc->dcbx_cap |= BNXT_DCB_CAP_DCBX_LLD_MANAGED;
 
-	bnxt_dcb_ieee_setets(softc, &ets);
-	bnxt_dcb_ieee_setpfc(softc, &pfc);
-
 }
 
 void