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
F106247597: D38721.id117736.diff
Fri, Dec 27, 9:22 PM
Unknown Object (File)
Oct 20 2024, 9:02 AM
Unknown Object (File)
Oct 20 2024, 9:02 AM
Unknown Object (File)
Oct 4 2024, 7:18 PM
Unknown Object (File)
Oct 3 2024, 11:46 PM
Unknown Object (File)
Oct 2 2024, 7:53 AM
Unknown Object (File)
Oct 2 2024, 1:37 AM
Unknown Object (File)
Oct 1 2024, 7:55 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 Passed
Unit
No Test Coverage
Build Status
Buildable 49943
Build 46835: arc lint + arc unit

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.