Re: [Qemu-devel] [PATCH 1/1] mach-virt: Set VM's SMBIOS system version to mc->desc

2018-03-22 Thread Daniel P . Berrangé
On Thu, Mar 22, 2018 at 02:49:35PM +0100, Andrew Jones wrote: > On Thu, Mar 22, 2018 at 11:07:12AM +, Daniel P. Berrangé wrote: > > On Wed, Mar 21, 2018 at 10:12:16PM -0500, Wei Huang wrote: > > > Instead of using "1.0" as the system version of SMBIOS, we should use > > > mc->desc for mach-virt

Re: [Qemu-devel] [PATCH 1/1] mach-virt: Set VM's SMBIOS system version to mc->desc

2018-03-22 Thread Andrew Jones
On Thu, Mar 22, 2018 at 11:07:12AM +, Daniel P. Berrangé wrote: > On Wed, Mar 21, 2018 at 10:12:16PM -0500, Wei Huang wrote: > > Instead of using "1.0" as the system version of SMBIOS, we should use > > mc->desc for mach-virt machine type. With this patch, "dmidecode -t 1" > > (e.g., "-M virt-2

Re: [Qemu-devel] [PATCH 1/1] mach-virt: Set VM's SMBIOS system version to mc->desc

2018-03-22 Thread Wei Huang
On 03/22/2018 05:53 AM, Peter Maydell wrote: > On 22 March 2018 at 08:01, Andrew Jones wrote: >> I agree we should change the useless 1.0, but shouldn't we use >> mc->name instead of mc->desc? mc->name would make it consistent >> with pc-piix and q35 and also be a less verbose "version". We'd >>

Re: [Qemu-devel] [PATCH 1/1] mach-virt: Set VM's SMBIOS system version to mc->desc

2018-03-22 Thread Daniel P . Berrangé
On Wed, Mar 21, 2018 at 10:12:16PM -0500, Wei Huang wrote: > Instead of using "1.0" as the system version of SMBIOS, we should use > mc->desc for mach-virt machine type. With this patch, "dmidecode -t 1" > (e.g., "-M virt-2.12,accel=kvm") will show: > > Handle 0x0100, DMI type 1, 27 bytes >

Re: [Qemu-devel] [PATCH 1/1] mach-virt: Set VM's SMBIOS system version to mc->desc

2018-03-22 Thread Peter Maydell
On 22 March 2018 at 08:01, Andrew Jones wrote: > I agree we should change the useless 1.0, but shouldn't we use > mc->name instead of mc->desc? mc->name would make it consistent > with pc-piix and q35 and also be a less verbose "version". We'd > get e.g. > > System Information > Manufactu

Re: [Qemu-devel] [PATCH 1/1] mach-virt: Set VM's SMBIOS system version to mc->desc

2018-03-22 Thread Andrew Jones
On Wed, Mar 21, 2018 at 10:12:16PM -0500, Wei Huang wrote: > Instead of using "1.0" as the system version of SMBIOS, we should use > mc->desc for mach-virt machine type. With this patch, "dmidecode -t 1" > (e.g., "-M virt-2.12,accel=kvm") will show: > > Handle 0x0100, DMI type 1, 27 bytes >

[Qemu-devel] [PATCH 1/1] mach-virt: Set VM's SMBIOS system version to mc->desc

2018-03-21 Thread Wei Huang
Instead of using "1.0" as the system version of SMBIOS, we should use mc->desc for mach-virt machine type. With this patch, "dmidecode -t 1" (e.g., "-M virt-2.12,accel=kvm") will show: Handle 0x0100, DMI type 1, 27 bytes System Information Manufacturer: QEMU Product