Re: [PATCH 0/2] SEV-SNP guest support fixes

["Lendacky, Thomas" <thomas.lendacky@amd.com>]

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.dong@intel.com>
> Cc:	Ray Ni <ray.ni@intel.com>
> Cc:	Rahul Kumar <rahul1.kumar@intel.com>
> Cc:	Gerd Hoffmann <kraxel@redhat.com>
> Cc:	Michael Roth <michael.roth@amd.com>
> Cc:	Ashish Kalra <Ashish.Kalra@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(-)
>