[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH v4 08/20] vhost: Route guest->host notification through s
From: |
Markus Armbruster |
Subject: |
Re: [RFC PATCH v4 08/20] vhost: Route guest->host notification through shadow virtqueue |
Date: |
Tue, 12 Oct 2021 07:19:53 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Eugenio Pérez <eperezma@redhat.com> writes:
> Shadow virtqueue notifications forwarding is disabled when vhost_dev
> stops, so code flow follows usual cleanup.
>
> Also, host notifiers must be disabled at SVQ start, and they will not
> start if SVQ has been enabled when device is stopped. This is trivial
> to address, but it is left out for simplicity at this moment.
>
> Signed-off-by: Eugenio Pérez <eperezma@redhat.com>
> ---
> qapi/net.json | 2 +-
> hw/virtio/vhost-shadow-virtqueue.h | 8 ++
> include/hw/virtio/vhost-vdpa.h | 4 +
> hw/virtio/vhost-shadow-virtqueue.c | 138 ++++++++++++++++++++++++++++-
> hw/virtio/vhost-vdpa.c | 116 +++++++++++++++++++++++-
> 5 files changed, 264 insertions(+), 4 deletions(-)
>
> diff --git a/qapi/net.json b/qapi/net.json
> index a2c30fd455..fe546b0e7c 100644
> --- a/qapi/net.json
> +++ b/qapi/net.json
> @@ -88,7 +88,7 @@
> #
> # @enable: true to use the alternate shadow VQ notifications
> #
> -# Returns: Always error, since SVQ is not implemented at the moment.
> +# Returns: Error if failure, or 'no error' for success.
Delete the whole line, please.
> #
> # Since: 6.2
> #
[...]
- [RFC PATCH v4 04/20] vhost: Make vhost_virtqueue_{start,stop} public, (continued)
[RFC PATCH v4 06/20] vhost: Add VhostShadowVirtqueue, Eugenio Pérez, 2021/10/01
[RFC PATCH v4 07/20] vdpa: Register vdpa devices in a list, Eugenio Pérez, 2021/10/01
[RFC PATCH v4 08/20] vhost: Route guest->host notification through shadow virtqueue, Eugenio Pérez, 2021/10/01
[RFC PATCH v4 09/20] vdpa: Save call_fd in vhost-vdpa, Eugenio Pérez, 2021/10/01
[RFC PATCH v4 10/20] vhost-vdpa: Take into account SVQ in vhost_vdpa_set_vring_call, Eugenio Pérez, 2021/10/01