[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Regression: virtio-pci: convert to ioeventfd callbacks
From: |
Cornelia Huck |
Subject: |
Re: [Qemu-devel] Regression: virtio-pci: convert to ioeventfd callbacks |
Date: |
Tue, 28 Jun 2016 09:42:45 +0200 |
On Tue, 28 Jun 2016 10:03:21 +0300
"Michael S. Tsirkin" <address@hidden> wrote:
> I notice cleanup is a bit weird:
>
> virtio_queue_set_host_notifier_fd_handler(vq, false, false);
> k->ioeventfd_assign(proxy, notifier, n, assign);
> event_notifier_cleanup(notifier);
>
> I think virtio_queue_set_host_notifier_fd_handler should happen
> after ioeventfd_assign for symmetry with init?
Looking at the pre-rework code, ccw used the order now in common code,
while pci and mmio used the order you suggest.
"Switch the handler back, then unassign the transport's ioeventfd
backing" made more sense to me (regardless of symmetry) - but we might
lose a notification?
Peter: Can you check whether your problem goes away if you switch the
two lines around?
Re: [Qemu-devel] Regression: virtio-pci: convert to ioeventfd callbacks, Jason Wang, 2016/06/28
Re: [Qemu-devel] Regression: virtio-pci: convert to ioeventfd callbacks, Cornelia Huck, 2016/06/29