Jiaxin, you are right! ExecuteFirstSmiInit doesn't belong to SmmRelocateBases. I will update commit message.
________________________________________ From: Ni, Ray <[email protected]> Sent: Monday, June 12, 2023 13:16 To: Wu, Jiaxin; [email protected] Cc: Dong, Eric; Kumar, Rahul R; Gerd Hoffmann Subject: Re: [edk2-devel] [PATCH V2 1/6] UefiCpuPkg/CpuSmm: Add perf-logging for time-consuming BSP procedures ________________________________________ From: Wu, Jiaxin <[email protected]> Sent: Friday, June 9, 2023 21:42 To: [email protected]; Ni, Ray Cc: Dong, Eric; Kumar, Rahul R; Gerd Hoffmann Subject: RE: [edk2-devel] [PATCH V2 1/6] UefiCpuPkg/CpuSmm: Add perf-logging for time-consuming BSP procedures > > The patch adds perf-logging for the following potential > time-consuming BSP procedures: > * PiCpuSmmEntry > - SmmRelocateBases > * ExecuteFirstSmiInit > > ExecuteFirstSmiInit is not belong to the SmmRelocateBases. Others good to me. ExecuteFirstSmiInit contributes to the overall running time when the CpuSmm driver relocates the SMM Base. > * BSPHandler > - SmmWaitForApArrival > - PerformRemainingTasks > * InitPaging > * SetMemMapAttributes > * SetUefiMemMapAttributes > * SetPageTableAttributes > * ConfigSmmCodeAccessCheck > * SmmCpuFeaturesCompleteSmmReadyToLock > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#105999): https://edk2.groups.io/g/devel/message/105999 Mute This Topic: https://groups.io/mt/99240108/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/leave/9847357/21656/1706620634/xyzzy [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
