On 3/10/23 11:03, Tom Lendacky wrote:
This patch series provides some fixes around AP creation:

- An erratum on AMD hardware requires that a VMSA not be aligned on a
   2MB boundary. To work around this issue, allocate 2 pages of memory
   and using the page that is not 2MB aligned and freeing the other.

- When parking APs after exiting boot services, the current SNP support
   will perform an allocation that will not be reflected in memory map
   being supplied to the OS. Instead of allocating new VMSAs each time,
   re-use the current VMSA.

BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=4353

---

These patches are based on commit:
f80f052277c8 ("OvmfPkg/RiscVVirt: Add Stack HOB")

Specified the wrong commit here...  should be:

9b94ebb0c826 ("DynamicTablesPkg: Add SMBIOS String table helper library")

Also, the first patch has a #define for IS_ALIGNED, which Gerd is trying to address with another patch series. Once his series is merged, I'll re-submit with the removal of the #define, but wanted to get this out for review now.

Thanks,
Tom


Cc:     Eric Dong <eric.d...@intel.com>
Cc:     Ray Ni <ray...@intel.com>
Cc:     Rahul Kumar <rahul1.ku...@intel.com>
Cc:     Gerd Hoffmann <kra...@redhat.com>
Cc:     Michael Roth <michael.r...@amd.com>
Cc:     Ashish Kalra <ashish.ka...@amd.com>

Tom Lendacky (2):
   UefiCpuPkg/MpInitLib: Ensure SEV-SNP VMSA allocations are not 2MB
     aligned
   UefiCpuPkg/MpInitLib: Reuse VMSA allocation to avoid unreserved
     allocation

  UefiCpuPkg/Library/MpInitLib/X64/AmdSev.c | 224 ++++++++++++++--------
  1 file changed, 144 insertions(+), 80 deletions(-)



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#101014): https://edk2.groups.io/g/devel/message/101014
Mute This Topic: https://groups.io/mt/97524216/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to