<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><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:DengXian;
        panose-1:2 1 6 0 3 1 1 1 1 1;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:"\@DengXian";
        panose-1:2 1 6 0 3 1 1 1 1 1;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:#0563C1;
        text-decoration:underline;}
p.msipfooter90245289, li.msipfooter90245289, div.msipfooter90245289
        {mso-style-name:msipfooter90245289;
        mso-margin-top-alt:auto;
        margin-right:0in;
        mso-margin-bottom-alt:auto;
        margin-left:0in;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
span.EmailStyle19
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="#0563C1" vlink="#954F72" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoNormal">Thanks for sharing your inputs Jiewen.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Dear TCG Experts,<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Can you please share your feedback on below proposals<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838"><o:p> </o:p></span></b></p>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838">Thanks & Regards<o:p></o:p></span></b></p>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838">Karunakar Poosapalli<o:p></o:p></span></b></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171">Firmware Principal Engineer, Client BIOS<o:p></o:p></span></p>
<p class="MsoNormal"><b><i><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#7F7F7F">Customer BIOS | Dell Core BIOS</span></i></b><b><i><span style="font-size:9.0pt;color:#7F7F7F"><o:p></o:p></span></i></b></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0070C0">CPG Software Engineering</span><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171"> |</span><b><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0076CE">
 Dell Technologies</span></b><span style="font-size:8.0pt;font-family:"Arial",sans-serif"><o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171">Mobile +91 9951902957<o:p></o:p></span></p>
<p class="MsoNormal"><i><u><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0070C0"><a href="mailto:Karunakar_poosapalli@Dell.Com">Karunakar_poosapalli@Dell.Com</a><o:p></o:p></span></u></i></p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="msipfooter90245289" style="margin:0in"><span style="font-size:7.0pt;color:#737373">Internal Use - Confidential</span><o:p></o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> Yao, Jiewen <jiewen.yao@intel.com> <br>
<b>Sent:</b> Thursday, August 17, 2023 8:30 AM<br>
<b>To:</b> Poosapalli, Karunakar; Gao, Liming; devel@edk2.groups.io<br>
<b>Subject:</b> RE: [edk2-devel] [PATCH] SecurityPkg: DxeTcg2PhysicalPresenceLib for Clear Graphics Screen To unblock and Display TPM messages
<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p><span style="color:#CE1126">[EXTERNAL EMAIL] <o:p></o:p></span></p>
</div>
<p class="MsoNormal">Hi Karunakar<o:p></o:p></p>
<p class="MsoNormal">Thanks for the patient.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">I think my concern is universal, no matter it is after user input the key or not.<o:p></o:p></p>
<p class="MsoNormal">The clear-screen behavior is a *<b>change</b>*. And it may break the compatibility.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Unless all BIOS consumer in EDKII community agree the behavior, the risk is always there.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">To move forward, I would like to hear the feedback from other OEM/ODM/IBV, who is consuming DxeTcg2PhysicalPresenceLib.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thank you<o:p></o:p></p>
<p class="MsoNormal">Yao, Jiewen<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></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">
<p class="MsoNormal"><b>From:</b> Poosapalli, Karunakar <<a href="mailto:KarunakarPoosapalli@Dell.com">KarunakarPoosapalli@Dell.com</a>>
<br>
<b>Sent:</b> Thursday, August 17, 2023 1:28 AM<br>
<b>To:</b> Yao, Jiewen <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>>; Gao, Liming <<a href="mailto:gaoliming@byosoft.com.cn">gaoliming@byosoft.com.cn</a>>;
<a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a><br>
<b>Subject:</b> RE: [edk2-devel] [PATCH] SecurityPkg: DxeTcg2PhysicalPresenceLib for Clear Graphics Screen To unblock and Display TPM messages
<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Hi <a id="OWAAM333B815A889D4FA4B2F9C4649AC63930" href="mailto:jiewen.yao@intel.com">
<span style="font-family:"Calibri",sans-serif;text-decoration:none">@Yao, Jiewen</span></a>,
<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Could you please share your thoughts. Please let me know if you have any queries or concerns.<o:p></o:p></p>
<div>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838"><o:p> </o:p></span></b></p>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838">Thanks & Regards<o:p></o:p></span></b></p>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838">Karunakar Poosapalli<o:p></o:p></span></b></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171">Firmware Principal Engineer, Client BIOS<o:p></o:p></span></p>
<p class="MsoNormal"><b><i><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#7F7F7F">Customer BIOS | Dell Core BIOS</span></i></b><b><i><span style="font-size:9.0pt;color:#7F7F7F"><o:p></o:p></span></i></b></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0070C0">CPG Software Engineering</span><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171"> |</span><b><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0076CE">
 Dell Technologies</span></b><span style="font-size:8.0pt;font-family:"Arial",sans-serif"><o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171">Mobile +91 9951902957<o:p></o:p></span></p>
<p class="MsoNormal"><i><u><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0070C0"><a href="mailto:Karunakar_poosapalli@Dell.Com">Karunakar_poosapalli@Dell.Com</a><o:p></o:p></span></u></i></p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="msipfooter90245289" style="margin:0in"><span style="font-size:7.0pt;color:#737373">Internal Use - Confidential</span><o:p></o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> Poosapalli, Karunakar <br>
<b>Sent:</b> Wednesday, August 9, 2023 9:59 AM<br>
<b>To:</b> Yao, Jiewen; Gao, Liming; <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a><br>
<b>Subject:</b> RE: [edk2-devel] [PATCH] SecurityPkg: DxeTcg2PhysicalPresenceLib for Clear Graphics Screen To unblock and Display TPM messages
<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Hi <a id="OWAAMA28BC70566604087BBDBE0893B3AB6EE" href="mailto:jiewen.yao@intel.com">
<span style="font-family:"Calibri",sans-serif;text-decoration:none">@Yao, Jiewen</span></a>,<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">The proposed solution to clear screen will be called only when user has to provide the input key. This call will not execute in other conditions.<o:p></o:p></p>
<p class="MsoNormal">When there is a pending request in Tcg2, system will wait until user press input key. If there is no proper UI to customer, they feel it’s system hang and go for customer support.
<o:p></o:p></p>
<p class="MsoNormal">Even BIOS vendors also won’t come to what is happening at customer box without any user information.
<o:p></o:p></p>
<p class="MsoNormal">This is generic issue and not specific to any platform.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thanks for sharing your thoughts.<o:p></o:p></p>
<div>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838"><o:p> </o:p></span></b></p>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838">Thanks & Regards<o:p></o:p></span></b></p>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838">Karunakar Poosapalli<o:p></o:p></span></b></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171">Firmware Principal Engineer, Client BIOS<o:p></o:p></span></p>
<p class="MsoNormal"><b><i><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#7F7F7F">Customer BIOS | Dell Core BIOS</span></i></b><b><i><span style="font-size:9.0pt;color:#7F7F7F"><o:p></o:p></span></i></b></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0070C0">CPG Software Engineering</span><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171"> |</span><b><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0076CE">
 Dell Technologies</span></b><span style="font-size:8.0pt;font-family:"Arial",sans-serif"><o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171">Mobile +91 9951902957<o:p></o:p></span></p>
<p class="MsoNormal"><i><u><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0070C0"><a href="mailto:Karunakar_poosapalli@Dell.Com">Karunakar_poosapalli@Dell.Com</a><o:p></o:p></span></u></i></p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="msipfooter90245289" style="margin:0in"><span style="font-size:7.0pt;color:#737373">Internal Use - Confidential</span><o:p></o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> Yao, Jiewen <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>>
<br>
<b>Sent:</b> Wednesday, August 9, 2023 4:44 AM<br>
<b>To:</b> Poosapalli, Karunakar; Gao, Liming; <a href="mailto:devel@edk2.groups.io">
devel@edk2.groups.io</a><br>
<b>Cc:</b> Yao, Jiewen<br>
<b>Subject:</b> RE: [edk2-devel] [PATCH] SecurityPkg: DxeTcg2PhysicalPresenceLib for Clear Graphics Screen To unblock and Display TPM messages
<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p><span style="color:#CE1126">[EXTERNAL EMAIL] <o:p></o:p></span></p>
</div>
<p class="MsoNormal">My concern is that you unconditionally clear the screen. What if someone did want to show something on the screen?<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">This seems an incompatible change. That is why I think it should be a platform policy.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Another way is that you may consider to duplicate the library for your platform and clear it for your platform.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thank you<o:p></o:p></p>
<p class="MsoNormal">Yao, Jiewen<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></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">
<p class="MsoNormal"><b>From:</b> Poosapalli, Karunakar <<a href="mailto:KarunakarPoosapalli@Dell.com">KarunakarPoosapalli@Dell.com</a>>
<br>
<b>Sent:</b> Wednesday, August 9, 2023 2:13 AM<br>
<b>To:</b> Yao, Jiewen <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>>; Gao, Liming <<a href="mailto:gaoliming@byosoft.com.cn">gaoliming@byosoft.com.cn</a>>;
<a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a>; Poosapalli, Karunakar <<a href="mailto:KarunakarPoosapalli@Dell.com">KarunakarPoosapalli@Dell.com</a>><br>
<b>Subject:</b> RE: [edk2-devel] [PATCH] SecurityPkg: DxeTcg2PhysicalPresenceLib for Clear Graphics Screen To unblock and Display TPM messages
<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Hi <a id="OWAAM4A973D9361764C039B66B1D68449BDB3" href="mailto:jiewen.yao@intel.com">
<span style="font-family:"Calibri",sans-serif;text-decoration:none">@Yao, Jiewen</span></a>,<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thanks for your review and feedback.<o:p></o:p></p>
<p class="MsoNormal">When there is a pending Tcg request, the control will be in Tcg2 library and it will not reach to BDS until user press the input key.<o:p></o:p></p>
<p class="MsoNormal">As there is no information the screen because of screen is blocked by some other messages/Logo.
<o:p></o:p></p>
<p class="MsoNormal">So user won’t able to press the key without any user information and control will not reach to BDS phase.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">As TCG user confirmation is the highest priority and it blocks the POST until the user presses the input key. Before TCG messages Print on the console, there should be logic added to clear the graphics screen<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Please let me know your thoughts. <o:p></o:p></p>
<div>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838"><o:p> </o:p></span></b></p>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838">Thanks & Regards<o:p></o:p></span></b></p>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838">Karunakar Poosapalli<o:p></o:p></span></b></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171">Firmware Principal Engineer, Client BIOS<o:p></o:p></span></p>
<p class="MsoNormal"><b><i><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#7F7F7F">Customer BIOS | Dell Core BIOS</span></i></b><b><i><span style="font-size:9.0pt;color:#7F7F7F"><o:p></o:p></span></i></b></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0070C0">CPG Software Engineering</span><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171"> |</span><b><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0076CE">
 Dell Technologies</span></b><span style="font-size:8.0pt;font-family:"Arial",sans-serif"><o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171">Mobile +91 9951902957<o:p></o:p></span></p>
<p class="MsoNormal"><i><u><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0070C0"><a href="mailto:Karunakar_poosapalli@Dell.Com">Karunakar_poosapalli@Dell.Com</a><o:p></o:p></span></u></i></p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="msipfooter90245289" style="margin:0in"><span style="font-size:7.0pt;color:#737373">Internal Use - Confidential</span><o:p></o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> Yao, Jiewen <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>>
<br>
<b>Sent:</b> Tuesday, July 25, 2023 5:48 PM<br>
<b>To:</b> Poosapalli, Karunakar; Gao, Liming; <a href="mailto:devel@edk2.groups.io">
devel@edk2.groups.io</a><br>
<b>Subject:</b> RE: [edk2-devel] [PATCH] SecurityPkg: DxeTcg2PhysicalPresenceLib for Clear Graphics Screen To unblock and Display TPM messages
<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p><span style="color:#CE1126">[EXTERNAL EMAIL] <o:p></o:p></span></p>
</div>
<p class="MsoNormal">Hello<o:p></o:p></p>
<p class="MsoNormal">I agree with you on the problem statement.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">But I don’t think this is a desired solution.<o:p></o:p></p>
<p class="MsoNormal">We expect Platform BDS to call the PhysicalPresenceLib. As such, why not clear the  screen in the platform BDS?<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Thank you<o:p></o:p></p>
<p class="MsoNormal">Yao, Jiewen<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></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">
<p class="MsoNormal"><b>From:</b> Poosapalli, Karunakar <<a href="mailto:KarunakarPoosapalli@Dell.com">KarunakarPoosapalli@Dell.com</a>>
<br>
<b>Sent:</b> Monday, July 24, 2023 11:26 PM<br>
<b>To:</b> Yao, Jiewen <<a href="mailto:jiewen.yao@intel.com">jiewen.yao@intel.com</a>>; Gao, Liming <<a href="mailto:gaoliming@byosoft.com.cn">gaoliming@byosoft.com.cn</a>>;
<a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a><br>
<b>Subject:</b> RE: [edk2-devel] [PATCH] SecurityPkg: DxeTcg2PhysicalPresenceLib for Clear Graphics Screen To unblock and Display TPM messages
<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Can you please review and share your feedback?<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838"><o:p> </o:p></span></b></p>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838">Thanks & Regards<o:p></o:p></span></b></p>
<p class="MsoNormal"><b><span style="font-size:9.0pt;color:#3B3838">Karunakar Poosapalli<o:p></o:p></span></b></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171">Firmware Principal Engineer, Client BIOS<o:p></o:p></span></p>
<p class="MsoNormal"><b><i><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#7F7F7F">Customer BIOS | Dell Core BIOS</span></i></b><b><i><span style="font-size:9.0pt;color:#7F7F7F"><o:p></o:p></span></i></b></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0070C0">CPG Software Engineering</span><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171"> |</span><b><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0076CE">
 Dell Technologies</span></b><span style="font-size:8.0pt;font-family:"Arial",sans-serif"><o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#767171">Mobile +91 9951902957<o:p></o:p></span></p>
<p class="MsoNormal"><i><u><span style="font-size:8.0pt;font-family:"Arial",sans-serif;color:#0070C0"><a href="mailto:Karunakar_poosapalli@Dell.Com">Karunakar_poosapalli@Dell.Com</a><o:p></o:p></span></u></i></p>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="msipfooter90245289" style="margin:0in"><span style="font-size:7.0pt;color:#737373">Internal Use - Confidential</span><o:p></o:p></p>
<div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in">
<p class="MsoNormal"><b>From:</b> Poosapalli, Karunakar <br>
<b>Sent:</b> Saturday, July 22, 2023 1:21 AM<br>
<b>To:</b> <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><br>
<b>Subject:</b> [edk2-devel] [PATCH] SecurityPkg: DxeTcg2PhysicalPresenceLib for Clear Graphics Screen To unblock and Display TPM messages
<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Patch review for Bugzilla -  <a href="https://urldefense.com/v3/__https:/bugzilla.tianocore.org/show_bug.cgi?id=4462__;!!LpKI!nIImq_Eo23HxKNbpCloDgiuP_82UEmFPLYEbrs-JRjj03rfEIF5h3MqG0lMX3qzdhStA76K3O2XvHbbFilQlXDMmwg$">https://bugzilla.tianocore.org/show_bug.cgi?id=4462
 [bugzilla.tianocore.org]</a> <o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">From c537f9c5c9e02c54e27466b96fe33555afccd358 Mon Sep 17 00:00:00 2001<o:p></o:p></p>
<p class="MsoNormal">From: Karunakar Poosapalli <a href="mailto:karunakar_poosapalli@dell.com">
karunakar_poosapalli@dell.com</a><o:p></o:p></p>
<p class="MsoNormal">Date: Sat, 22 Jul 2023 01:13:44 +0530<o:p></o:p></p>
<p class="MsoNormal">Subject: [PATCH] Patch - Enhance Tcg2 to clear graphics before printing the<o:p></o:p></p>
<p class="MsoNormal">messages on screen<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">[Background]<o:p></o:p></p>
<p class="MsoNormal">Tcg2UserConfirm() Function is used to display any user conformation messages on the console<o:p></o:p></p>
<p class="MsoNormal">Function Definition Full path - SecurityPkg\Library\DxeTcg2PhysicalPresenceLib\DxeTcg2PhysicalPresenceLib.c<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">[Issue]<o:p></o:p></p>
<p class="MsoNormal">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></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">But there is no logic added to clear the graphics before printing the messages on the screen.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">There are some scenarios where Tcg messages might have been blocked by some other GUI or messages on Console.<o:p></o:p></p>
<p class="MsoNormal">  1. When there are some messages or logos already displayed in the content on the console,<o:p></o:p></p>
<p class="MsoNormal">     TCG message will NOT be displayed or corrupted to the End user.<o:p></o:p></p>
<p class="MsoNormal">  2. There could be a Custom logo displaying on the screen which actually blocks the screen.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">[Solution]<o:p></o:p></p>
<p class="MsoNormal">  1. As TCG user confirmation is the highest priority and it blocks the POST until the user<o:p></o:p></p>
<p class="MsoNormal">     presses the input key. Before TCG messages Print on the console, there should be logic<o:p></o:p></p>
<p class="MsoNormal">     added to clear the graphics screen<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">Cc: gaoliming <a href="mailto:gaoliming@byosoft.com.cn">gaoliming@byosoft.com.cn</a><o:p></o:p></p>
<p class="MsoNormal">Cc: Jiewen  <a href="mailto:Jiewen.yao@intel.com">Jiewen.yao@intel.com</a><o:p></o:p></p>
<p class="MsoNormal">REF: <a href="https://urldefense.com/v3/__https:/bugzilla.tianocore.org/show_bug.cgi?id=4462__;!!LpKI!nIImq_Eo23HxKNbpCloDgiuP_82UEmFPLYEbrs-JRjj03rfEIF5h3MqG0lMX3qzdhStA76K3O2XvHbbFilQlXDMmwg$">
https://bugzilla.tianocore.org/show_bug.cgi?id=4462 [bugzilla.tianocore.org]</a><o:p></o:p></p>
<p class="MsoNormal">Signed-off-by: Karunakar Poosapalli <a href="mailto:karunakar_poosapalli@dell.com">
karunakar_poosapalli@dell.com</a><o:p></o:p></p>
<p class="MsoNormal">---<o:p></o:p></p>
<p class="MsoNormal">.../DxeTcg2PhysicalPresenceLib.c              | 74 +++++++++++++++++++<o:p></o:p></p>
<p class="MsoNormal">.../DxeTcg2PhysicalPresenceLib.inf            |  1 +<o:p></o:p></p>
<p class="MsoNormal">2 files changed, 75 insertions(+)<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">diff --git a/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.c b/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.c<o:p></o:p></p>
<p class="MsoNormal">index de4f5e583d..bd486f3b5b 100644<o:p></o:p></p>
<p class="MsoNormal">--- a/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.c<o:p></o:p></p>
<p class="MsoNormal">+++ b/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.c<o:p></o:p></p>
<p class="MsoNormal">@@ -251,6 +251,74 @@ Tcg2ExecutePhysicalPresence (<o:p></o:p></p>
<p class="MsoNormal">   }<o:p></o:p></p>
<p class="MsoNormal">}<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">+/**<o:p></o:p></p>
<p class="MsoNormal">+  Clear Graphics Screen To unblock and Display TPM messages 
<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+  @param[in]  VOID<o:p></o:p></p>
<p class="MsoNormal">+  @retval     EFI_STATUS<o:p></o:p></p>
<p class="MsoNormal">+**/<o:p></o:p></p>
<p class="MsoNormal">+EFI_STATUS <o:p></o:p></p>
<p class="MsoNormal">+ClearGraphicsScreenToDisplayTpmMessages()<o:p></o:p></p>
<p class="MsoNormal">+{<o:p></o:p></p>
<p class="MsoNormal">+  EFI_GRAPHICS_OUTPUT_PROTOCOL *Gop;<o:p></o:p></p>
<p class="MsoNormal">+  EFI_STATUS Status;<o:p></o:p></p>
<p class="MsoNormal">+  EFI_HANDLE                              *HandleBuffer;<o:p></o:p></p>
<p class="MsoNormal">+  UINTN                                   NumberOfHandles;<o:p></o:p></p>
<p class="MsoNormal">+  UINTN                                   Index;<o:p></o:p></p>
<p class="MsoNormal">+  EFI_DEVICE_PATH_PROTOCOL                *GopDevicePath;<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+  DEBUG((DEBUG_INFO, "ClearGraphicsScreenToDisplayTpmMessages Entry...\n"));<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+  Status = gBS->LocateHandleBuffer (<o:p></o:p></p>
<p class="MsoNormal">+            ByProtocol,<o:p></o:p></p>
<p class="MsoNormal">+            &gEfiGraphicsOutputProtocolGuid,<o:p></o:p></p>
<p class="MsoNormal">+            NULL,<o:p></o:p></p>
<p class="MsoNormal">+            &NumberOfHandles,<o:p></o:p></p>
<p class="MsoNormal">+            &HandleBuffer<o:p></o:p></p>
<p class="MsoNormal">+            );<o:p></o:p></p>
<p class="MsoNormal">+  DEBUG((DEBUG_INFO, "LocateHandleBuffer Status = %r, NumberOfHandles = %x\n", Status, NumberOfHandles));
<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+  if (EFI_ERROR (Status))<o:p></o:p></p>
<p class="MsoNormal">+  {<o:p></o:p></p>
<p class="MsoNormal">+    return Status;<o:p></o:p></p>
<p class="MsoNormal">+  }<o:p></o:p></p>
<p class="MsoNormal">+  <o:p></o:p></p>
<p class="MsoNormal">+  for (Index = 0; Index < NumberOfHandles; Index++)<o:p></o:p></p>
<p class="MsoNormal">+    {<o:p></o:p></p>
<p class="MsoNormal">+      Status = gBS->HandleProtocol (<o:p></o:p></p>
<p class="MsoNormal">+                HandleBuffer[Index],<o:p></o:p></p>
<p class="MsoNormal">+                &gEfiDevicePathProtocolGuid,<o:p></o:p></p>
<p class="MsoNormal">+                (VOID *)&GopDevicePath<o:p></o:p></p>
<p class="MsoNormal">+                );<o:p></o:p></p>
<p class="MsoNormal">+      DEBUG((DEBUG_INFO, "HandleProtocol GopDevicePath Status = %r, Index = %x\n", Status, Index));<o:p></o:p></p>
<p class="MsoNormal">+      if (EFI_ERROR (Status))<o:p></o:p></p>
<p class="MsoNormal">+        {<o:p></o:p></p>
<p class="MsoNormal">+          continue;<o:p></o:p></p>
<p class="MsoNormal">+        }<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+      Status = gBS->HandleProtocol (<o:p></o:p></p>
<p class="MsoNormal">+                HandleBuffer[Index],<o:p></o:p></p>
<p class="MsoNormal">+                &gEfiGraphicsOutputProtocolGuid,<o:p></o:p></p>
<p class="MsoNormal">+                (VOID **) &Gop<o:p></o:p></p>
<p class="MsoNormal">+                );<o:p></o:p></p>
<p class="MsoNormal">+      DEBUG((DEBUG_INFO, "HandleProtocol Gop Status = %r, Index = %x\n", Status, Index));<o:p></o:p></p>
<p class="MsoNormal">+      if (EFI_ERROR(Status))<o:p></o:p></p>
<p class="MsoNormal">+        {<o:p></o:p></p>
<p class="MsoNormal">+            continue;<o:p></o:p></p>
<p class="MsoNormal">+        }<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+      // Clear the graphics screen to black<o:p></o:p></p>
<p class="MsoNormal">+      Status = Gop->Blt(Gop, NULL, EfiBltVideoFill, 0, 0, 0, 0, Gop->Mode->Info->HorizontalResolution, Gop->Mode->Info->VerticalResolution, 0);<o:p></o:p></p>
<p class="MsoNormal">+      DEBUG((DEBUG_INFO, "Gop->Blt Status = %r, Index = %x\n", Status, Index));<o:p></o:p></p>
<p class="MsoNormal">+      if (EFI_ERROR(Status)) {<o:p></o:p></p>
<p class="MsoNormal">+        continue;<o:p></o:p></p>
<p class="MsoNormal">+      }<o:p></o:p></p>
<p class="MsoNormal">+    }<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">+  gBS->FreePool (HandleBuffer);<o:p></o:p></p>
<p class="MsoNormal">+  DEBUG((DEBUG_INFO, "ClearGraphicsScreenToDisplayTpmMessages Exit...\n"));<o:p></o:p></p>
<p class="MsoNormal">+  return Status;<o:p></o:p></p>
<p class="MsoNormal">+}<o:p></o:p></p>
<p class="MsoNormal">/**<o:p></o:p></p>
<p class="MsoNormal">   Read the specified key for user confirmation.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">@@ -576,6 +644,12 @@ Tcg2UserConfirm (<o:p></o:p></p>
<p class="MsoNormal">   BufSize -= StrSize (ConfirmText);<o:p></o:p></p>
<p class="MsoNormal">   UnicodeSPrint (ConfirmText + StrLen (ConfirmText), BufSize, TmpStr1, TmpStr2);<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">+  //<o:p></o:p></p>
<p class="MsoNormal">+  //Clear Graphics Screen To unblock and Display TPM messages
<o:p></o:p></p>
<p class="MsoNormal">+  // <o:p></o:p></p>
<p class="MsoNormal">+  Status = ClearGraphicsScreenToDisplayTpmMessages();<o:p></o:p></p>
<p class="MsoNormal">+  DEBUG((DEBUG_INFO, "ClearGraphicsScreenToDisplayTpmMessages Status = %r\n", Status));<o:p></o:p></p>
<p class="MsoNormal">+<o:p></o:p></p>
<p class="MsoNormal">   DstStr[80] = L'\0';<o:p></o:p></p>
<p class="MsoNormal">   for (Index = 0; Index < StrLen (ConfirmText); Index += 80) {<o:p></o:p></p>
<p class="MsoNormal">     StrnCpyS (DstStr, sizeof (DstStr) / sizeof (CHAR16), ConfirmText + Index, sizeof (DstStr) / sizeof (CHAR16) - 1);<o:p></o:p></p>
<p class="MsoNormal">diff --git a/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.inf b/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.inf<o:p></o:p></p>
<p class="MsoNormal">index e1c7c20d52..de423cfd13 100644<o:p></o:p></p>
<p class="MsoNormal">--- a/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.inf<o:p></o:p></p>
<p class="MsoNormal">+++ b/SecurityPkg/Library/DxeTcg2PhysicalPresenceLib/DxeTcg2PhysicalPresenceLib.inf<o:p></o:p></p>
<p class="MsoNormal">@@ -54,6 +54,7 @@<o:p></o:p></p>
<p class="MsoNormal">[Protocols]<o:p></o:p></p>
<p class="MsoNormal">   gEfiTcg2ProtocolGuid                 ## SOMETIMES_CONSUMES<o:p></o:p></p>
<p class="MsoNormal">   gEdkiiVariableLockProtocolGuid       ## SOMETIMES_CONSUMES<o:p></o:p></p>
<p class="MsoNormal">+  gEfiGraphicsOutputProtocolGuid       ## CONSUMES<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"> [Pcd]<o:p></o:p></p>
<p class="MsoNormal">   gEfiSecurityPkgTokenSpaceGuid.PcdTcg2PhysicalPresenceFlags       ## SOMETIMES_CONSUMES<o:p></o:p></p>
<p class="MsoNormal">-- <o:p></o:p></p>
<p class="MsoNormal">2.17.0.windows.1<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="msipfooter90245289" style="margin:0in"><span style="font-size:7.0pt;color:#737373">Internal Use - Confidential</span><o:p></o:p></p>
</div>
</div>
</div>
</div>
</body>
</html>


<div width="1" style="color:white;clear:both">_._,_._,_</div>
<hr>


Groups.io Links:<p>


  
    You receive all messages sent to this group.
  
  


<p>
<a target="_blank" href="https://edk2.groups.io/g/devel/message/108087">View/Reply Online (#108087)</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>

<br>




<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]<br>
<div width="1" style="color:white;clear:both">_._,_._,_</div>