From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35957) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bmhRu-0007VG-NN for qemu-devel@nongnu.org; Wed, 21 Sep 2016 09:20:35 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bmhRo-0001De-Us for qemu-devel@nongnu.org; Wed, 21 Sep 2016 09:20:30 -0400 Received: from mail-wm0-f65.google.com ([74.125.82.65]:34532) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bmhRo-0001DQ-L8 for qemu-devel@nongnu.org; Wed, 21 Sep 2016 09:20:24 -0400 Received: by mail-wm0-f65.google.com with SMTP id l132so8541827wmf.1 for ; Wed, 21 Sep 2016 06:20:24 -0700 (PDT) Sender: Paolo Bonzini From: Paolo Bonzini Date: Wed, 21 Sep 2016 15:18:59 +0200 Message-Id: <1474463939-12223-13-git-send-email-pbonzini@redhat.com> In-Reply-To: <1474463939-12223-1-git-send-email-pbonzini@redhat.com> References: <1474463939-12223-1-git-send-email-pbonzini@redhat.com> Subject: [Qemu-devel] [PATCH 12/12] virtio: inline set_host_notifier_internal List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: borntraeger@de.ibm.com, cornelia.huck@de.ibm.com, mst@redhat.com, famz@redhat.com This is only called from virtio_bus_set_host_notifier. Signed-off-by: Paolo Bonzini --- hw/virtio/virtio-bus.c | 60 +++++++++++++++++++++----------------------------- 1 file changed, 25 insertions(+), 35 deletions(-) diff --git a/hw/virtio/virtio-bus.c b/hw/virtio/virtio-bus.c index b8f0527..1f3526d 100644 --- a/hw/virtio/virtio-bus.c +++ b/hw/virtio/virtio-bus.c @@ -146,40 +146,6 @@ void virtio_bus_set_vdev_config(VirtioBusState *bus, uint8_t *config) } } -static int set_host_notifier_internal(DeviceState *proxy, VirtioBusState *bus, - int n, bool assign) -{ - VirtIODevice *vdev = virtio_bus_get_device(bus); - VirtioBusClass *k = VIRTIO_BUS_GET_CLASS(bus); - VirtQueue *vq = virtio_get_queue(vdev, n); - EventNotifier *notifier = virtio_queue_get_host_notifier(vq); - int r = 0; - - if (assign) { - r = event_notifier_init(notifier, 1); - if (r < 0) { - error_report("%s: unable to init event notifier: %d", __func__, r); - return r; - } - r = k->ioeventfd_assign(proxy, notifier, n, true); - if (r < 0) { - error_report("%s: unable to assign ioeventfd: %d", __func__, r); - goto cleanup_event_notifier; - } - return 0; - } else { - k->ioeventfd_assign(proxy, notifier, n, false); - } - -cleanup_event_notifier: - /* Test and clear notifier before after disabling event, - * in case poll callback didn't have time to run. - */ - virtio_queue_host_notifier_read(notifier); - event_notifier_cleanup(notifier); - return r; -} - int virtio_bus_start_ioeventfd(VirtioBusState *bus) { VirtioBusClass *k = VIRTIO_BUS_GET_CLASS(bus); @@ -229,20 +195,44 @@ bool virtio_bus_ioeventfd_enabled(VirtioBusState *bus) */ int virtio_bus_set_host_notifier(VirtioBusState *bus, int n, bool assign) { + VirtIODevice *vdev = virtio_bus_get_device(bus); VirtioBusClass *k = VIRTIO_BUS_GET_CLASS(bus); DeviceState *proxy = DEVICE(BUS(bus)->parent); + VirtQueue *vq = virtio_get_queue(vdev, n); + EventNotifier *notifier = virtio_queue_get_host_notifier(vq); + int r = 0; if (!k->ioeventfd_assign) { return -ENOSYS; } + if (assign) { assert(!bus->ioeventfd_started); + r = event_notifier_init(notifier, 1); + if (r < 0) { + error_report("%s: unable to init event notifier: %d", __func__, r); + return r; + } + r = k->ioeventfd_assign(proxy, notifier, n, true); + if (r < 0) { + error_report("%s: unable to assign ioeventfd: %d", __func__, r); + goto cleanup_event_notifier; + } + return 0; } else { if (!bus->ioeventfd_started) { return 0; } + k->ioeventfd_assign(proxy, notifier, n, false); } - return set_host_notifier_internal(proxy, bus, n, assign); + +cleanup_event_notifier: + /* Test and clear notifier before after disabling event, + * in case poll callback didn't have time to run. + */ + virtio_queue_host_notifier_read(notifier); + event_notifier_cleanup(notifier); + return r; } static char *virtio_bus_get_dev_path(DeviceState *dev) -- 2.7.4