dasharo/edk2
0
0
Fork 0
mirror of https://github.com/Dasharo/edk2.git synced 2026-06-13 10:16:24 -07:00
Code Issues Packages Projects Releases Wiki Activity

StandaloneMmPkg: StandaloneMmCore: Log MM Entry/Exit Protocol at Verbose

Browse Source 
Currently, StandaloneMmCore prints for every MM entry and exit
as well as producing protocols for MM entry and exit that get
logged. So there are four logs for every MM entry/exit pair.

In a sample OVMF run with StMM enabled, this produced 6800 logs
to log MM entry/exit out of 16,400 logs. In other words, > 1/3
of logs were for these prints.

Protocol install notifications are useful, but not for these ones
that are just entry and exit notifies.

This commit only logs at verbose level if the protocol installed
is the entry or exit protocol.

Signed-off-by: Oliver Smith-Denny <osde@microsoft.com>
This commit is contained in:
Oliver Smith-Denny
2026-01-22 07:31:58 -08:00
committed by mergify[bot]
parent ffa6fc362a
commit f0eade5984
1 changed files with 8 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
StandaloneMmPkg/Core/Handle.c
+8 -2
View File
@@ -233,9 +233,15 @@ MmInstallProtocolInterfaceNotify (
}
//
// Print debug message
// Print debug message unless installing MM entry/exit notify protocol as that spams the log
//
DEBUG ((DEBUG_LOAD | DEBUG_INFO, "MmInstallProtocolInterface: %g %p\n", Protocol, Interface));
if (!CompareGuid (Protocol, &gEfiMmEntryNotifyProtocolGuid) &&
!CompareGuid (Protocol, &gEfiMmExitNotifyProtocolGuid))
{
DEBUG ((DEBUG_LOAD | DEBUG_INFO, "MmInstallProtocolInterface: Installing non-entry/exit protocol %g %p\n", Protocol, Interface));
} else {
DEBUG ((DEBUG_VERBOSE, "MmInstallProtocolInterface: Installing entry/exit protocol %g %p\n", Protocol, Interface));
}
Status = EFI_OUT_OF_RESOURCES;
Prot = NULL;