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

Use virtio-serial on newer kernels #466

Closed
wants to merge 1 commit into from

Conversation

bmwiedemann
Copy link
Member

@bmwiedemann bmwiedemann commented Sep 24, 2018

Assuming that all 4.x target kernels have virtio-serial
This avoids such ugly+noisy messages in output:
serial8250: too much work for irq4

It makes diffing of build logs easier to help reproducible builds.

https://bugzilla.redhat.com/show_bug.cgi?id=986761
https://bugs.launchpad.net/qemu/+bug/1719339

Assuming that all 4.x target kernels have virtio-serial
This avoids such ugly+noisy messages in output:
  serial8250: too much work for irq4

It makes diffing of build logs easier to help reproducible builds.

https://bugzilla.redhat.com/show_bug.cgi?id=986761
https://bugs.launchpad.net/qemu/+bug/1719339
@adrianschroeter
Copy link
Member

hm, all kernels since version 4 can have virtio-serial, but there is no guarantee that it is compiled in or that the matching module is available in initrd. Not sure about this...

@bmwiedemann
Copy link
Member Author

if the associated kernel config was available somewhere, we could grep there.

rpm -qf /boot/config-4.12.14-lp150.12.16-default
kernel-default-4.12.14-lp150.12.16.1.x86_64

@bmwiedemann
Copy link
Member Author

or maybe we can use lsinitrd | grep virtio_console.ko ... will give that a try.

@bmwiedemann
Copy link
Member Author

I tried

if xzcat $vm_initrd | cpio -t | grep -q 'kernel/drivers/char/virtio_console.ko$' ; then

but the driver is not part of the initrd. But virtio_serial still works, because the relevant work happens after initrd in the rootfs.

@adrianschroeter
Copy link
Member

This has been implemented here now: #566

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.

None yet

2 participants