[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>
Thanks!
Thomas