Page MenuHomeFreeBSD

libsa: smbios: probe: BCD revision parsing is v2-only code; Expand comments
ClosedPublic

Authored by olce on Mar 7 2025, 4:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 7, 3:57 AM
Unknown Object (File)
Sat, Mar 28, 11:49 AM
Unknown Object (File)
Sun, Mar 22, 7:14 PM
Unknown Object (File)
Sat, Mar 14, 3:57 PM
Unknown Object (File)
Mar 3 2026, 11:34 AM
Unknown Object (File)
Mar 2 2026, 7:59 AM
Unknown Object (File)
Feb 17 2026, 3:26 PM
Unknown Object (File)
Feb 8 2026, 2:40 PM
Subscribers

Details

Summary

The code parsing the BCD revision is only meaningful on v2, so move it
away into the appropriate 'if' branch to ease reading (and to avoid
a useless test).

Expand comments. In particular, make it clear that setting
'smbios.count' to '-1' removes the limit of the number of structures to
parse.

No functional change.

Diff Detail

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

Event Timeline

olce requested review of this revision.Mar 7 2025, 4:57 PM
This revision is now accepted and ready to land.Mar 7 2025, 5:04 PM
markj added inline comments.
stand/libsa/smbios.c
586

"No." is a somewhat common abbreviation for "number". "Nb" looks like a typo to me. If you're going to change it, I'd rather use "No." or just "Number".