[edk2-devel] [PATCH - resend] MdeModulePkg/Universal/SmbiosDxe: Scan for existing tables

Guo Dong guo.dong at intel.com
Wed Mar 3 17:53:47 UTC 2021


hi Ray,

Just saw the discussion on this patch.
Both coreboot and SBL would build the whole SMBIOS table and report it to payloads.

For UEFI payload, I think it is not necessary to let other driver (BlSupportDxe) to split the whole SMBIOS table into records.
I would prefer SMBIOS DXE diver could support the whole SMBIOS table from PEI/bootloader.
But it is also possible to support individual records if required by checking AnchorString to know if it is whole table.

Thanks,
Guo

> -----Original Message-----
> From: Ni, Ray <ray.ni at intel.com>
> Sent: Wednesday, March 3, 2021 3:04 AM
> To: Patrick Rudolph <patrick.rudolph at 9elements.com>
> Cc: devel at edk2.groups.io; Chaganty, Rangasai V
> <rangasai.v.chaganty at intel.com>; Bi, Dandan <dandan.bi at intel.com>; Zeng,
> Star <star.zeng at intel.com>; Gao, Zhichao <zhichao.gao at intel.com>; You,
> Benjamin <benjamin.you at intel.com>;
> philipp.deppenwiese at 9elements.com; Ma, Maurice
> <maurice.ma at intel.com>; Dong, Guo <guo.dong at intel.com>
> Subject: RE: [edk2-devel] [PATCH - resend]
> MdeModulePkg/Universal/SmbiosDxe: Scan for existing tables
> 
> >
> > 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
> > solution.
> 
> 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 reason I like this approach is it doesn't require the other bootloaders to
> write
> a SMBIOS driver that merges all SMBIOS structures together into one table.
> 
> Thanks,
> Ray


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#72398): https://edk2.groups.io/g/devel/message/72398
Mute This Topic: https://groups.io/mt/80998570/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-






More information about the edk2-devel-archive mailing list