REF: https://bugzilla.tianocore.org/show_bug.cgi?id=2443
1. Add two interfaces Tcg2PpVendorLibExecutePendingRequestEx and Tcg2PpVendorLibSubmitRequestToPreOSFunctionEx to Tcg2PpVendorLib. It has one more parameter PPData (type EFI_TCG2_PHYSICAL_PRESENCE) to transfer more data. 2. Use the Ex version instead of original one in Tcg2PhysicalPresenceLib 3. Add a pcd PcdPhysicalPresenceUserConfirmTimeout to control the user confirm input key timeout. 4. Add FunctionIndex to structure type EFI_TCG2_PHYSICAL_PRESENCE to transfer mTcgNvs->PhysicalPresence.Parameter data. 5. Add parameter FunctionIndex to Tcg2PhysicalPresenceLibSubmitRequestToPreOSFunctionEx to initialize the PPdata. 6. Move Tcg2ConfigNvData.h from SecurityPkg/Tcg/Tcg2Config to SecurityPkg/Include. It is useful for platform code to implement their own Tcg2PhysicalPresenceLib. 7. Replace the ASSERT with error code return in TpmPhysicalPresenceLib 8. Fix one operation (PHYSICAL_PRESENCE_DEACTIVATE_DISABLE_OWNER_FALSE) flow of TcgPhysicalPresenceLib (refer to Physical Presence Interface Spec Page 37). Cc: Jiewen Yao <jiewen....@intel.com> Cc: Jian J Wang <jian.j.w...@intel.com> Cc: Chao Zhang <chao.b.zh...@intel.com> Cc: Jordan Justen <jordan.l.jus...@intel.com> Cc: Laszlo Ersek <ler...@redhat.com> Cc: Ard Biesheuvel <ard.biesheu...@linaro.org> Cc: Marc-André Lureau <marcandre.lur...@redhat.com> Cc: Stefan Berger <stef...@linux.ibm.com> Signed-off-by: Zhichao Gao <zhichao....@intel.com> Zhichao Gao (13): SecurityPkg/Tcg2PpVerndorLib: Add two Ex function to handle PPdata SecurityPkg/Tcg2PpVendorLib: Add implementation of new Ex function SecurityPkg/Tcg2PhysicalPresenceLib: Use the new Ex function SecurityPkg/SmmTcg2PhysicalPresenceLib: Use the new Ex function SecurityPkg/dec: Add a pcd for user response wait time OvmfPkg/Tcg2PhysicalPresenceLib: Use pcd for user response wait time SecurityPkg/Tcg2PhysicalPresenceLib: Use Pcd for user resp wait time SecurityPkg/TcgPyhsicalPresenceLib: Use Pcd for user resp wait time SecurityPkg/Tcg2PhysicalPresenceData.h: Add FunctionIndex for PPdata SecurityPkg/Tcg2PhysicalPresenceLib: Extend the submit preOS func SecurityPkg: Move the Tcg2ConfigNvData.h to Include folder SecurityPkg/TcgPhysicalPresenceLib: Replace the ASSERT with error code SecurityPkg/TcgPhysicalPresenceLib: Fix the operation of 11 .../DxeTcg2PhysicalPresenceLib.c | 63 +++++++--- .../DxeTcg2PhysicalPresenceLib.inf | 6 +- .../Include/Guid/Tcg2PhysicalPresenceData.h | 3 +- .../Include/Library/Tcg2PhysicalPresenceLib.h | 4 +- SecurityPkg/Include/Library/Tcg2PpVendorLib.h | 54 ++++++++- .../Tcg2Config => Include}/Tcg2ConfigNvData.h | 2 +- .../DxeTcg2PhysicalPresenceLib.c | 68 ++++++++--- .../DxeTcg2PhysicalPresenceLib.inf | 4 +- .../DxeTcgPhysicalPresenceLib.c | 110 ++++++++++++------ .../DxeTcgPhysicalPresenceLib.inf | 6 +- .../SmmTcg2PhysicalPresenceLib.c | 15 ++- .../Tcg2PpVendorLibNull/Tcg2PpVendorLibNull.c | 61 +++++++++- SecurityPkg/SecurityPkg.dec | 7 +- SecurityPkg/SecurityPkg.uni | 7 +- SecurityPkg/Tcg/Tcg2Config/Tcg2Config.vfr | 4 +- SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigDxe.inf | 3 +- SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigImpl.h | 4 +- SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigPei.inf | 3 +- SecurityPkg/Tcg/Tcg2Config/Tcg2ConfigPeim.c | 4 +- SecurityPkg/Tcg/Tcg2Config/TpmDetection.c | 4 +- SecurityPkg/Tcg/Tcg2Smm/Tcg2Smm.c | 10 +- 21 files changed, 347 insertions(+), 95 deletions(-) rename SecurityPkg/{Tcg/Tcg2Config => Include}/Tcg2ConfigNvData.h (94%) -- 2.21.0.windows.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#52707): https://edk2.groups.io/g/devel/message/52707 Mute This Topic: https://groups.io/mt/69392326/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-