Skip to content

sys-kernel/coreos-modules: Enable VFIO for arm64 - #2484

Merged
tormath1 merged 2 commits into
flatcar:mainfrom
genesiscloud:arm64-vfio
Dec 12, 2024
Merged

tormath1 merged 2 commits into
flatcar:mainfrom
genesiscloud:arm64-vfio

Conversation

@lukasstockner

Copy link
Copy Markdown
Contributor

Enable VFIO for arm64

VFIO was configured as x86-only, but it also works and is useful on ARM.
CONFIG_VFIO_PCI_VGA is x86-specific, so it remains in the amd64 config.

How to use

  • Build and run Flatcar image
  • modprobe vfio_pci
  • Bind a PCI device to vfio-pci

Testing done

  • Changelog entries added in the respective changelog/ directory (user-facing change, bug fix, security fix, update)
  • Inspected CI output for image differences: /boot and /usr size, packages, list files for any missing binaries, kernel modules, config files, kernel modules, etc.

VFIO was configured as x86-only, but it also works and is useful on ARM.
CONFIG_VFIO_PCI_VGA is x86-specific, so it remains in the amd64 config.

@jepio jepio left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you for this contribution! We'll let CI pass and then this can be merged.

@lukasstockner

Copy link
Copy Markdown
Contributor Author

Thanks for the quick review!
Looks like the CI failed, but I don't see how the error is related to the change. Is this something you've seen before?

@ader1990

Copy link
Copy Markdown
Contributor

Thanks for the quick review! Looks like the CI failed, but I don't see how the error is related to the change. Is this something you've seen before?

Hello, the CI failed because it could not use the code from your branch, it is expected when making a PR from a fork.

@github-actions

github-actions Bot commented Nov 27, 2024 •

Copy link
Copy Markdown

Build action triggered: https://github.com/flatcar/scripts/actions/runs/12293243395

Comment thread changelog/changes/2024-11-26-enable-vfio-on-arm.md Outdated
@tormath1
tormath1 merged commit 4dd088c into flatcar:main Dec 12, 2024
@tormath1

Copy link
Copy Markdown
Contributor

Thanks for your contribution :)

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants