[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/8] virtio: add AioContext-specific function fo
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] [PATCH 3/8] virtio: add AioContext-specific function for host notifiers |
Date: |
Tue, 16 Feb 2016 10:42:40 +0100 |
On Tue, 16 Feb 2016 15:20:17 +0800
Fam Zheng <address@hidden> wrote:
> On Sun, 02/14 18:17, Paolo Bonzini wrote:
> > +void virtio_queue_aio_set_host_notifier_handler(VirtQueue *vq, AioContext
> > *ctx,
> > + bool assign, bool
> > set_handler)
> > +{
> > + if (assign && set_handler) {
> > + aio_set_event_notifier(ctx, &vq->host_notifier, true,
> > + virtio_queue_host_notifier_read);
> > + } else {
> > + aio_set_event_notifier(ctx, &vq->host_notifier, true, NULL);
> > + }
> > + if (!assign) {
> > + /* Test and clear notifier before after disabling event,
>
> Does "before after" mean "after"? :)
I think that was copied verbatim from
virtio_queue_set_host_notifier_fd_handler :)
- [Qemu-devel] [PATCH 1/8] block-migration: acquire AioContext as necessary, (continued)
- [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
- [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