Skip to content

enable CONFIG_{ACPI, PCI} for x86 - #138

Open
jakecorrenti wants to merge 1 commit into
libkrun:mainfrom
jakecorrenti:add-acpi
Open

enable CONFIG_{ACPI, PCI} for x86#138
jakecorrenti wants to merge 1 commit into
libkrun:mainfrom
jakecorrenti:add-acpi

Conversation

@jakecorrenti

Copy link
Copy Markdown
Contributor

Enable PCI and ACPI in the kernel so that we can add support for them in libkrun and eventully support additional firmware options.

@jakecorrenti
jakecorrenti marked this pull request as draft August 6, 2026 14:36
Enable PCI and ACPI in the x86 kernel so that we can add support for them in
libkrun and eventully support additional firmware options.

Signed-off-by: Jake Correnti <jakecorrenti+github@proton.me>
@jakecorrenti
jakecorrenti marked this pull request as ready for review August 6, 2026 15:29
@jakecorrenti jakecorrenti changed the title enable CONFIG_{ACPI, PCI} enable CONFIG_{ACPI, PCI} for x86 Aug 6, 2026
CONFIG_LEGACY_VSYSCALL_XONLY=y
# CONFIG_LEGACY_VSYSCALL_NONE is not set
CONFIG_CMDLINE_BOOL=y
CONFIG_CMDLINE="reboot=k panic=-1 panic_print=0 nomodules console=hvc0 earlyprintk=hvc0 rootfstype=virtiofs rw no-kvmapf tsi_hijack init=/init.krun virtio_mmio.device=4K@0xd0000000:5 virtio_mmio.device=4K@0xd0001000:6 virtio_mmio.device=4K@0xd0002000:7 virtio_mmio.device=4K@0xd0003000:8 virtio_mmio.device=4K@0xd0004000:9 no-kvmclock"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Can we safely drop the manual virtio_mmio.device configuration?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Would keeping the manual config generate a conflict for the ACPI PR? I think it'd be safer keeping it, at least for a while.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I will double check

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