[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] Print a warning message if the user specifie
From: |
Thomas Huth |
Subject: |
Re: [Qemu-devel] [PATCH v2] Print a warning message if the user specified a legacy USB option |
Date: |
Tue, 2 May 2017 10:40:49 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 |
On 02.05.2017 10:18, Paolo Bonzini wrote:
>
>
> On 02/05/2017 10:07, Thomas Huth wrote:
>> '-usb' and '-usbdevice' are considered as deprecated nowadays and
>> we might want to remove these options in a future version of QEMU.
>> So mark this options as deprecated in the documenation and print out
>> a warning if they are used to tell the user what to use instead.
>>
>> Signed-off-by: Thomas Huth <address@hidden>
>> ---
>> v2: Also update the documentation accordingly
>>
>> qemu-options.hx | 8 +++++---
>> vl.c | 4 ++++
>> 2 files changed, 9 insertions(+), 3 deletions(-)
>>
>> diff --git a/qemu-options.hx b/qemu-options.hx
>> index f68829f..57bacc5 100644
>> --- a/qemu-options.hx
>> +++ b/qemu-options.hx
>> @@ -946,12 +946,13 @@ STEXI
>> ETEXI
>>
>> DEF("usb", 0, QEMU_OPTION_usb,
>> - "-usb enable the USB driver (will be the default soon)\n",
>> + "-usb enable the USB driver (if it is not used by default
>> yet)\n",
>> QEMU_ARCH_ALL)
>> STEXI
>> @item -usb
>> @findex -usb
>> -Enable the USB driver (will be the default soon)
>> +Enable the USB driver (if it is not used by default yet). Note that this
>> +option is deprecated, please use @code{-M usb=on} instead.
>
> This is pointless, "-usb" is just syntactic sugar and doesn't add any
> complex logic.
Ok.
>> ETEXI
>>
>> DEF("usbdevice", HAS_ARG, QEMU_OPTION_usbdevice,
>> @@ -961,7 +962,8 @@ STEXI
>>
>> @item -usbdevice @var{devname}
>> @findex -usbdevice
>> -Add the USB device @var{devname}. @xref{usb_devices}.
>> +Add the USB device @var{devname}. Note that this option is deperecated,
>> +please use @code{-device usb-...} instead. See also @xref{usb_devices}.
>>
>> @table @option
>
> This makes sense, but the usb_devices section of the manual still
> documents -usbdevice instead of "-device usb-...".
I noticed that, too, after sending my patch ... actually, that section
is really out of date, e.g. it also says that QEMU only "emulates a PCI
UHCI USB controller" which is just not true nowadays anymore... Looks
like we need to spend some effort here to update it properly...
Thomas