From 7b9b55b2ef7be13608605dc58a54b9ca04be4e40 Mon Sep 17 00:00:00 2001 From: Star Zeng Date: Fri, 8 Dec 2017 18:44:40 +0800 Subject: [PATCH] MdeModulePkg PiSmmCore: Only install EndOfS3Resume during S3 resume Otherwise, it may be triggered wrongly by other code in OS. This patch is to use S3 entry callback to determine if it will be during S3 resume, and check it in SmmReadyToBootHandler(). Cc: Jiewen Yao Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Star Zeng Reviewed-by: Jiewen Yao --- MdeModulePkg/Core/PiSmmCore/PiSmmCore.c | 79 +++++++++++++++++++++-- MdeModulePkg/Core/PiSmmCore/PiSmmCore.h | 1 + MdeModulePkg/Core/PiSmmCore/PiSmmCore.inf | 1 + 3 files changed, 77 insertions(+), 4 deletions(-) diff --git a/MdeModulePkg/Core/PiSmmCore/PiSmmCore.c b/MdeModulePkg/Core/PiSmmCore/PiSmmCore.c index 0b9c7958c7..4aef9b70a1 100644 --- a/MdeModulePkg/Core/PiSmmCore/PiSmmCore.c +++ b/MdeModulePkg/Core/PiSmmCore/PiSmmCore.c @@ -71,6 +71,12 @@ EFI_SMM_SYSTEM_TABLE2 gSmmCoreSmst = { // BOOLEAN mInLegacyBoot = FALSE; +// +// Flag to determine if it is during S3 resume. +// It will be set in S3 entry callback and cleared at EndOfS3Resume. +// +BOOLEAN mDuringS3Resume = FALSE; + // // Table of SMI Handlers that are registered by the SMM Core when it is initialized // @@ -212,6 +218,37 @@ SmmExitBootServicesHandler ( return Status; } +/** + Main entry point for an SMM handler dispatch or communicate-based callback. + + @param[in] DispatchHandle The unique handle assigned to this handler by SmiHandlerRegister(). + @param[in] Context 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-SMM environment into an SMM 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 +SmmS3EntryCallBack ( + IN EFI_HANDLE DispatchHandle, + IN CONST VOID *Context OPTIONAL, + IN OUT VOID *CommBuffer OPTIONAL, + IN OUT UINTN *CommBufferSize OPTIONAL + ) +{ + mDuringS3Resume = TRUE; + return EFI_SUCCESS; +} + /** Software SMI handler that is called when an Ready To Boot event is signalled. Then the SMM Core also install SMM Ready To Boot protocol to notify SMM driver @@ -235,8 +272,11 @@ SmmReadyToBootHandler ( IN OUT UINTN *CommBufferSize OPTIONAL ) { - EFI_STATUS Status; - EFI_HANDLE SmmHandle; + EFI_STATUS Status; + EFI_HANDLE SmmHandle; + EFI_SMM_SX_DISPATCH2_PROTOCOL *SxDispatch; + EFI_SMM_SX_REGISTER_CONTEXT EntryRegisterContext; + EFI_HANDLE S3EntryHandle; // // Install SMM Ready To Boot protocol. @@ -251,7 +291,31 @@ SmmReadyToBootHandler ( SmiHandlerUnRegister (DispatchHandle); - return Status; + // + // Locate SmmSxDispatch2 protocol. + // + Status = SmmLocateProtocol ( + &gEfiSmmSxDispatch2ProtocolGuid, + NULL, + &SxDispatch + ); + if (!EFI_ERROR (Status)) { + // + // Register a S3 entry callback function to + // determine if it will be during S3 resume. + // + EntryRegisterContext.Type = SxS3; + EntryRegisterContext.Phase = SxEntry; + Status = SxDispatch->Register ( + SxDispatch, + SmmS3EntryCallBack, + &EntryRegisterContext, + &S3EntryHandle + ); + ASSERT_EFI_ERROR (Status); + } + + return EFI_SUCCESS; } /** @@ -409,7 +473,12 @@ SmmEndOfS3ResumeHandler ( EFI_STATUS Status; EFI_HANDLE SmmHandle; - DEBUG ((EFI_D_INFO, "SmmEndOfS3ResumeHandler\n")); + DEBUG ((DEBUG_INFO, "SmmEndOfS3ResumeHandler\n")); + + if (!mDuringS3Resume) { + DEBUG ((DEBUG_ERROR, "It is not during S3 resume\n")); + return EFI_SUCCESS; + } // // Install SMM EndOfS3Resume protocol @@ -434,6 +503,8 @@ SmmEndOfS3ResumeHandler ( ); ASSERT_EFI_ERROR (Status); + mDuringS3Resume = FALSE; + return Status; } diff --git a/MdeModulePkg/Core/PiSmmCore/PiSmmCore.h b/MdeModulePkg/Core/PiSmmCore/PiSmmCore.h index 2729a434d8..8c10d833e2 100644 --- a/MdeModulePkg/Core/PiSmmCore/PiSmmCore.h +++ b/MdeModulePkg/Core/PiSmmCore/PiSmmCore.h @@ -33,6 +33,7 @@ #include #include #include +#include #include #include diff --git a/MdeModulePkg/Core/PiSmmCore/PiSmmCore.inf b/MdeModulePkg/Core/PiSmmCore/PiSmmCore.inf index de0037fe4a..5c04e851f9 100644 --- a/MdeModulePkg/Core/PiSmmCore/PiSmmCore.inf +++ b/MdeModulePkg/Core/PiSmmCore/PiSmmCore.inf @@ -92,6 +92,7 @@ gEfiSmmUsbDispatch2ProtocolGuid ## SOMETIMES_CONSUMES gEfiSmmCpuProtocolGuid ## SOMETIMES_CONSUMES gEdkiiSmmMemoryAttributeProtocolGuid ## CONSUMES + gEfiSmmSxDispatch2ProtocolGuid ## SOMETIMES_CONSUMES [Pcd] gEfiMdeModulePkgTokenSpaceGuid.PcdLoadFixAddressSmmCodePageNumber ## SOMETIMES_CONSUMES -- 2.39.2