[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/8] vring: make vring_enable_notification retur
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH 2/8] vring: make vring_enable_notification return void |
Date: |
Tue, 16 Feb 2016 15:18:08 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Sun, 02/14 18:17, Paolo Bonzini wrote:
> Make the API more similar to the regular virtqueue API. This will
> help when modifying the code to not use vring.c anymore.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> hw/block/dataplane/virtio-blk.c | 3 ++-
> hw/virtio/dataplane/vring.c | 3 +--
> include/hw/virtio/dataplane/vring.h | 2 +-
> 3 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/hw/block/dataplane/virtio-blk.c b/hw/block/dataplane/virtio-blk.c
> index 0d99781..03b81bc 100644
> --- a/hw/block/dataplane/virtio-blk.c
> +++ b/hw/block/dataplane/virtio-blk.c
> @@ -128,7 +128,8 @@ static void handle_notify(EventNotifier *e)
> /* Re-enable guest->host notifies and stop processing the vring.
> * But if the guest has snuck in more descriptors, keep
> processing.
> */
> - if (vring_enable_notification(s->vdev, &s->vring)) {
> + vring_enable_notification(s->vdev, &s->vring);
> + if (!vring_more_avail(s->vdev, &s->vring)) {
> break;
> }
> } else { /* fatal error */
> diff --git a/hw/virtio/dataplane/vring.c b/hw/virtio/dataplane/vring.c
> index 4308d9f..157e8b8 100644
> --- a/hw/virtio/dataplane/vring.c
> +++ b/hw/virtio/dataplane/vring.c
> @@ -175,7 +175,7 @@ void vring_disable_notification(VirtIODevice *vdev, Vring
> *vring)
> *
> * Return true if the vring is empty, false if there are more requests.
> */
> -bool vring_enable_notification(VirtIODevice *vdev, Vring *vring)
> +void vring_enable_notification(VirtIODevice *vdev, Vring *vring)
> {
> if (virtio_vdev_has_feature(vdev, VIRTIO_RING_F_EVENT_IDX)) {
> vring_avail_event(&vring->vr) = vring->vr.avail->idx;
> @@ -183,7 +183,6 @@ bool vring_enable_notification(VirtIODevice *vdev, Vring
> *vring)
> vring_clear_used_flags(vdev, vring, VRING_USED_F_NO_NOTIFY);
> }
> smp_mb(); /* ensure update is seen before reading avail_idx */
> - return !vring_more_avail(vdev, vring);
> }
>
> /* This is stolen from linux/drivers/vhost/vhost.c:vhost_notify() */
> diff --git a/include/hw/virtio/dataplane/vring.h
> b/include/hw/virtio/dataplane/vring.h
> index e80985e..e1c2a65 100644
> --- a/include/hw/virtio/dataplane/vring.h
> +++ b/include/hw/virtio/dataplane/vring.h
> @@ -42,7 +42,7 @@ static inline void vring_set_broken(Vring *vring)
> bool vring_setup(Vring *vring, VirtIODevice *vdev, int n);
> void vring_teardown(Vring *vring, VirtIODevice *vdev, int n);
> void vring_disable_notification(VirtIODevice *vdev, Vring *vring);
> -bool vring_enable_notification(VirtIODevice *vdev, Vring *vring);
> +void vring_enable_notification(VirtIODevice *vdev, Vring *vring);
> bool vring_should_notify(VirtIODevice *vdev, Vring *vring);
> void *vring_pop(VirtIODevice *vdev, Vring *vring, size_t sz);
> void vring_push(VirtIODevice *vdev, Vring *vring, VirtQueueElement *elem,
> --
> 1.8.3.1
>
>
>
Reviewed-by: Fam Zheng <address@hidden>
- [Qemu-devel] [PATCH 0/8] virtio: allow migration with dataplane, Paolo Bonzini, 2016/02/14
- [Qemu-devel] [PATCH 1/8] block-migration: acquire AioContext as necessary, Paolo Bonzini, 2016/02/14
- [Qemu-devel] [PATCH 5/8] virtio-blk: fix "disabled data plane" mode, Paolo Bonzini, 2016/02/14
- [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