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

2018-06-27 Thread Gao, Liming
Reviewed-by: Liming Gao 

>-Original Message-
>From: Bi, Dandan
>Sent: Wednesday, June 27, 2018 2:16 PM
>To: edk2-devel@lists.01.org
>Cc: Gao, Liming ; Zeng, Star 
>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 
>Cc: Star Zeng 
>Contributed-under: TianoCore Contribution Agreement 1.1
>Signed-off-by: Dandan Bi 
>---
> 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

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


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

2018-06-27 Thread Dandan Bi
The perf measurement entry in SmmEntryPoint function
doesn't have significant meaning. So remove it now.

Cc: Liming Gao 
Cc: Star Zeng 
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi 
---
 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

___
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel