On 30.05.2017 07:20, Markus Armbruster wrote:
> Thomas Huth <th...@redhat.com> 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 <th...@redhat.com>
>> ---
[...]
>> /* 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 serial=@var{serial}
>> This option specifies the serial number to assign to the device.
>> @item addr=@var{addr}
>> -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 werror=@var{action},rerror=@var{action}
>> 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 <arm...@redhat.com>
Thanks!
Thomas
