Reviewed-by: Liming Gao <gaolim...@byosoft.com.cn> > -----邮件原件----- > 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Sunny Wang > 发送时间: 2021年7月9日 17:09 > 收件人: Grzegorz Bernacki <g...@semihalf.com>; devel@edk2.groups.io > 抄送: l...@nuviainc.com; ardb+tianoc...@kernel.org; Samer > El-Haj-Mahmoud <samer.el-haj-mahm...@arm.com>; m...@semihalf.com; > upstr...@semihalf.com; jiewen....@intel.com; jian.j.w...@intel.com; > min.m...@intel.com; ler...@redhat.com; Sami Mujawar > <sami.muja...@arm.com>; af...@apple.com; ray...@intel.com; > jordan.l.jus...@intel.com; rebe...@bsdio.com; gre...@freebsd.org; > Thomas Abraham <thomas.abra...@arm.com>; chasel.c...@intel.com; > nathaniel.l.desim...@intel.com; gaolim...@byosoft.com.cn; > eric.d...@intel.com; michael.d.kin...@intel.com; zailiang....@intel.com; > yi.q...@intel.com; gra...@nuviainc.com; r...@semihalf.com; p...@akeo.ie; > Sunny Wang <sunny.w...@arm.com> > 主题: Re: [edk2-devel] [edk2-platforms PATCH v5 1/4] Intel Platforms: add > SecureBootVariableLib class resolution > > Looks good to me. > Reviewed-by: Sunny Wang <sunny.w...@arm.com> > > -----Original Message----- > From: Grzegorz Bernacki <g...@semihalf.com> > Sent: Thursday, July 1, 2021 5:21 PM > To: devel@edk2.groups.io > Cc: l...@nuviainc.com; ardb+tianoc...@kernel.org; Samer El-Haj-Mahmoud > <samer.el-haj-mahm...@arm.com>; Sunny Wang <sunny.w...@arm.com>; > m...@semihalf.com; upstr...@semihalf.com; jiewen....@intel.com; > jian.j.w...@intel.com; min.m...@intel.com; ler...@redhat.com; Sami > Mujawar <sami.muja...@arm.com>; af...@apple.com; ray...@intel.com; > jordan.l.jus...@intel.com; rebe...@bsdio.com; gre...@freebsd.org; > Thomas Abraham <thomas.abra...@arm.com>; chasel.c...@intel.com; > nathaniel.l.desim...@intel.com; gaolim...@byosoft.com.cn; > eric.d...@intel.com; michael.d.kin...@intel.com; zailiang....@intel.com; > yi.q...@intel.com; gra...@nuviainc.com; r...@semihalf.com; p...@akeo.ie; > Grzegorz Bernacki <g...@semihalf.com> > Subject: [edk2-platforms PATCH v5 1/4] Intel Platforms: add > SecureBootVariableLib class resolution > > The edk2 patch > SecurityPkg: Create library for setting Secure Boot variables. > > removes generic functions from SecureBootConfigDxe and places > them into SecureBootVariableLib. This patch adds SecureBootVariableLib > mapping for each Intel platform which uses SecureBootConfigDxe. > > Signed-off-by: Grzegorz Bernacki <g...@semihalf.com> > --- > Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc | 1 + > Platform/Intel/QuarkPlatformPkg/Quark.dsc | 1 + > Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc | 1 + > Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc | 3 ++- > 4 files changed, 5 insertions(+), 1 deletion(-) > > diff --git a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc > b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc > index b154f9615d..5157c87a9a 100644 > --- a/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc > +++ b/Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc > @@ -139,6 +139,7 @@ > > !if gMinPlatformPkgTokenSpaceGuid.PcdUefiSecureBootEnable == TRUE > AuthVariableLib|SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf > + > SecureBootVariableLib|SecurityPkg/Library/SecureBootVariableLib/SecureBo > otVariableLib.inf > !endif > > SafeIntLib|MdePkg/Library/BaseSafeIntLib/BaseSafeIntLib.inf > diff --git a/Platform/Intel/QuarkPlatformPkg/Quark.dsc > b/Platform/Intel/QuarkPlatformPkg/Quark.dsc > index cc1eba4df4..35f99429f7 100644 > --- a/Platform/Intel/QuarkPlatformPkg/Quark.dsc > +++ b/Platform/Intel/QuarkPlatformPkg/Quark.dsc > @@ -175,6 +175,7 @@ > !if $(SECURE_BOOT_ENABLE) > > PlatformSecureLib|QuarkPlatformPkg/Library/PlatformSecureLib/PlatformSec > ureLib.inf > AuthVariableLib|SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf > + > SecureBootVariableLib|SecurityPkg/Library/SecureBootVariableLib/SecureBo > otVariableLib.inf > !else > > AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLi > bNull.inf > !endif > diff --git a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > index d15da40819..5a0d3e31e1 100644 > --- a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > +++ b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > @@ -227,6 +227,7 @@ > !if $(SECURE_BOOT_ENABLE) == TRUE > > PlatformSecureLib|SecurityPkg/Library/PlatformSecureLibNull/PlatformSecur > eLibNull.inf > AuthVariableLib|SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf > + > SecureBootVariableLib|SecurityPkg/Library/SecureBootVariableLib/SecureBo > otVariableLib.inf > !else > > AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLi > bNull.inf > !endif > diff --git a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > index 4a5548b80e..36a5ae333c 100644 > --- a/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > +++ b/Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > @@ -1,4 +1,4 @@ > -#/** @file > +e > # Platform description. > # > # Copyright (c) 2012 - 2021, Intel Corporation. All rights reserved.<BR> > @@ -229,6 +229,7 @@ > !if $(SECURE_BOOT_ENABLE) == TRUE > > PlatformSecureLib|SecurityPkg/Library/PlatformSecureLibNull/PlatformSecur > eLibNull.inf > AuthVariableLib|SecurityPkg/Library/AuthVariableLib/AuthVariableLib.inf > + > SecureBootVariableLib|SecurityPkg/Library/SecureBootVariableLib/SecureBo > otVariableLib.inf > !else > > AuthVariableLib|MdeModulePkg/Library/AuthVariableLibNull/AuthVariableLi > bNull.inf > !endif > -- > 2.25.1 > > IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended recipient, > please notify the sender immediately and do not disclose the contents to any > other person, use it for any purpose, or store or copy the information in any > medium. Thank you. > > > >
-=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#77697): https://edk2.groups.io/g/devel/message/77697 Mute This Topic: https://groups.io/mt/84153183/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
