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

fix(mkimage): log console boot messages to hvc0 #112

Merged
merged 1 commit into from
Sep 2, 2023

Conversation

ryancurrah
Copy link
Contributor

VZ has support for serial console logging but the device is different from Qemu. VZ uses the hypervisor virtual console device instead. This change modifies the bootloader to add hvc0 as a device to log console messages to on boot.

Note this has no effect on the tty serial logging.

Issue: lima-vm/lima#1659

VZ has support for serial console logging but the device is different from Qemu. VZ uses the hypervisor virtual console device instead. This change modifies the bootloader to add hvc0 as a device to log console messages to on boot.

Note this has no effect on the tty serial logging.

Issue: lima-vm/lima#1659
Signed-off-by: Ryan Currah <[email protected]>
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

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

Thanks

@AkihiroSuda AkihiroSuda requested a review from jandubois August 30, 2023 19:58
Copy link
Member

@jandubois jandubois left a comment

Choose a reason for hiding this comment

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

Thanks, LGTM

Tested with QEMU on Monterey Intel, and VZ on Ventura M1

@jandubois jandubois merged commit 5cd42ea into lima-vm:main Sep 2, 2023
@jandubois
Copy link
Member

I assume you wanted a release with this change: https://github.com/lima-vm/alpine-lima/releases/tag/v0.2.32

@ryancurrah
Copy link
Contributor Author

Thanks @jandubois and @AkihiroSuda! Just a note on functionality in the Alpine Lima image. Because hvc0 is not configured as the primary console, messages to hvc0 will only work during boot. It's better then no messages, but maybe in the future we could revisit this.

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.

3 participants