]> git.ipfire.org Git - thirdparty/qemu.git/commit
virtio: notify virtqueue via host notifier when available
authorStefan Hajnoczi <stefanha@redhat.com>
Tue, 5 Nov 2019 14:09:46 +0000 (15:09 +0100)
committerMichael S. Tsirkin <mst@redhat.com>
Wed, 6 Nov 2019 11:35:00 +0000 (06:35 -0500)
commitfcccb271e0894bc04078ababb29d3d5e06b79892
tree3e1b5e8ea018b85f6542cda0f8839f55b70bae98
parent977aff1045b01579e73020a271336e559cdd6b58
virtio: notify virtqueue via host notifier when available

Host notifiers are used in several cases:
1. Traditional ioeventfd where virtqueue notifications are handled in
   the main loop thread.
2. IOThreads (aio_handle_output) where virtqueue notifications are
   handled in an IOThread AioContext.
3. vhost where virtqueue notifications are handled by kernel vhost or
   a vhost-user device backend.

Most virtqueue notifications from the guest use the ioeventfd mechanism,
but there are corner cases where QEMU code calls virtio_queue_notify().
This currently honors the host notifier for the IOThreads
aio_handle_output case, but not for the vhost case.  The result is that
vhost does not receive virtqueue notifications from QEMU when
virtio_queue_notify() is called.

This patch extends virtio_queue_notify() to set the host notifier
whenever it is enabled instead of calling the vq->(aio_)handle_output()
function directly.  We track the host notifier state for each virtqueue
separately since some devices may use it only for certain virtqueues.

This fixes the vhost case although it does add a trip through the
eventfd for the traditional ioeventfd case.  I don't think it's worth
adding a fast path for the traditional ioeventfd case because calling
virtio_queue_notify() is rare when ioeventfd is enabled.

Reported-by: Felipe Franciosi <felipe@nutanix.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Message-Id: <20191105140946.165584-1-stefanha@redhat.com>
Reviewed-by: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
hw/virtio/virtio-bus.c
hw/virtio/virtio.c
include/hw/virtio/virtio.h