Page MenuHomeFreeBSD

Fix bhyve type19 SMBIOS handling to avoid losing 4GB
ClosedPublic

Authored by bcran on Nov 24 2020, 4:51 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 16 2024, 1:36 PM
Unknown Object (File)
Dec 20 2023, 7:14 AM
Unknown Object (File)
Dec 12 2023, 9:54 PM
Unknown Object (File)
Nov 13 2023, 3:24 AM
Unknown Object (File)
Nov 12 2023, 4:26 PM
Unknown Object (File)
Nov 7 2023, 1:03 AM
Unknown Object (File)
Oct 6 2023, 12:08 AM
Unknown Object (File)
Oct 4 2023, 3:20 PM
Subscribers

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 34993
Build 31983: arc lint + arc unit

Event Timeline

bcran requested review of this revision.Nov 24 2020, 4:51 AM

This fixes the amount of memory displayed in the UiApp to be the same as passed on the bhyve command line. Otherwise, 8GB gets displayed as 4, 32GB as 28 etc.

This revision is now accepted and ready to land.Nov 24 2020, 1:17 PM
usr.sbin/bhyve/smbiostbl.c
760

Isnt this tied to the xsaddr setting in the preceding line, such that this should more correctly be written:

type19->xeaddr = guest_himem + type19->xsaddr;
jhb added a subscriber: jhb.
jhb added inline comments.
usr.sbin/bhyve/smbiostbl.c
760

Perhaps, though that's relatively minor (and it won't ever move). The other nitpick one could observe is to perhaps order it as 'start + size' instead of 'size + start', i.e. xsaddr + guest_himem, but any of the variants is fine with me TBH.

usr.sbin/bhyve/smbiostbl.c
760

I agree, @rgrimes - I'll fix it to be type19->xsaddr + guest_himem;