Reviewed-by: Jiewen Yao <[email protected]>

> -----Original Message-----
> From: Xu, Min M <[email protected]>
> Sent: Sunday, June 12, 2022 7:25 PM
> To: [email protected]
> Cc: Xu, Min M <[email protected]>; Aktas, Erdem
> <[email protected]>; Gerd Hoffmann <[email protected]>; James
> Bottomley <[email protected]>; Yao, Jiewen <[email protected]>; Tom
> Lendacky <[email protected]>
> Subject: [PATCH 1/2] OvmfPkg: Use PcdOvmfWorkAreaBase instead of
> PcdSevEsWorkAreaBase
> 
> From: Min M Xu <[email protected]>
> 
> It is an typo error that HobList pointer should be stored at
> PcdOvmfWorkAreaBase, not PcdSevEsWorkAreaBase.
> 
> Cc: Erdem Aktas <[email protected]>
> Cc: Gerd Hoffmann <[email protected]>
> Cc: James Bottomley <[email protected]>
> Cc: Jiewen Yao <[email protected]>
> Cc: Tom Lendacky <[email protected]>
> Signed-off-by: Min Xu <[email protected]>
> ---
>  .../IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointer.c  | 4 ++--
>  .../PrePiHobListPointerLibTdx/PrePiHobListPointerLibTdx.inf   | 2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/OvmfPkg/IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointer.c
> b/OvmfPkg/IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointer.c
> index 1b41a3983c99..ecd271c0bdff 100644
> --- a/OvmfPkg/IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointer.c
> +++ b/OvmfPkg/IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointer.c
> @@ -27,7 +27,7 @@ PrePeiGetHobList (
>  {
>    TDX_WORK_AREA  *TdxWorkArea;
> 
> -  TdxWorkArea = (TDX_WORK_AREA *)(UINTN)FixedPcdGet32
> (PcdSevEsWorkAreaBase);
> +  TdxWorkArea = (TDX_WORK_AREA *)(UINTN)FixedPcdGet32
> (PcdOvmfWorkAreaBase);
>    ASSERT (TdxWorkArea != NULL);
>    ASSERT (TdxWorkArea->SecTdxWorkArea.HobList != 0);
> 
> @@ -48,7 +48,7 @@ PrePeiSetHobList (
>  {
>    TDX_WORK_AREA  *TdxWorkArea;
> 
> -  TdxWorkArea = (TDX_WORK_AREA *)(UINTN)FixedPcdGet32
> (PcdSevEsWorkAreaBase);
> +  TdxWorkArea = (TDX_WORK_AREA *)(UINTN)FixedPcdGet32
> (PcdOvmfWorkAreaBase);
>    ASSERT (TdxWorkArea != NULL);
> 
>    TdxWorkArea->SecTdxWorkArea.HobList = (UINTN)HobList;
> diff --git
> a/OvmfPkg/IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointerLibTdx.inf
> b/OvmfPkg/IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointerLibTdx.inf
> index 2667f841cea2..fe7ee23fdc8a 100644
> ---
> a/OvmfPkg/IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointerLibTdx.inf
> +++
> b/OvmfPkg/IntelTdx/PrePiHobListPointerLibTdx/PrePiHobListPointerLibTdx.inf
> @@ -23,7 +23,7 @@
>    UefiCpuPkg/UefiCpuPkg.dec
> 
>  [Pcd]
> -  gUefiCpuPkgTokenSpaceGuid.PcdSevEsWorkAreaBase
> +  gUefiOvmfPkgTokenSpaceGuid.PcdOvmfWorkAreaBase
> 
>  [LibraryClasses]
>    PcdLib
> --
> 2.29.2.windows.2



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#90463): https://edk2.groups.io/g/devel/message/90463
Mute This Topic: https://groups.io/mt/91704052/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to