Re: [PATCH v3 1/9] OvmfPkg: Add CpuLib to module INFs that depend on UefiCpuLib.

["Zhiguang Liu" <zhiguang.liu@intel.com>]

Hi OvmfPkg Maintainers,
Could you help review this patch, and also the patch "[PATCH v3 4/9] OvmfPkg: Remove UefiCpuLib from module INFs." ?

Thanks
Zhiguang

> -----Original Message-----
> From: Liu, Zhiguang <zhiguang.liu@intel.com>
> Sent: Tuesday, November 1, 2022 3:59 PM
> To: devel@edk2.groups.io
> Cc: Liu, Zhiguang <zhiguang.liu@intel.com>; Ard Biesheuvel
> <ardb+tianocore@kernel.org>; Yao, Jiewen <jiewen.yao@intel.com>; Justen,
> Jordan L <jordan.l.justen@intel.com>; Gerd Hoffmann <kraxel@redhat.com>
> Subject: [PATCH v3 1/9] OvmfPkg: Add CpuLib to module INFs that depend
> on UefiCpuLib.
> 
> There are two libraries: MdePkg/CpuLib and UefiCpuPkg/UefiCpuLib and
> UefiCpuPkg/UefiCpuLib will be merged to MdePkg/CpuLib. To avoid build
> failure, add CpuLib dependency to all modules that depend on UefiCpuLib.
> 
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Gerd Hoffmann <kraxel@redhat.com>
> Signed-off-by: Zhiguang Liu <zhiguang.liu@intel.com>
> ---
>  OvmfPkg/IntelTdx/Sec/SecMain.c             | 1 +
>  OvmfPkg/Library/TdxMailboxLib/TdxMailbox.c | 1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/OvmfPkg/IntelTdx/Sec/SecMain.c
> b/OvmfPkg/IntelTdx/Sec/SecMain.c index ab01ec9ab1..eef47f17e4 100644
> --- a/OvmfPkg/IntelTdx/Sec/SecMain.c
> +++ b/OvmfPkg/IntelTdx/Sec/SecMain.c
> @@ -17,6 +17,7 @@
>  #include <Library/BaseMemoryLib.h>
>  #include <Library/PcdLib.h>
>  #include <Library/UefiCpuLib.h>
> +#include <Library/CpuLib.h>
>  #include <Library/DebugAgentLib.h>
>  #include <Library/IoLib.h>
>  #include <Library/PeCoffLib.h>
> diff --git a/OvmfPkg/Library/TdxMailboxLib/TdxMailbox.c
> b/OvmfPkg/Library/TdxMailboxLib/TdxMailbox.c
> index 74cb55611f..0e2515ca38 100644
> --- a/OvmfPkg/Library/TdxMailboxLib/TdxMailbox.c
> +++ b/OvmfPkg/Library/TdxMailboxLib/TdxMailbox.c
> @@ -11,6 +11,7 @@
>  #include <Library/BaseMemoryLib.h>
>  #include <Library/PcdLib.h>
>  #include <Library/UefiCpuLib.h>
> +#include <Library/CpuLib.h>
>  #include <Library/SynchronizationLib.h>  #include <Uefi/UefiBaseType.h>
> #include <Library/TdxLib.h>
> --
> 2.31.1.windows.1