Re: [patch] MdeModulePkg/PiSmmCore: Remove SMM Perf entry

["Gao, Liming" <liming.gao@intel.com>]

Reviewed-by: Liming Gao <liming.gao@intel.com>

>-----Original Message-----
>From: Bi, Dandan
>Sent: Wednesday, June 27, 2018 2:16 PM
>To: edk2-devel@lists.01.org
>Cc: Gao, Liming <liming.gao@intel.com>; Zeng, Star <star.zeng@intel.com>
>Subject: [patch] MdeModulePkg/PiSmmCore: Remove SMM Perf entry
>
>The perf measurement entry in SmmEntryPoint function
>doesn't have significant meaning. So remove it now.
>
>Cc: Liming Gao <liming.gao@intel.com>
>Cc: Star Zeng <star.zeng@intel.com>
>Contributed-under: TianoCore Contribution Agreement 1.1
>Signed-off-by: Dandan Bi <dandan.bi@intel.com>
>---
> MdeModulePkg/Core/PiSmmCore/PiSmmCore.c | 4 ----
> 1 file changed, 4 deletions(-)
>
>diff --git a/MdeModulePkg/Core/PiSmmCore/PiSmmCore.c
>b/MdeModulePkg/Core/PiSmmCore/PiSmmCore.c
>index 45f40e0e60c..2fb401e47e7 100644
>--- a/MdeModulePkg/Core/PiSmmCore/PiSmmCore.c
>+++ b/MdeModulePkg/Core/PiSmmCore/PiSmmCore.c
>@@ -642,12 +642,10 @@ SmmEntryPoint (
>   BOOLEAN                     InLegacyBoot;
>   BOOLEAN                     IsOverlapped;
>   VOID                        *CommunicationBuffer;
>   UINTN                       BufferSize;
>
>-  PERF_INMODULE_BEGIN ("SMM");
>-
>   //
>   // Update SMST with contents of the SmmEntryContext structure
>   //
>   gSmmCoreSmst.SmmStartupThisAp      = SmmEntryContext-
>>SmmStartupThisAp;
>   gSmmCoreSmst.CurrentlyExecutingCpu = SmmEntryContext-
>>CurrentlyExecutingCpu;
>@@ -736,12 +734,10 @@ SmmEntryPoint (
>     //
>     // Clear the InSmm flag as we are going to leave SMM
>     //
>     gSmmCorePrivate->InSmm = FALSE;
>   }
>-
>-  PERF_INMODULE_END ("SMM");
> }
>
> /**
>   Install LoadedImage protocol for SMM Core.
> **/
>--
>2.14.3.windows.1