Hi SmbiosDxe maintainers, Do you have any comments on this patch? Maybe it is a more clean way to check if SMBIOS table HOB exists in its entry point instead of checking

Yes, i don't see the assembler code stub for the hypercall in the OVMF patches, so i was wondering the same. Thanks, Ashish

You are correct that we don't support migration for SEV-ES machines, although our approach can potentially be adapted for SEV-ES. I was on the fence about including this patch, because we don't

EDK2 port of DMTF libredfish project. We clone the necessary files from open source project libredfish (https://github.com/DMTF/libredfish) tag v1.0.0 and revise it to incorporate with edk2 firmware

Hi Ard, Leif, This patch series is creating 2 new folders Platform/StMMRpmb & Drivers/OpTeeRpmb. - Should these be in Drivers\StandaloneMmRpmbPkg similar to Drivers\OptionRomPkg ? - Also, the

Hello Ilias, My review is mostly about coding style, but I would have some (inlined) remarks about your patch, Regards, Pierre I think: is -> in I think there should not be a space here. I think

Hello Ilias, I would have some (inlined) remarks about your patch, Regards, Pierre I think we are at the revision 0x0001001C, cf

Hi Patrick, I checked the code in deep. The HOB is not created by coreboot. It's the PayloadEntry that creates the HOB. Can we update PayloadEntry to create multiple HOBs? Guo, Any comments? The

I have 5 more comments embedded, can you read and reply?

Hi Mike & Zhiguang & Liming, Could you help review this patch? Thanks, Jiaxin

Hi Ray, thanks for your feedback. Currently a single HOB containing all the SMBIOS table is exported by coreboot. As coreboot doesn't support multiple HOBs with the same ID, #2 isn't a

In general, I agree this solution that lets SMBIOS driver directly absorbs the SMBIOS table from PEI. This can eliminate the needs of a separate driver that consumes the HOB and calls SMBIOS protocol

I am OK with the patch. But the patch is using incorrect “Line ending”: python ./BaseTools/Scripts/PatchCheck.py -1 Checking git commit: HEAD ShellPkg: add more items for smbiosview -t 3 . The

https://bugzilla.tianocore.org/show_bug.cgi?id=3177 Add ContainedElementCount, ContainedElementRecordLength and ContainedElements for smbiosview type 3. Signed-off-by: Mars CC Lin

Fix typo in Readme.md Signed-off-by: Masahisa Kojima <masahisa.kojima@...> Reviewed-by: Leif Lindholm <leif@...> --- Platform/Qemu/SbsaQemu/Readme.md | 2 +- 1 file changed, 1

This commit adds the standalone MM build instruction to enable UEFI secure boot. Signed-off-by: Masahisa Kojima <masahisa.kojima@...> --- Platform/Qemu/SbsaQemu/Readme.md | 35

This implements support for UEFI secure boot on SbsaQemu using the standalone MM framework. This moves all of the software handling of the UEFI authenticated variable store into the standalone

Add the build infrastructure for compilation of StandaloneMm image. SbsaQemu.fdf is modified to extend the FLASH0 region enough big to contain StandaloneMM image(BL32). Signed-off-by: Masahisa Kojima

This patch series implment the UEFI secure boot on SbsaQemu. Cc: Leif Lindholm <leif@...> Cc: Graeme Gregory <graeme@...> Cc: Radoslaw Biernacki <rad@...> Cc: Shashi

I created both "memory" and "/reserved-memory" parsing helper function. Could you check the next version of patch? Thanks, Masahisa