[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 6/7] vhost-user: add multiple queue support
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 6/7] vhost-user: add multiple queue support |
Date: |
Tue, 8 Sep 2015 15:22:30 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 |
On 09/08/2015 01:38 AM, Yuanhan Liu wrote:
> From: Ouyang Changchun <address@hidden>
>
> This patch is initially based a patch from Nikolay Nikolaev.
>
> Here is the latest version for adding vhost-user multiple queue support,
> by creating a nc and vhost_net pair for each queue.
>
Reviewing grammar and interface only:
> +++ b/docs/specs/vhost-user.txt
> @@ -135,6 +135,19 @@ As older slaves don't support negotiating protocol
> features,
> a feature bit was dedicated for this purpose:
> #define VHOST_USER_F_PROTOCOL_FEATURES 30
>
> +Multiple queue support
> +-------------------
> +Multiple queue is treated as a protocal extension, hence the slave has to
s/protocal/protocol/
> +implement protocol features first. Multiple queues is supported only when
> +the protocol feature VHOST_USER_PROTOCOL_F_MQ(bit 0) is set.
> +
> +The max # of queues the slave support can be queried with message
s/#/number/
s/support/supports/
> +VHOST_USER_GET_PROTOCOL_FEATURES. Master should stop when the # of requested
s/#/number/
> +queues is bigger than that.
> +
> +As all queues share one connection, the master use a unique index for each
s/use/uses/
> +queue in the sent message to identify one specified queue.
> +
> +++ b/qapi-schema.json
> @@ -2480,12 +2480,16 @@
> #
> # @vhostforce: #optional vhost on for non-MSIX virtio guests (default:
> false).
> #
> +# @queues: #optional number of queues to be created for multiqueue vhost-user
> +# (default: 1) (Since 2.5)
> +#
> # Since 2.1
> ##
> { 'struct': 'NetdevVhostUserOptions',
> 'data': {
> 'chardev': 'str',
> - '*vhostforce': 'bool' } }
> + '*vhostforce': 'bool',
> + '*queues': 'int' } }
Looks okay.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH 5/7] vhost_net: move vhost_net_set_vq_index ahead at vhost_net_init, (continued)
- [Qemu-devel] [PATCH 2/7] vhost-user: add protocol feature negotiation, Yuanhan Liu, 2015/09/08
- [Qemu-devel] [PATCH 6/7] vhost-user: add multiple queue support, Yuanhan Liu, 2015/09/08
- Re: [Qemu-devel] [PATCH 6/7] vhost-user: add multiple queue support,
Eric Blake <=
- Re: [Qemu-devel] [PATCH 6/7] vhost-user: add multiple queue support, Ouyang, Changchun, 2015/09/09
- Re: [Qemu-devel] [PATCH 6/7] vhost-user: add multiple queue support, Michael S. Tsirkin, 2015/09/09
- Re: [Qemu-devel] [PATCH 6/7] vhost-user: add multiple queue support, Michael S. Tsirkin, 2015/09/09
- Re: [Qemu-devel] [PATCH 6/7] vhost-user: add multiple queue support, Jason Wang, 2015/09/14