[Top][All Lists]

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

Re: [Qemu-block] [Qemu-devel] [PATCH v2] blockdev: Print a warning for l

From: Thomas Huth
Subject: Re: [Qemu-block] [Qemu-devel] [PATCH v2] blockdev: Print a warning for legacy drive options that belong to -device
Date: Tue, 30 May 2017 09:45:33 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.1.0

On 30.05.2017 07:20, Markus Armbruster wrote:
> Thomas Huth <address@hidden> writes:
>> We likely do not want to carry these legacy -drive options along forever.
>> Let's emit a deprecation warning for the -drive options that have a
>> replacement with the -device option, so that the (hopefully few) remaining
>> users are aware of this and can adapt their scripts / behaviour accordingly.
>> Signed-off-by: Thomas Huth <address@hidden>
>> ---
>>      /* Change legacy command line options into QMP ones */
>>      static const struct {
>> @@ -880,6 +884,16 @@ DriveInfo *drive_new(QemuOpts *all_opts, 
>> BlockInterfaceType block_default_type)
>        if (qemu_opt_get(legacy_opts, "boot") != NULL) {
>            fprintf(stderr, "qemu-kvm: boot=on|off is deprecated and will be "
>                    "ignored. Future versions will reject this parameter. 
> Please "
>>                  "update your scripts.\n");
> Unrelated to this patch: this is ugly.  It's also almost three years
> old.  Can we bury the corpse already?

If you like to get rid of this now, feel free to send a patch ...
otherwise I'll make sure that it'll go away with QEMU v3.0 (it's on the
to-be-removed list on http://wiki.qemu.org/Features/LegacyRemoval already)

>> @@ -631,7 +633,8 @@ an untrusted format header.
>>  @item address@hidden
>>  This option specifies the serial number to assign to the device.
>>  @item address@hidden
>> -Specify the controller's PCI address (if=virtio only).
>> +Specify the controller's PCI address (if=virtio only). This parameter is
>> +deprecated, use the corresponding parameter of @code{-device} instead.
>>  @item address@hidden,address@hidden
>>  Specify which @var{action} to take on write and read errors. Valid actions 
>> are:
>>  "ignore" (ignore the error and try to continue), "stop" (pause QEMU),
> Reviewed-by: Markus Armbruster <address@hidden>


reply via email to

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