git: 31cbd8f27d68 - main - bnxt_en: dcb: stop zeroing ETS and PFC config in bnxt_dcb_init()
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
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