[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 14/14] vdpa: Add x-svq to NetdevVhostVDPAOptions
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v4 14/14] vdpa: Add x-svq to NetdevVhostVDPAOptions |
Date: |
Fri, 04 Mar 2022 07:35:05 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Eugenio Pérez <eperezma@redhat.com> writes:
> Finally offering the possibility to enable SVQ from the command line.
>
> Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
> ---
> qapi/net.json | 8 +++++++-
> net/vhost-vdpa.c | 48 ++++++++++++++++++++++++++++++++++++++++--------
> 2 files changed, 47 insertions(+), 9 deletions(-)
>
> diff --git a/qapi/net.json b/qapi/net.json
> index 7fab2e7cd8..06a74d4224 100644
> --- a/qapi/net.json
> +++ b/qapi/net.json
> @@ -445,12 +445,18 @@
> # @queues: number of queues to be created for multiqueue vhost-vdpa
> # (default: 1)
> #
> +# @x-svq: Start device with (experimental) shadow virtqueue. (Since 7.0)
> +#
> +# Features:
> +# @unstable: Member @x-svq could change in future revisions.
Elsewhere we document "Member @foo is experimental." Does your
different phrasing indicate a difference in intent?
> +#
> # Since: 5.1
> ##
> { 'struct': 'NetdevVhostVDPAOptions',
> 'data': {
> '*vhostdev': 'str',
> - '*queues': 'int' } }
> + '*queues': 'int',
> + '*x-svq': {'type': 'bool', 'features' : [ 'unstable'] } } }
>
> ##
> # @NetClientDriver:
Do you hope to make @x-svq stable eventually? If yes: you'll want to
rename it to @svq then, which could be a bother. Can be avoided by
naming it @svq now. Up to you.
[...]
- [PATCH v4 07/14] vhost: Shadow virtqueue buffers forwarding, (continued)
- [PATCH v4 07/14] vhost: Shadow virtqueue buffers forwarding, Eugenio Pérez, 2022/03/03
- [PATCH v4 08/14] util: Add iova_tree_alloc_map, Eugenio Pérez, 2022/03/03
- [PATCH v4 09/14] vhost: Add VhostIOVATree, Eugenio Pérez, 2022/03/03
- [PATCH v4 10/14] vdpa: Add custom IOTLB translations to SVQ, Eugenio Pérez, 2022/03/03
- [PATCH v4 11/14] vdpa: Adapt vhost_vdpa_get_vring_base to SVQ, Eugenio Pérez, 2022/03/03
- [PATCH v4 12/14] vdpa: Never set log_base addr if SVQ is enabled, Eugenio Pérez, 2022/03/03
- [PATCH v4 13/14] vdpa: Expose VHOST_F_LOG_ALL on SVQ, Eugenio Pérez, 2022/03/03
- [PATCH v4 14/14] vdpa: Add x-svq to NetdevVhostVDPAOptions, Eugenio Pérez, 2022/03/03
- Re: [PATCH v4 14/14] vdpa: Add x-svq to NetdevVhostVDPAOptions,
Markus Armbruster <=