diff options
| author | Peter Maydell <peter.maydell@linaro.org> | 2012-11-22 06:48:45 +0000 |
|---|---|---|
| committer | Alexander Graf <agraf@suse.de> | 2012-11-26 19:53:26 +0100 |
| commit | c4d88267ae76810420295a1682cf779c49f3ddec (patch) | |
| tree | b6f45be057908324f285e817858f27392474447a | |
| parent | 70c68cf638b4cf44a42a937622deac3ad3c7833c (diff) | |
| download | focaccia-qemu-c4d88267ae76810420295a1682cf779c49f3ddec.tar.gz focaccia-qemu-c4d88267ae76810420295a1682cf779c49f3ddec.zip | |
vl.c: Fix broken -usb option
Commit 094b287f0b accidentally broke the "-usb" command line
option, so it would have no effect if the user had not specified
any machine options at that point. (the return value from
'qemu_opts_find(qemu_find_opts("machine"), 0);' is NULL if there
are no user specified options, so it is only to be used for
looking up an option, not when trying to set one.) Similarly,
would '-usbdevice' no longer cause USB to default to enabled.
Fix this regression by using the same style of code for forcing
the usb=on machine option that we use for other aliases such as
'-enable-kvm'.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Alexander Graf <agraf@suse.de>
| -rw-r--r-- | vl.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/vl.c b/vl.c index c8e9c782d6..a3ab3841a7 100644 --- a/vl.c +++ b/vl.c @@ -3273,16 +3273,12 @@ int main(int argc, char **argv, char **envp) break; } case QEMU_OPTION_usb: - machine_opts = qemu_opts_find(qemu_find_opts("machine"), 0); - if (machine_opts) { - qemu_opt_set_bool(machine_opts, "usb", true); - } + olist = qemu_find_opts("machine"); + qemu_opts_parse(olist, "usb=on", 0); break; case QEMU_OPTION_usbdevice: - machine_opts = qemu_opts_find(qemu_find_opts("machine"), 0); - if (machine_opts) { - qemu_opt_set_bool(machine_opts, "usb", true); - } + olist = qemu_find_opts("machine"); + qemu_opts_parse(olist, "usb=on", 0); add_device_config(DEV_USB, optarg); break; case QEMU_OPTION_device: |