Thanks for the comments. In the latest V3 patch set, I have moved CpuPageTableLib definition from UefiCpuPkg to MdePkg. So that MdeModulePkg won't depend on UefiCpuPkg.
Thanks, Dun -----Original Message----- From: Wang, Jian J <jian.j.w...@intel.com> Sent: Friday, April 14, 2023 5:10 PM To: Tan, Dun <dun....@intel.com>; devel@edk2.groups.io Cc: Gao, Liming <gaolim...@byosoft.com.cn>; Ni, Ray <ray...@intel.com> Subject: RE: [Patch V2 3/8] MdeModulePkg: Add CpuPageTableLib required by DxeIpl in DSC This will cause that MdeModulePkg depends on UefiCpuPkg. Please double check if there's any side effect introduced by breaking unidirectional dependency. Acked-by: Jian J Wang <jian.j.w...@intel.com> > -----Original Message----- > From: Tan, Dun <dun....@intel.com> > Sent: Friday, March 31, 2023 5:34 PM > To: devel@edk2.groups.io > Cc: Wang, Jian J <jian.j.w...@intel.com>; Gao, Liming > <gaolim...@byosoft.com.cn>; Ni, Ray <ray...@intel.com> > Subject: [Patch V2 3/8] MdeModulePkg: Add CpuPageTableLib required by > DxeIpl in DSC > > Add CpuPageTableLib instance required by DxeIpl in MdeModulePkg.dsc. > > Signed-off-by: Dun Tan <dun....@intel.com> > Cc: Jian J Wang <jian.j.w...@intel.com> > Cc: Liming Gao <gaolim...@byosoft.com.cn> > Cc: Ray Ni <ray...@intel.com> > --- > MdeModulePkg/MdeModulePkg.dsc | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/MdeModulePkg/MdeModulePkg.dsc > b/MdeModulePkg/MdeModulePkg.dsc index 1014598f31..d95acabe83 100644 > --- a/MdeModulePkg/MdeModulePkg.dsc > +++ b/MdeModulePkg/MdeModulePkg.dsc > @@ -2,7 +2,7 @@ > # EFI/PI Reference Module Package for All Architectures # # (C) > Copyright 2014 Hewlett-Packard Development Company, L.P.<BR> -# > Copyright (c) 2007 - 2021, Intel Corporation. All rights reserved.<BR> > +# Copyright (c) 2007 - 2023, Intel Corporation. All rights > +reserved.<BR> > # Copyright (c) Microsoft Corporation. > # Copyright (C) 2023 Advanced Micro Devices, Inc. All rights > reserved.<BR> # @@ -106,6 +106,7 @@ > > MmUnblockMemoryLib|MdePkg/Library/MmUnblockMemoryLib/MmUnblockM > emoryLibNull.inf > > VariableFlashInfoLib|MdeModulePkg/Library/BaseVariableFlashInfoLib/Bas > VariableFlashInfoLib|eVar > iableFlashInfoLib.inf > > IpmiCommandLib|MdeModulePkg/Library/BaseIpmiCommandLibNull/BaseIpmi > CommandLibNull.inf > + > + CpuPageTableLib|UefiCpuPkg/Library/CpuPageTableLib/CpuPageTableLib.i > + nf > > [LibraryClasses.EBC.PEIM] > IoLib|MdePkg/Library/PeiIoLibCpuIo/PeiIoLibCpuIo.inf > -- > 2.31.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#103473): https://edk2.groups.io/g/devel/message/103473 Mute This Topic: https://groups.io/mt/97969854/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
