[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 4/8] virtio: export vring_notify as virtio_shoul
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH 4/8] virtio: export vring_notify as virtio_should_notify |
Date: |
Tue, 16 Feb 2016 15:21:03 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sun, 02/14 18:17, Paolo Bonzini wrote:
> Virtio dataplane needs to trigger the irq manually through the
> guest notifier. Export virtio_should_notify so that it can be
> used around event_notifier_set.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> hw/virtio/virtio.c | 4 ++--
> include/hw/virtio/virtio.h | 1 +
> 2 files changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/hw/virtio/virtio.c b/hw/virtio/virtio.c
> index 3a5cca4..abb97f4 100644
> --- a/hw/virtio/virtio.c
> +++ b/hw/virtio/virtio.c
> @@ -1162,7 +1162,7 @@ void virtio_irq(VirtQueue *vq)
> virtio_notify_vector(vq->vdev, vq->vector);
> }
>
> -static bool vring_notify(VirtIODevice *vdev, VirtQueue *vq)
> +bool virtio_should_notify(VirtIODevice *vdev, VirtQueue *vq)
> {
> uint16_t old, new;
> bool v;
> @@ -1187,7 +1187,7 @@ static bool vring_notify(VirtIODevice *vdev, VirtQueue
> *vq)
>
> void virtio_notify(VirtIODevice *vdev, VirtQueue *vq)
> {
> - if (!vring_notify(vdev, vq)) {
> + if (!virtio_should_notify(vdev, vq)) {
> return;
> }
>
> diff --git a/include/hw/virtio/virtio.h b/include/hw/virtio/virtio.h
> index 4ce01a1..5884228 100644
> --- a/include/hw/virtio/virtio.h
> +++ b/include/hw/virtio/virtio.h
> @@ -162,6 +162,7 @@ void virtqueue_get_avail_bytes(VirtQueue *vq, unsigned
> int *in_bytes,
> unsigned int *out_bytes,
> unsigned max_in_bytes, unsigned
> max_out_bytes);
>
> +bool virtio_should_notify(VirtIODevice *vdev, VirtQueue *vq);
> void virtio_notify(VirtIODevice *vdev, VirtQueue *vq);
>
> void virtio_save(VirtIODevice *vdev, QEMUFile *f);
> --
> 1.8.3.1
>
>
>
Reviewed-by: Fam Zheng <address@hidden>
- Re: [Qemu-devel] [PATCH 5/8] virtio-blk: fix "disabled data plane" mode, (continued)
- [Qemu-devel] [PATCH 2/8] vring: make vring_enable_notification return void, Paolo Bonzini, 2016/02/14
- [Qemu-devel] [PATCH 3/8] virtio: add AioContext-specific function for host notifiers, Paolo Bonzini, 2016/02/14
- [Qemu-devel] [PATCH 4/8] virtio: export vring_notify as virtio_should_notify, Paolo Bonzini, 2016/02/14
- [Qemu-devel] [PATCH 6/8] virtio-blk: do not use vring in dataplane, Paolo Bonzini, 2016/02/14
- [Qemu-devel] [PATCH 7/8] virtio-scsi: do not use vring in dataplane, Paolo Bonzini, 2016/02/14
- [Qemu-devel] [PATCH 8/8] vring: remove, Paolo Bonzini, 2016/02/14
- Re: [Qemu-devel] [PATCH 0/8] virtio: allow migration with dataplane, Christian Borntraeger, 2016/02/16
- Re: [Qemu-devel] [PATCH 0/8] virtio: allow migration with dataplane, Michael S. Tsirkin, 2016/02/19