Skip to content

rfc: Add Unaccepted Hot-Plug Memory Hint attribute#41

Open
pratiksampat wants to merge 1 commit into
tianocore:mainfrom
pratiksampat:rfc-unaccepted-hot-plug-memory
Open

rfc: Add Unaccepted Hot-Plug Memory Hint attribute#41
pratiksampat wants to merge 1 commit into
tianocore:mainfrom
pratiksampat:rfc-unaccepted-hot-plug-memory

Conversation

@pratiksampat

Copy link
Copy Markdown

Propose EFI_MEMORY_UHP and EFI_RESOURCE_ATTRIBUTE_UNACCEPTED_HOT_PLUG so firmware can mark unaccepted memory belonging to a memory hot-plug window, distinct from present-but-unvalidated unaccepted memory. This lets a confidential-computing guest OS defer acceptance of that window until the memory is actually hot-added, with an AMD SEV-SNP reference implementation on OVMF/QEMU.

Propose EFI_MEMORY_UHP and EFI_RESOURCE_ATTRIBUTE_UNACCEPTED_HOT_PLUG so
firmware can mark unaccepted memory belonging to a memory hot-plug window,
distinct from present-but-unvalidated unaccepted memory. This lets a
confidential-computing guest OS defer acceptance of that window until the
memory is actually hot-added, with an AMD SEV-SNP reference implementation
on OVMF/QEMU.

Signed-off-by: Pratik R. Sampat <prsampat@amd.com>
@pratiksampat
pratiksampat requested review from a team, makubacki and mdkinney as code owners July 15, 2026 16:44
@pratiksampat

Copy link
Copy Markdown
Author

Link to edk2 Github issue: tianocore/edk2#12810
Link to Code-first edk2 changes: https://github.com/pratiksampat/edk2/tree/GI12810-unaccepted_memory_hotplug

@os-d

os-d commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

@pratiksampat imo, this does not need an RFC. I think the change is scoped enough and doesn't raise fundamental edk2 questions, just PI/UEFI spec questions that we can discuss this fully in the Code First PR.

@pratiksampat

Copy link
Copy Markdown
Author

Sure thing, we can definitely close this out and continue the discussion over edk2's PR.

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