Hi Sachin
I like this clean up. Thanks for doing this.
I saw this patch is 6/6, but I did not see any other such as 1/6 ~ 5/6 in my
mailbox. Not sure what is happening on my side.
Just double confirm, have you sent those patches?
Thank you
Yao, Jiewen
> -----Original Message-----
> From: Sachin Ganesh <sachingan...@ami.com>
> Sent: Thursday, April 18, 2024 3:45 AM
> To: devel@edk2.groups.io
> Cc: gaolim...@byosoft.com.cn; ardb+tianoc...@kernel.org; kra...@redhat.com;
> Yao, Jiewen <jiewen....@intel.com>; Aktas, Erdem <erdemak...@google.com>;
> Xu, Min M <min.m...@intel.com>; thomas.lenda...@amd.com; POLUDOV,
> FELIX <fel...@ami.com>; Dhanaraj V <vdhana...@ami.com>; Sachin Ganesh
> <sachingan...@ami.com>
> Subject: [PATCH 6/6] OvmfPkg: Use newly defined Unaccepted Memory Type
>
> EFI_RESOURCE_MEMORY_UNACCEPTED has been officially defined in the PI
> 1.8 specification. So all temporary solutions have been replaced with
> the actual definition.
>
> Cc: Felix Polyudov <fel...@ami.com>
> Cc: Dhanaraj V <vdhana...@ami.com>
> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org>
> Cc: Jiewen Yao <jiewen....@intel.com>
> Cc: Gerd Hoffmann <kra...@redhat.com>
> Cc: Erdem Aktas <erdemak...@google.com>
> Cc: Min Xu <min.m...@intel.com>
> Cc: Tom Lendacky <thomas.lenda...@amd.com>
> Signed-off-by: Sachin Ganesh <sachingan...@ami.com>
> ---
> OvmfPkg/AmdSevDxe/AmdSevDxe.c | 4 ++--
> OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c | 8 ++++----
> OvmfPkg/Library/PeilessStartupLib/Hob.c | 4 ++--
> OvmfPkg/Library/PlatformInitLib/IntelTdx.c | 8 ++++----
> OvmfPkg/PlatformPei/AmdSev.c | 4 ++--
> 5 files changed, 14 insertions(+), 14 deletions(-)
>
> diff --git a/OvmfPkg/AmdSevDxe/AmdSevDxe.c
> b/OvmfPkg/AmdSevDxe/AmdSevDxe.c
> index db3675ae86..d497a343d3 100644
> --- a/OvmfPkg/AmdSevDxe/AmdSevDxe.c
> +++ b/OvmfPkg/AmdSevDxe/AmdSevDxe.c
> @@ -20,7 +20,7 @@
> #include <Library/UefiBootServicesTableLib.h>
>
> #include <Guid/ConfidentialComputingSevSnpBlob.h>
>
> #include <Library/PcdLib.h>
>
> -#include <Pi/PrePiDxeCis.h>
>
> +#include <Pi/PiDxeCis.h>
>
> #include <Protocol/SevMemoryAcceptance.h>
>
> #include <Protocol/MemoryAccept.h>
>
> #include <Uefi/UefiSpec.h>
>
> @@ -119,7 +119,7 @@ AcceptAllMemory (
> CONST EFI_GCD_MEMORY_SPACE_DESCRIPTOR *Desc;
>
>
>
> Desc = &AllDescMap[Index];
>
> - if (Desc->GcdMemoryType != EFI_GCD_MEMORY_TYPE_UNACCEPTED) {
>
> + if (Desc->GcdMemoryType != EfiGcdMemoryTypeUnaccepted) {
>
> continue;
>
> }
>
>
>
> diff --git a/OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c
> b/OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c
> index 3372cee2f7..b6085eab44 100644
> --- a/OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c
> +++ b/OvmfPkg/IntelTdx/TdxHelperLib/SecTdxHelper.c
> @@ -19,7 +19,7 @@
> #include <Library/TdxLib.h>
>
> #include <Library/TdxMailboxLib.h>
>
> #include <Library/SynchronizationLib.h>
>
> -#include <Pi/PrePiHob.h>
>
> +#include <Pi/PiHob.h>
>
> #include <WorkArea.h>
>
> #include <ConfidentialComputingGuestAttr.h>
>
> #include <Library/TdxHelperLib.h>
>
> @@ -351,7 +351,7 @@ AcceptMemoryForAPsStack (
> if (Hob.Header->HobType == EFI_HOB_TYPE_RESOURCE_DESCRIPTOR) {
>
> DEBUG ((DEBUG_INFO, "\nResourceType: 0x%x\n", Hob.ResourceDescriptor-
> >ResourceType));
>
>
>
> - if (Hob.ResourceDescriptor->ResourceType ==
> BZ3937_EFI_RESOURCE_MEMORY_UNACCEPTED) {
>
> + if (Hob.ResourceDescriptor->ResourceType ==
> EFI_RESOURCE_MEMORY_UNACCEPTED) {
>
> ResourceLength = Hob.ResourceDescriptor->ResourceLength;
>
> PhysicalStart = Hob.ResourceDescriptor->PhysicalStart;
>
> PhysicalEnd = PhysicalStart + ResourceLength;
>
> @@ -427,7 +427,7 @@ AcceptMemory (
> //
>
> while (!END_OF_HOB_LIST (Hob)) {
>
> if (Hob.Header->HobType == EFI_HOB_TYPE_RESOURCE_DESCRIPTOR) {
>
> - if (Hob.ResourceDescriptor->ResourceType ==
> BZ3937_EFI_RESOURCE_MEMORY_UNACCEPTED) {
>
> + if (Hob.ResourceDescriptor->ResourceType ==
> EFI_RESOURCE_MEMORY_UNACCEPTED) {
>
> PhysicalStart = Hob.ResourceDescriptor->PhysicalStart;
>
> PhysicalEnd = PhysicalStart +
> Hob.ResourceDescriptor->ResourceLength;
>
>
>
> @@ -563,7 +563,7 @@ ValidateHobList (
> EFI_RESOURCE_MEMORY_MAPPED_IO_PORT,
>
> EFI_RESOURCE_MEMORY_RESERVED,
>
> EFI_RESOURCE_IO_RESERVED,
>
> - BZ3937_EFI_RESOURCE_MEMORY_UNACCEPTED
>
> + EFI_RESOURCE_MEMORY_UNACCEPTED
>
> };
>
>
>
> if (VmmHobList == NULL) {
>
> diff --git a/OvmfPkg/Library/PeilessStartupLib/Hob.c
> b/OvmfPkg/Library/PeilessStartupLib/Hob.c
> index 318b74c95d..725927da73 100644
> --- a/OvmfPkg/Library/PeilessStartupLib/Hob.c
> +++ b/OvmfPkg/Library/PeilessStartupLib/Hob.c
> @@ -20,7 +20,7 @@
> #include <IndustryStandard/UefiTcgPlatform.h>
>
> #include <Library/PlatformInitLib.h>
>
> #include <OvmfPlatforms.h>
>
> -#include <Pi/PrePiHob.h>
>
> +#include <Pi/PiHob.h>
>
> #include "PeilessStartupInternal.h"
>
>
>
> /**
>
> @@ -92,7 +92,7 @@ ConstructFwHobList (
> //
>
> while (!END_OF_HOB_LIST (Hob)) {
>
> if (Hob.Header->HobType == EFI_HOB_TYPE_RESOURCE_DESCRIPTOR) {
>
> - if (Hob.ResourceDescriptor->ResourceType ==
> BZ3937_EFI_RESOURCE_MEMORY_UNACCEPTED) {
>
> + if (Hob.ResourceDescriptor->ResourceType ==
> EFI_RESOURCE_MEMORY_UNACCEPTED) {
>
> PhysicalEnd = Hob.ResourceDescriptor->PhysicalStart +
> Hob.ResourceDescriptor->ResourceLength;
>
> ResourceLength = Hob.ResourceDescriptor->ResourceLength;
>
>
>
> diff --git a/OvmfPkg/Library/PlatformInitLib/IntelTdx.c
> b/OvmfPkg/Library/PlatformInitLib/IntelTdx.c
> index ada8592ddd..e561cee30b 100644
> --- a/OvmfPkg/Library/PlatformInitLib/IntelTdx.c
> +++ b/OvmfPkg/Library/PlatformInitLib/IntelTdx.c
> @@ -17,7 +17,7 @@
> #include <IndustryStandard/Tdx.h>
>
> #include <IndustryStandard/IntelTdx.h>
>
> #include <Library/PeiServicesLib.h>
>
> -#include <Pi/PrePiHob.h>
>
> +#include <Pi/PiHob.h>
>
> #include <WorkArea.h>
>
> #include <ConfidentialComputingGuestAttr.h>
>
>
>
> @@ -40,9 +40,9 @@ BuildResourceDescriptorHobForUnacceptedMemory (
> EFI_RESOURCE_ATTRIBUTE_TYPE ResourceAttribute;
>
> UINT64 MaxAcceptedMemoryAddress;
>
>
>
> - ASSERT (Hob->ResourceType ==
> BZ3937_EFI_RESOURCE_MEMORY_UNACCEPTED);
>
> + ASSERT (Hob->ResourceType == EFI_RESOURCE_MEMORY_UNACCEPTED);
>
>
>
> - ResourceType = BZ3937_EFI_RESOURCE_MEMORY_UNACCEPTED;
>
> + ResourceType = EFI_RESOURCE_MEMORY_UNACCEPTED;
>
> ResourceAttribute = Hob->ResourceAttribute;
>
> PhysicalStart = Hob->PhysicalStart;
>
> ResourceLength = Hob->ResourceLength;
>
> @@ -104,7 +104,7 @@ TransferTdxHobList (
> ResourceType = Hob.ResourceDescriptor->ResourceType;
>
> ResourceAttribute = Hob.ResourceDescriptor->ResourceAttribute;
>
>
>
> - if (ResourceType == BZ3937_EFI_RESOURCE_MEMORY_UNACCEPTED) {
>
> + if (ResourceType == EFI_RESOURCE_MEMORY_UNACCEPTED) {
>
> BuildResourceDescriptorHobForUnacceptedMemory
> (Hob.ResourceDescriptor);
>
> } else {
>
> BuildResourceDescriptorHob (
>
> diff --git a/OvmfPkg/PlatformPei/AmdSev.c b/OvmfPkg/PlatformPei/AmdSev.c
> index e6b602d79a..52fb4a7202 100644
> --- a/OvmfPkg/PlatformPei/AmdSev.c
> +++ b/OvmfPkg/PlatformPei/AmdSev.c
> @@ -16,7 +16,7 @@
> #include <Library/MemEncryptSevLib.h>
>
> #include <Library/MemoryAllocationLib.h>
>
> #include <Library/PcdLib.h>
>
> -#include <Pi/PrePiHob.h>
>
> +#include <Pi/PiHob.h>
>
> #include <PiPei.h>
>
> #include <Register/Amd/Msr.h>
>
> #include <Register/Intel/SmramSaveStateMap.h>
>
> @@ -67,7 +67,7 @@ AmdSevSnpInitialize (
>
>
> if (ResourceHob->ResourceType == EFI_RESOURCE_SYSTEM_MEMORY) {
>
> if (ResourceHob->PhysicalStart >= SIZE_4GB) {
>
> - ResourceHob->ResourceType =
> BZ3937_EFI_RESOURCE_MEMORY_UNACCEPTED;
>
> + ResourceHob->ResourceType = EFI_RESOURCE_MEMORY_UNACCEPTED;
>
> continue;
>
> }
>
>
>
> --
> 2.24.1.windows.2
> -The information contained in this message may be confidential and proprietary
> to American Megatrends (AMI). This communication is intended to be read only
> by
> the individual or entity to whom it is addressed or by their designee. If the
> reader
> of this message is not the intended recipient, you are on notice that any
> distribution of this message, in any form, is strictly prohibited. Please
> promptly
> notify the sender by reply e-mail or by telephone at 770-246-8600, and then
> delete or destroy all copies of the transmission.
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#117938): https://edk2.groups.io/g/devel/message/117938
Mute This Topic: https://groups.io/mt/105585462/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
