<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=utf-8">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<!--[if !mso]><style>v\:* {behavior:url(#default#VML);}
o\:* {behavior:url(#default#VML);}
w\:* {behavior:url(#default#VML);}
.shape {behavior:url(#default#VML);}
</style><![endif]--><style><!--
/* Font Definitions */
@font-face
        {font-family:Helvetica;
        panose-1:2 11 6 4 2 2 2 2 2 4;}
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:"Microsoft Yi Baiti";
        panose-1:3 0 5 0 0 0 0 0 0 0;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
p.MsoListParagraph, li.MsoListParagraph, div.MsoListParagraph
        {mso-style-priority:34;
        margin-top:0in;
        margin-right:0in;
        margin-bottom:0in;
        margin-left:.5in;
        margin-bottom:.0001pt;
        font-size:11.0pt;
        font-family:"Calibri",sans-serif;}
span.EmailStyle18
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:windowtext;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;
        font-family:"Calibri",sans-serif;
        mso-fareast-language:KO;}
@page WordSection1
        {size:8.5in 11.0in;
        margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
        {page:WordSection1;}
/* List Definitions */
@list l0
        {mso-list-id:1651979178;
        mso-list-type:hybrid;
        mso-list-template-ids:-1329271182 67698705 67698713 67698715 67698703 67698713 67698715 67698703 67698713 67698715;}
@list l0:level1
        {mso-level-text:"%1\)";
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l0:level2
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l0:level3
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        text-indent:-9.0pt;}
@list l0:level4
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l0:level5
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l0:level6
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        text-indent:-9.0pt;}
@list l0:level7
        {mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l0:level8
        {mso-level-number-format:alpha-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:left;
        text-indent:-.25in;}
@list l0:level9
        {mso-level-number-format:roman-lower;
        mso-level-tab-stop:none;
        mso-level-number-position:right;
        text-indent:-9.0pt;}
ol
        {margin-bottom:0in;}
ul
        {margin-bottom:0in;}
--></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="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="mso-fareast-language:KO"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO">Hi Marcello,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO">The patch passed all checks now .<a href="https://github.com/tianocore/edk2/pull/1109">https://github.com/tianocore/edk2/pull/1109</a><o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO">And I just send the patch out for code review. Please comments if you have question.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO">I briefly reviewed your secureboot work, both BlSMMStoreDxe.inf and  SecureBootEnrollDefaultKeys are DXE driver, I don’t see any dependency to PEI.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO">And here is my though for the overall flow for the default key enrollment:<o:p></o:p></span></p>
<ol style="margin-top:0in" start="1" type="1">
<li class="MsoListParagraph" style="margin-left:0in;mso-list:l0 level1 lfo1"><span style="mso-fareast-language:KO">Bootloader pass the NVS variable region to UEFI payload via HOB<o:p></o:p></span></li><ol style="margin-top:0in" start="1" type="a">
<li class="MsoListParagraph" style="margin-left:0in;mso-list:l0 level2 lfo1"><span style="mso-fareast-language:KO">Bootloader might provision the default keys in variable region.<o:p></o:p></span></li></ol>
<li class="MsoListParagraph" style="margin-left:0in;mso-list:l0 level1 lfo1"><span style="mso-fareast-language:KO">UEFI payload retrieve the variable region from HOB.<o:p></o:p></span></li><ol style="margin-top:0in" start="1" type="a">
<li class="MsoListParagraph" style="margin-left:0in;mso-list:l0 level2 lfo1"><span style="mso-fareast-language:KO">This could be done via a PlatformSupportLib instance, or add to ParseLib.<o:p></o:p></span></li></ol>
<li class="MsoListParagraph" style="margin-left:0in;mso-list:l0 level1 lfo1"><span style="mso-fareast-language:KO">UEFI payload BlSupportDxe modules could set variable related PCDs if the variable HOB found.<o:p></o:p></span></li><li class="MsoListParagraph" style="margin-left:0in;mso-list:l0 level1 lfo1"><span style="mso-fareast-language:KO">UEFI payload FVB driver or a new driver (SecureBootEnrollDefaultKeys in your case) could check if variable region is initialized. UEFI payload
 could setup the default Keys only when the variable region is not initialized. This should only happens once if the bootloader doesn’t provision default keys.<o:p></o:p></span></li><li class="MsoListParagraph" style="margin-left:0in;mso-list:l0 level1 lfo1"><span style="mso-fareast-language:KO">Use the EDKII variable driver should work.<o:p></o:p></span></li></ol>
<p class="MsoNormal"><span style="mso-fareast-language:KO">Feel free to contact me if you want to discuss it further.<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO"><o:p> </o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO">Thanks,<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO">Guo<o:p></o:p></span></p>
<p class="MsoNormal"><span style="mso-fareast-language:KO"><o:p> </o:p></span></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> devel@edk2.groups.io <devel@edk2.groups.io> <b>
On Behalf Of </b>Marcello Sylvester Bauer<br>
<b>Sent:</b> Tuesday, November 10, 2020 6:24 AM<br>
<b>To:</b> Dong, Guo <guo.dong@intel.com><br>
<b>Cc:</b> devel@edk2.groups.io; Ma, Maurice <maurice.ma@intel.com>; patrick.rudolph@9elements.com; You, Benjamin <benjamin.you@intel.com><br>
<b>Subject:</b> Re: [edk2-devel] [PATCH v2 0/1] UefiPayloadPkg: Set default PciBaseSize on Ia32<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<p class="MsoNormal">Hi Guo,<o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">> Removing the PEI phase could simplify the UEFI payload, the PEI FV will be replaced by a single UefiPayloadEntry module which only does minimal work and passes control to DXE core. So we don’t need have a PEI core and other PEI modules
 in UEFI payload. It would reduce payload size and improve the boot performance.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Okay, this makes very much sense. Is there an approximate estimate of when it will be finished? It would be great if we could still merge this patch anyway somehow. <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal" style="margin-bottom:12.0pt">> I don’t know how your upcoming patches and your secureboot works in UEFI payload PEI. I could take a look and give comments if you have the change list.<br>
<br>
It is still WIP but could be adapted: <a href="https://github.com/9elements/edk2/tree/feature/secureboot">
https://github.com/9elements/edk2/tree/feature/secureboot</a><br>
<br>
Thanks,<br>
Marcello<br>
<br>
<br>
<br>
<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal">On Tue, Nov 10, 2020 at 5:14 AM Dong, Guo <<a href="mailto:guo.dong@intel.com" target="_blank">guo.dong@intel.com</a>> wrote:<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt">
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Hi Marcello,<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Removing the PEI phase could simplify the UEFI payload, the PEI FV will be replaced by a single UefiPayloadEntry module which only does minimal work and passes control to DXE core.
 So we don’t need have a PEI core and other PEI modules in UEFI payload. It would reduce payload size and improve the boot performance.<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">I don’t know how your upcoming patches and your secureboot works in UEFI payload PEI. I could take a look and give comments if you have the change list.<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Thank you to point out the failure. I will update the patch and create a PR.<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Thanks,<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Guo<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <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" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b>From:</b> Marcello Sylvester Bauer <<a href="mailto:marcello.bauer@9elements.com" target="_blank">marcello.bauer@9elements.com</a>>
<br>
<b>Sent:</b> Monday, November 9, 2020 3:47 AM<br>
<b>To:</b> Dong, Guo <<a href="mailto:guo.dong@intel.com" target="_blank">guo.dong@intel.com</a>><br>
<b>Cc:</b> <a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>; Ma, Maurice <<a href="mailto:maurice.ma@intel.com" target="_blank">maurice.ma@intel.com</a>>;
<a href="mailto:patrick.rudolph@9elements.com" target="_blank">patrick.rudolph@9elements.com</a>; You, Benjamin <<a href="mailto:benjamin.you@intel.com" target="_blank">benjamin.you@intel.com</a>><br>
<b>Subject:</b> Re: [edk2-devel] [PATCH v2 0/1] UefiPayloadPkg: Set default PciBaseSize on Ia32<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Hi Guo,<o:p></o:p></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;margin-bottom:12.0pt">Sounds good to remove the IA32 target and stick to a single DSC file. However, Is there an advantage to remove the PEI phase? It breaks many of our upcoming patches, which currently rely
 on the PEI phase (e.g. SecureBoot support).<br>
<br>
In addition to the failed PatchCheck, it does not build on Linux gcc: <a href="https://github.com/9elements/edk2/runs/1373473040" target="_blank">
https://github.com/9elements/edk2/runs/1373473040</a><br>
<br>
thanks,<br>
Marcello<o:p></o:p></p>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">On Sat, Oct 24, 2020 at 12:36 AM Dong, Guo <<a href="mailto:guo.dong@intel.com" target="_blank">guo.dong@intel.com</a>> wrote:<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt">
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Hi Marcello,<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">It looks there is issue for the CI tool to complete all the checks. So I just closed this PR.<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">And I just created another PR
<a href="https://github.com/tianocore/edk2/pull/1046" target="_blank">https://github.com/tianocore/edk2/pull/1046</a> to remove PEI phase from UEFI payload.<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">If the new patch is approved, we don’t need this patch to update DSC file.<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">In the new patch, UEFI payload would use a single DSC file to support X64, IA32X64 and possibly IA32 build.<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Please have a look at that PR and let me know if you have any comments.<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Thanks,<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Guo<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <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" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b>From:</b>
<a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a> <<a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>>
<b>On Behalf Of </b>Guo Dong<br>
<b>Sent:</b> Thursday, October 22, 2020 7:49 PM<br>
<b>To:</b> Marcello Sylvester Bauer <<a href="mailto:marcello.bauer@9elements.com" target="_blank">marcello.bauer@9elements.com</a>>; Ma, Maurice <<a href="mailto:maurice.ma@intel.com" target="_blank">maurice.ma@intel.com</a>><br>
<b>Cc:</b> <a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>;
<a href="mailto:patrick.rudolph@9elements.com" target="_blank">patrick.rudolph@9elements.com</a>; You, Benjamin <<a href="mailto:benjamin.you@intel.com" target="_blank">benjamin.you@intel.com</a>><br>
<b>Subject:</b> Re: [edk2-devel] [PATCH v2 0/1] UefiPayloadPkg: Set default PciBaseSize on Ia32<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Hi Marcello,<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Sorry for late response. I created a pull request
<a href="https://github.com/tianocore/edk2/pull/1044" target="_blank">https://github.com/tianocore/edk2/pull/1044</a> to merge this patch 2 hours ago.<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Hopefully it could be merged soon after all the checks.<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Thanks,<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Guo<o:p></o:p></p>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <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" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b>From:</b> Marcello Sylvester Bauer <<a href="mailto:marcello.bauer@9elements.com" target="_blank">marcello.bauer@9elements.com</a>>
<br>
<b>Sent:</b> Thursday, October 22, 2020 1:25 AM<br>
<b>To:</b> Ma, Maurice <<a href="mailto:maurice.ma@intel.com" target="_blank">maurice.ma@intel.com</a>><br>
<b>Cc:</b> <a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a>;
<a href="mailto:patrick.rudolph@9elements.com" target="_blank">patrick.rudolph@9elements.com</a>; Dong, Guo <<a href="mailto:guo.dong@intel.com" target="_blank">guo.dong@intel.com</a>>; You, Benjamin <<a href="mailto:benjamin.you@intel.com" target="_blank">benjamin.you@intel.com</a>><br>
<b>Subject:</b> Re: [PATCH v2 0/1] UefiPayloadPkg: Set default PciBaseSize on Ia32<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Hi,<br>
<br>
As already mentioned, this patch fixes the current master build for the UefiPayloadPkgIa32 platform.<br>
Is it possible to merge this change soon?<o:p></o:p></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Sorry for the circumstances.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;margin-bottom:12.0pt"><br>
thanks,<br>
Marcello<o:p></o:p></p>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">On Tue, Oct 13, 2020 at 9:02 PM Ma, Maurice <<a href="mailto:maurice.ma@intel.com" target="_blank">maurice.ma@intel.com</a>> wrote:<o:p></o:p></p>
</div>
<blockquote style="border:none;border-left:solid #CCCCCC 1.0pt;padding:0in 0in 0in 6.0pt;margin-left:4.8pt;margin-top:5.0pt;margin-right:0in;margin-bottom:5.0pt">
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">Reviewed-by:
<br>
Maurice Ma <<a href="mailto:maurice.ma@intel.com" target="_blank">maurice.ma@intel.com</a>><br>
<br>
> -----Original Message-----<br>
> From: Marcello Sylvester Bauer <<a href="mailto:marcello.bauer@9elements.com" target="_blank">marcello.bauer@9elements.com</a>><br>
> Sent: Tuesday, October 13, 2020 6:34<br>
> To: <a href="mailto:devel@edk2.groups.io" target="_blank">devel@edk2.groups.io</a><br>
> Cc: Marcello Sylvester Bauer <<a href="mailto:marcello.bauer@9elements.com" target="_blank">marcello.bauer@9elements.com</a>>;<br>
> <a href="mailto:patrick.rudolph@9elements.com" target="_blank">patrick.rudolph@9elements.com</a>; Ma, Maurice <<a href="mailto:maurice.ma@intel.com" target="_blank">maurice.ma@intel.com</a>>; Dong,<br>
> Guo <<a href="mailto:guo.dong@intel.com" target="_blank">guo.dong@intel.com</a>>; You, Benjamin <<a href="mailto:benjamin.you@intel.com" target="_blank">benjamin.you@intel.com</a>><br>
> Subject: [PATCH v2 0/1] UefiPayloadPkg: Set default PciBaseSize on Ia32<br>
> <br>
> This commit fix UefiPayloadPkgIa32 build in master.<br>
> <br>
> In commit 8028b2907e20b21cd7d69639a36ac82a77c81dc1 I did forget to set<br>
> the default value for PcdPciExpressBaseSize on Ia32 Targets. This patch does<br>
> insert it afterwards. It would be great if it could be merged asap.<br>
> <br>
> PS: I added the Ia32 target to our CI to avoid this issue in future. Sorry for the<br>
> misfortune.<br>
> <br>
> v2:<br>
>   * Remove no longer required build-time PcdPciExpressBaseAddress<br>
> <br>
> Branch: <a href="https://github.com/9elements/edk2/tree/fix/UefiPayloadPkgIa32_V2" target="_blank">
https://github.com/9elements/edk2/tree/fix/UefiPayloadPkgIa32_V2</a><br>
> PR:     <a href="https://github.com/tianocore/edk2/pull/1008" target="_blank">https://github.com/tianocore/edk2/pull/1008</a><br>
> <br>
> Marcello Sylvester Bauer (1):<br>
>   UefiPayloadPkg: Set default PciBaseSize on Ia32<br>
> <br>
>  UefiPayloadPkg/UefiPayloadPkgIa32.dsc | 3 +--<br>
>  1 file changed, 1 insertion(+), 2 deletions(-)<br>
> <br>
> --<br>
> 2.28.0<o:p></o:p></p>
</blockquote>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><br clear="all">
<o:p></o:p></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">--
<o:p></o:p></p>
<div>
<div>
<div>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b><span style="font-size:12.0pt;font-family:"Helvetica",sans-serif;color:black">[Marcello Sylvester Bauer]</span></b><span style="font-size:9.5pt;font-family:"Arial",sans-serif;color:#222222"> </span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:9.5pt;font-family:"Arial",sans-serif;color:#222222"> </span><o:p></o:p></p>
</div>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;margin-bottom:13.5pt"><span style="font-size:13.5pt;font-family:"Times New Roman",serif;color:black"><img border="0" width="150" height="29" style="width:1.5625in;height:.302in" id="_x0000_i1033" src="http://static.9elements.com/logo-signature.png"></span><o:p></o:p></p>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:black">9elements Agency GmbH, Kortumstraße 19-21, 44787 Bochum, Germany</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:8.5pt;font-family:"Arial",sans-serif;color:#222222">Email:  </span><a href="https://static.9elements.com/email_signatur.html" target="_blank"><span style="font-size:8.5pt;font-family:"Arial",sans-serif;color:#1155CC">[DEINE
 EMAIL ADDRESSE]</span></a><o:p></o:p></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:black">Phone: </span><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:#222222"> </span><a href="tel:+492346894188" target="_blank"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif">+49
 234 68 94 188</span></a><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:black">Mobile: </span><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:#222222"> </span><a href="tel:+491722847618" target="_blank"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif">+49
 1722847618</span></a><o:p></o:p></p>
</div>
</div>
</div>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto;line-height:8.25pt">
<span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Sitz der Gesellschaft: Bochum</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto;line-height:8.25pt">
<span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Handelsregister: Amtsgericht Bochum, HRB 17519</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto;line-height:8.25pt">
<span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Geschäftsführung: Sebastian Deutsch, Eray Basar</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:13.5pt;font-family:"Helvetica",sans-serif;color:#222222"><br>
</span><a href="https://9elements.com/privacy" target="_blank"><span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Datenschutzhinweise nach Art. 13 DSGVO</span></a><o:p></o:p></p>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</blockquote>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><br clear="all">
<o:p></o:p></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto">--
<o:p></o:p></p>
<div>
<div>
<div>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><b><span style="font-size:12.0pt;font-family:"Helvetica",sans-serif;color:black">[Marcello Sylvester Bauer]</span></b><span style="font-size:9.5pt;font-family:"Arial",sans-serif;color:#222222"> </span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:9.5pt;font-family:"Arial",sans-serif;color:#222222"> </span><o:p></o:p></p>
</div>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;margin-bottom:13.5pt"><span style="font-size:13.5pt;font-family:"Times New Roman",serif;color:black"><img border="0" width="150" height="29" style="width:1.5625in;height:.302in" id="_x0000_i1032" src="http://static.9elements.com/logo-signature.png"></span><o:p></o:p></p>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:black">9elements Agency GmbH, Kortumstraße 19-21, 44787 Bochum, Germany</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:8.5pt;font-family:"Arial",sans-serif;color:#222222">Email:  </span><a href="https://static.9elements.com/email_signatur.html" target="_blank"><span style="font-size:8.5pt;font-family:"Arial",sans-serif;color:#1155CC">[DEINE
 EMAIL ADDRESSE]</span></a><o:p></o:p></p>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:black">Phone: </span><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:#222222"> </span><a href="tel:+492346894188" target="_blank"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif">+49
 234 68 94 188</span></a><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:black">Mobile: </span><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:#222222"> </span><a href="tel:+491722847618" target="_blank"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif">+49
 1722847618</span></a><o:p></o:p></p>
</div>
</div>
</div>
</div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"> <o:p></o:p></p>
<div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto;line-height:8.25pt">
<span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Sitz der Gesellschaft: Bochum</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto;line-height:8.25pt">
<span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Handelsregister: Amtsgericht Bochum, HRB 17519</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto;line-height:8.25pt">
<span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Geschäftsführung: Sebastian Deutsch, Eray Basar</span><o:p></o:p></p>
</div>
<div>
<p class="MsoNormal" style="mso-margin-top-alt:auto;mso-margin-bottom-alt:auto"><span style="font-size:13.5pt;font-family:"Helvetica",sans-serif;color:#222222"><br>
</span><a href="https://9elements.com/privacy" target="_blank"><span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Datenschutzhinweise nach Art. 13 DSGVO</span></a><o:p></o:p></p>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</blockquote>
</div>
<p class="MsoNormal"><br clear="all">
<o:p></o:p></p>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<p class="MsoNormal">-- <o:p></o:p></p>
<div>
<div>
<div>
<div>
<div>
<p class="MsoNormal"><b><span style="font-size:12.0pt;font-family:"Helvetica",sans-serif;color:black">[Marcello Sylvester Bauer]</span></b><span style="font-size:9.5pt;font-family:"Arial",sans-serif;color:#222222"> <o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:9.5pt;font-family:"Arial",sans-serif;color:#222222"><o:p> </o:p></span></p>
</div>
</div>
<div>
<p class="MsoNormal" style="margin-bottom:13.5pt"><span style="font-size:13.5pt;font-family:"Times New Roman",serif;color:black"><img border="0" width="150" height="29" style="width:1.5625in;height:.302in" id="_x0000_i1031" src="http://static.9elements.com/logo-signature.png"></span><span style="font-size:13.5pt;font-family:"Times New Roman",serif;color:black"><o:p></o:p></span></p>
<div>
<div>
<p class="MsoNormal"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:black">9elements Agency GmbH, Kortumstraße 19-21, 44787 Bochum, Germany</span><span style="font-size:9.5pt;font-family:"Arial",sans-serif;color:#222222"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:8.5pt;font-family:"Arial",sans-serif;color:#222222">Email:  </span><a href="https://static.9elements.com/email_signatur.html" target="_blank"><span style="font-size:8.5pt;font-family:"Arial",sans-serif;color:#1155CC">[DEINE
 EMAIL ADDRESSE]</span></a><span style="font-size:13.5pt;font-family:"Times New Roman",serif;color:black"><o:p></o:p></span></p>
<div>
<p class="MsoNormal"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:black">Phone: </span><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:#222222"> </span><a href="tel:+492346894188" target="_blank"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif">+49
 234 68 94 188</span></a><span style="font-size:13.5pt;font-family:"Times New Roman",serif;color:black"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:black">Mobile: </span><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif;color:#222222"> </span><a href="tel:+491722847618" target="_blank"><span style="font-size:8.5pt;font-family:"Helvetica",sans-serif">+49
 1722847618</span></a><span style="font-size:9.5pt;font-family:"Arial",sans-serif;color:#222222"><o:p></o:p></span></p>
</div>
</div>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal" style="line-height:8.25pt"><span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Sitz der Gesellschaft: Bochum</span><span style="font-size:13.5pt;font-family:"Helvetica",sans-serif;color:#222222"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="line-height:8.25pt"><span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Handelsregister: Amtsgericht Bochum, HRB 17519</span><span style="font-size:13.5pt;font-family:"Helvetica",sans-serif;color:#222222"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="line-height:8.25pt"><span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Geschäftsführung: Sebastian Deutsch, Eray Basar</span><span style="font-size:13.5pt;font-family:"Helvetica",sans-serif;color:#222222"><o:p></o:p></span></p>
</div>
<div>
<p class="MsoNormal" style="mso-line-height-alt:8.25pt"><span style="font-size:13.5pt;font-family:"Helvetica",sans-serif;color:#222222"><br>
</span><a href="https://9elements.com/privacy" target="_blank"><span style="font-size:7.5pt;font-family:"Helvetica",sans-serif;color:#B4B7B8">Datenschutzhinweise nach Art. 13 DSGVO</span></a><span style="font-size:13.5pt;font-family:"Helvetica",sans-serif;color:#222222"><o:p></o:p></span></p>
</div>
</div>
</div>
</div>
</div>
<div>
<p class="MsoNormal"></o:p></span></p>
</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/67250">View/Reply Online (#67250)</a> |    |  <a target="_blank" href="https://groups.io/mt/77482867/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>