From 504dfa9dbbbd8d485468aee58eb45cb295d9b031 Mon Sep 17 00:00:00 2001 From: qianouyang Date: Wed, 12 Sep 2012 10:20:34 +0000 Subject: [PATCH] Add Tper Reset Logic by using MOR bit. Signed-off-by: Ouyang, Qian Reviewed-by: Tian, Feng Dong, Guo git-svn-id: https://edk2.svn.sourceforge.net/svnroot/edk2/trunk/edk2@13728 6f19259b-4bc3-4df7-8a09-765794883524 --- .../Tcg/MemoryOverwriteControl/TcgMor.c | 91 ++++++++++++++----- .../Tcg/MemoryOverwriteControl/TcgMor.h | 3 +- .../Tcg/MemoryOverwriteControl/TcgMor.inf | 3 +- 3 files changed, 72 insertions(+), 25 deletions(-) diff --git a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.c b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.c index ce53112dc5..def965f138 100644 --- a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.c +++ b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.c @@ -4,7 +4,7 @@ This driver initilize MemoryOverwriteRequestControl variable. It will clear MOR_CLEAR_MEMORY_BIT bit if it is set. -Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.
+Copyright (c) 2009 - 2012, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -17,6 +17,50 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. #include "TcgMor.h" +UINT8 mMorControl; + +/** + Ready to Boot Event notification handler. + + Sequence of OS boot events is measured in this event notification handler. + + @param[in] Event Event whose notification function is being invoked + @param[in] Context Pointer to the notification function's context + +**/ +VOID +EFIAPI +OnReadyToBoot ( + IN EFI_EVENT Event, + IN VOID *Context + ) +{ + UINTN DataSize; + + if (MOR_CLEAR_MEMORY_VALUE (mMorControl) == 0x0) { + // + // MorControl is expected, directly return to avoid unnecessary variable operation + // + return ; + } + // + // Clear MOR_CLEAR_MEMORY_BIT + // + DEBUG ((EFI_D_INFO, "TcgMor: Clear MorClearMemory bit\n")); + mMorControl &= 0xFE; + + DataSize = sizeof (mMorControl); + gRT->SetVariable ( + MEMORY_OVERWRITE_REQUEST_VARIABLE_NAME, + &gEfiMemoryOverwriteControlDataGuid, + EFI_VARIABLE_NON_VOLATILE | EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_RUNTIME_ACCESS, + DataSize, + &mMorControl + ); + +} + + /** Entry Point for TCG MOR Control driver. @@ -34,48 +78,49 @@ MorDriverEntryPoint ( ) { EFI_STATUS Status; - UINT8 MorControl; UINTN DataSize; + EFI_EVENT Event; /// /// The firmware is required to create the MemoryOverwriteRequestControl UEFI variable. /// - DataSize = sizeof (MorControl); + DataSize = sizeof (mMorControl); Status = gRT->GetVariable ( MEMORY_OVERWRITE_REQUEST_VARIABLE_NAME, &gEfiMemoryOverwriteControlDataGuid, NULL, &DataSize, - &MorControl + &mMorControl ); if (EFI_ERROR (Status)) { // // Set default value to 0 // - MorControl = 0; + mMorControl = 0; + DEBUG ((EFI_D_INFO, "TcgMor: Create gEfiMemoryOverwriteControlDataGuid!\n")); + Status = gRT->SetVariable ( + MEMORY_OVERWRITE_REQUEST_VARIABLE_NAME, + &gEfiMemoryOverwriteControlDataGuid, + EFI_VARIABLE_NON_VOLATILE | EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_RUNTIME_ACCESS, + DataSize, + &mMorControl + ); + ASSERT_EFI_ERROR (Status); + } else { - if (MOR_CLEAR_MEMORY_VALUE (MorControl) == 0x0) { - // - // MorControl is expected, directly return to avoid unnecessary variable operation - // - return EFI_SUCCESS; - } // - // Clear MOR_CLEAR_MEMORY_BIT + // Create a Ready To Boot Event and Clear the MorControl bit in the call back function. // - DEBUG ((EFI_D_INFO, "TcgMor: Clear MorClearMemory bit\n")); - MorControl &= 0xFE; - } + DEBUG ((EFI_D_INFO, "TcgMor: Create ReadyToBoot Event for MorControl Bit cleanning!\n")); + Status = EfiCreateEventReadyToBootEx ( + TPL_CALLBACK, + OnReadyToBoot, + NULL, + &Event + ); + } - Status = gRT->SetVariable ( - MEMORY_OVERWRITE_REQUEST_VARIABLE_NAME, - &gEfiMemoryOverwriteControlDataGuid, - EFI_VARIABLE_NON_VOLATILE | EFI_VARIABLE_BOOTSERVICE_ACCESS | EFI_VARIABLE_RUNTIME_ACCESS, - DataSize, - &MorControl - ); - ASSERT_EFI_ERROR (Status); return Status; } diff --git a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.h b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.h index bfad2cf2f0..24ecdf6a06 100644 --- a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.h +++ b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.h @@ -1,7 +1,7 @@ /** @file The header file for TcgMor. -Copyright (c) 2009, Intel Corporation. All rights reserved.
+Copyright (c) 2009 - 2012, Intel Corporation. All rights reserved.
This program and the accompanying materials are licensed and made available under the terms and conditions of the BSD License which accompanies this distribution. The full text of the license may be found at @@ -22,6 +22,7 @@ WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. #include #include #include +#include #endif diff --git a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf index e62a90447c..6cf51b78da 100644 --- a/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf +++ b/SecurityPkg/Tcg/MemoryOverwriteControl/TcgMor.inf @@ -1,7 +1,7 @@ ## @file # Component description file for Memory Overwrite Control driver. # -# Copyright (c) 2009 - 2011, Intel Corporation. All rights reserved.
+# Copyright (c) 2009 - 2012, Intel Corporation. All rights reserved.
# This program and the accompanying materials # are licensed and made available under the terms and conditions of the BSD License # which accompanies this distribution. The full text of the license may be found at @@ -39,6 +39,7 @@ UefiRuntimeServicesTableLib ReportStatusCodeLib DebugLib + UefiLib [Guids] gEfiMemoryOverwriteControlDataGuid # GUID ALWAYS_CONSUMED