Yes, I'm working on a patch to use GetProcessorNumber() instead of using the APIC ID. Thanks, Tom >

Ah, yes, I did change that order. I could submit one more version that puts the ALIGN_VALUE back to the original position and fix the PcdGet32 space if that is desired (but, as you determined, it does

From: Tom Lendacky <thomas.lendacky@...> The AP reset vector stack allocation is only required if running as an SEV-ES guest. Since the reset vector allocation is below 1MB in memory, eliminate th

Re. #4, the code uses the APIC ID to calculate the stack start, so the calculation, now in GetApResetVectorSize(), uses the full value of PcdGet32(PcdCpuMaxLogicalProcessorNumber) instead of subtracti

Yup, will do. Thanks, Tom

From: Tom Lendacky <thomas.lendacky@...> The AP reset vector stack allocation is only required if running as an SEV-ES guest. Since the reset vector allocation is below 1MB in memory, eliminate th

Acked-by: Tom Lendacky <thomas.lendacky@...> Thanks, Tom

From: Tom Lendacky <thomas.lendacky@...> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2901 The DoDecrement variable in ApWakeupFunction () wasn't always being initialized. Update the code t

Hi Liming, It didn't feel quite right to keep your Tested-by: and Reviewed-by: for the change in v2 that I'm about to send. So please re-test / re-review. Thanks, Tom

Oh, I didn't realize you were waiting on another version of the bug fix. I'll send a v2 with the change over to: DoDecrement = (BOOLEAN) (CpuMpData->InitFlag == ApInitConfig); Thanks, Tom

From: Tom Lendacky <thomas.lendacky@...> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2901 The DoDecrement variable in ApWakeupFunction () wasn't always being initialized. Update the code t

Hi Liming, Yes, fix is on the way. Thanks, Tom

Yes, a big thank you to everyone involved! Tom

From: Tom Lendacky <thomas.lendacky@...> Register reviewers for the SEV-related files in OvmfPkg. Cc: Andrew Fish <afish@...> Cc: Laszlo Ersek <lersek@...> Cc: Leif Lindholm <leif@nuv

From: Tom Lendacky <thomas.lendacky@...> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2198 Before UEFI transfers control to the OS, it must park the AP. This is done using the AsmRelocateApL

From: Tom Lendacky <thomas.lendacky@...> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2198 After having transitioned from UEFI to the OS, the OS will need to boot the APs. For an SEV-ES gues

From: Tom Lendacky <thomas.lendacky@...> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2198 A hypervisor is not allowed to update an SEV-ES guest's register state, so when booting an SEV-ES g

From: Tom Lendacky <thomas.lendacky@...> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2198 Typically, an AP is booted using the INIT-SIPI-SIPI sequence. This sequence is intercepted by the h

From: Tom Lendacky <thomas.lendacky@...> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2198 When starting APs in an SMP configuration, the AP needs to know if it is running as an SEV-ES guest

From: Tom Lendacky <thomas.lendacky@...> BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=2198 A hypervisor is not allowed to update an SEV-ES guests register state, so when booting an SEV-ES gu