smbios: Make multiple -smbios type= accumulate sanely
authorMarkus Armbruster <armbru@redhat.com>
Fri, 16 Aug 2013 13:18:31 +0000 (15:18 +0200)
committerMichael S. Tsirkin <mst@redhat.com>
Sat, 28 Sep 2013 20:49:39 +0000 (23:49 +0300)
commitfc3b32958a80bca13309e2695de07b43dd788421
treeca421cb339cb55a86ae5c5a74cac65ab16d373f4
parentec2df8c10a4585ba4641ae482cf2f5f13daa810e
smbios: Make multiple -smbios type= accumulate sanely

Currently, -smbios type=T,NAME=VAL,... adds one field (T,NAME) with
value VAL to fw_cfg for each unique NAME.  If NAME occurs multiple
times, the last one's VAL is used (before the QemuOpts conversion, the
first one was used).

Multiple -smbios can add multiple fields with the same (T, NAME).
SeaBIOS reads all of them from fw_cfg, but uses only the first field
(T, NAME).  The others are ignored.

"First one wins, subsequent ones get ignored silently" isn't nice.  We
commonly let the last option win.  Useful, because it lets you
-readconfig first, then selectively override with command line
options.

Clean up -smbios to work the common way.  Accumulate the settings,
with later ones overwriting earlier ones.  Put the result into fw_cfg
(no more useless duplicates).

Bonus cleanup: qemu_uuid_parse() no longer sets SMBIOS system uuid by
side effect.

Signed-off-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
arch_init.c
hw/i386/smbios.c
include/hw/i386/smbios.h
include/sysemu/sysemu.h
vl.c