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

Enabling hvc for KVM #1172

Merged
merged 1 commit into from
Jul 11, 2020
Merged

Enabling hvc for KVM #1172

merged 1 commit into from
Jul 11, 2020

Conversation

rvs
Copy link
Contributor

@rvs rvs commented Jul 7, 2020

I'm still testing it, so hold off on merging, but it is fully ready for review

This one moves us to using virtio-console as opposed to an emulated serial console for KVM domains. The virtio-console gets a fixed PCI ID #3 (hence renumbering of all PCI IDs in the tests) and is activated for all KVM domains. On the QEMU side we're now mapping that console to both a socket in the /run/hyperviosr/kvm/DOMAIN-NAME/cons AND to the stdout of the QEMU task. This has a nice benefit of collecting all the logs for the console in the cloud while at the same time allowing socat-style connection to the console for interactive debugging (your interactive shenanigans will be logged and sent to the cloud too!).

All of this allows us functionality that is on-part with Xen for containers AND also simplifies our life on ARM a great deal.

Signed-off-by: Roman Shaposhnik <rvs@zededa.com>
@rvs rvs requested a review from cshari-zededa July 7, 2020 19:51
@rvs rvs requested a review from eriknordmark as a code owner July 7, 2020 19:51
Copy link
Contributor

@eriknordmark eriknordmark left a comment

Choose a reason for hiding this comment

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

LGTM
Pull it when your testing is done

@rvs rvs merged commit 60add6f into lf-edge:master Jul 11, 2020
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