Yes. Merged. 499c4608b13df0738213329722cdcb0ac838f3ad.. bd298d75934a7a0cf6b9e2863a5e4975e9714285
> -----Original Message----- > From: Stefan Berger <stef...@linux.ibm.com> > Sent: Thursday, September 30, 2021 3:15 AM > To: devel@edk2.groups.io; Yao, Jiewen <jiewen....@intel.com> > Cc: mhaeu...@posteo.de; spbro...@outlook.com; > marcandre.lur...@redhat.com; kra...@redhat.com > Subject: Re: [edk2-devel] [PATCH v2 0/4] OvmfPkg: Disable the TPM 2 platform > hierarchy > > > On 9/14/21 6:26 PM, Yao, Jiewen wrote: > > Reviewed-by: Jiewen Yao <jiewen....@intel.com> > > > > I will wait for a week, to see if there is any feedback from AMD or Bhyve > reviewer. > > I didn't see any more comments to v3 > (https://listman.redhat.com/archives/edk2-devel-archive/2021- > September/msg00641.html) > So maybe you can apply it to the repo. > > > Stefan > > > > > > Thank you > > Yao Jiewen > > > > > >> -----Original Message----- > >> From: Stefan Berger <stef...@linux.ibm.com> > >> Sent: Tuesday, September 14, 2021 10:18 PM > >> To: devel@edk2.groups.io > >> Cc: mhaeu...@posteo.de; spbro...@outlook.com; > >> marcandre.lur...@redhat.com; kra...@redhat.com; Yao, Jiewen > >> <jiewen....@intel.com>; Stefan Berger <stef...@linux.ibm.com> > >> Subject: [PATCH v2 0/4] OvmfPkg: Disable the TPM 2 platform hierarchy > >> > >> This series of patches adds support for disabling the TPM 2 platform > >> hierarchy to Ovmf. To be able to do this we have to handle TPM 2 > >> physical presence interface (PPI) opcodes before the TPM 2 platform > >> hierarchy is disabled otherwise TPM 2 commands that are sent due to the > >> PPI opcodes may fail if the platform hierarchy is already disabled. > >> Therefore, we need to invoke the handler function > >> Tcg2PhysicalPresenceLibProcessRequest from within > >> PlatformBootManagerBeforeConsole. Since handling of PPI opcodes may > >> require > >> interaction with the user, we also move PlatformInitializeConsole > >> to before the handling of PPI codes so that the keyboard is available > >> when needed. The PPI handling code will activate the default consoles > >> only if it requires user interaction. > >> > >> Regards, > >> Stefan > >> > >> v2: > >> - 1/4: Added missing link library > >> - 2/4: Modified other BdsPlatform.c files as well > >> - Added Yao's comments to 1/2 and 2/2 > >> > >> Stefan Berger (4): > >> OvmfPkg/TPM PPI: Connect default consoles for user interaction > >> OvmfPkg: Handle TPM 2 physical presence opcodes much earlier > >> OvmfPkg: Reference new Tcg2PlatformDxe in the build system for > >> compilation > >> OvmfPkg: Reference new Tcg2PlatformPei in the build system > >> > >> OvmfPkg/AmdSev/AmdSevX64.dsc | 8 ++++++++ > >> OvmfPkg/AmdSev/AmdSevX64.fdf | 2 ++ > >> .../PlatformBootManagerLib/BdsPlatform.c | 19 +++++++++++-------- > >> .../PlatformBootManagerLibBhyve/BdsPlatform.c | 16 +++++++++------- > >> .../PlatformBootManagerLibGrub/BdsPlatform.c | 16 +++++++++------- > >> .../DxeTcg2PhysicalPresenceLib.c | 5 +++++ > >> .../DxeTcg2PhysicalPresenceLib.inf | 1 + > >> OvmfPkg/OvmfPkgIa32.dsc | 8 ++++++++ > >> OvmfPkg/OvmfPkgIa32.fdf | 2 ++ > >> OvmfPkg/OvmfPkgIa32X64.dsc | 8 ++++++++ > >> OvmfPkg/OvmfPkgIa32X64.fdf | 2 ++ > >> OvmfPkg/OvmfPkgX64.dsc | 8 ++++++++ > >> OvmfPkg/OvmfPkgX64.fdf | 2 ++ > >> 13 files changed, 75 insertions(+), 22 deletions(-) > >> > >> -- > >> 2.31.1 > > > > > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#81294): https://edk2.groups.io/g/devel/message/81294 Mute This Topic: https://groups.io/mt/85602992/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
