MdeModulePkg/FPDT: Lock boot performance table address variable at EndOfDxeedk2-stable202111-rc1

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2957

1. Allocate performance data table at EndOfDxe and then lock the varible
   which store the table address at EndOfDxe.

2. Enlarge PCD gEfiMdeModulePkgTokenSpaceGuid.PcdExtFpdtBootRecordPadSize
   from 0x20000 to 0x30000 in order to hold the Delta performance data
   between EndOfDxe and ReadyToBoot.

3. SMM performance data is collected by DXE modules through SMM communication
   at ReadyToBoot before.
   Now to do SMM communication twice, one for allocating the performance
   size at EndOfDxe, another is at ReadyToBoot to get SMM performance data.

4. Make SmmCorePerformanceLib rather than FirmwarePerformanceSmm to communicate
   with DxeCorePerformanceLib for SMM performance data and size.

Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Jian J Wang <jian.j.wang@intel.com>
Signed-off-by: Dandan Bi <dandan.bi@intel.com>
Reviewed-by: Hao A Wu <hao.a.wu@intel.com>

8 files changed, 88 insertions, 225 deletions

diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/FirmwarePerformanceDxe.c b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/FirmwarePerformanceDxe.c
index 61a7704..6875555 100644
--- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/FirmwarePerformanceDxe.c
+++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/FirmwarePerformanceDxe.c
@@ -5,7 +5,7 @@
   for Firmware Basic Boot Performance Record and other boot performance records,

   and install FPDT to ACPI table.

 

-  Copyright (c) 2011 - 2019, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2011 - 2021, Intel Corporation. All rights reserved.<BR>

   SPDX-License-Identifier: BSD-2-Clause-Patent

 

 **/

@@ -16,6 +16,7 @@
 #include <Protocol/AcpiTable.h>

 #include <Protocol/LockBox.h>

 #include <Protocol/Variable.h>

+#include <Protocol/VariablePolicy.h>

 

 #include <Guid/Acpi.h>

 #include <Guid/FirmwarePerformance.h>

@@ -32,6 +33,8 @@
 #include <Library/HobLib.h>

 #include <Library/LockBoxLib.h>

 #include <Library/UefiLib.h>

+#include <Library/VariablePolicyHelperLib.h>

+#include <Library/PerformanceLib.h>

 

 #define SMM_BOOT_RECORD_COMM_SIZE (OFFSET_OF (EFI_SMM_COMMUNICATE_HEADER, Data) + sizeof(SMM_BOOT_RECORD_COMMUNICATE))

 

@@ -278,11 +281,12 @@ InstallFirmwarePerformanceDataTable (
   VOID

   )

 {

-  EFI_STATUS                    Status;

-  EFI_ACPI_TABLE_PROTOCOL       *AcpiTableProtocol;

-  UINTN                         BootPerformanceDataSize;

-  FIRMWARE_PERFORMANCE_VARIABLE PerformanceVariable;

-  UINTN                         Size;

+  EFI_STATUS                      Status;

+  EFI_ACPI_TABLE_PROTOCOL         *AcpiTableProtocol;

+  UINTN                           BootPerformanceDataSize;

+  FIRMWARE_PERFORMANCE_VARIABLE   PerformanceVariable;

+  UINTN                           Size;

+  EDKII_VARIABLE_POLICY_PROTOCOL  *VariablePolicyProtocol;

 

   //

   // Get AcpiTable Protocol.

@@ -292,6 +296,14 @@ InstallFirmwarePerformanceDataTable (
     return Status;

   }

 

+  //

+  // Get VariablePolicy Protocol.

+  //

+  Status = gBS->LocateProtocol(&gEdkiiVariablePolicyProtocolGuid, NULL, (VOID **)&VariablePolicyProtocol);

+  if (EFI_ERROR (Status)) {

+    return Status;

+  }

+

   if (mReceivedAcpiBootPerformanceTable != NULL) {

     mAcpiBootPerformanceTable = mReceivedAcpiBootPerformanceTable;

     mAcpiBootPerformanceTable->BasicBoot.ResetEnd = mBootPerformanceTableTemplate.BasicBoot.ResetEnd;

@@ -370,6 +382,24 @@ InstallFirmwarePerformanceDataTable (
         );

 

   //

+  // Lock the variable which stores the Performance Table pointers.

+  //

+  Status = RegisterBasicVariablePolicy (

+             VariablePolicyProtocol,

+             &gEfiFirmwarePerformanceGuid,

+             EFI_FIRMWARE_PERFORMANCE_VARIABLE_NAME,

+             VARIABLE_POLICY_NO_MIN_SIZE,

+             VARIABLE_POLICY_NO_MAX_SIZE,

+             VARIABLE_POLICY_NO_MUST_ATTR,

+             VARIABLE_POLICY_NO_CANT_ATTR,

+             VARIABLE_POLICY_TYPE_LOCK_NOW

+             );

+  if (EFI_ERROR(Status)) {

+    DEBUG((DEBUG_ERROR, "[FirmwarePerformanceDxe] Error when lock variable %s, Status = %r\n", EFI_FIRMWARE_PERFORMANCE_VARIABLE_NAME, Status));

+    ASSERT_EFI_ERROR(Status);

+  }

+

+  //

   // Publish Firmware Performance Data Table.

   //

   FpdtAcpiTableChecksum ((UINT8 *) &mFirmwarePerformanceTableTemplate, mFirmwarePerformanceTableTemplate.Header.Length);

@@ -501,18 +531,12 @@ FpdtStatusCodeListenerDxe (
     DEBUG ((EFI_D_INFO, "FPDT: Boot Performance - OsLoaderStartImageStart = %ld\n", mAcpiBootPerformanceTable->BasicBoot.OsLoaderStartImageStart));

     DEBUG ((EFI_D_INFO, "FPDT: Boot Performance - ExitBootServicesEntry   = 0\n"));

     DEBUG ((EFI_D_INFO, "FPDT: Boot Performance - ExitBootServicesExit    = 0\n"));

-  } else if (Value == (EFI_SOFTWARE_DXE_BS_DRIVER | EFI_SW_DXE_BS_PC_READY_TO_BOOT_EVENT)) {

-    if (mAcpiBootPerformanceTable == NULL) {

-      //

-      // ACPI Firmware Performance Data Table not installed yet, install it now.

-      //

-      InstallFirmwarePerformanceDataTable ();

-    }

   } else if (Data != NULL && CompareGuid (&Data->Type, &gEdkiiFpdtExtendedFirmwarePerformanceGuid)) {

     //

     // Get the Boot performance table and then install it to ACPI table.

     //

     CopyMem (&mReceivedAcpiBootPerformanceTable, Data + 1, Data->Size);

+    InstallFirmwarePerformanceDataTable ();

   } else if (Data != NULL && CompareGuid (&Data->Type, &gEfiFirmwarePerformanceGuid)) {

     DEBUG ((DEBUG_ERROR, "FpdtStatusCodeListenerDxe: Performance data reported through gEfiFirmwarePerformanceGuid will not be collected by FirmwarePerformanceDataTableDxe\n"));

     Status = EFI_UNSUPPORTED;

@@ -526,6 +550,32 @@ FpdtStatusCodeListenerDxe (
   return Status;

 }

 

+/**

+  Notify function for event EndOfDxe.

+

+  This is used to install ACPI Firmware Performance Data Table for basic boot records.

+

+  @param[in]  Event   The Event that is being processed.

+  @param[in]  Context The Event Context.

+

+**/

+VOID

+EFIAPI

+FpdtEndOfDxeEventNotify (

+  IN EFI_EVENT        Event,

+  IN VOID             *Context

+  )

+{

+  //

+  // When performance is enabled, the FPDT will be installed when DxeCorePerformanceLib report the data to FimwarePerformanceDxe.

+  // This is used to install the FPDT for the basic boot recods when performance infrastructure is not enabled.

+  //

+  if ((PcdGet8(PcdPerformanceLibraryPropertyMask) & PERFORMANCE_LIBRARY_PROPERTY_MEASUREMENT_ENABLED) != 0) {

+    return;

+  }

+  ASSERT (mReceivedAcpiBootPerformanceTable == NULL);

+  InstallFirmwarePerformanceDataTable ();

+}

 

 /**

   Notify function for event EVT_SIGNAL_EXIT_BOOT_SERVICES. This is used to record

@@ -596,6 +646,7 @@ FirmwarePerformanceDxeEntryPoint (
   FIRMWARE_SEC_PERFORMANCE *Performance;

   VOID                     *Registration;

   UINT64                   OemTableId;

+  EFI_EVENT                EndOfDxeEvent;

 

   CopyMem (

     mFirmwarePerformanceTableTemplate.Header.OemId,

@@ -621,6 +672,19 @@ FirmwarePerformanceDxeEntryPoint (
   ASSERT_EFI_ERROR (Status);

 

   //

+  // Register the notify function to install FPDT at EndOfDxe.

+  //

+  Status = gBS->CreateEventEx (

+                  EVT_NOTIFY_SIGNAL,

+                  TPL_NOTIFY,

+                  FpdtEndOfDxeEventNotify,

+                  NULL,

+                  &gEfiEndOfDxeEventGroupGuid,

+                  &EndOfDxeEvent

+                  );

+  ASSERT_EFI_ERROR (Status);

+

+  //

   // Register the notify function to update FPDT on ExitBootServices Event.

   //

   Status = gBS->CreateEventEx (

diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/FirmwarePerformanceDxe.inf b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/FirmwarePerformanceDxe.inf
index 1debb01..0411a22 100644
--- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/FirmwarePerformanceDxe.inf
+++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableDxe/FirmwarePerformanceDxe.inf
@@ -5,7 +5,7 @@
 #  for Firmware Basic Boot Performance Record and other boot performance records,

 #  and install FPDT to ACPI table.

 #

-#  Copyright (c) 2011 - 2018, Intel Corporation. All rights reserved.<BR>

+#  Copyright (c) 2011 - 2021, Intel Corporation. All rights reserved.<BR>

 #  SPDX-License-Identifier: BSD-2-Clause-Patent

 #

 ##

@@ -46,12 +46,14 @@
   HobLib

   LockBoxLib

   UefiLib

+  VariablePolicyHelperLib

 

 [Protocols]

   gEfiAcpiTableProtocolGuid                     ## CONSUMES

   gEfiRscHandlerProtocolGuid                    ## CONSUMES

   gEfiVariableArchProtocolGuid                  ## CONSUMES

   gEfiLockBoxProtocolGuid                       ## CONSUMES

+  gEdkiiVariablePolicyProtocolGuid              ## CONSUMES

 

 [Guids]

   gEfiEventExitBootServicesGuid                 ## CONSUMES             ## Event

@@ -63,6 +65,7 @@
   gEfiFirmwarePerformanceGuid

   gEdkiiFpdtExtendedFirmwarePerformanceGuid     ## SOMETIMES_CONSUMES ## UNDEFINED # StatusCode Data

   gFirmwarePerformanceS3PointerGuid             ## PRODUCES ## UNDEFINED # SaveLockBox

+  gEfiEndOfDxeEventGroupGuid                    ## CONSUMES ## Event

 

 [Pcd]

   gEfiMdeModulePkgTokenSpaceGuid.PcdProgressCodeOsLoaderLoad    ## CONSUMES

@@ -72,6 +75,7 @@
   gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultOemRevision      ## CONSUMES

   gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultCreatorId        ## CONSUMES

   gEfiMdeModulePkgTokenSpaceGuid.PcdAcpiDefaultCreatorRevision  ## CONSUMES

+  gEfiMdePkgTokenSpaceGuid.PcdPerformanceLibraryPropertyMask    ## CONSUMES

 

 [FeaturePcd]

   gEfiMdeModulePkgTokenSpaceGuid.PcdFirmwarePerformanceDataTableS3Support   ## CONSUMES

diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.c b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.c
index ecadef8..47db9a2 100644
--- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.c
+++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.c
@@ -11,7 +11,7 @@
 

   FpdtSmiHandler() will receive untrusted input and do basic validation.

 

-  Copyright (c) 2011 - 2018, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2011 - 2021, Intel Corporation. All rights reserved.<BR>

   SPDX-License-Identifier: BSD-2-Clause-Patent

 

 **/

@@ -29,20 +29,12 @@
 #include <Library/LockBoxLib.h>

 #include <Library/PcdLib.h>

 #include <Library/BaseMemoryLib.h>

-#include <Library/MemoryAllocationLib.h>

-#include <Library/SynchronizationLib.h>

 #include "FirmwarePerformanceCommon.h"

 

-SMM_BOOT_PERFORMANCE_TABLE    *mMmBootPerformanceTable = NULL;

 

 EFI_MM_RSC_HANDLER_PROTOCOL   *mRscHandlerProtocol    = NULL;

 UINT64                        mSuspendStartTime       = 0;

 BOOLEAN                       mS3SuspendLockBoxSaved  = FALSE;

-UINT32                        mBootRecordSize = 0;

-UINT8                         *mBootRecordBuffer = NULL;

-

-SPIN_LOCK                     mMmFpdtLock;

-BOOLEAN                       mMmramIsOutOfResource = FALSE;

 

 /**

   Report status code listener for MM. This is used to record the performance

@@ -84,21 +76,6 @@ FpdtStatusCodeListenerMm (
     return EFI_UNSUPPORTED;

   }

 

-  //

-  // Collect one or more Boot records in boot time

-  //

-  if (Data != NULL && CompareGuid (&Data->Type, &gEdkiiFpdtExtendedFirmwarePerformanceGuid)) {

-    AcquireSpinLock (&mMmFpdtLock);

-    //

-    // Get the boot performance data.

-    //

-    CopyMem (&mMmBootPerformanceTable, Data + 1, Data->Size);

-    mBootRecordBuffer = ((UINT8 *) (mMmBootPerformanceTable)) + sizeof (SMM_BOOT_PERFORMANCE_TABLE);

-

-    ReleaseSpinLock (&mMmFpdtLock);

-    return EFI_SUCCESS;

-  }

-

   if (Data != NULL && CompareGuid (&Data->Type, &gEfiFirmwarePerformanceGuid)) {

     DEBUG ((DEBUG_ERROR, "FpdtStatusCodeListenerMm: Performance data reported through gEfiFirmwarePerformanceGuid will not be collected by FirmwarePerformanceDataTableMm\n"));

     return EFI_UNSUPPORTED;

@@ -154,118 +131,6 @@ FpdtStatusCodeListenerMm (
 }

 

 /**

-  Communication service SMI Handler entry.

-

-  This SMI handler provides services for report MM boot records.

-

-  Caution: This function may receive untrusted input.

-  Communicate buffer and buffer size are external input, so this function will do basic validation.

-

-  @param[in]     DispatchHandle  The unique handle assigned to this handler by SmiHandlerRegister().

-  @param[in]     RegisterContext Points to an optional handler context which was specified when the

-                                 handler was registered.

-  @param[in, out] CommBuffer     A pointer to a collection of data in memory that will

-                                 be conveyed from a non-MM environment into an MM environment.

-  @param[in, out] CommBufferSize The size of the CommBuffer.

-

-  @retval EFI_SUCCESS                         The interrupt was handled and quiesced. No other handlers

-                                              should still be called.

-  @retval EFI_WARN_INTERRUPT_SOURCE_QUIESCED  The interrupt has been quiesced but other handlers should

-                                              still be called.

-  @retval EFI_WARN_INTERRUPT_SOURCE_PENDING   The interrupt is still pending and other handlers should still

-                                              be called.

-  @retval EFI_INTERRUPT_PENDING               The interrupt could not be quiesced.

-

-**/

-EFI_STATUS

-EFIAPI

-FpdtSmiHandler (

-  IN     EFI_HANDLE                   DispatchHandle,

-  IN     CONST VOID                   *RegisterContext,

-  IN OUT VOID                         *CommBuffer,

-  IN OUT UINTN                        *CommBufferSize

-  )

-{

-  EFI_STATUS                   Status;

-  SMM_BOOT_RECORD_COMMUNICATE  *SmmCommData;

-  UINTN                        BootRecordOffset;

-  UINTN                        BootRecordSize;

-  VOID                         *BootRecordData;

-  UINTN                        TempCommBufferSize;

-

-  //

-  // If input is invalid, stop processing this SMI

-  //

-  if (CommBuffer == NULL || CommBufferSize == NULL) {

-    return EFI_SUCCESS;

-  }

-

-  TempCommBufferSize = *CommBufferSize;

-

-  if(TempCommBufferSize < sizeof (SMM_BOOT_RECORD_COMMUNICATE)) {

-    return EFI_SUCCESS;

-  }

-

-  if (!IsBufferOutsideMmValid ((UINTN)CommBuffer, TempCommBufferSize)) {

-    DEBUG ((DEBUG_ERROR, "FpdtSmiHandler: MM communication data buffer in MMRAM or overflow!\n"));

-    return EFI_SUCCESS;

-  }

-

-  SmmCommData = (SMM_BOOT_RECORD_COMMUNICATE*)CommBuffer;

-

-  Status = EFI_SUCCESS;

-

-  switch (SmmCommData->Function) {

-    case SMM_FPDT_FUNCTION_GET_BOOT_RECORD_SIZE :

-      if (mMmBootPerformanceTable != NULL) {

-        mBootRecordSize = mMmBootPerformanceTable->Header.Length - sizeof (SMM_BOOT_PERFORMANCE_TABLE);

-      }

-      SmmCommData->BootRecordSize = mBootRecordSize;

-      break;

-

-    case SMM_FPDT_FUNCTION_GET_BOOT_RECORD_DATA :

-      Status = EFI_UNSUPPORTED;

-      break;

-

-    case SMM_FPDT_FUNCTION_GET_BOOT_RECORD_DATA_BY_OFFSET :

-      BootRecordOffset = SmmCommData->BootRecordOffset;

-      BootRecordData   = SmmCommData->BootRecordData;

-      BootRecordSize   = SmmCommData->BootRecordSize;

-      if (BootRecordData == NULL || BootRecordOffset >= mBootRecordSize) {

-        Status = EFI_INVALID_PARAMETER;

-        break;

-      }

-

-      //

-      // Sanity check

-      //

-      if (BootRecordSize > mBootRecordSize - BootRecordOffset) {

-        BootRecordSize = mBootRecordSize - BootRecordOffset;

-      }

-      SmmCommData->BootRecordSize = BootRecordSize;

-      if (!IsBufferOutsideMmValid ((UINTN)BootRecordData, BootRecordSize)) {

-        DEBUG ((DEBUG_ERROR, "FpdtSmiHandler: MM Data buffer in MMRAM or overflow!\n"));

-        Status = EFI_ACCESS_DENIED;

-        break;

-      }

-

-      CopyMem (

-       (UINT8*)BootRecordData,

-       mBootRecordBuffer + BootRecordOffset,

-       BootRecordSize

-       );

-      break;

-

-    default:

-      Status = EFI_UNSUPPORTED;

-  }

-

-  SmmCommData->ReturnStatus = Status;

-

-  return EFI_SUCCESS;

-}

-

-/**

   The module Entry Point of the Firmware Performance Data Table MM driver.

 

   @retval EFI_SUCCESS    The entry point is executed successfully.

@@ -278,12 +143,6 @@ FirmwarePerformanceCommonEntryPoint (
   )

 {

   EFI_STATUS                Status;

-  EFI_HANDLE                Handle;

-

-  //

-  // Initialize spin lock

-  //

-  InitializeSpinLock (&mMmFpdtLock);

 

   //

   // Get MM Report Status Code Handler Protocol.

@@ -301,12 +160,5 @@ FirmwarePerformanceCommonEntryPoint (
   Status = mRscHandlerProtocol->Register (FpdtStatusCodeListenerMm);

   ASSERT_EFI_ERROR (Status);

 

-  //

-  // Register SMI handler.

-  //

-  Handle = NULL;

-  Status = gMmst->MmiHandlerRegister (FpdtSmiHandler, &gEfiFirmwarePerformanceGuid, &Handle);

-  ASSERT_EFI_ERROR (Status);

-

   return Status;

 }

diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.h b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.h
index 0fbdac0..0640c48 100644
--- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.h
+++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.h
@@ -11,7 +11,7 @@
 

   FpdtSmiHandler() will receive untrusted input and do basic validation.

 

-  Copyright (c) 2011 - 2018, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2011 - 2021, Intel Corporation. All rights reserved.<BR>

   Copyright (c), Microsoft Corporation.

   SPDX-License-Identifier: BSD-2-Clause-Patent

 

@@ -21,21 +21,6 @@
 #define _FW_PERF_COMMON_H_

 

 /**

-  This function is an abstraction layer for implementation specific Mm buffer validation routine.

-

-  @param Buffer  The buffer start address to be checked.

-  @param Length  The buffer length to be checked.

-

-  @retval TRUE  This buffer is valid per processor architecture and not overlap with SMRAM.

-  @retval FALSE This buffer is not valid per processor architecture or overlap with SMRAM.

-**/

-BOOLEAN

-IsBufferOutsideMmValid (

-  IN EFI_PHYSICAL_ADDRESS  Buffer,

-  IN UINT64                Length

-  );

-

-/**

   The module Entry Point of the Firmware Performance Data Table MM driver.

 

   @retval EFI_SUCCESS    The entry point is executed successfully.

diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceSmm.inf b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceSmm.inf
index b7194bd..a4da0ba 100644
--- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceSmm.inf
+++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceSmm.inf
@@ -4,7 +4,7 @@
 #  This module registers report status code listener to collect performance data

 #  for SMM boot performance records and S3 Suspend Performance Record.

 #

-#  Copyright (c) 2011 - 2018, Intel Corporation. All rights reserved.<BR>

+#  Copyright (c) 2011 - 2021, Intel Corporation. All rights reserved.<BR>

 #  SPDX-License-Identifier: BSD-2-Clause-Patent

 #

 ##

@@ -52,10 +52,8 @@
 

 [Guids]

   ## SOMETIMES_PRODUCES   ## UNDEFINED # SaveLockBox

-  ## PRODUCES             ## UNDEFINED # SmiHandlerRegister

   ## SOMETIMES_CONSUMES   ## UNDEFINED # StatusCode Data

   gEfiFirmwarePerformanceGuid

-  gEdkiiFpdtExtendedFirmwarePerformanceGuid  ## SOMETIMES_PRODUCES ## UNDEFINED # StatusCode Data

 

 [Pcd]

   gEfiMdeModulePkgTokenSpaceGuid.PcdProgressCodeS3SuspendStart  ## CONSUMES

diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceStandaloneMm.c b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceStandaloneMm.c
index d7da61c..be52d8c 100644
--- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceStandaloneMm.c
+++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceStandaloneMm.c
@@ -11,7 +11,7 @@
 

   FpdtSmiHandler() will receive untrusted input and do basic validation.

 

-  Copyright (c) 2011 - 2018, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2011 - 2021, Intel Corporation. All rights reserved.<BR>

   Copyright (c), Microsoft Corporation.

   SPDX-License-Identifier: BSD-2-Clause-Patent

 

@@ -19,28 +19,9 @@
 

 #include <PiMm.h>

 

-#include <Library/StandaloneMmMemLib.h>

 #include "FirmwarePerformanceCommon.h"

 

 /**

-  This function is an abstraction layer for implementation specific Mm buffer validation routine.

-

-  @param Buffer  The buffer start address to be checked.

-  @param Length  The buffer length to be checked.

-

-  @retval TRUE  This buffer is valid per processor architecture and not overlap with SMRAM.

-  @retval FALSE This buffer is not valid per processor architecture or overlap with SMRAM.

-**/

-BOOLEAN

-IsBufferOutsideMmValid (

-  IN EFI_PHYSICAL_ADDRESS  Buffer,

-  IN UINT64                Length

-  )

-{

-  return MmIsBufferOutsideMmValid (Buffer, Length);

-}

-

-/**

   The module Entry Point of the Firmware Performance Data Table MM driver.

 

   @param[in]  ImageHandle    The firmware allocated handle for the EFI image.

diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceStandaloneMm.inf b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceStandaloneMm.inf
index e6aad88..fabe30b 100644
--- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceStandaloneMm.inf
+++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceStandaloneMm.inf
@@ -4,7 +4,7 @@
 #  This module registers report status code listener to collect performance data

 #  for SMM boot performance records and S3 Suspend Performance Record.

 #

-#  Copyright (c) 2011 - 2018, Intel Corporation. All rights reserved.<BR>

+#  Copyright (c) 2011 - 2021, Intel Corporation. All rights reserved.<BR>

 #  Copyright (c) Microsoft Corporation.

 #  SPDX-License-Identifier: BSD-2-Clause-Patent

 #

@@ -53,10 +53,8 @@
 

 [Guids]

   ## SOMETIMES_PRODUCES   ## UNDEFINED # SaveLockBox

-  ## PRODUCES             ## UNDEFINED # SmiHandlerRegister

   ## SOMETIMES_CONSUMES   ## UNDEFINED # StatusCode Data

   gEfiFirmwarePerformanceGuid

-  gEdkiiFpdtExtendedFirmwarePerformanceGuid  ## SOMETIMES_PRODUCES ## UNDEFINED # StatusCode Data

 

 [Pcd]

   gEfiMdeModulePkgTokenSpaceGuid.PcdProgressCodeS3SuspendStart  ## CONSUMES

diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceTraditional.c b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceTraditional.c
index 43c050d..5bd38b0 100644
--- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceTraditional.c
+++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceTraditional.c
@@ -11,7 +11,7 @@
 

   FpdtSmiHandler() will receive untrusted input and do basic validation.

 

-  Copyright (c) 2011 - 2018, Intel Corporation. All rights reserved.<BR>

+  Copyright (c) 2011 - 2021, Intel Corporation. All rights reserved.<BR>

   Copyright (c), Microsoft Corporation.

   SPDX-License-Identifier: BSD-2-Clause-Patent

 

@@ -19,28 +19,9 @@
 

 #include <PiSmm.h>

 

-#include <Library/SmmMemLib.h>

 #include "FirmwarePerformanceCommon.h"

 

 /**

-  This function is an abstraction layer for implementation specific Mm buffer validation routine.

-

-  @param Buffer  The buffer start address to be checked.

-  @param Length  The buffer length to be checked.

-

-  @retval TRUE  This buffer is valid per processor architecture and not overlap with SMRAM.

-  @retval FALSE This buffer is not valid per processor architecture or overlap with SMRAM.

-**/

-BOOLEAN

-IsBufferOutsideMmValid (

-  IN EFI_PHYSICAL_ADDRESS  Buffer,

-  IN UINT64                Length

-  )

-{

-  return SmmIsBufferOutsideSmmValid (Buffer, Length);

-}

-

-/**

   The module Entry Point of the Firmware Performance Data Table MM driver.

 

   @param[in]  ImageHandle    The firmware allocated handle for the EFI image.