]> git.proxmox.com Git - mirror_edk2.git/commitdiff
MdeModulePkg: Use new added Perf macros
authorBi, Dandan <dandan.bi@intel.com>
Fri, 22 Jun 2018 08:56:21 +0000 (16:56 +0800)
committerLiming Gao <liming.gao@intel.com>
Tue, 26 Jun 2018 07:36:59 +0000 (15:36 +0800)
Replace old Perf macros with the new added ones.

Cc: Liming Gao <liming.gao@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Cc: Eric Dong <eric.dong@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
MdeModulePkg/Core/Dxe/Dispatcher/Dispatcher.c
MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c
MdeModulePkg/Core/Dxe/Hand/DriverSupport.c
MdeModulePkg/Core/Dxe/Image/Image.c
MdeModulePkg/Core/Pei/Dispatcher/Dispatcher.c
MdeModulePkg/Core/Pei/PeiMain/PeiMain.c
MdeModulePkg/Core/PiSmmCore/Dispatcher.c
MdeModulePkg/Core/PiSmmCore/PiSmmCore.c
MdeModulePkg/Library/UefiBootManagerLib/BmBoot.c
MdeModulePkg/Library/UefiBootManagerLib/BmConsole.c
MdeModulePkg/Universal/BdsDxe/BdsEntry.c

index 088cd81063137a269e0d9f2957713bd9cc239fc0..4c6ca761a6054b7b2605ffd1a4e7ddb7c61accab 100644 (file)
@@ -419,6 +419,7 @@ CoreDispatcher (
   BOOLEAN                         ReadyToRun;\r
   EFI_EVENT                       DxeDispatchEvent;\r
   \r
+  PERF_FUNCTION_BEGIN ();\r
 \r
   if (gDispatcherRunning) {\r
     //\r
@@ -584,6 +585,8 @@ CoreDispatcher (
 \r
   gDispatcherRunning = FALSE;\r
 \r
+  PERF_FUNCTION_END ();\r
+\r
   return ReturnStatus;\r
 }\r
 \r
@@ -1437,6 +1440,8 @@ CoreInitializeDispatcher (
   VOID\r
   )\r
 {\r
+  PERF_FUNCTION_BEGIN ();\r
+\r
   mFwVolEvent = EfiCreateProtocolNotifyEvent (\r
                   &gEfiFirmwareVolume2ProtocolGuid,\r
                   TPL_CALLBACK,\r
@@ -1444,6 +1449,8 @@ CoreInitializeDispatcher (
                   NULL,\r
                   &mFwVolEventRegistration\r
                   );\r
+\r
+  PERF_FUNCTION_END ();\r
 }\r
 \r
 //\r
index 07c740346242fb229b077575766402fb3667da8f..3f753738a273fca1c0dc312878b97cb886543078 100644 (file)
@@ -301,8 +301,8 @@ DxeMain (
   // Call constructor for all libraries\r
   //\r
   ProcessLibraryConstructorList (gDxeCoreImageHandle, gDxeCoreST);\r
-  PERF_END   (NULL,"PEI", NULL, 0) ;\r
-  PERF_START (NULL,"DXE", NULL, 0) ;\r
+  PERF_CROSSMODULE_END   ("PEI");\r
+  PERF_CROSSMODULE_BEGIN ("DXE");\r
 \r
   //\r
   // Report DXE Core image information to the PE/COFF Extra Action Library\r
@@ -499,16 +499,12 @@ DxeMain (
   //\r
   // Initialize the DXE Dispatcher\r
   //\r
-  PERF_START (NULL,"CoreInitializeDispatcher", "DxeMain", 0) ;\r
   CoreInitializeDispatcher ();\r
-  PERF_END (NULL,"CoreInitializeDispatcher", "DxeMain", 0) ;\r
 \r
   //\r
   // Invoke the DXE Dispatcher\r
   //\r
-  PERF_START (NULL, "CoreDispatcher", "DxeMain", 0);\r
   CoreDispatcher ();\r
-  PERF_END (NULL, "CoreDispatcher", "DxeMain", 0);\r
 \r
   //\r
   // Display Architectural protocols that were not loaded if this is DEBUG build\r
index 33dd0bd0b7ca807b0831a749ecc8ac42bcc2b0f3..ab3cc0c07e57b7400af148a3bb458e4fd3fab840 100644 (file)
@@ -627,13 +627,13 @@ CoreConnectSingleController (
     for (Index = 0; (Index < NumberOfSortedDriverBindingProtocols) && !DriverFound; Index++) {\r
       if (SortedDriverBindingProtocols[Index] != NULL) {\r
         DriverBinding = SortedDriverBindingProtocols[Index];\r
-        PERF_START (DriverBinding->DriverBindingHandle, "DB:Support:", NULL, 0);\r
+        PERF_DRIVER_BINDING_SUPPORT_BEGIN (DriverBinding->DriverBindingHandle, ControllerHandle);\r
         Status = DriverBinding->Supported(\r
                                   DriverBinding,\r
                                   ControllerHandle,\r
                                   RemainingDevicePath\r
                                   );\r
-        PERF_END (DriverBinding->DriverBindingHandle, "DB:Support:", NULL, 0);\r
+        PERF_DRIVER_BINDING_SUPPORT_END (DriverBinding->DriverBindingHandle, ControllerHandle);\r
         if (!EFI_ERROR (Status)) {\r
           SortedDriverBindingProtocols[Index] = NULL;\r
           DriverFound = TRUE;\r
@@ -642,13 +642,13 @@ CoreConnectSingleController (
           // A driver was found that supports ControllerHandle, so attempt to start the driver\r
           // on ControllerHandle.\r
           //\r
-          PERF_START (DriverBinding->DriverBindingHandle, "DB:Start:", NULL, 0);\r
+          PERF_DRIVER_BINDING_START_BEGIN (DriverBinding->DriverBindingHandle, ControllerHandle);\r
           Status = DriverBinding->Start (\r
                                     DriverBinding,\r
                                     ControllerHandle,\r
                                     RemainingDevicePath\r
                                     );\r
-          PERF_END (DriverBinding->DriverBindingHandle, "DB:Start:", NULL, 0);\r
+          PERF_DRIVER_BINDING_START_END (DriverBinding->DriverBindingHandle, ControllerHandle);\r
 \r
           if (!EFI_ERROR (Status)) {\r
             //\r
index c49ddfcc81d16cae42ff0f315dfe8fc405c8d6bf..adeb4bf313a4009f028edb99a8dfc8cebc37725a 100644 (file)
@@ -1436,13 +1436,9 @@ CoreLoadImage (
   )\r
 {\r
   EFI_STATUS    Status;\r
-  UINT64        Tick;\r
   EFI_HANDLE    Handle;\r
 \r
-  Tick = 0;\r
-  PERF_CODE (\r
-    Tick = GetPerformanceCounter ();\r
-  );\r
+  PERF_LOAD_IMAGE_BEGIN (NULL);\r
 \r
   Status = CoreLoadImageCommon (\r
              BootPolicy,\r
@@ -1465,8 +1461,7 @@ CoreLoadImage (
     Handle = *ImageHandle;\r
   }\r
 \r
-  PERF_START (Handle, "LoadImage:", NULL, Tick);\r
-  PERF_END (Handle, "LoadImage:", NULL, 0);\r
+  PERF_LOAD_IMAGE_END (Handle);\r
 \r
   return Status;\r
 }\r
@@ -1526,13 +1521,9 @@ CoreLoadImageEx (
   )\r
 {\r
   EFI_STATUS    Status;\r
-  UINT64        Tick;\r
   EFI_HANDLE    Handle;\r
 \r
-  Tick = 0;\r
-  PERF_CODE (\r
-    Tick = GetPerformanceCounter ();\r
-  );\r
+  PERF_LOAD_IMAGE_BEGIN (NULL);\r
 \r
   Status = CoreLoadImageCommon (\r
            TRUE,\r
@@ -1555,8 +1546,7 @@ CoreLoadImageEx (
     Handle = *ImageHandle;\r
   }\r
 \r
-  PERF_START (Handle, "LoadImage:", NULL, Tick);\r
-  PERF_END (Handle, "LoadImage:", NULL, 0);\r
+  PERF_LOAD_IMAGE_END (Handle);\r
 \r
   return Status;\r
 }\r
@@ -1594,10 +1584,8 @@ CoreStartImage (
   LOADED_IMAGE_PRIVATE_DATA     *LastImage;\r
   UINT64                        HandleDatabaseKey;\r
   UINTN                         SetJumpFlag;\r
-  UINT64                        Tick;\r
   EFI_HANDLE                    Handle;\r
 \r
-  Tick = 0;\r
   Handle = ImageHandle;\r
 \r
   Image = CoreLoadedImageInfo (ImageHandle);\r
@@ -1621,9 +1609,7 @@ CoreStartImage (
     return EFI_UNSUPPORTED;\r
   }\r
 \r
-  PERF_CODE (\r
-    Tick = GetPerformanceCounter ();\r
-  );\r
+  PERF_START_IMAGE_BEGIN (Handle);\r
 \r
 \r
   //\r
@@ -1647,8 +1633,7 @@ CoreStartImage (
     // Image may be unloaded after return with failure,\r
     // then ImageHandle may be invalid, so use NULL handle to record perf log.\r
     //\r
-    PERF_START (NULL, "StartImage:", NULL, Tick);\r
-    PERF_END (NULL, "StartImage:", NULL, 0);\r
+    PERF_START_IMAGE_END (NULL);\r
 \r
     //\r
     // Pop the current start image context\r
@@ -1763,8 +1748,7 @@ CoreStartImage (
   //\r
   // Done\r
   //\r
-  PERF_START (Handle, "StartImage:", NULL, Tick);\r
-  PERF_END (Handle, "StartImage:", NULL, 0);\r
+  PERF_START_IMAGE_END (Handle);\r
   return Status;\r
 }\r
 \r
index 0ac3fdde6a9a7440b6ccc570028ba23a0f2b078e..c4f629dadffe91d1e05f9c54510b1b6cffaf690e 100644 (file)
@@ -1000,9 +1000,9 @@ PeiDispatcher (
             //\r
             PeimEntryPoint = (EFI_PEIM_ENTRY_POINT2)(UINTN)EntryPoint;\r
 \r
-            PERF_START (PeimFileHandle, "PEIM", NULL, 0);\r
+            PERF_START_IMAGE_BEGIN (PeimFileHandle);\r
             PeimEntryPoint(PeimFileHandle, (const EFI_PEI_SERVICES **) &Private->Ps);\r
-            PERF_END (PeimFileHandle, "PEIM", NULL, 0);\r
+            PERF_START_IMAGE_END (PeimFileHandle);\r
           }\r
 \r
           //\r
@@ -1109,7 +1109,7 @@ PeiDispatcher (
                 // The PEIM has its dependencies satisfied, and its entry point\r
                 // has been found, so invoke it.\r
                 //\r
-                PERF_START (PeimFileHandle, "PEIM", NULL, 0);\r
+                PERF_START_IMAGE_BEGIN (PeimFileHandle);\r
 \r
                 REPORT_STATUS_CODE_WITH_EXTENDED_DATA (\r
                   EFI_PROGRESS_CODE,\r
@@ -1145,7 +1145,7 @@ PeiDispatcher (
                   (VOID *)(&PeimFileHandle),\r
                   sizeof (PeimFileHandle)\r
                   );\r
-                PERF_END (PeimFileHandle, "PEIM", NULL, 0);\r
+                PERF_START_IMAGE_END (PeimFileHandle);\r
 \r
               }\r
             }\r
index 775bf18ce9386e3ab293353241432826999a801e..a52e6f020176811e0a4086db8e0a2f9fc2c44879 100644 (file)
@@ -352,18 +352,17 @@ PeiCore (
   // Update performance measurements \r
   //\r
   if (OldCoreData == NULL) {\r
-    PERF_START (NULL, "SEC", NULL, 1);\r
-    PERF_END   (NULL, "SEC", NULL, 0);\r
+    PERF_EVENT ("SEC"); // Means the end of SEC phase.\r
 \r
     //\r
     // If first pass, start performance measurement.\r
     //\r
-    PERF_START (NULL,"PEI",    NULL, 0);\r
-    PERF_START (NULL,"PreMem", NULL, 0);\r
+    PERF_CROSSMODULE_BEGIN ("PEI");\r
+    PERF_INMODULE_BEGIN ("PreMem");\r
 \r
   } else {\r
-    PERF_END   (NULL,"PreMem",  NULL, 0);\r
-    PERF_START (NULL,"PostMem", NULL, 0);\r
+    PERF_INMODULE_END ("PreMem");\r
+    PERF_INMODULE_BEGIN ("PostMem");\r
   }\r
 \r
   //\r
@@ -411,7 +410,7 @@ PeiCore (
     //\r
     // Alert any listeners that there is permanent memory available\r
     //\r
-    PERF_START (NULL,"DisMem", NULL, 0);\r
+    PERF_INMODULE_BEGIN ("DisMem");\r
     Status = PeiServicesInstallPpi (&mMemoryDiscoveredPpi);\r
 \r
     //\r
@@ -419,7 +418,7 @@ PeiCore (
     //\r
     ProcessNotifyList (&PrivateData);\r
 \r
-    PERF_END (NULL,"DisMem", NULL, 0);\r
+    PERF_INMODULE_END ("DisMem");\r
   }\r
 \r
   //\r
@@ -437,7 +436,7 @@ PeiCore (
   //\r
   // Measure PEI Core execution time.\r
   //\r
-  PERF_END (NULL, "PostMem", NULL, 0);\r
+  PERF_INMODULE_END ("PostMem");\r
 \r
   //\r
   // Lookup DXE IPL PPI\r
index f32bbbd1b4b3da6abaadb2c504ef963039ce330e..0b3b5924d383be4be8c2a134052213d5db0e272c 100644 (file)
@@ -321,13 +321,9 @@ SmmLoadImage (
   EFI_DEVICE_PATH_PROTOCOL       *HandleFilePath;\r
   EFI_FIRMWARE_VOLUME2_PROTOCOL  *Fv;\r
   PE_COFF_LOADER_IMAGE_CONTEXT   ImageContext;\r
-  UINT64                         Tick;\r
 \r
-  Tick = 0;\r
-  PERF_CODE (\r
-    Tick = GetPerformanceCounter ();\r
-  );\r
-   \r
+  PERF_LOAD_IMAGE_BEGIN (DriverEntry->ImageHandle);\r
+\r
   Buffer               = NULL;\r
   Size                 = 0;\r
   Fv                   = DriverEntry->Fv;\r
@@ -641,8 +637,7 @@ SmmLoadImage (
              &DriverEntry->SmmLoadedImage\r
              );\r
 \r
-  PERF_START (DriverEntry->ImageHandle, "LoadImage:", NULL, Tick);\r
-  PERF_END (DriverEntry->ImageHandle, "LoadImage:", NULL, 0);\r
+  PERF_LOAD_IMAGE_END (DriverEntry->ImageHandle);\r
 \r
   //\r
   // Print the load address and the PDB file name if it is available\r
@@ -909,9 +904,9 @@ SmmDispatcher (
       // For each SMM driver, pass NULL as ImageHandle\r
       //\r
       RegisterSmramProfileImage (DriverEntry, TRUE);\r
-      PERF_START (DriverEntry->ImageHandle, "StartImage:", NULL, 0);\r
+      PERF_START_IMAGE_BEGIN (DriverEntry->ImageHandle);\r
       Status = ((EFI_IMAGE_ENTRY_POINT)(UINTN)DriverEntry->ImageEntryPoint)(DriverEntry->ImageHandle, gST);\r
-      PERF_END (DriverEntry->ImageHandle, "StartImage:", NULL, 0);\r
+      PERF_START_IMAGE_END (DriverEntry->ImageHandle);\r
       if (EFI_ERROR(Status)){\r
         UnregisterSmramProfileImage (DriverEntry, TRUE);\r
         SmmFreePages(DriverEntry->ImageBuffer, DriverEntry->NumberOfPage);\r
index 686b9b45a5564f710554e2eb4b1b3823a6931cd9..45f40e0e60c7839ee731114a7b430d16a631094e 100644 (file)
@@ -644,7 +644,7 @@ SmmEntryPoint (
   VOID                        *CommunicationBuffer;\r
   UINTN                       BufferSize;\r
 \r
-  PERF_START (NULL, "SMM", NULL, 0) ;\r
+  PERF_INMODULE_BEGIN ("SMM");\r
 \r
   //\r
   // Update SMST with contents of the SmmEntryContext structure\r
@@ -739,7 +739,7 @@ SmmEntryPoint (
     gSmmCorePrivate->InSmm = FALSE;\r
   }\r
 \r
-  PERF_END (NULL, "SMM", NULL, 0) ;\r
+  PERF_INMODULE_END ("SMM");\r
 }\r
 \r
 /**\r
index b842d5824aed669551154fa34c20146064b84629..59bd1e4c6c00ca3c59fbade40c075c86d636345a 100644 (file)
@@ -45,7 +45,7 @@ BmEndOfBdsPerfCode (
   //\r
   // Record the performance data for End of BDS\r
   //\r
-  PERF_END(NULL, "BDS", NULL, 0);\r
+  PERF_CROSSMODULE_END("BDS");\r
 \r
   return ;\r
 }\r
index 80511814eec5b0a3bdac13895c3bbfafcdb94956..d85b603d64a09acf04311a3fcb3805c422a6181c 100644 (file)
@@ -722,23 +722,20 @@ EfiBootManagerConnectAllDefaultConsoles (
   if (!EFI_ERROR (Status)) {\r
     OneConnected = TRUE;\r
   }\r
-  PERF_START (NULL, "ConOutReady", "BDS", 1);\r
-  PERF_END   (NULL, "ConOutReady", "BDS", 0);\r
+  PERF_EVENT ("ConOutReady");\r
 \r
   \r
   Status = EfiBootManagerConnectConsoleVariable (ConIn);\r
   if (!EFI_ERROR (Status)) {\r
     OneConnected = TRUE;\r
   }\r
-  PERF_START (NULL, "ConInReady", "BDS", 1);\r
-  PERF_END   (NULL, "ConInReady", "BDS", 0);\r
+  PERF_EVENT ("ConInReady");\r
 \r
   Status = EfiBootManagerConnectConsoleVariable (ErrOut);\r
   if (!EFI_ERROR (Status)) {\r
     OneConnected = TRUE;\r
   }\r
-  PERF_START (NULL, "ErrOutReady", "BDS", 1);\r
-  PERF_END   (NULL, "ErrOutReady", "BDS", 0);\r
+  PERF_EVENT ("ErrOutReady");\r
 \r
   SystemTableUpdated = FALSE;\r
   //\r
index 3191a986304b2aa2752ecff696b8736c8943fd3b..acb25abfcd486aa3c4f6fbc8ea52dddd7ac7e608 100644 (file)
@@ -725,8 +725,8 @@ BdsEntry (
   //\r
   // Insert the performance probe\r
   //\r
-  PERF_END (NULL, "DXE", NULL, 0);\r
-  PERF_START (NULL, "BDS", NULL, 0);\r
+  PERF_CROSSMODULE_END("DXE");\r
+  PERF_CROSSMODULE_BEGIN("BDS");\r
   DEBUG ((EFI_D_INFO, "[Bds] Entry...\n"));\r
 \r
   //\r
@@ -888,9 +888,9 @@ BdsEntry (
   // > Signal ReadyToLock event\r
   // > Authentication action: 1. connect Auth devices; 2. Identify auto logon user.\r
   //\r
-  PERF_START (NULL, "PlatformBootManagerBeforeConsole", "BDS", 0);\r
+  PERF_INMODULE_BEGIN("PlatformBootManagerBeforeConsole");\r
   PlatformBootManagerBeforeConsole ();\r
-  PERF_END   (NULL, "PlatformBootManagerBeforeConsole", "BDS", 0);\r
+  PERF_INMODULE_END("PlatformBootManagerBeforeConsole");\r
 \r
   //\r
   // Initialize hotkey service\r
@@ -907,7 +907,7 @@ BdsEntry (
   //\r
   // Connect consoles\r
   //\r
-  PERF_START (NULL, "EfiBootManagerConnectAllDefaultConsoles", "BDS", 0);\r
+  PERF_INMODULE_BEGIN("EfiBootManagerConnectAllDefaultConsoles");\r
   if (PcdGetBool (PcdConInConnectOnDemand)) {\r
     EfiBootManagerConnectConsoleVariable (ConOut);\r
     EfiBootManagerConnectConsoleVariable (ErrOut);\r
@@ -917,7 +917,7 @@ BdsEntry (
   } else {\r
     EfiBootManagerConnectAllDefaultConsoles ();\r
   }\r
-  PERF_END   (NULL, "EfiBootManagerConnectAllDefaultConsoles", "BDS", 0);\r
+  PERF_INMODULE_END("EfiBootManagerConnectAllDefaultConsoles");\r
 \r
   //\r
   // Do the platform specific action after the console is ready\r
@@ -930,9 +930,9 @@ BdsEntry (
   // > Dispatch aditional option roms\r
   // > Special boot: e.g.: USB boot, enter UI\r
   // \r
-  PERF_START (NULL, "PlatformBootManagerAfterConsole", "BDS", 0);\r
+  PERF_INMODULE_BEGIN("PlatformBootManagerAfterConsole");\r
   PlatformBootManagerAfterConsole ();\r
-  PERF_END   (NULL, "PlatformBootManagerAfterConsole", "BDS", 0);\r
+  PERF_INMODULE_END("PlatformBootManagerAfterConsole");\r
   //\r
   // Boot to Boot Manager Menu when EFI_OS_INDICATIONS_BOOT_TO_FW_UI is set. Skip HotkeyBoot\r
   //\r
@@ -1025,10 +1025,9 @@ BdsEntry (
     //\r
     // Execute Key####\r
     //\r
-    PERF_START (NULL, "BdsWait", "BDS", 0);\r
+    PERF_INMODULE_BEGIN ("BdsWait");\r
     BdsWait (HotkeyTriggered);\r
-    PERF_END   (NULL, "BdsWait", "BDS", 0);\r
-\r
+    PERF_INMODULE_END ("BdsWait");\r
     //\r
     // BdsReadKeys() can be removed after all keyboard drivers invoke callback in timer callback.\r
     //\r