[edk2-devel] [PATCH v2 0/4] OvmfPkg: Disable the TPM 2 platform hierarchy
Yao, Jiewen
jiewen.yao at intel.com
Thu Sep 30 00:01:56 UTC 2021
Yes. Merged.
499c4608b13df0738213329722cdcb0ac838f3ad.. bd298d75934a7a0cf6b9e2863a5e4975e9714285
> -----Original Message-----
> From: Stefan Berger <stefanb at linux.ibm.com>
> Sent: Thursday, September 30, 2021 3:15 AM
> To: devel at edk2.groups.io; Yao, Jiewen <jiewen.yao at intel.com>
> Cc: mhaeuser at posteo.de; spbrogan at outlook.com;
> marcandre.lureau at redhat.com; kraxel at 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.yao at 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 <stefanb at linux.ibm.com>
> >> Sent: Tuesday, September 14, 2021 10:18 PM
> >> To: devel at edk2.groups.io
> >> Cc: mhaeuser at posteo.de; spbrogan at outlook.com;
> >> marcandre.lureau at redhat.com; kraxel at redhat.com; Yao, Jiewen
> >> <jiewen.yao at intel.com>; Stefan Berger <stefanb at 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/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list