REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4651
In FirmwarePerformanceCommonEntryPoint there is a MmLocateProtocol which will locate the gEfiMmRscHandlerProtocolGuid and then ASSERT_EFI_ERROR(Status) if the locate protocol fails when build is debug enabled. if the build is debug disabled then code will execute - register the report status code using the API provided by gEfiMmRscHandlerProtocolGuid. Adding a proper error handling mechanism to handle the error gracefully when debug is not enabled and to properly report the error if it cannot continue. Signed-off-by: Sasikanth Valaparla <sasikanth.valapa...@intel.com> --- .../FirmwarePerformanceCommon.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.c b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.c index 9046896c55..7b13215ade 100644 --- a/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.c +++ b/MdeModulePkg/Universal/Acpi/FirmwarePerformanceDataTableSmm/FirmwarePerformanceCommon.c @@ -152,7 +152,10 @@ FirmwarePerformanceCommonEntryPoint ( NULL, (VOID **)&mRscHandlerProtocol ); - ASSERT_EFI_ERROR (Status); + if (EFI_ERROR(Status)) { + ASSERT_EFI_ERROR (Status); + return Status; + } // // Register report status code listener for BootRecords and S3 Suspend Start and End. -- 2.26.2.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#116036): https://edk2.groups.io/g/devel/message/116036 Mute This Topic: https://groups.io/mt/104601540/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-