[AMD Official Use Only - General] Hi Mike, I think 5/9 was added for those functions AMD has no implementation as the initial commit, while 6/9 was updated for the functions with AMD implementations. @Attar, AbdulLateef (Abdul Lateef)can confirm this.
Thanks Abner > -----Original Message----- > From: Kinney, Michael D <michael.d.kin...@intel.com> > Sent: Tuesday, April 11, 2023 12:29 AM > To: Attar, AbdulLateef (Abdul Lateef) <abdullateef.at...@amd.com>; > devel@edk2.groups.io > Cc: Grimes, Paul <paul.gri...@amd.com>; Kirkendall, Garrett > <garrett.kirkend...@amd.com>; Chang, Abner <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>; 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>; Kinney, Michael D > <michael.d.kin...@intel.com> > Subject: RE: [PATCH v8 0/9] Adds AmdSmmCpuFeaturesLib and > SmmSmramSaveStateLib > > Caution: This message originated from an External Source. Use proper > caution when opening attachments, clicking links, or responding. > > > 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.i > nf > > 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 (#102789): https://edk2.groups.io/g/devel/message/102789 Mute This Topic: https://groups.io/mt/98172946/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-