Skip to content

openhcl_boot: Serial output in the boot shim in SNP #1378

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

Open
wants to merge 30 commits into
base: main
Choose a base branch
from

Conversation

romank-msft
Copy link
Contributor

@romank-msft romank-msft commented May 17, 2025

Might be useful in other scenarios when running virtually baremetal SNP-isolated.
All the machinery inside VTL2 works (mapping the GHCB page, (un)accepting, doing GHCB calls), and the host prints the logs from the boot shim running in SNP.

image

@romank-msft romank-msft marked this pull request as ready for review May 20, 2025 20:10
@romank-msft romank-msft requested review from a team as code owners May 20, 2025 20:10
@mebersol mebersol added the backport_2505 Change should be backported to the release/2505 branch label May 20, 2025
@romank-msft
Copy link
Contributor Author

Don't approve just yet folks, will look more at the page acceptance code to make sure it matches the manuals.

@romank-msft
Copy link
Contributor Author

@jstarks Addressed your comments, appreciated very much! pvalidate makes the VM shut down when accepting the GHCB page back when unintializating. Looking what's up, followed the usual procedure, shuffled the steps around just in case. @chris-oo @mebersol @sluck-msft could you look to see what's suspicious?

@romank-msft
Copy link
Contributor Author

@jstarks Addressed your comments, appreciated very much! pvalidate makes the VM shut down when accepting the GHCB page back when unintializating. Looking what's up, followed the usual procedure, shuffled the steps around just in case. @chris-oo @mebersol @sluck-msft could you look to see what's suspicious?

found a way to rseolve

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport_2505 Change should be backported to the release/2505 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants