Skip to content

Commit

Permalink
StandaloneMmCore: Change log level when mCommunicationBuffer is NULL
Browse files Browse the repository at this point in the history
On Arm all requests are handled as Asynchronous events by the Root
MMI handler.
Since the communication data is conveyed using either the NS shared
buffer or the Secure shared buffer, the Arm implementation does not
setup the mCommunicationBuffer. Therefore, the mCommunicationBuffer
being NULL is not an error case.

Moreover, the existing code switches to Asynchronous event processing
when the mCommunicationBuffer is NULL, which means that the log is an
info log rather than an error.

Therefore, change the log level from ERROR to INFO when the
mCommunicationBuffer is NULL.

Signed-off-by: Levi Yun <[email protected]>
(cherry picked from commit 6016c52)
  • Loading branch information
LeviYeoReum authored and kuqin12 committed Jan 25, 2025
1 parent 2e9c16c commit 99d438a
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions StandaloneMmPkg/Core/StandaloneMmCore.c
Original file line number Diff line number Diff line change
Expand Up @@ -397,6 +397,8 @@ MmEntryPoint (
gMmCorePrivate->CommunicationBuffer = 0;
gMmCorePrivate->ReturnStatus = (Status == EFI_SUCCESS) ? EFI_SUCCESS : EFI_NOT_FOUND;
}
} else {
DEBUG ((DEBUG_INFO, "No valid communication buffer, no Synchronous MMI will be processed\n"));
}

//
Expand Down

0 comments on commit 99d438a

Please sign in to comment.