[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/2] Add usb option in machine option

From: Andreas Färber
Subject: Re: [Qemu-ppc] [Qemu-devel] [PATCH 1/2] Add usb option in machine options to enable/disable usb
Date: Wed, 27 Jun 2012 15:24:15 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:12.0) Gecko/20120421 Thunderbird/12.0

Am 27.06.2012 15:13, schrieb Li Zhang:
> On Wed, Jun 27, 2012 at 8:00 PM, Andreas Färber <address@hidden> wrote:
>> Am 18.06.2012 11:22, schrieb Li Zhang:
>>> +    bool usb_on = false;
>> Didn't you want this to be true for sPAPR in absence of -machine?
> It is set in the following:
>      if (machine_opts)
>         usb_on = qemu_opt_get_bool(machine_opts, "usb", true);
> It means that when using "-machine" option, usb_on is set as true if
> usb option is not specified.

What I mean is:

-machine usb=on => usb_on == true
-machine usb=off => usb_on == false
-machine => usb_on == true
(nothing) => usb_on == false  <-- this !machine_opts case

There you should assign true as default, false will be assigned only
from qemu_opt_get_bool().

>> Other than that looking good to me now. Where's 2/2?
> 2/2's title is as the following:
> [Qemu-devel][PATCH 2/2] spapr: Add support for -vga option. :)

Please thread the messages together then when sending out the next
version. :)


SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg

reply via email to

[Prev in Thread] Current Thread [Next in Thread]