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...@intel.com> > Sent: Wednesday, March 3, 2021 3:04 AM > To: Patrick Rudolph <patrick.rudo...@9elements.com> > Cc: devel@edk2.groups.io; Chaganty, Rangasai V > <rangasai.v.chaga...@intel.com>; Bi, Dandan <dandan...@intel.com>; Zeng, > Star <star.z...@intel.com>; Gao, Zhichao <zhichao....@intel.com>; You, > Benjamin <benjamin....@intel.com>; > philipp.deppenwi...@9elements.com; Ma, Maurice > <maurice...@intel.com>; Dong, Guo <guo.d...@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/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
