On 20.02.2018 23:35, Eric Blake wrote:
> On 02/20/2018 11:40 AM, Thomas Huth wrote:
>> "-net dump" has been marked as deprecated since QEMU v2.10, since it
>> only works with the deprecated 'vlan' parameter (or hubs). Network
>> dumping should be done with "-object filter-dump" nowadays instead.
>> Since nobody complained so far about the deprecation message, let's
>> finally get rid of "-net dump" now.
>>
>> Reviewed-by: Paolo Bonzini <pbonz...@redhat.com>
>> Reviewed-by: Eric Blake <ebl...@redhat.com>
>> Signed-off-by: Thomas Huth <th...@redhat.com>
>> ---
>
>> +++ b/qapi/net.json
>> @@ -39,8 +39,10 @@
>> #
>> # Add a network backend.
>> #
>> -# @type: the type of network backend. Current valid values are
>> 'user', 'tap',
>> -# 'vde', 'socket', 'dump' and 'bridge'
>> +# @type: the type of network backend. Possible values in version
>> +# 2.11: 'user', 'tap', 'vde', 'socket', 'hubport', 'bridge',
>> +# 'dump', 'l2tpv3', 'netmap', 'vhost-user'
>> +# 2.12: 'dump' dropped
>> #
>
> That's a bit fuzzy, especially since the command has been around since
> 0.14. It might be easier to word it as:
>
> @type: the type of network backend. Possible values are listed in
> NetClientDriver.
That's a really good idea to avoid that we have to maintain the list
multiple times...
> Then defer the actual listing by version...
>
>> @@ -468,7 +453,7 @@
>> # Since: 2.7
>> ##
>> { 'enum': 'NetClientDriver',
>> - 'data': [ 'none', 'nic', 'user', 'tap', 'l2tpv3', 'socket', 'vde',
>> 'dump',
>> + 'data': [ 'none', 'nic', 'user', 'tap', 'l2tpv3', 'socket', 'vde',
>> 'bridge', 'hubport', 'netmap', 'vhost-user' ] }
>
> ...to here
... but 'none' and 'nic' are not valid for netdev_add. Well, I guess I
can simply write something like "Possible values are listed in
NetClientDriver (excluding 'none' and 'nic')". I'll send a v3 with that.
Thomas
PS: By the way, now that you've mentioned this, I think "dump" was also
not valid for netdev_add, so the comment was even wrong there.
