Created attachment 219209 [details] patch filling manufacturer and family smbios field bhyve is not reporting smbios.system.maker neither smbios.system.family: It currently uses the standard method of one space character. But reporting this non-visible character create not-so-funny debugging session with tools parsing those field. So, to prevent returning this non-visible character my proposal is to improve bhyve to return those strings: smbios.system.family="Virtual Machine" smbios.system.maker="BHYVE" For information, Amazon Instances are reporting: smbios.system.family="Default string" smbios.system.maker="Amazon EC2" and VirtualBox VMs: smbios.system.family="Virtual Machine" smbios.system.maker="innotek GmbH"
Looks fine to me: I'd say go ahead and commit this.
> "BHYVE", /* manufacturer string */ Why not: > "FreeBSD", /* manufacturer string */ ?
(In reply to VVD from comment #2) Yes this seems a better idea to use 'FreeBSD' as manufacturer: With my previous idea: smbios.system.family="Virtual Machine" smbios.system.maker="BHYVE" smbios.system.product="BHYVE" smbios.system.serial="None" smbios.system.sku="None" smbios.system.uuid="3c7d953f-dce9-11ea-b840-0cc47ad8c988" smbios.system.version="1.0" By using 'FreeBSD' : smbios.system.family="Virtual Machine" smbios.system.maker="FreeBSD" smbios.system.product="BHYVE" smbios.system.serial="None" smbios.system.sku="None" smbios.system.uuid="3c7d953f-dce9-11ea-b840-0cc47ad8c988" smbios.system.version="1.0"
A commit references this bug: Author: olivier Date: Fri Oct 30 00:03:59 UTC 2020 New revision: 367152 URL: https://svnweb.freebsd.org/changeset/base/367152 Log: bhyve currently reports each of "smbios.system.maker" and "smbios.system.family" as " ". This presents challenges for both humans and tools when trying to parse output that uses those results. The new values reported are now: smbios.system.family="Virtual Machine" smbios.system.maker="FreeBSD" PR: 250728 Approved by: grehan@FreeBSD.org Sponsored by: Netflix Changes: head/usr.sbin/bhyve/smbiostbl.c