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

Accept MADT and SRAT from the host via DevicePlatformSettings #656

Merged

Conversation

mebersol
Copy link
Collaborator

Prefer the MADT/SRAT tables offered by the host through DevicePlatformSettings over self-built tables, when the host is trusted.

This allows OpenHCL to support scenarios in which the VMM builds these tables in a device- or scenario-specific way.

@mebersol mebersol requested a review from a team as a code owner January 11, 2025 00:11
@daprilik
Copy link
Contributor

daprilik commented Jan 11, 2025

you've got some clippy, rustfmt, etc... stuff to fix, but otherwise this seems fine #Resolved

};

// Build the ACPI tables as specified.
if (build_madt) {
Copy link
Member

@jstarks jstarks Jan 11, 2025

Choose a reason for hiding this comment

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

No parens. #Resolved

jstarks
jstarks previously approved these changes Jan 11, 2025
@mebersol mebersol added the backport_2411 Change should be backported to the release/2411 branch label Jan 11, 2025
@mebersol mebersol merged commit 8da6f71 into microsoft:main Jan 11, 2025
25 checks passed
mebersol added a commit to mebersol/openvmm that referenced this pull request Jan 11, 2025
…oft#656)

Prefer the MADT/SRAT tables offered by the host through
DevicePlatformSettings over self-built tables, when the host is trusted.

This allows OpenHCL to support scenarios in which the VMM builds these
tables in a device- or scenario-specific way.
mebersol added a commit that referenced this pull request Jan 11, 2025
…656) (#657)

Prefer the MADT/SRAT tables offered by the host through
DevicePlatformSettings over self-built tables, when the host is trusted.

This allows OpenHCL to support scenarios in which the VMM builds these
tables in a device- or scenario-specific way.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport_2411 Change should be backported to the release/2411 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants