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: Disable drivers for USB-attached network interfaces #2328

Merged

Conversation

markusboehme
Copy link
Member

Issue number: n/a

Description of changes: There is no need to support USB-attached network interfaces in the cloud, and such devices are equally unlikely to be encountered in servers. Make sure not to build any drivers for them as they would just be dead weight.

We had those drivers for the 5.4 kernel variants (last aws-k8s-1.19, now gone) already and picked them up for the 5.10 kernel variants (any variants except *-dev) with release 1.9.0. This reverses course for the 5.10 kernel and ensures future upstream changes won't bring those drivers into the 5.15 kernel.

Testing done:

  • Builds and boots
  • Config differences in final builds are only those expected

The full list of resulting config changes can be found in this Gist.

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.

There is no need to support USB-attached network interfaces in the
cloud, and such devices are equally unlikely to be encountered in
servers. Make sure not to build any drivers for them as they would just
be dead weight.

Signed-off-by: Markus Boehme <[email protected]>
@stmcginnis
Copy link
Contributor

Do we still want these for metal and homelab types of deployments? They may be needed for those types of hardware setups.

cc @mchaker

@mchaker
Copy link
Contributor

mchaker commented Aug 10, 2022

In the case of a homelab variant, a separate kernel config would be necessary -- there would be a lot of hardware to support that shouldn't weigh down the official metal variant kernel config.

@markusboehme markusboehme merged commit 781e6b4 into bottlerocket-os:develop Aug 11, 2022
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.

5 participants