Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

kernel 5.10: Add bnxt module for Broadcom 10/25Gb network #2243

Merged
merged 1 commit into from
Jun 28, 2022

Conversation

zmrow
Copy link
Contributor

@zmrow zmrow commented Jun 24, 2022

Description of changes:
Adds the bnxt kernel module to support some Broadcom 10/15Gb network adapters.

Testing done:
Received a confirmed report a machine using this adapter was able to boot with this module.

Terms of contribution:

By submitting this pull request, I agree that this contribution is dual-licensed under the terms of both the Apache License, version 2.0, and the MIT license.

Copy link
Contributor

@bcressey bcressey left a comment

Choose a reason for hiding this comment

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

Looks good. Can you post a diff of the kernel configs before and after to confirm that nothing unexpected changed?

@zmrow
Copy link
Contributor Author

zmrow commented Jun 27, 2022

Looks good. Can you post a diff of the kernel configs before and after to confirm that nothing unexpected changed?

Looks like only the 5.10 x86_64 kernel changed.

--- a/config-bottlerocket-x86_64-kernel-5.10-5.10.118-1.x86_64.rpm
+++ b/config-bottlerocket-x86_64-kernel-5.10-5.10.118-1.x86_64.rpm
@@ -2369,7 +2369,11 @@ CONFIG_TIGON3=m
 CONFIG_TIGON3_HWMON=y
 # CONFIG_BNX2X is not set
 # CONFIG_SYSTEMPORT is not set
-# CONFIG_BNXT is not set
+CONFIG_BNXT=m
+CONFIG_BNXT_SRIOV=y
+CONFIG_BNXT_FLOWER_OFFLOAD=y
+# CONFIG_BNXT_DCB is not set
+CONFIG_BNXT_HWMON=y
 # CONFIG_NET_VENDOR_BROCADE is not set
 CONFIG_NET_VENDOR_CADENCE=y
 # CONFIG_MACB is not set

@zmrow zmrow merged commit b2e6448 into bottlerocket-os:develop Jun 28, 2022
@zmrow zmrow deleted the broadcom_bnxt branch June 28, 2022 20:03
foersleo added a commit to foersleo/bottlerocket that referenced this pull request Jul 6, 2022
Split out bare metal specific config options. All these are enabling
hardware support, mostly for specific storage or networking hardware
that is not available in settings where the non-metal variants are
used. For now split out the metal specific config options to later
enable only building those drivers for the metal variants and keep
the non-metal variants lean.

This moves the settings originally introduced in commits

    b2e6448 Merge pull request bottlerocket-os#2243 from zmrow/broadcom_bnxt
    6ab5fb6 kernel-5.10: enable mellanox modules
    3be82c9 kernel 5.10: Add `bnxt` module for Broadcom 10/25Gb network
    58d15ce kernel-5.10: add support for Microsemi PQI
    a69bcfa kernel: adjust option for megaraid sas
    d0d2c68 kernel-5.10: enable support for broadcom ethernet cards
    31d5062 kernel: add support for MegaRAID SAS
    928a481 kernel 5.10: Add config to support additional hardware

Signed-off-by: Leonard Foerster <[email protected]>
parth3sh pushed a commit to parth3sh/bottlerocket that referenced this pull request Aug 3, 2022
Split out bare metal specific config options. All these are enabling
hardware support, mostly for specific storage or networking hardware
that is not available in settings where the non-metal variants are
used. For now split out the metal specific config options to later
enable only building those drivers for the metal variants and keep
the non-metal variants lean.

This moves the settings originally introduced in commits

    b2e6448 Merge pull request bottlerocket-os#2243 from zmrow/broadcom_bnxt
    6ab5fb6 kernel-5.10: enable mellanox modules
    3be82c9 kernel 5.10: Add `bnxt` module for Broadcom 10/25Gb network
    58d15ce kernel-5.10: add support for Microsemi PQI
    a69bcfa kernel: adjust option for megaraid sas
    d0d2c68 kernel-5.10: enable support for broadcom ethernet cards
    31d5062 kernel: add support for MegaRAID SAS
    928a481 kernel 5.10: Add config to support additional hardware

Signed-off-by: Leonard Foerster <[email protected]>
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.

4 participants