<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="#0563C1" vlink="#954F72" style="word-wrap:break-word"> <div class="WordSection1"> Hello<o:p></o:p> I agree with you on the problem statement.<o:p></o:p> <o:p> </o:p> But I don’t think this is a desired solution.<o:p></o:p> We expect Platform BDS to call the PhysicalPresenceLib. As such, why not clear the screen in the platform BDS?<o:p></o:p> <o:p> </o:p> Thank you<o:p></o:p> Yao, Jiewen<o:p></o:p> <o:p> </o:p> <div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: Poosapalli, Karunakar <KarunakarPoosapalli@Dell.com> Sent: Monday, July 24, 2023 11:26 PM To: Yao, Jiewen <jiewen.yao@intel.com>; Gao, Liming <gaoliming@byosoft.com.cn>; devel@edk2.groups.io Subject: RE: [edk2-devel] [PATCH] SecurityPkg: DxeTcg2PhysicalPresenceLib for Clear Graphics Screen To unblock and Display TPM messages <o:p></o:p> </div> </div> <o:p> </o:p> Can you please review and share your feedback?<o:p></o:p> <o:p> </o:p> <div> <o:p> </o:p> Thanks & Regards<o:p></o:p> Karunakar Poosapalli<o:p></o:p> Firmware Principal Engineer, Client BIOS<o:p></o:p> Customer BIOS | Dell Core BIOS<o:p></o:p> CPG Software Engineering | Dell Technologies<o:p></o:p> Mobile +91 9951902957<o:p></o:p> <a href="mailto:Karunakar_poosapalli@Dell.Com">Karunakar_poosapalli@Dell.Com</a><o:p></o:p> </div> <o:p> </o:p> <o:p> </o:p> Internal Use - Confidential<o:p></o:p> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> From: Poosapalli, Karunakar Sent: Saturday, July 22, 2023 1:21 AM To: <a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>; gaoliming; <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a> Subject: [edk2-devel] [PATCH] SecurityPkg: DxeTcg2PhysicalPresenceLib for Clear Graphics Screen To unblock and Display TPM messages <o:p></o:p> </div> </div> <o:p> </o:p> Patch review for Bugzilla - <a href="https://bugzilla.tianocore.org/show_bug.cgi?id=4462">https://bugzilla.tianocore.org/show_bug.cgi?id=4462</a> <o:p></o:p> <o:p> </o:p> From c537f9c5c9e02c54e27466b96fe33555afccd358 Mon Sep 17 00:00:00 2001<o:p></o:p> From: Karunakar Poosapalli <a href="mailto:karunakar_poosapalli@dell.com"> karunakar_poosapalli@dell.com</a><o:p></o:p> Date: Sat, 22 Jul 2023 01:13:44 +0530<o:p></o:p> Subject: [PATCH] Patch - Enhance Tcg2 to clear graphics before printing the<o:p></o:p> messages on screen<o:p></o:p> <o:p> </o:p> [Background]<o:p></o:p> Tcg2UserConfirm() Function is used to display any user conformation messages on the console<o:p></o:p> Function Definition Full path - SecurityPkg\Library\DxeTcg2PhysicalPresenceLib\DxeTcg2PhysicalPresenceLib.c<o:p></o:p> <o:p> </o:p> [Issue]<o:p></o:p> In the current Tcg2UserConfirm() implementation, This function forms a destination string to be displayed on the console and directly Print the message on Console.<o:p></o:p> <o:p> </o:p> But there is no logic added to clear the graphics before printing the messages on the screen.<o:p></o:p> <o:p> </o:p> There are some scenarios where Tcg messages might have been blocked by some other GUI or messages on Console.<o:p></o:p> 1. When there are some messages or logos already displayed in the content on the console,<o:p></o:p> TCG message will NOT be displayed or corrupted to the End user.<o:p></o:p> 2. There could be a Custom logo displaying on the screen which actually blocks the screen.<o:p></o:p> <o:p> </o:p> [Solution]<o:p></o:p> 1. As TCG user confirmation is the highest priority and it blocks the POST until the user<o:p></o:p> presses the input key. Before TCG messages Print on the console, there should be logic<o:p></o:p> added to clear the graphics screen<o:p></o:p> <o:p> </o:p> Cc: gaoliming <a href="mailto:gaoliming@byosoft.com.cn">gaoliming@byosoft.com.cn</a><o:p></o:p> Cc: Jiewen <a href="mailto:Jiewen.yao@intel.com">Jiewen.yao@intel.com</a><o:p></o:p> REF: <a href="https://bugzilla.tianocore.org/show_bug.cgi?id=4462"> https://bugzilla.tianocore.org/show_bug.cgi?id=4462</a><o:p></o:p> Signed-off-by: Karunakar Poosapalli <a href="mailto:karunakar_poosapalli@dell.com"> karunakar_poosapalli@dell.com</a><o:p></o:p> ---<o:p></o:p> .../DxeTcg2PhysicalPresenceLib.c | 74 +++++++++++++++++++<o:p></o:p> .../DxeTcg2PhysicalPresenceLib.inf | 1 +<o:p></o:p> 2 files changed, 75 insertions(+)<o:p></o:p> <o:p> </o:p> diff --git a/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.c b/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.c<o:p></o:p> index de4f5e583d..bd486f3b5b 100644<o:p></o:p> --- a/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.c<o:p></o:p> +++ b/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.c<o:p></o:p> @@ -251,6 +251,74 @@ Tcg2ExecutePhysicalPresence (<o:p></o:p> }<o:p></o:p> }<o:p></o:p> <o:p> </o:p> +/**<o:p></o:p> + Clear Graphics Screen To unblock and Display TPM messages <o:p></o:p> +<o:p></o:p> + @param[in] VOID<o:p></o:p> + @retval EFI_STATUS<o:p></o:p> +**/<o:p></o:p> +EFI_STATUS <o:p></o:p> +ClearGraphicsScreenToDisplayTpmMessages()<o:p></o:p> +{<o:p></o:p> + EFI_GRAPHICS_OUTPUT_PROTOCOL *Gop;<o:p></o:p> + EFI_STATUS Status;<o:p></o:p> + EFI_HANDLE *HandleBuffer;<o:p></o:p> + UINTN NumberOfHandles;<o:p></o:p> + UINTN Index;<o:p></o:p> + EFI_DEVICE_PATH_PROTOCOL *GopDevicePath;<o:p></o:p> +<o:p></o:p> + DEBUG((DEBUG_INFO, "ClearGraphicsScreenToDisplayTpmMessages Entry...\n"));<o:p></o:p> +<o:p></o:p> + Status = gBS->LocateHandleBuffer (<o:p></o:p> + ByProtocol,<o:p></o:p> + &gEfiGraphicsOutputProtocolGuid,<o:p></o:p> + NULL,<o:p></o:p> + &NumberOfHandles,<o:p></o:p> + &HandleBuffer<o:p></o:p> + );<o:p></o:p> + DEBUG((DEBUG_INFO, "LocateHandleBuffer Status = %r, NumberOfHandles = %x\n", Status, NumberOfHandles)); <o:p></o:p> +<o:p></o:p> + if (EFI_ERROR (Status))<o:p></o:p> + {<o:p></o:p> + return Status;<o:p></o:p> + }<o:p></o:p> + <o:p></o:p> + for (Index = 0; Index < NumberOfHandles; Index++)<o:p></o:p> + {<o:p></o:p> + Status = gBS->HandleProtocol (<o:p></o:p> + HandleBuffer[Index],<o:p></o:p> + &gEfiDevicePathProtocolGuid,<o:p></o:p> + (VOID *)&GopDevicePath<o:p></o:p> + );<o:p></o:p> + DEBUG((DEBUG_INFO, "HandleProtocol GopDevicePath Status = %r, Index = %x\n", Status, Index));<o:p></o:p> + if (EFI_ERROR (Status))<o:p></o:p> + {<o:p></o:p> + continue;<o:p></o:p> + }<o:p></o:p> +<o:p></o:p> + Status = gBS->HandleProtocol (<o:p></o:p> + HandleBuffer[Index],<o:p></o:p> + &gEfiGraphicsOutputProtocolGuid,<o:p></o:p> + (VOID **) &Gop<o:p></o:p> + );<o:p></o:p> + DEBUG((DEBUG_INFO, "HandleProtocol Gop Status = %r, Index = %x\n", Status, Index));<o:p></o:p> + if (EFI_ERROR(Status))<o:p></o:p> + {<o:p></o:p> + continue;<o:p></o:p> + }<o:p></o:p> +<o:p></o:p> + // Clear the graphics screen to black<o:p></o:p> + Status = Gop->Blt(Gop, NULL, EfiBltVideoFill, 0, 0, 0, 0, Gop->Mode->Info->HorizontalResolution, Gop->Mode->Info->VerticalResolution, 0);<o:p></o:p> + DEBUG((DEBUG_INFO, "Gop->Blt Status = %r, Index = %x\n", Status, Index));<o:p></o:p> + if (EFI_ERROR(Status)) {<o:p></o:p> + continue;<o:p></o:p> + }<o:p></o:p> + }<o:p></o:p> +<o:p></o:p> + gBS->FreePool (HandleBuffer);<o:p></o:p> + DEBUG((DEBUG_INFO, "ClearGraphicsScreenToDisplayTpmMessages Exit...\n"));<o:p></o:p> + return Status;<o:p></o:p> +}<o:p></o:p> /**<o:p></o:p> Read the specified key for user confirmation.<o:p></o:p> <o:p> </o:p> @@ -576,6 +644,12 @@ Tcg2UserConfirm (<o:p></o:p> BufSize -= StrSize (ConfirmText);<o:p></o:p> UnicodeSPrint (ConfirmText + StrLen (ConfirmText), BufSize, TmpStr1, TmpStr2);<o:p></o:p> <o:p> </o:p> + //<o:p></o:p> + //Clear Graphics Screen To unblock and Display TPM messages <o:p></o:p> + // <o:p></o:p> + Status = ClearGraphicsScreenToDisplayTpmMessages();<o:p></o:p> + DEBUG((DEBUG_INFO, "ClearGraphicsScreenToDisplayTpmMessages Status = %r\n", Status));<o:p></o:p> +<o:p></o:p> DstStr[80] = L'\0';<o:p></o:p> for (Index = 0; Index < StrLen (ConfirmText); Index += 80) {<o:p></o:p> StrnCpyS (DstStr, sizeof (DstStr) / sizeof (CHAR16), ConfirmText + Index, sizeof (DstStr) / sizeof (CHAR16) - 1);<o:p></o:p> diff --git a/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.inf b/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.inf<o:p></o:p> index e1c7c20d52..de423cfd13 100644<o:p></o:p> --- a/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.inf<o:p></o:p> +++ b/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.inf<o:p></o:p> @@ -54,6 +54,7 @@<o:p></o:p> [Protocols]<o:p></o:p> gEfiTcg2ProtocolGuid ## SOMETIMES_CONSUMES<o:p></o:p> gEdkiiVariableLockProtocolGuid ## SOMETIMES_CONSUMES<o:p></o:p> + gEfiGraphicsOutputProtocolGuid ## CONSUMES<o:p></o:p> <o:p> </o:p> [Pcd]<o:p></o:p> gEfiSecurityPkgTokenSpaceGuid.PcdTcg2PhysicalPresenceFlags ## SOMETIMES_CONSUMES<o:p></o:p> -- <o:p></o:p> 2.17.0.windows.1<o:p></o:p> <o:p> </o:p> <o:p> </o:p> Internal Use - Confidential<o:p></o:p> </div> </div> </body> </html> <div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links: You receive all messages sent to this group. <a target="_blank" href="https://edk2.groups.io/g/devel/message/107244">View/Reply Online (#107244)</a> | | <a target="_blank" href="https://groups.io/mt/100333271/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com] <div width="1" style="color:white;clear:both">_._,_._,_</div>