Series Reviewed-by: Jiaxin Wu <[email protected]> The series patches are good to me now. Thanks Dun.
Thanks, Jiaxin > -----Original Message----- > From: Tan, Dun <[email protected]> > Sent: Friday, May 17, 2024 5:46 PM > To: [email protected] > Cc: Liming Gao <[email protected]>; Wu, Jiaxin > <[email protected]>; Ni, Ray <[email protected]>; Ard Biesheuvel > <[email protected]>; Yao, Jiewen <[email protected]>; Gerd > Hoffmann <[email protected]>; Kumar, Rahul R <[email protected]> > Subject: [Patch V2 00/18] Remove some S3 related code in CpuS3.c of smm > cpu driver > > Comparing to V1 patchs set, the V2 patch set only adjusts the commits > ordering and modifies copy right year in some files. > This patch set is to remove some S3 related code in CpuS3.c of smm cpu > driver. It contain commits to: > 1) S3 MTRRs operation: > > MdeModulePkg: Add gEdkiiS3MtrrSettingGuid > > OvmfPkg: Save MTRR by lockbox in CpuS3DataDxe > > UefiCpuPkg: Add locbox lib instance in DSC > > UefiCpuPkg: Save MTRR by lockbox in CpuS3DataDxe > > UefiCpuPkg: LoadMtrrData for all cpu in S3Resume > > UefiCpuPkg: Remove code to load mtrr setting > > 2) AP page table unavailiable issue fix: > > UefiCpuPkg: Disable PG in IA32 ApLoopCode > > 3) Register table cleanup: > > UefiCpuPkg:Set PcdCpuFeaturesInitOnS3Resume to TRUE > > UefiCpuPkg: Remove code to set register table > > 4) S3 ApHltLoopCode Operation: > > UefiCpuPkg:Abstract some DxeMpLib code to function > > UefiCpuPkg:Move some code in DxeMpLib to common place > > UefiCpuPkg: Install gEdkiiEndOfS3ResumeGuid in S3Resume > > UefiCpuPkg:Relocate AP to new safe buffer in PeiMpLib > > UefiCpuPkg:Remove code to handle APIC setting and Interrupt > > UefiCpuPkg:Rremove code to wakeup AP and relocate ap > > UefiCpuPkg: Remove the duplicated mpservice locate > > MdeModulePkg: remove MpService2Ppi field in SMM_S3_RESUME_STATE > > 5) Remove code to get AcpiCpuData: > > UefiCpuPkg: Remove GetAcpiCpuData() in CpuS3.c > > With this patch set, CpuS3.c in smm CPU driver can be simplified. > The whole patch set has been reviewed-by Ray. > PR for review: https://github.com/tianocore/edk2/pull/5606 > > Cc: Liming Gao <[email protected]> > Cc: Jiaxin Wu <[email protected]> > Cc: Ray Ni <[email protected]> > Cc: Ard Biesheuvel <[email protected]> > Cc: Jiewen Yao <[email protected]> > Cc: Gerd Hoffmann <[email protected]> > Cc: Rahul Kumar <[email protected]> > Cc: Gerd Hoffmann <[email protected]> > > Dun Tan (18): > MdeModulePkg: Add gEdkiiS3MtrrSettingGuid > OvmfPkg: Save MTRR by lockbox in CpuS3DataDxe > UefiCpuPkg: Add locbox lib instance in DSC > UefiCpuPkg: Save MTRR by lockbox in CpuS3DataDxe > UefiCpuPkg: LoadMtrrData for all cpu in S3Resume > UefiCpuPkg: Remove code to load mtrr setting > UefiCpuPkg:Set PcdCpuFeaturesInitOnS3Resume to TRUE > UefiCpuPkg: Remove code to set register table > UefiCpuPkg: Disable PG in IA32 ApLoopCode > UefiCpuPkg:Abstract some DxeMpLib code to function > UefiCpuPkg:Move some code in DxeMpLib to common place > UefiCpuPkg: Install gEdkiiEndOfS3ResumeGuid in S3Resume > UefiCpuPkg:Relocate AP to new safe buffer in PeiMpLib > UefiCpuPkg:Remove code to handle APIC setting and Interrupt > UefiCpuPkg:Remove code to wakeup AP and relocate ap > UefiCpuPkg: Remove unneeded MpService2Ppi assignment > MdeModulePkg:Remove MpService2Ppi field in SMM_S3_RESUME_STATE > UefiCpuPkg: Remove GetAcpiCpuData() in CpuS3.c > > MdeModulePkg/Include/Guid/AcpiS3Context.h | 3 +-- > MdeModulePkg/MdeModulePkg.dec | 5 ++++- > OvmfPkg/CpuS3DataDxe/CpuS3Data.c | 13 ++++++++++++- > OvmfPkg/CpuS3DataDxe/CpuS3DataDxe.inf | 4 +++- > UefiCpuPkg/CpuS3DataDxe/CpuS3Data.c | 13 ++++++++++++- > UefiCpuPkg/CpuS3DataDxe/CpuS3DataDxe.inf | 4 +++- > UefiCpuPkg/Library/MpInitLib/DxeMpLib.c | 183 > ++++++++++++++++++++++++++++++++++++++++++++++++----------------- > ---------------------------------------------------------------------------------------------- > -------------- > UefiCpuPkg/Library/MpInitLib/Ia32/MpFuncs.nasm | 6 +++++- > UefiCpuPkg/Library/MpInitLib/MpLib.c | 144 > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > +++++++++++++++++++- > UefiCpuPkg/Library/MpInitLib/MpLib.h | 56 > +++++++++++++++++++++++++++++++++++++++++++++++++++-- > UefiCpuPkg/Library/MpInitLib/PeiMpInitLib.inf | 6 +++++- > UefiCpuPkg/Library/MpInitLib/PeiMpLib.c | 154 > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > +++++++++++++++++++++++++++++- > UefiCpuPkg/PiSmmCpuDxeSmm/CpuS3.c | 993 > +++++++++------- > ---------------------------------------------------------------------------------------------- > ----------------------------------------------------------------- > UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/MpFuncs.nasm | 153 ------------ > ---------------------------------------------------------------------------------------------- > --------------------------------------- > UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/SmmFuncsArch.c | 29 +----------- > ---------------- > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c | 8 +++----- > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h | 26 +---------- > -------------- > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf | 5 +---- > UefiCpuPkg/PiSmmCpuDxeSmm/X64/MpFuncs.nasm | 189 ------------ > ---------------------------------------------------------------------------------------------- > ------------------------------------------------------------------------ > UefiCpuPkg/PiSmmCpuDxeSmm/X64/SmmFuncsArch.c | 30 +----------- > ----------------- > UefiCpuPkg/UefiCpuPkg.dec | 4 ++-- > UefiCpuPkg/UefiCpuPkg.dsc | 3 ++- > UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume.c | 78 > ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > +--------------- > UefiCpuPkg/Universal/Acpi/S3Resume2Pei/S3Resume2Pei.inf | 4 +++- > 24 files changed, 531 insertions(+), 1582 deletions(-) > delete mode 100644 UefiCpuPkg/PiSmmCpuDxeSmm/Ia32/MpFuncs.nasm > delete mode 100644 UefiCpuPkg/PiSmmCpuDxeSmm/X64/MpFuncs.nasm > > -- > 2.31.1.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#119182): https://edk2.groups.io/g/devel/message/119182 Mute This Topic: https://groups.io/mt/106150757/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
