Re: [edk2-devel] [PATCH v5 09/19] MdeModulePkg: Reference Null ProtectedVariableLib
Reviewed-by: Jian J Wang Regards, Jian > -Original Message- > From: Vang, Judah > Sent: Sunday, November 06, 2022 3:35 PM > To: devel@edk2.groups.io > Cc: Wang, Jian J ; Gao, Liming > ; Mistry, Nishant C > Subject: [PATCH v5 09/19] MdeModulePkg: Reference Null ProtectedVariableLib > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2594 > > V5: Add reference to new Protected Variable libs. > > V1: Make reference to new Null ProtectVariableLib. > The null ProtectedVariableLib is used by default. > > Cc: Jian J Wang > Cc: Liming Gao > Cc: Nishant C Mistry > Signed-off-by: Jian J Wang > Signed-off-by: Nishant C Mistry > Signed-off-by: Judah Vang > --- > MdeModulePkg/MdeModulePkg.dsc | 20 +++- > MdeModulePkg/Test/MdeModulePkgHostTest.dsc | 8 > 2 files changed, 27 insertions(+), 1 deletion(-) > > diff --git a/MdeModulePkg/MdeModulePkg.dsc > b/MdeModulePkg/MdeModulePkg.dsc > index 659482ab737f..65ec6d1e0918 100644 > --- a/MdeModulePkg/MdeModulePkg.dsc > +++ b/MdeModulePkg/MdeModulePkg.dsc > @@ -2,7 +2,7 @@ > # EFI/PI Reference Module Package for All Architectures > # > # (C) Copyright 2014 Hewlett-Packard Development Company, L.P. > -# Copyright (c) 2007 - 2021, Intel Corporation. All rights reserved. > +# Copyright (c) 2007 - 2022, Intel Corporation. All rights reserved. > # Copyright (c) Microsoft Corporation. > # > #SPDX-License-Identifier: BSD-2-Clause-Patent > @@ -104,6 +104,7 @@ [LibraryClasses] > > VariablePolicyHelperLib|MdeModulePkg/Library/VariablePolicyHelperLib/Variab > lePolicyHelperLib.inf > > MmUnblockMemoryLib|MdePkg/Library/MmUnblockMemoryLib/MmUnblockM > emoryLibNull.inf > > VariableFlashInfoLib|MdeModulePkg/Library/BaseVariableFlashInfoLib/BaseVar > iableFlashInfoLib.inf > + > ProtectedVariableLib|MdeModulePkg/Library/ProtectedVariableLibNull/Protect > edVariableLibNull.inf > > [LibraryClasses.EBC.PEIM] >IoLib|MdePkg/Library/PeiIoLibCpuIo/PeiIoLibCpuIo.inf > @@ -318,6 +319,7 @@ [Components] > > MdeModulePkg/Library/PlatformBootManagerLibNull/PlatformBootManagerLi > bNull.inf >MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf > > MdeModulePkg/Library/TpmMeasurementLibNull/TpmMeasurementLibNull.inf > + > MdeModulePkg/Library/ProtectedVariableLibNull/ProtectedVariableLibNull.inf >MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLibNull.inf >MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLib.inf >MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLibRuntimeDxe.inf > @@ -397,6 +399,7 @@ [Components] >MdeModulePkg/Application/VariableInfo/VariableInfo.inf >MdeModulePkg/Universal/FaultTolerantWritePei/FaultTolerantWritePei.inf >MdeModulePkg/Universal/Variable/Pei/VariablePei.inf > + MdeModulePkg/Universal/Variable/Protected/Pei/VariablePei.inf >MdeModulePkg/Universal/WatchdogTimerDxe/WatchdogTimer.inf >MdeModulePkg/Universal/TimestampDxe/TimestampDxe.inf >MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteDxe.inf > @@ -461,6 +464,7 @@ [Components.IA32, Components.X64, > Components.ARM, Components.AARCH64] > !if $(TOOL_CHAIN_TAG) != "XCODE5" > > MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteStandalon > eMm.inf >MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.inf > + > MdeModulePkg/Universal/Variable/Protected/RuntimeDxe/VariableStandalone > Mm.inf > !endif > > [Components.IA32, Components.X64] > @@ -475,13 +479,27 @@ [Components.IA32, Components.X64] >NULL|MdeModulePkg/Library/VarCheckHiiLib/VarCheckHiiLib.inf >NULL|MdeModulePkg/Library/VarCheckPcdLib/VarCheckPcdLib.inf >} > + MdeModulePkg/Universal/Variable/Protected/RuntimeDxe/VariableSmm.inf { > + > + NULL|MdeModulePkg/Library/VarCheckPolicyLib/VarCheckPolicyLib.inf > + NULL|MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLib.inf > + NULL|MdeModulePkg/Library/VarCheckHiiLib/VarCheckHiiLib.inf > + NULL|MdeModulePkg/Library/VarCheckPcdLib/VarCheckPcdLib.inf > + } >MdeModulePkg/Universal/Variable/RuntimeDxe/VariableRuntimeDxe.inf { > >NULL|MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLib.inf >NULL|MdeModulePkg/Library/VarCheckHiiLib/VarCheckHiiLib.inf >NULL|MdeModulePkg/Library/VarCheckPcdLib/VarCheckPcdLib.inf >} > + > MdeModulePkg/Universal/Variable/Protected/RuntimeDxe/VariableRuntimeDxe > .inf { > + > + NULL|MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLib.inf > + NULL|MdeModulePkg/Library/VarCheckHiiLib/VarCheckHiiLib.inf > + NULL|MdeModulePkg/Library/VarCheckPcdLib/VarCheckPcdLib.inf > + } >MdeModulePkg/Universal/Variable/RuntimeDxe/VariableSmmRuntimeDxe.inf > + > MdeModulePkg/Universal/Variable/Protected/RuntimeDxe/VariableSmmRuntim > eDxe.inf > > MdeModulePkg/Library/SmmReportStatusCodeLib/SmmReportStatusCodeLib.in > f > > MdeModulePkg/Library/SmmReportStatusCodeLib/StandaloneMmReportStatus > CodeLib.inf > >
[edk2-devel] [PATCH v5 09/19] MdeModulePkg: Reference Null ProtectedVariableLib
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2594 V5: Add reference to new Protected Variable libs. V1: Make reference to new Null ProtectVariableLib. The null ProtectedVariableLib is used by default. Cc: Jian J Wang Cc: Liming Gao Cc: Nishant C Mistry Signed-off-by: Jian J Wang Signed-off-by: Nishant C Mistry Signed-off-by: Judah Vang --- MdeModulePkg/MdeModulePkg.dsc | 20 +++- MdeModulePkg/Test/MdeModulePkgHostTest.dsc | 8 2 files changed, 27 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/MdeModulePkg.dsc b/MdeModulePkg/MdeModulePkg.dsc index 659482ab737f..65ec6d1e0918 100644 --- a/MdeModulePkg/MdeModulePkg.dsc +++ b/MdeModulePkg/MdeModulePkg.dsc @@ -2,7 +2,7 @@ # EFI/PI Reference Module Package for All Architectures # # (C) Copyright 2014 Hewlett-Packard Development Company, L.P. -# Copyright (c) 2007 - 2021, Intel Corporation. All rights reserved. +# Copyright (c) 2007 - 2022, Intel Corporation. All rights reserved. # Copyright (c) Microsoft Corporation. # #SPDX-License-Identifier: BSD-2-Clause-Patent @@ -104,6 +104,7 @@ [LibraryClasses] VariablePolicyHelperLib|MdeModulePkg/Library/VariablePolicyHelperLib/VariablePolicyHelperLib.inf MmUnblockMemoryLib|MdePkg/Library/MmUnblockMemoryLib/MmUnblockMemoryLibNull.inf VariableFlashInfoLib|MdeModulePkg/Library/BaseVariableFlashInfoLib/BaseVariableFlashInfoLib.inf + ProtectedVariableLib|MdeModulePkg/Library/ProtectedVariableLibNull/ProtectedVariableLibNull.inf [LibraryClasses.EBC.PEIM] IoLib|MdePkg/Library/PeiIoLibCpuIo/PeiIoLibCpuIo.inf @@ -318,6 +319,7 @@ [Components] MdeModulePkg/Library/PlatformBootManagerLibNull/PlatformBootManagerLibNull.inf MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf MdeModulePkg/Library/TpmMeasurementLibNull/TpmMeasurementLibNull.inf + MdeModulePkg/Library/ProtectedVariableLibNull/ProtectedVariableLibNull.inf MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLibNull.inf MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLib.inf MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLibRuntimeDxe.inf @@ -397,6 +399,7 @@ [Components] MdeModulePkg/Application/VariableInfo/VariableInfo.inf MdeModulePkg/Universal/FaultTolerantWritePei/FaultTolerantWritePei.inf MdeModulePkg/Universal/Variable/Pei/VariablePei.inf + MdeModulePkg/Universal/Variable/Protected/Pei/VariablePei.inf MdeModulePkg/Universal/WatchdogTimerDxe/WatchdogTimer.inf MdeModulePkg/Universal/TimestampDxe/TimestampDxe.inf MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteDxe.inf @@ -461,6 +464,7 @@ [Components.IA32, Components.X64, Components.ARM, Components.AARCH64] !if $(TOOL_CHAIN_TAG) != "XCODE5" MdeModulePkg/Universal/FaultTolerantWriteDxe/FaultTolerantWriteStandaloneMm.inf MdeModulePkg/Universal/Variable/RuntimeDxe/VariableStandaloneMm.inf + MdeModulePkg/Universal/Variable/Protected/RuntimeDxe/VariableStandaloneMm.inf !endif [Components.IA32, Components.X64] @@ -475,13 +479,27 @@ [Components.IA32, Components.X64] NULL|MdeModulePkg/Library/VarCheckHiiLib/VarCheckHiiLib.inf NULL|MdeModulePkg/Library/VarCheckPcdLib/VarCheckPcdLib.inf } + MdeModulePkg/Universal/Variable/Protected/RuntimeDxe/VariableSmm.inf { + + NULL|MdeModulePkg/Library/VarCheckPolicyLib/VarCheckPolicyLib.inf + NULL|MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLib.inf + NULL|MdeModulePkg/Library/VarCheckHiiLib/VarCheckHiiLib.inf + NULL|MdeModulePkg/Library/VarCheckPcdLib/VarCheckPcdLib.inf + } MdeModulePkg/Universal/Variable/RuntimeDxe/VariableRuntimeDxe.inf { NULL|MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLib.inf NULL|MdeModulePkg/Library/VarCheckHiiLib/VarCheckHiiLib.inf NULL|MdeModulePkg/Library/VarCheckPcdLib/VarCheckPcdLib.inf } + MdeModulePkg/Universal/Variable/Protected/RuntimeDxe/VariableRuntimeDxe.inf { + + NULL|MdeModulePkg/Library/VarCheckUefiLib/VarCheckUefiLib.inf + NULL|MdeModulePkg/Library/VarCheckHiiLib/VarCheckHiiLib.inf + NULL|MdeModulePkg/Library/VarCheckPcdLib/VarCheckPcdLib.inf + } MdeModulePkg/Universal/Variable/RuntimeDxe/VariableSmmRuntimeDxe.inf + MdeModulePkg/Universal/Variable/Protected/RuntimeDxe/VariableSmmRuntimeDxe.inf MdeModulePkg/Library/SmmReportStatusCodeLib/SmmReportStatusCodeLib.inf MdeModulePkg/Library/SmmReportStatusCodeLib/StandaloneMmReportStatusCodeLib.inf MdeModulePkg/Universal/StatusCodeHandler/Smm/StatusCodeHandlerSmm.inf diff --git a/MdeModulePkg/Test/MdeModulePkgHostTest.dsc b/MdeModulePkg/Test/MdeModulePkgHostTest.dsc index c9ec835df65d..c0ca9be71e8c 100644 --- a/MdeModulePkg/Test/MdeModulePkgHostTest.dsc +++ b/MdeModulePkg/Test/MdeModulePkgHostTest.dsc @@ -42,6 +42,14 @@ [Components] gEfiMdeModulePkgTokenSpaceGuid.PcdAllowVariablePolicyEnforcementDisable|TRUE } +