[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/12] tap: multiqueue support
From: |
Jason Wang |
Subject: |
Re: [Qemu-devel] [PATCH 01/12] tap: multiqueue support |
Date: |
Wed, 09 Jan 2013 23:25:24 +0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 |
On 01/09/2013 05:56 PM, Stefan Hajnoczi wrote:
> On Fri, Dec 28, 2012 at 06:31:53PM +0800, Jason Wang wrote:
>> diff --git a/qapi-schema.json b/qapi-schema.json
>> index 5dfa052..583eb7c 100644
>> --- a/qapi-schema.json
>> +++ b/qapi-schema.json
>> @@ -2465,7 +2465,7 @@
>> { 'type': 'NetdevTapOptions',
>> 'data': {
>> '*ifname': 'str',
>> - '*fd': 'str',
>> + '*fd': ['String'],
> This change is not backwards-compatible. You need to add a '*fds':
> ['String'] field instead.
I'm not quite understand this case, I think it still work when we we
just specify one fd.
>> '*script': 'str',
>> '*downscript': 'str',
>> '*helper': 'str',
>> @@ -2473,7 +2473,8 @@
>> '*vnet_hdr': 'bool',
>> '*vhost': 'bool',
>> '*vhostfd': 'str',
>> - '*vhostforce': 'bool' } }
>> + '*vhostforce': 'bool',
>> + '*queues': 'uint32'} }
> The 'queues' parameter should not be necessary when fd passing is used
> since we can learn the number of queues by looking at the list length.
Ok.
>
> Stefan