Page MenuHomeFreeBSD

Support SMBIOS v3 for 64-bit entry systems
ClosedPublic

Authored by jmg on Feb 22 2023, 2:31 AM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 26 2024, 10:00 PM
Unknown Object (File)
Jan 26 2024, 9:59 PM
Unknown Object (File)
Dec 20 2023, 7:11 AM
Unknown Object (File)
Dec 16 2023, 2:30 PM
Unknown Object (File)
Nov 26 2023, 5:38 AM
Unknown Object (File)
Nov 26 2023, 5:35 AM
Unknown Object (File)
Nov 6 2023, 5:47 AM
Unknown Object (File)
Oct 9 2023, 2:46 PM
Subscribers

Details

Summary

Under QEMU on arm64 systems, the smbios table is above 4GB
requiring a 64-bit address to access.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jmg requested review of this revision.Feb 22 2023, 2:31 AM

I think the changes are good. All my comments are purely style and/or portability.
Thanks for doing this...

stand/libsa/smbios.c
37

HAS_SMBV3

would be more in line with what the rest of the loader does.

90

This and below need to be #ifdef or else we get warnings about testing undefined macros

111–113

I don't like the '4' here either...

121

Don't like the 5 here or the 4 above... Suggest sizeof(SMBIOS3_SIG) - 1.

486–506

shouldn't these be intended one level?

489

I can't tell from phab, but it looks like these comments don't line up...

jmg marked 5 inline comments as done.Feb 22 2023, 3:23 AM

Ok, will update the patch in a second, also forgot to include the output of the result:

root@freebsd:~ # kenv | grep ^smbios
smbios.bios.reldate="02/06/2015"
smbios.bios.revision="0.0"
smbios.bios.vendor="EFI Development Kit II / OVMF"
smbios.bios.version="0.0.0"
smbios.chassis.maker="QEMU"
smbios.chassis.type="Other"
smbios.chassis.version="virt-7.1"
smbios.memory.enabled="4194304"
smbios.socket.enabled="1"
smbios.socket.populated="1"
smbios.system.maker="QEMU"
smbios.system.product="QEMU Virtual Machine"
smbios.system.version="virt-7.1"
smbios.version="3.0"

wrong change made it, this fully addresses comments

Looks great! Thanks for such a fast turn around.

This revision is now accepted and ready to land.Feb 22 2023, 3:52 AM
This revision was automatically updated to reflect the committed changes.