Fan: This version patch is good to me. Reviewed-by: Liming Gao <gaolim...@byosoft.com.cn> for this patch set.
Thanks Liming > -----邮件原件----- > 发件人: Wang, Fan <fan.w...@intel.com> > 发送时间: 2023年12月14日 22:15 > 收件人: Gao, Liming <gaolim...@byosoft.com.cn>; Kumar, Rahul R > <rahul.r.ku...@intel.com> > 抄送: Wang, Fan <fan.w...@intel.com>; Wang, Jian J > <jian.j.w...@intel.com>; Ni, Ray <ray...@intel.com>; Kinney, Michael D > <michael.d.kin...@intel.com>; Jiang, Guomin <guomin.ji...@intel.com>; > devel@edk2.groups.io > 主题: RE: [edk2-devel] [PATCH v5 0/2] Support customized FV Migration > Information > > Hi Liming and Kumar > > Could you help review this v5 patch: > > MdeModulePkg: Support customized FV Migration Information: > https://github.com/fanwang2intel/edk2/commit/83c55a73107bfb13df1d8b5 > 22e0ea1d18ef3d86b > > SecurityPkg: Support customized FV Migration Information > https://github.com/fanwang2intel/edk2/commit/6f7e955f1d63a875f69ebc08 > 4885e76610acc722 > > v5: > - Remove RemoveFvHobsInTemporaryMemory() since no consumer will call > this API now. > - Separate patches to different packages. > > Best Regards > Fan > > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Wang Fan > Sent: Thursday, December 14, 2023 9:53 PM > To: devel@edk2.groups.io > Cc: Wang, Fan <fan.w...@intel.com> > Subject: [edk2-devel] [PATCH v5 0/2] Support customized FV Migration > Information > > There are use cases which not all FVs need be migrated from TempRam to > permanent memory before TempRam tears down. This new guid is introduced > to avoid unnecessary FV migration to improve boot performance. Platform can > publish MigrationInfo hob with this guid to customize FV migration info, and > PeiCore will only migrate FVs indicated by this Hob info. > > This is a backwards compatible change, PeiCore will check MigrationInfo hob > before migration. If MigrationInfo hobs exists, only migrate FVs recorded by > hobs. If MigrationInfo hobs not exists, migrate all FVs to permanent memory. > > In Tcg driver, when MigratedFvInfo hob is detected, existing code logic is > assuming FV raw data is already copied, and raw data base address is also > recorded. Due to the new PeiCore change, the platform can publish hob to > indicate raw data need be copied or not along with FV migration. > > Two cases need be considered to skip copy for boot performance: The first > case is FV is not expected to be measured in post-mem phase, we should use > MeasurementExcludedPpiGuid to skip measurement. The second case is FV > raw data has no need to do rebase operation after migration, then > measurement should calculate hash directly from FV base address. > > Fan Wang (2): > MdeModulePkg: Support customized FV Migration Information > SecurityPkg: Support customized FV Migration Information > > MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c | 84 > ++++++++++++++----- > MdeModulePkg/Core/Pei/Memory/MemoryServices.c | 40 --------- > MdeModulePkg/Core/Pei/PeiMain.h | 11 --- > MdeModulePkg/Core/Pei/PeiMain.inf | 1 + > MdeModulePkg/Include/Guid/MigratedFvInfo.h | 42 +++++++++- > MdeModulePkg/MdeModulePkg.dec | 3 +- > SecurityPkg/Tcg/Tcg2Pei/Tcg2Pei.c | 7 +- > SecurityPkg/Tcg/TcgPei/TcgPei.c | 7 +- > 8 files changed, 116 insertions(+), 79 deletions(-) > > -- > 2.29.2.windows.2 > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#112693): https://edk2.groups.io/g/devel/message/112693 Mute This Topic: https://groups.io/mt/103258717/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-