From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by mx.groups.io with SMTP id smtpd.web09.3857.1628043703685285203 for ; Tue, 03 Aug 2021 19:21:45 -0700 Authentication-Results: mx.groups.io; dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Wed, 04 Aug 2021 10:21:10 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 58.246.60.130 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: "'Sunny Wang'" , "'Grzegorz Bernacki'" , , Cc: , "'Samer El-Haj-Mahmoud'" , , , , , , , "'Sami Mujawar'" , , , , , , "'Thomas Abraham'" , , , , , , , , , References: <20210714123045.1340940-1-gjb@semihalf.com> In-Reply-To: Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1wbGF0Zm9ybXMgUEFUQ0ggdjYgMC80XSBTZWN1cmUgQm9vdCBkZWZhdWx0IGtleXM=?= Date: Wed, 4 Aug 2021 10:21:09 +0800 Message-ID: <01b601d788d7$64ca88a0$2e5f99e0$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQKbTgAo5Q/XFcmjvTV3cxLr+ev1DQJdYsIRqcgniSA= Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Sunny: I am OK to merge the reviewed patched first.=20 Thanks Liming > -----=D3=CA=BC=FE=D4=AD=BC=FE----- > =B7=A2=BC=FE=C8=CB: Sunny Wang > =B7=A2=CB=CD=CA=B1=BC=E4: 2021=C4=EA8=D4=C23=C8=D5 22:11 > =CA=D5=BC=FE=C8=CB: Grzegorz Bernacki ; = devel@edk2.groups.io; > ardb+tianocore@kernel.org > =B3=AD=CB=CD: leif@nuviainc.com; Samer El-Haj-Mahmoud > ; mw@semihalf.com; > upstream@semihalf.com; jiewen.yao@intel.com; jian.j.wang@intel.com; > min.m.xu@intel.com; lersek@redhat.com; Sami Mujawar > ; afish@apple.com; ray.ni@intel.com; > jordan.l.justen@intel.com; rebecca@bsdio.com; grehan@freebsd.org; > Thomas Abraham ; chasel.chiu@intel.com; > nathaniel.l.desimone@intel.com; gaoliming@byosoft.com.cn; > eric.dong@intel.com; michael.d.kinney@intel.com; = zailiang.sun@intel.com; > yi.qian@intel.com; graeme@nuviainc.com; rad@semihalf.com; = pete@akeo.ie; > Sunny Wang > =D6=F7=CC=E2: RE: [edk2-platforms PATCH v6 0/4] Secure Boot default = keys >=20 > Hi Ard and Maintainers, >=20 > For this patchset, > 1/4 - Intel Platforms: add SecureBootVariableLib class resolution > 2/4 - ARM Silicon and Platforms: add SecureBootVariableLib class resolution > 3/4 - RISC-V Platforms: add SecureBootVariableLib class resolution > 4/4 - Platform/RaspberryPi: Enable default Secure Boot variables > initialization >=20 > Only Intel platform patch (1/4) hasn't got all Review-bys. I offline = sent a > reminder to Intel platform Maintainers. I think they may be busy with other > things or need more time to review it. >=20 > Therefore, how about we merge another three patches (2/4, 3/4, and = 4/4) > first? >=20 > Best Regards, > Sunny Wang >=20 > -----Original Message----- > From: Grzegorz Bernacki > Sent: Wednesday, July 14, 2021 8:31 PM > To: devel@edk2.groups.io > Cc: leif@nuviainc.com; ardb+tianocore@kernel.org; Samer El-Haj-Mahmoud > ; Sunny Wang ; > mw@semihalf.com; upstream@semihalf.com; jiewen.yao@intel.com; > jian.j.wang@intel.com; min.m.xu@intel.com; lersek@redhat.com; Sami > Mujawar ; afish@apple.com; ray.ni@intel.com; > jordan.l.justen@intel.com; rebecca@bsdio.com; grehan@freebsd.org; > Thomas Abraham ; chasel.chiu@intel.com; > nathaniel.l.desimone@intel.com; gaoliming@byosoft.com.cn; > eric.dong@intel.com; michael.d.kinney@intel.com; = zailiang.sun@intel.com; > yi.qian@intel.com; graeme@nuviainc.com; rad@semihalf.com; = pete@akeo.ie; > Grzegorz Bernacki > Subject: [edk2-platforms PATCH v6 0/4] Secure Boot default keys >=20 > This patchset is a consequence of "Secure Boot default keys" > patchset in edk2. It adds SecureBootVariableLib class resolution > for each platform which uses SecureBootConfigDxe and also > enables Secure Boot variables initialization for RPi4. > Previously these commits were part of edk2 patchset, but since > number of commits increased in v5 version, it is now separate > patchset. >=20 > Changes related to both edk2 & edk-platform versions: > Changes since v1: > - change names: > SecBootVariableLib =3D> SecureBootVariableLib > SecBootDefaultKeysDxe =3D> SecureBootDefaultKeysDxe > SecEnrollDefaultKeysApp =3D> EnrollFromDefaultKeysApp > - change name of function CheckSetupMode to GetSetupMode > - remove ShellPkg dependecy from EnrollFromDefaultKeysApp > - rebase to master >=20 > Changes since v2: > - fix coding style for functions headers in SecureBootVariableLib.h > - add header to SecureBootDefaultKeys.fdf.inc > - remove empty line spaces in SecureBootDefaultKeysDxe files > - revert FAIL macro in EnrollFromDefaultKeysApp > - remove functions duplicates and add SecureBootVariableLib > to platforms which used it >=20 > Changes since v3: > - move SecureBootDefaultKeys.fdf.inc to ArmPlatformPkg > - leave duplicate of CreateTimeBasedPayload in PlatformVarCleanupLib > - fix typo in guid description >=20 > Changes since v4: > - reorder patches to make it bisectable > - split commits related to more than one platform > - move edk2-platform commits to separate patchset >=20 > Changes since v5: > - split SecureBootVariableLib into SecureBootVariableLib and > SecureBootVariableProvisionLib >=20 > Grzegorz Bernacki (4): > Intel Platforms: add SecureBootVariableLib class resolution > ARM Silicon and Platforms: add SecureBootVariableLib class = resolution > RISC-V Platforms: add SecureBootVariableLib class resolution > Platform/RaspberryPi: Enable default Secure Boot variables > initialization >=20 > Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc > | 2 ++ > Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc > | 2 ++ > Platform/Intel/QuarkPlatformPkg/Quark.dsc > | 2 ++ > Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc > | 2 ++ > Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc > | 2 ++ > Platform/Qemu/SbsaQemu/SbsaQemu.dsc > | 2 ++ > Platform/RaspberryPi/RPi3/RPi3.dsc > | 2 ++ > Platform/RaspberryPi/RPi4/RPi4.dsc > | 4 ++++ > Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc > | 2 ++ >=20 > Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc > | 2 ++ > Platform/Socionext/DeveloperBox/DeveloperBox.dsc > | 5 +++++ > Platform/RaspberryPi/RPi4/RPi4.fdf > | 2 ++ > 12 files changed, 29 insertions(+) >=20 > -- > 2.25.1 >=20 > 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.