Skip to content

enable Broadcom RoCE driver (bnxt_re) as module on x86_64#17874

Open
AZaugg wants to merge 1 commit into
microsoft:4.0from
AZaugg:azaugg/4.0/bnxt_re-enable
Open

enable Broadcom RoCE driver (bnxt_re) as module on x86_64#17874
AZaugg wants to merge 1 commit into
microsoft:4.0from
AZaugg:azaugg/4.0/bnxt_re-enable

Conversation

@AZaugg

@AZaugg AZaugg commented Jun 30, 2026

Copy link
Copy Markdown

Set CONFIG_INFINIBAND_BNXT_RE=m, reenabling RDMA support on Broadcom NetXtream NICS

Set CONFIG_INFINIBAND_BNXT_RE=m, reenabling RDMA support on
Broadcom NetXtream NICS
@AZaugg AZaugg requested a review from a team as a code owner June 30, 2026 20:19
Copilot AI review requested due to automatic review settings June 30, 2026 20:19
@github-actions

Copy link
Copy Markdown

Hello, and thank you for opening this pull request! 👋🏼 We appreciate the contribution.

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Commits in this repo will typically be prefixed with fix:, feat:, docs:, chore:, refactor:, test:, or ci: to indicate the type of change being proposed. The linked specification has more details.

Details:

No release type found in pull request title "enable Broadcom RoCE driver (bnxt_re) as module on x86_64". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to re-enable Broadcom NetXtream RoCE (bnxt_re) by setting CONFIG_INFINIBAND_BNXT_RE=m for the x86_64 Azure Linux kernel build.

Changes:

  • Enables CONFIG_INFINIBAND_BNXT_RE as a loadable module in the x86_64 kernel config (currently done in the rendered specs/ output).

Comment on lines 6179 to 6183
CONFIG_INFINIBAND_ADDR_TRANS=y
CONFIG_INFINIBAND_ADDR_TRANS_CONFIGFS=y
CONFIG_INFINIBAND_VIRT_DMA=y
# CONFIG_INFINIBAND_BNXT_RE is not set
CONFIG_INFINIBAND_BNXT_RE=m
# CONFIG_INFINIBAND_CXGB4 is not set
@github-actions

Copy link
Copy Markdown

📄❌ Rendered specs are out of date

FIX: — run this and commit the result:

azldev component render kernel

Or download the fix patch and apply it:

gh run download 28473262451 -R microsoft/azurelinux -n rendered-specs-patch
git apply rendered-specs.patch
Category Count
Content diffs 1
Extra files (untracked) 0
Missing files (deleted) 0

Content diffs

`specs/k/kernel/6.18-x86_64-azl.config`
--- committed/specs/k/kernel/6.18-x86_64-azl.config
+++ rendered/specs/k/kernel/6.18-x86_64-azl.config
@@ -6179,7 +6179,7 @@
 CONFIG_INFINIBAND_ADDR_TRANS=y
 CONFIG_INFINIBAND_ADDR_TRANS_CONFIGFS=y
 CONFIG_INFINIBAND_VIRT_DMA=y
-CONFIG_INFINIBAND_BNXT_RE=m
+# CONFIG_INFINIBAND_BNXT_RE is not set
 # CONFIG_INFINIBAND_CXGB4 is not set
 CONFIG_INFINIBAND_EFA=m
 CONFIG_INFINIBAND_ERDMA=m

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants