Gerd, Can you kindly provide "Reviewed-by" for the patch series? Thanks, Ray
> -----Original Message----- > From: Ni, Ray > Sent: Tuesday, June 13, 2023 3:56 PM > To: devel@edk2.groups.io; abdat...@amd.com > Cc: Paul Grimes <paul.gri...@amd.com>; Abner Chang > <abner.ch...@amd.com>; Dong, Eric <eric.d...@intel.com>; Kumar, Rahul R > <rahul.r.ku...@intel.com>; Gerd Hoffmann <kra...@redhat.com>; Kinney, > Michael D <michael.d.kin...@intel.com>; Gao, Liming > <gaolim...@byosoft.com.cn>; Liu, Zhiguang <zhiguang....@intel.com>; Ard > Biesheuvel <ardb+tianoc...@kernel.org>; Yao, Jiewen > <jiewen....@intel.com>; Justen, Jordan L <jordan.l.jus...@intel.com> > Subject: RE: [edk2-devel] [PATCH v14 0/8] Adds AmdSmmCpuFeaturesLib and > MmSaveStateLib > > Reviewed-by: Ray Ni <ray...@intel.com> > > > -----Original Message----- > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Abdul > Lateef > > Attar via groups.io > > Sent: Tuesday, June 6, 2023 2:07 PM > > To: devel@edk2.groups.io > > Cc: Abdul Lateef Attar <abdat...@amd.com>; Paul Grimes > > <paul.gri...@amd.com>; Abner Chang <abner.ch...@amd.com>; Dong, > Eric > > <eric.d...@intel.com>; Ni, Ray <ray...@intel.com>; Kumar, Rahul R > > <rahul.r.ku...@intel.com>; Gerd Hoffmann <kra...@redhat.com>; Kinney, > > Michael D <michael.d.kin...@intel.com>; Gao, Liming > > <gaolim...@byosoft.com.cn>; Liu, Zhiguang <zhiguang....@intel.com>; Ard > > Biesheuvel <ardb+tianoc...@kernel.org>; Yao, Jiewen > <jiewen....@intel.com>; > > Justen, Jordan L <jordan.l.jus...@intel.com> > > Subject: [edk2-devel] [PATCH v14 0/8] Adds AmdSmmCpuFeaturesLib and > > MmSaveStateLib > > > > Backward-compatibility changes: > > This patch series removes the SmmCpuFeaturesReadSaveStateRegister > > and SmmCpuFeaturesWriteSaveStateRegister interface/function. > > SmmReadSaveState() and SmmWriteSaveState() now directly invokes > > MmSaveStateLib > > routines to save/restore registers. > > > > PR: https://github.com/tianocore/edk2/pull/4519 > > > > V14: Delta changes > > Added @note to the MmSaveStateLib.h. > > SaveState(Read/Write) of > > > EFI_SMM_SAVE_STATE_REGISTER_PROCESSOR_ID/EFI_MM_SAVE_STATE_REG > IS > > TER_PROCESSOR_ID > > is handled by PiSmmCpuDxeSmm driver. > > Fixed PatchCheck warnings. > > V13: Delta changes > > Address review comments from Ray Ni > > Changed the BASE _NAME of AmdSmmCpuFeaturesLib. > > Removed EFIAPI from local function. > > Removed CpuIndex parameter from MmSaveStateGetRegisterLma > > Modifed MmSaveStateGetRegisterIndex () to accept RegOffset > > as second parameter. > > Removed FILE_GUID library instance for intel implemention from > > UefiCpuPkg.dsc. > > V12: > > Addressed review comments from Michael. > > Added LibraryClasses to .inf file. > > removed duplicate MACRO definations. > > Moved related MACRO defination to respective file. > > V11: Delta changes > > Drop the OVMF implementation of MmSaveStateLib > > V10: Delta changes: > > Addressed review comments from Abner. > > V9: Delta changes: > > Addressed review comments. > > Rename to MmSaveStateLib. > > Also rename SMM_ defines to MM_. > > Implemented OVMF MmSaveStateLib. > > Removes SmmCpuFeaturesReadSaveStateRegister and > > SmmCpuFeaturesWriteSaveStateRegister > > function interface. > > V8 delta changes: > > Addressed review comments from Abner, > > Fix the whitespace error. > > Seperate the Ovmf changes to another patch > > V7 delta changes: > > Adds SmmSmramSaveStateLib for Intel processor. > > Integrate SmmSmramSaveStateLib library. > > V6 delta changes: > > Addressed review comments for Ray NI. > > removed unnecessary EFIAPI. > > V5 delta changes: > > rebase to master branch. > > updated Reviewed-by > > V4 delta changes: > > rebase to master branch. > > added reviewed-by. > > V3 delta changes: > > Addressed review comments from Abner chang. > > Re-arranged patch order. > > > > Cc: Paul Grimes <paul.gri...@amd.com> > > Cc: Abner Chang <abner.ch...@amd.com> > > Cc: Eric Dong <eric.d...@intel.com> > > Cc: Ray Ni <ray...@intel.com> > > Cc: Rahul Kumar <rahul1.ku...@intel.com> > > Cc: Gerd Hoffmann <kra...@redhat.com> > > Cc: Michael D Kinney <michael.d.kin...@intel.com> > > Cc: Liming Gao <gaolim...@byosoft.com.cn> > > Cc: Zhiguang Liu <zhiguang....@intel.com> > > Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> > > Cc: Jiewen Yao <jiewen....@intel.com> > > Cc: Jordan Justen <jordan.l.jus...@intel.com> > > Cc: Abdul Lateef Attar <abdat...@amd.com> > > > > Abdul Lateef Attar (8): > > MdePkg: Adds AMD SMRAM save state map > > UefiCpuPkg: Adds MmSaveStateLib library class > > UefiCpuPkg: Implements MmSaveStateLib library instance > > UefiCpuPkg/SmmCpuFeaturesLib: Restructure arch-dependent code > > UefiCpuPkg: Implements SmmCpuFeaturesLib for AMD Family > > UefiCpuPkg: Implements MmSaveStateLib for Intel > > UefiCpuPkg: Removes SmmCpuFeaturesReadSaveStateRegister > > OvmfPkg: Uses MmSaveStateLib library > > > > UefiCpuPkg/UefiCpuPkg.dec | 4 + > > OvmfPkg/OvmfPkgIa32.dsc | 1 + > > OvmfPkg/OvmfPkgIa32X64.dsc | 3 + > > OvmfPkg/OvmfPkgX64.dsc | 1 + > > UefiCpuPkg/UefiCpuPkg.dsc | 13 + > > .../MmSaveStateLib/AmdMmSaveStateLib.inf | 34 + > > .../MmSaveStateLib/IntelMmSaveStateLib.inf | 34 + > > .../AmdSmmCpuFeaturesLib.inf | 38 + > > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf | 2 + > > .../Include/Register/Amd/SmramSaveStateMap.h | 194 +++++ > > UefiCpuPkg/Include/Library/MmSaveStateLib.h | 74 ++ > > .../Include/Library/SmmCpuFeaturesLib.h | 52 -- > > .../Library/MmSaveStateLib/MmSaveState.h | 94 +++ > > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h | 56 +- > > .../SmmCpuFeaturesLib/SmmCpuFeaturesLib.c | 767 ------------------ > > .../Library/MmSaveStateLib/AmdMmSaveState.c | 309 +++++++ > > .../Library/MmSaveStateLib/IntelMmSaveState.c | 410 ++++++++++ > > .../MmSaveStateLib/MmSaveStateCommon.c | 132 +++ > > .../SmmCpuFeaturesLib/AmdSmmCpuFeaturesLib.c | 387 +++++++++ > > .../IntelSmmCpuFeaturesLib.c | 70 ++ > > .../SmmCpuFeaturesLibCommon.c | 128 --- > > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c | 11 +- > > UefiCpuPkg/PiSmmCpuDxeSmm/SmramSaveState.c | 500 +----------- > > MdePkg/MdePkg.ci.yaml | 4 +- > > 24 files changed, 1810 insertions(+), 1508 deletions(-) > > create mode 100644 > > UefiCpuPkg/Library/MmSaveStateLib/AmdMmSaveStateLib.inf > > create mode 100644 > > UefiCpuPkg/Library/MmSaveStateLib/IntelMmSaveStateLib.inf > > create mode 100644 > > UefiCpuPkg/Library/SmmCpuFeaturesLib/AmdSmmCpuFeaturesLib.inf > > create mode 100644 > MdePkg/Include/Register/Amd/SmramSaveStateMap.h > > create mode 100644 UefiCpuPkg/Include/Library/MmSaveStateLib.h > > create mode 100644 UefiCpuPkg/Library/MmSaveStateLib/MmSaveState.h > > create mode 100644 > UefiCpuPkg/Library/MmSaveStateLib/AmdMmSaveState.c > > create mode 100644 > UefiCpuPkg/Library/MmSaveStateLib/IntelMmSaveState.c > > create mode 100644 > > UefiCpuPkg/Library/MmSaveStateLib/MmSaveStateCommon.c > > create mode 100644 > > UefiCpuPkg/Library/SmmCpuFeaturesLib/AmdSmmCpuFeaturesLib.c > > > > -- > > 2.25.1 > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#106132): https://edk2.groups.io/g/devel/message/106132 Mute This Topic: https://groups.io/mt/99358069/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-