Is there a reason why patches 5 and 6 where not combined? Mike
> -----Original Message----- > From: Abdul Lateef Attar <abdat...@amd.com> > Sent: Monday, April 10, 2023 4:10 AM > To: devel@edk2.groups.io > Cc: Abdul Lateef Attar <abdat...@amd.com>; Paul Grimes <paul.gri...@amd.com>; > Garrett Kirkendall <garrett.kirkend...@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: [PATCH v8 0/9] Adds AmdSmmCpuFeaturesLib and SmmSmramSaveStateLib > > PR: https://github.com/tianocore/edk2/pull/4258 > > 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: Garrett Kirkendall <garrett.kirkend...@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 (9): > MdePkg: Adds AMD SMRAM save state map > UefiCpuPkg: Adds SmmSmramSaveStateLib library class > UefiCpuPkg: Implements SmmSmramSaveStateLib library class > UefiCpuPkg/SmmCpuFeaturesLib: Restructure arch-dependent code > UefiCpuPkg: Initial implementation of AMD's SmmCpuFeaturesLib > UefiCpuPkg: Implements SmmCpuFeaturesLib for AMD Family > UefiCpuPkg: Implements SmmSmramSaveStateLib for Intel > UefiCpuPkg: Uses SmmSmramSaveStateLib library > OvmfPkg: Uses SmmSmramSaveStateLib library > > UefiCpuPkg/UefiCpuPkg.dec | 4 + > OvmfPkg/OvmfPkgIa32X64.dsc | 2 + > UefiCpuPkg/UefiCpuPkg.dsc | 15 + > .../AmdSmmCpuFeaturesLib.inf | 38 ++ > .../AmdSmmSmramSaveStateLib.inf | 28 + > .../IntelSmmSmramSaveStateLib.inf | 28 + > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf | 2 + > .../Include/Register/Amd/SmramSaveStateMap.h | 194 +++++++ > .../Include/Library/SmmSmramSaveStateLib.h | 70 +++ > .../SmmSmramSaveStateLib/SmramSaveState.h | 100 ++++ > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h | 2 + > .../SmmCpuFeaturesLib/AmdSmmCpuFeaturesLib.c | 445 ++++++++++++++++ > .../IntelSmmCpuFeaturesLib.c | 128 +++++ > .../SmmCpuFeaturesLibCommon.c | 128 ----- > .../SmmSmramSaveStateLib/AmdSmramSaveState.c | 286 ++++++++++ > .../IntelSmramSaveState.c | 359 +++++++++++++ > .../SmramSaveStateCommon.c | 232 ++++++++ > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c | 5 +- > UefiCpuPkg/PiSmmCpuDxeSmm/SmramSaveState.c | 500 +----------------- > MdePkg/MdePkg.ci.yaml | 4 +- > 20 files changed, 1941 insertions(+), 629 deletions(-) > create mode 100644 > UefiCpuPkg/Library/SmmCpuFeaturesLib/AmdSmmCpuFeaturesLib.inf > create mode 100644 > UefiCpuPkg/Library/SmmSmramSaveStateLib/AmdSmmSmramSaveStateLib.inf > create mode 100644 > UefiCpuPkg/Library/SmmSmramSaveStateLib/IntelSmmSmramSaveStateLib.inf > create mode 100644 MdePkg/Include/Register/Amd/SmramSaveStateMap.h > create mode 100644 UefiCpuPkg/Include/Library/SmmSmramSaveStateLib.h > create mode 100644 UefiCpuPkg/Library/SmmSmramSaveStateLib/SmramSaveState.h > create mode 100644 > UefiCpuPkg/Library/SmmCpuFeaturesLib/AmdSmmCpuFeaturesLib.c > create mode 100644 > UefiCpuPkg/Library/SmmSmramSaveStateLib/AmdSmramSaveState.c > create mode 100644 > UefiCpuPkg/Library/SmmSmramSaveStateLib/IntelSmramSaveState.c > create mode 100644 > UefiCpuPkg/Library/SmmSmramSaveStateLib/SmramSaveStateCommon.c > > -- > 2.25.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#102780): https://edk2.groups.io/g/devel/message/102780 Mute This Topic: https://groups.io/mt/98172946/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
