> Status = EFI_NOT_FOUND; > // > // Add QEMU SMBIOS data if found > // > SmbiosTables = GetQemuSmbiosTables (); > - if (SmbiosTables != NULL) { > - Status = InstallAllStructures (SmbiosTables); > + if (SmbiosTables == NULL) { > + SmbiosTables = GetCloudHvSmbiosTables ();
Check PcdOvmfHostBridgePciDevId here to figure whenever GetQemuSmbiosTables or GetCloudHvSmbiosTables should be called? Should be more robust and give us a slightly less messy code flow, specifically when it comes to flip FreeTables to true for the qemu case. take care, Gerd -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#84206): https://edk2.groups.io/g/devel/message/84206 Mute This Topic: https://groups.io/mt/87429115/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-