]> git.proxmox.com Git - mirror_edk2.git/blobdiff - MdeModulePkg/Universal/Acpi/BootScriptExecutorDxe/ScriptExecute.c
MdeModulePkg DxeCore/PiSmmCore: Add UEFI memory and SMRAM profile support.
[mirror_edk2.git] / MdeModulePkg / Universal / Acpi / BootScriptExecutorDxe / ScriptExecute.c
index d4d2a93492dda4b53c4b03a4e7bef49e8daa63df..3fbdf5cd838e8e734d78521103fab2377621c02a 100644 (file)
@@ -1,10 +1,10 @@
 /** @file\r
   This is the code for Boot Script Executer module.\r
 \r
-  This driver is dispatched by Dxe core and the driver will reload itself to ACPI NVS memory\r
+  This driver is dispatched by Dxe core and the driver will reload itself to ACPI reserved memory\r
   in the entry point. The functionality is to interpret and restore the S3 boot script\r
 \r
-Copyright (c) 2006 - 2011, Intel Corporation. All rights reserved.<BR>\r
+Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>\r
 \r
 This program and the accompanying materials\r
 are licensed and made available under the terms and conditions of the BSD License\r
@@ -18,86 +18,11 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
 \r
 #include "ScriptExecute.h"\r
 \r
-EFI_PHYSICAL_ADDRESS  mPerfDataMemAddress;\r
-UINT64                mS3BootScriptEntryTick;\r
-UINT64                mScriptStartTick;\r
-UINT64                mScriptEndTick;\r
-\r
 EFI_GUID              mBootScriptExecutorImageGuid = {\r
-  0x9a8d3433, 0x9fe8, 0x42b6, 0x87, 0xb, 0x1e, 0x31, 0xc8, 0x4e, 0xbe, 0x3b\r
+  0x9a8d3433, 0x9fe8, 0x42b6, { 0x87, 0xb, 0x1e, 0x31, 0xc8, 0x4e, 0xbe, 0x3b }\r
 };\r
 \r
-/**\r
-  The event callback is used to get the base address of boot performance data structure on\r
-  LegacyBoot event and ExitBootServices event.\r
-\r
-  @param  Event    The event handle.\r
-  @param  Context  The event context.\r
-\r
-**/\r
-VOID\r
-EFIAPI\r
-OnBootEvent (\r
-  IN EFI_EVENT        Event,\r
-  IN VOID             *Context\r
-  )\r
-{\r
-  EFI_STATUS Status;\r
-  UINTN      VarSize;\r
-\r
-  VarSize = sizeof (EFI_PHYSICAL_ADDRESS);\r
-  Status = gRT->GetVariable (\r
-                  L"PerfDataMemAddr",\r
-                  &gPerformanceProtocolGuid,\r
-                  NULL,\r
-                  &VarSize,\r
-                  &mPerfDataMemAddress\r
-                  );\r
-  if (EFI_ERROR (Status)) {\r
-    mPerfDataMemAddress = 0;\r
-  }\r
-}\r
-\r
-/**\r
-  Record S3 Script execution time and adjust total S3 resume time for script running.\r
-**/\r
-VOID\r
-WriteToOsS3PerformanceData (\r
-  VOID\r
-  )\r
-{\r
-  UINT64               Ticker;\r
-  UINT64               StartValue;\r
-  UINT64               EndValue;\r
-  UINT64               Freq;\r
-  UINT64               ScriptExecuteTicks;\r
-  PERF_HEADER          *PerfHeader;\r
-  PERF_DATA            *PerfData;\r
-\r
-  Ticker = GetPerformanceCounter ();\r
-\r
-  PerfHeader = (PERF_HEADER *)(UINTN)mPerfDataMemAddress;\r
-  if (PerfHeader == NULL) {\r
-    return;\r
-  }\r
-\r
-  Freq   = GetPerformanceCounterProperties (&StartValue, &EndValue);\r
-  Freq   = DivU64x32 (Freq, 1000);\r
-\r
-  if (EndValue >= StartValue) {\r
-    ScriptExecuteTicks = mScriptEndTick - mScriptStartTick;\r
-    PerfHeader->S3Resume += Ticker - mS3BootScriptEntryTick;\r
-  } else {\r
-    ScriptExecuteTicks = mScriptStartTick - mScriptEndTick;\r
-    PerfHeader->S3Resume += mS3BootScriptEntryTick - Ticker;\r
-  }\r
-  if (PerfHeader->S3EntryNum < PERF_PEI_ENTRY_MAX_NUM) {\r
-    PerfData = &PerfHeader->S3Entry[PerfHeader->S3EntryNum];\r
-    PerfData->Duration = (UINT32) DivU64x32 (ScriptExecuteTicks, (UINT32) Freq);;\r
-    AsciiStrnCpy (PerfData->Token, "ScriptExec", PERF_TOKEN_LENGTH);\r
-    PerfHeader->S3EntryNum++;\r
-  }\r
-}\r
+BOOLEAN               mPage1GSupport = FALSE;\r
 \r
 /**\r
   Entry function of Boot script exector. This function will be executed in\r
@@ -122,38 +47,37 @@ S3BootScriptExecutorEntryFunction (
   UINTN                                         TempStackTop;\r
   UINTN                                         TempStack[0x10];\r
   UINTN                                         AsmTransferControl16Address;\r
-\r
-  PERF_CODE (\r
-    mS3BootScriptEntryTick = GetPerformanceCounter ();\r
-    );\r
+  IA32_DESCRIPTOR                               IdtDescriptor;\r
 \r
   //\r
   // Disable interrupt of Debug timer, since new IDT table cannot handle it.\r
   //\r
   SaveAndSetDebugTimerInterrupt (FALSE);\r
 \r
+  AsmReadIdtr (&IdtDescriptor);\r
   //\r
   // Restore IDT for debug\r
   //\r
   SetIdtEntry (AcpiS3Context);\r
 \r
   //\r
-  // Initialize Debug Agent to support source level debug in S3 path.\r
+  // Initialize Debug Agent to support source level debug in S3 path, it will disable interrupt and Debug Timer.\r
   //\r
-  InitializeDebugAgent (DEBUG_AGENT_INIT_S3, NULL, NULL);\r
+  InitializeDebugAgent (DEBUG_AGENT_INIT_S3, (VOID *)&IdtDescriptor, NULL);\r
 \r
   //\r
   // Because not install BootScriptExecute PPI(used just in this module), So just pass NULL\r
   // for that parameter.\r
   //\r
-  PERF_CODE (\r
-    mScriptStartTick = GetPerformanceCounter ();\r
-    );\r
   Status = S3BootScriptExecute ();\r
-  PERF_CODE (\r
-    mScriptEndTick = GetPerformanceCounter ();\r
-    );\r
+  \r
+  //\r
+  // If invalid script table or opcode in S3 boot script table.\r
+  //\r
+  ASSERT_EFI_ERROR (Status);\r
+  \r
   if (EFI_ERROR (Status)) {\r
+    CpuDeadLoop ();\r
     return Status;\r
   }\r
 \r
@@ -164,17 +88,15 @@ S3BootScriptExecutorEntryFunction (
   //\r
   Facs = (EFI_ACPI_4_0_FIRMWARE_ACPI_CONTROL_STRUCTURE *) ((UINTN) (AcpiS3Context->AcpiFacsTable));\r
 \r
-  if ((Facs == NULL) ||\r
-      (Facs->Signature != EFI_ACPI_4_0_FIRMWARE_ACPI_CONTROL_STRUCTURE_SIGNATURE) ||\r
-      ((Facs->FirmwareWakingVector == 0) && (Facs->XFirmwareWakingVector == 0)) ) {\r
-    CpuDeadLoop();\r
-    return EFI_INVALID_PARAMETER;\r
-  }\r
-\r
   //\r
   // We need turn back to S3Resume - install boot script done ppi and report status code on S3resume.\r
   //\r
   if (PeiS3ResumeState != 0) {\r
+    //\r
+    // Need report status back to S3ResumePeim. \r
+    // If boot script execution is failed, S3ResumePeim wil report the error status code.\r
+    //\r
+    PeiS3ResumeState->ReturnStatus = (UINT64)(UINTN)Status;\r
     if (FeaturePcdGet (PcdDxeIplSwitchToLongMode)) {\r
       //\r
       // X64 S3 Resume\r
@@ -182,12 +104,16 @@ S3BootScriptExecutorEntryFunction (
       DEBUG ((EFI_D_ERROR, "Call AsmDisablePaging64() to return to S3 Resume in PEI Phase\n"));\r
       PeiS3ResumeState->AsmTransferControl = (EFI_PHYSICAL_ADDRESS)(UINTN)AsmTransferControl32;\r
 \r
-      //\r
-      // more step needed - because relative address is handled differently between X64 and IA32.\r
-      //\r
-      AsmTransferControl16Address = (UINTN)AsmTransferControl16;\r
-      AsmFixAddress16 = (UINT32)AsmTransferControl16Address;\r
-      AsmJmpAddr32 = (UINT32)((Facs->FirmwareWakingVector & 0xF) | ((Facs->FirmwareWakingVector & 0xFFFF0) << 12));\r
+      if ((Facs != NULL) &&\r
+          (Facs->Signature == EFI_ACPI_4_0_FIRMWARE_ACPI_CONTROL_STRUCTURE_SIGNATURE) &&\r
+          (Facs->FirmwareWakingVector != 0) ) {\r
+        //\r
+        // more step needed - because relative address is handled differently between X64 and IA32.\r
+        //\r
+        AsmTransferControl16Address = (UINTN)AsmTransferControl16;\r
+        AsmFixAddress16 = (UINT32)AsmTransferControl16Address;\r
+        AsmJmpAddr32 = (UINT32)((Facs->FirmwareWakingVector & 0xF) | ((Facs->FirmwareWakingVector & 0xFFFF0) << 12));\r
+      }\r
 \r
       AsmDisablePaging64 (\r
         PeiS3ResumeState->ReturnCs,\r
@@ -217,11 +143,10 @@ S3BootScriptExecutorEntryFunction (
     CpuDeadLoop();\r
     return EFI_UNSUPPORTED;\r
   }\r
-\r
-  PERF_CODE (\r
-    WriteToOsS3PerformanceData ();\r
-    );\r
-\r
+  \r
+  //\r
+  // S3ResumePeim does not provide a way to jump back to itself, so resume to OS here directly\r
+  //\r
   if (Facs->XFirmwareWakingVector != 0) {\r
     //\r
     // Switch to native waking vector\r
@@ -282,6 +207,170 @@ S3BootScriptExecutorEntryFunction (
   CpuDeadLoop();\r
   return EFI_UNSUPPORTED;\r
 }\r
+\r
+/**\r
+  Register image to memory profile.\r
+\r
+  @param FileName       File name of the image.\r
+  @param ImageBase      Image base address.\r
+  @param ImageSize      Image size.\r
+  @param FileType       File type of the image.\r
+\r
+**/\r
+VOID\r
+RegisterMemoryProfileImage (\r
+  IN EFI_GUID                       *FileName,\r
+  IN PHYSICAL_ADDRESS               ImageBase,\r
+  IN UINT64                         ImageSize,\r
+  IN EFI_FV_FILETYPE                FileType\r
+  )\r
+{\r
+  EFI_STATUS                        Status;\r
+  EDKII_MEMORY_PROFILE_PROTOCOL     *ProfileProtocol;\r
+  MEDIA_FW_VOL_FILEPATH_DEVICE_PATH *FilePath;\r
+  UINT8                             TempBuffer[sizeof (MEDIA_FW_VOL_FILEPATH_DEVICE_PATH) + sizeof (EFI_DEVICE_PATH_PROTOCOL)];\r
+\r
+  if ((PcdGet8 (PcdMemoryProfilePropertyMask) & BIT0) != 0) {\r
+\r
+    FilePath = (MEDIA_FW_VOL_FILEPATH_DEVICE_PATH *)TempBuffer;\r
+    Status = gBS->LocateProtocol (&gEdkiiMemoryProfileGuid, NULL, (VOID **) &ProfileProtocol);\r
+    if (!EFI_ERROR (Status)) {\r
+      EfiInitializeFwVolDevicepathNode (FilePath, FileName);\r
+      SetDevicePathEndNode (FilePath + 1);\r
+\r
+      Status = ProfileProtocol->RegisterImage (\r
+                                  ProfileProtocol,\r
+                                  (EFI_DEVICE_PATH_PROTOCOL *) FilePath,\r
+                                  ImageBase,\r
+                                  ImageSize,\r
+                                  FileType\r
+                                  );\r
+    }\r
+  }\r
+}\r
+\r
+/**\r
+  This is the Event notification function to reload BootScriptExecutor image\r
+  to RESERVED mem and save it to LockBox.\r
+  \r
+  @param    Event   Pointer to this event\r
+  @param    Context Event handler private data \r
+ **/\r
+VOID\r
+EFIAPI\r
+ReadyToLockEventNotify (\r
+  IN EFI_EVENT  Event,\r
+  IN VOID       *Context\r
+  )\r
+{\r
+  EFI_STATUS                                    Status;\r
+  VOID                                          *Interface;\r
+  UINT8                                         *Buffer;\r
+  UINTN                                         BufferSize;\r
+  EFI_HANDLE                                    NewImageHandle;\r
+  UINTN                                         Pages;\r
+  EFI_PHYSICAL_ADDRESS                          FfsBuffer;\r
+  PE_COFF_LOADER_IMAGE_CONTEXT                  ImageContext;\r
+\r
+  Status = gBS->LocateProtocol (&gEfiDxeSmmReadyToLockProtocolGuid, NULL, &Interface);\r
+  if (EFI_ERROR (Status)) {\r
+    return;\r
+  }\r
+\r
+  //\r
+  // A workaround: Here we install a dummy handle\r
+  //\r
+  NewImageHandle = NULL;\r
+  Status = gBS->InstallProtocolInterface (\r
+                  &NewImageHandle,\r
+                  &gEfiCallerIdGuid,\r
+                  EFI_NATIVE_INTERFACE,\r
+                  NULL\r
+                  );\r
+  ASSERT_EFI_ERROR (Status);\r
+\r
+  //\r
+  // Reload BootScriptExecutor image itself to RESERVED mem\r
+  //\r
+  Status = GetSectionFromAnyFv  (\r
+             &gEfiCallerIdGuid,\r
+             EFI_SECTION_PE32,\r
+             0,\r
+             (VOID **) &Buffer,\r
+             &BufferSize\r
+             );\r
+  ASSERT_EFI_ERROR (Status);\r
+  ImageContext.Handle    = Buffer;\r
+  ImageContext.ImageRead = PeCoffLoaderImageReadFromMemory;\r
+  //\r
+  // Get information about the image being loaded\r
+  //\r
+  Status = PeCoffLoaderGetImageInfo (&ImageContext);\r
+  ASSERT_EFI_ERROR (Status);\r
+  Pages = EFI_SIZE_TO_PAGES(BufferSize + ImageContext.SectionAlignment);\r
+  FfsBuffer = 0xFFFFFFFF;\r
+  Status = gBS->AllocatePages (\r
+                  AllocateMaxAddress,\r
+                  EfiReservedMemoryType,\r
+                  Pages,\r
+                  &FfsBuffer\r
+                  );\r
+  ASSERT_EFI_ERROR (Status);\r
+  ImageContext.ImageAddress = (PHYSICAL_ADDRESS)(UINTN)FfsBuffer;\r
+  //\r
+  // Align buffer on section boundry\r
+  //\r
+  ImageContext.ImageAddress += ImageContext.SectionAlignment - 1;\r
+  ImageContext.ImageAddress &= ~((EFI_PHYSICAL_ADDRESS)(ImageContext.SectionAlignment - 1));\r
+  //\r
+  // Load the image to our new buffer\r
+  //\r
+  Status = PeCoffLoaderLoadImage (&ImageContext);\r
+  ASSERT_EFI_ERROR (Status);\r
+\r
+  //\r
+  // Relocate the image in our new buffer\r
+  //\r
+  Status = PeCoffLoaderRelocateImage (&ImageContext);\r
+  ASSERT_EFI_ERROR (Status);\r
+\r
+  //\r
+  // Free the buffer allocated by ReadSection since the image has been relocated in the new buffer\r
+  //\r
+  gBS->FreePool (Buffer);\r
+\r
+  //\r
+  // Flush the instruction cache so the image data is written before we execute it\r
+  //\r
+  InvalidateInstructionCacheRange ((VOID *)(UINTN)ImageContext.ImageAddress, (UINTN)ImageContext.ImageSize);\r
+\r
+  RegisterMemoryProfileImage (\r
+    &gEfiCallerIdGuid,\r
+    ImageContext.ImageAddress,\r
+    ImageContext.ImageSize,\r
+    EFI_FV_FILETYPE_DRIVER\r
+    );\r
+\r
+  Status = ((EFI_IMAGE_ENTRY_POINT)(UINTN)(ImageContext.EntryPoint)) (NewImageHandle, gST);\r
+  ASSERT_EFI_ERROR (Status);\r
+\r
+  //\r
+  // Additional step for BootScript integrity\r
+  // Save BootScriptExecutor image\r
+  //\r
+  Status = SaveLockBox (\r
+             &mBootScriptExecutorImageGuid,\r
+             (VOID *)(UINTN)ImageContext.ImageAddress,\r
+             (UINTN)ImageContext.ImageSize\r
+             );\r
+  ASSERT_EFI_ERROR (Status);\r
+\r
+  Status = SetLockBoxAttributes (&mBootScriptExecutorImageGuid, LOCK_BOX_ATTRIBUTE_RESTORE_IN_PLACE);\r
+  ASSERT_EFI_ERROR (Status);\r
+\r
+  gBS->CloseEvent (Event);\r
+}\r
+\r
 /**\r
   Entrypoint of Boot script exector driver, this function will be executed in\r
   normal boot phase and invoked by DXE dispatch.\r
@@ -299,16 +388,16 @@ BootScriptExecutorEntryPoint (
   IN EFI_SYSTEM_TABLE     *SystemTable\r
   )\r
 {\r
-  UINT8                                         *Buffer;\r
   UINTN                                         BufferSize;\r
   UINTN                                         Pages;\r
-  EFI_PHYSICAL_ADDRESS                          FfsBuffer;\r
-  PE_COFF_LOADER_IMAGE_CONTEXT                  ImageContext;\r
   BOOT_SCRIPT_EXECUTOR_VARIABLE                 *EfiBootScriptExecutorVariable;\r
   EFI_PHYSICAL_ADDRESS                          BootScriptExecutorBuffer;\r
   EFI_STATUS                                    Status;\r
   VOID                                          *DevicePath;\r
-  EFI_HANDLE                                    NewImageHandle;\r
+  EFI_EVENT                                     ReadyToLockEvent;\r
+  VOID                                          *Registration;\r
+  UINT32                                        RegEax;\r
+  UINT32                                        RegEdx;\r
 \r
   //\r
   // Test if the gEfiCallerIdGuid of this image is already installed. if not, the entry\r
@@ -317,113 +406,43 @@ BootScriptExecutorEntryPoint (
   //\r
   Status = gBS->LocateProtocol (&gEfiCallerIdGuid, NULL, &DevicePath);\r
   if (EFI_ERROR (Status)) {\r
-\r
-      //\r
-      // This is the first-time loaded by DXE core. reload itself to NVS mem\r
-      //\r
-      //\r
-      // A workarouond: Here we install a dummy handle\r
-      //\r
-      NewImageHandle = NULL;\r
-      Status = gBS->InstallProtocolInterface (\r
-                  &NewImageHandle,\r
-                  &gEfiCallerIdGuid,\r
-                  EFI_NATIVE_INTERFACE,\r
-                  NULL\r
-                  );\r
-\r
-      Status = GetSectionFromAnyFv  (\r
-                 &gEfiCallerIdGuid,\r
-                 EFI_SECTION_PE32,\r
-                 0,\r
-                 (VOID **) &Buffer,\r
-                 &BufferSize\r
-                 );\r
-      ImageContext.Handle    = Buffer;\r
-      ImageContext.ImageRead = PeCoffLoaderImageReadFromMemory;\r
-      //\r
-      // Get information about the image being loaded\r
-      //\r
-      Status = PeCoffLoaderGetImageInfo (&ImageContext);\r
-      if (EFI_ERROR (Status)) {\r
-        return Status;\r
-      }\r
-      Pages = EFI_SIZE_TO_PAGES(BufferSize + ImageContext.SectionAlignment);\r
-      FfsBuffer = 0xFFFFFFFF;\r
-      Status = gBS->AllocatePages (\r
-                    AllocateMaxAddress,\r
-                    EfiACPIMemoryNVS,\r
-                    Pages,\r
-                    &FfsBuffer\r
-                    );\r
-      if (EFI_ERROR (Status)) {\r
-        return EFI_OUT_OF_RESOURCES;\r
-      }\r
-      ImageContext.ImageAddress = (PHYSICAL_ADDRESS)(UINTN)FfsBuffer;\r
-      //\r
-      // Align buffer on section boundry\r
-      //\r
-      ImageContext.ImageAddress += ImageContext.SectionAlignment - 1;\r
-      ImageContext.ImageAddress &= ~(ImageContext.SectionAlignment - 1);\r
-      //\r
-      // Load the image to our new buffer\r
       //\r
-      Status = PeCoffLoaderLoadImage (&ImageContext);\r
-      if (EFI_ERROR (Status)) {\r
-        gBS->FreePages (FfsBuffer, Pages);\r
-        return Status;\r
-      }\r
-\r
-      //\r
-      // Relocate the image in our new buffer\r
+      // Create ReadyToLock event to reload BootScriptExecutor image\r
+      // to RESERVED mem and save it to LockBox.\r
       //\r
-      Status = PeCoffLoaderRelocateImage (&ImageContext);\r
-\r
-      if (EFI_ERROR (Status)) {\r
-        PeCoffLoaderUnloadImage (&ImageContext);\r
-        gBS->FreePages (FfsBuffer, Pages);\r
-        return Status;\r
-      }\r
+      ReadyToLockEvent = EfiCreateProtocolNotifyEvent  (\r
+                           &gEfiDxeSmmReadyToLockProtocolGuid,\r
+                           TPL_NOTIFY,\r
+                           ReadyToLockEventNotify,\r
+                           NULL,\r
+                           &Registration\r
+                           );\r
+      ASSERT (ReadyToLockEvent != NULL);\r
+    } else {\r
       //\r
-      // Flush the instruction cache so the image data is written before we execute it\r
+      // the entry point is invoked after reloading. following code only run in RESERVED mem\r
       //\r
-      InvalidateInstructionCacheRange ((VOID *)(UINTN)ImageContext.ImageAddress, (UINTN)ImageContext.ImageSize);\r
-      Status = ((EFI_IMAGE_ENTRY_POINT)(UINTN)(ImageContext.EntryPoint)) (NewImageHandle, SystemTable);\r
-      if (EFI_ERROR (Status)) {\r
-        gBS->FreePages (FfsBuffer, Pages);\r
-        return Status;\r
+      if (PcdGetBool(PcdUse1GPageTable)) {\r
+        AsmCpuid (0x80000000, &RegEax, NULL, NULL, NULL);\r
+        if (RegEax >= 0x80000001) {\r
+          AsmCpuid (0x80000001, NULL, NULL, NULL, &RegEdx);\r
+          if ((RegEdx & BIT26) != 0) {\r
+            mPage1GSupport = TRUE;\r
+          }\r
+        }\r
       }\r
-      //\r
-      // Additional step for BootScript integrity\r
-      // Save BootScriptExecutor image\r
-      //\r
-      Status = SaveLockBox (\r
-                 &mBootScriptExecutorImageGuid,\r
-                 (VOID *)(UINTN)ImageContext.ImageAddress,\r
-                 (UINTN)ImageContext.ImageSize\r
-                 );\r
-      ASSERT_EFI_ERROR (Status);\r
 \r
-      Status = SetLockBoxAttributes (&mBootScriptExecutorImageGuid, LOCK_BOX_ATTRIBUTE_RESTORE_IN_PLACE);\r
-      ASSERT_EFI_ERROR (Status);\r
-\r
-    } else {\r
-      //\r
-      // the entry point is invoked after reloading. following code only run in  ACPI NVS\r
-      //\r
       BufferSize = sizeof (BOOT_SCRIPT_EXECUTOR_VARIABLE);\r
 \r
       BootScriptExecutorBuffer = 0xFFFFFFFF;\r
       Pages = EFI_SIZE_TO_PAGES(BufferSize);\r
       Status = gBS->AllocatePages (\r
                       AllocateMaxAddress,\r
-                      EfiACPIMemoryNVS,\r
+                      EfiReservedMemoryType,\r
                       Pages,\r
                       &BootScriptExecutorBuffer\r
                       );\r
-      if (EFI_ERROR (Status)) {\r
-        return EFI_OUT_OF_RESOURCES;\r
-      }\r
+      ASSERT_EFI_ERROR (Status);\r
 \r
       EfiBootScriptExecutorVariable = (BOOT_SCRIPT_EXECUTOR_VARIABLE *)(UINTN)BootScriptExecutorBuffer;\r
       EfiBootScriptExecutorVariable->BootScriptExecutorEntrypoint = (UINTN) S3BootScriptExecutorEntryFunction ;\r
@@ -448,30 +467,8 @@ BootScriptExecutorEntryPoint (
 \r
       Status = SetLockBoxAttributes (&gEfiBootScriptExecutorContextGuid, LOCK_BOX_ATTRIBUTE_RESTORE_IN_PLACE);\r
       ASSERT_EFI_ERROR (Status);\r
-\r
-      PERF_CODE (\r
-        EFI_EVENT Event;\r
-\r
-        gBS->CreateEventEx (\r
-              EVT_NOTIFY_SIGNAL,\r
-              TPL_NOTIFY,\r
-              OnBootEvent,\r
-              NULL,\r
-              &gEfiEventExitBootServicesGuid,\r
-              &Event\r
-              );\r
-\r
-        EfiCreateEventLegacyBootEx(\r
-          TPL_NOTIFY,\r
-          OnBootEvent,\r
-          NULL,\r
-          &Event\r
-          );\r
-        );\r
     }\r
 \r
     return EFI_SUCCESS;\r
 }\r
 \r
-\r
-\r