Hi, Please find my response inline. Regards, Sayanta

Pushed as 3ccc21d918c..2f0188b56ef4 Thanks to Grzegorz and others involved for finding a way to address this boot protocol issue elegantly! -- Ard.

Yeah, go for the patch. - Bret From: Samer El-Haj-Mahmoud via groups.io Sent: Tuesday, August 3, 2021 6:10 AM To: Nelson, Eric; Bret Barkelew; devel@edk2.groups.io;G Edhaya Chandran;

Reviewed-by: Samer El-Haj-Mahmoud <Samer.El-Haj-Mahmoud@...>

Hi Omkar, How did you check that the HEST table is populated correctly? There is no HEST parser in Acpiview at the moment. Do you plan to add an HEST parser to Acpiview? Regards, Sami Mujawar

Yes. The unmeasured here means the contents of the page is not measured but the PAGE_INFO is measured for all the pages added before the VM launch. The page info of both the secrets and cpuid page

Hi Omkar, Thank you for this patch. I have a minor suggestion marked inline as [SAMI], other than that this patch looks good to me. Reviewed-by: Sami Mujawar<sami.mujawar@...> Regards, Sami

Yes, patches were missing because the mailing list chose to unsort and retain non-consecutive patches (they were sent in order) after they exceeded the autoresponder mail loop guard of 40

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 Updates usage of gPchSpiPpiGuid to use the new interface that identifies SPI flash

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 The library instance has moved to IntelSiliconPkg. Cc: Nate DeSimone

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 Updates TigerlakeOpenBoardPkg to use the SmmSpiFlashCommonLib instance in

Thank you Jiewen for your feedback. I will try to address the comments in next version. I don't know why we choose "Ovmf" from the LaunchSecretsBase PCD. I thought PCD's specific the Uefi typically

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 Updates PCDs to use the IntelSiliconPkg PCD tokenspace now that the PCDs are declared

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 Updates PCDs to use the IntelSiliconPkg PCD tokenspace now that the PCDs are declared

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 Adds a Traditional MM and Standalone MM SPI FVB Service driver to IntelSiliconPkg.

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 SpiFlashCommonLib is duplicated in multiple places across the MinPlatform design in

Hi Omkar, Please find my response inline marked [SAMI]. Regards, Sami Mujawar On 10/07/2021 05:18 PM, Omkar Anand Kulkarni wrote: [SAMI] Should this patch

Hi Omkar, Please find my response below marked [SAMI] Regards, Sami Mujawar On 10/07/2021 05:18 PM, Omkar Anand Kulkarni wrote: [SAMI] Not sure if a

Hi Ard and Maintainers, For this patchset, 1/4 - Intel Platforms: add SecureBootVariableLib class resolution 2/4 - ARM Silicon and Platforms: add SecureBootVariableLib class resolution 3/4 -

Thanks Leif, I remove the description from the package name. Abner