[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v2 08/16] hw/xen: do not use aio_set_fd_handler(is_external=true) in xen_xenstore
- To: qemu-devel@xxxxxxxxxx
- From: Stefan Hajnoczi <stefanha@xxxxxxxxxx>
- Date: Wed, 19 Apr 2023 13:28:09 -0400
- Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>, Marcel Apfelbaum <marcel.apfelbaum@xxxxxxxxx>, Fam Zheng <fam@xxxxxxxxxx>, Stefan Hajnoczi <stefanha@xxxxxxxxxx>, Julia Suvorova <jusual@xxxxxxxxxx>, Hanna Reitz <hreitz@xxxxxxxxxx>, Daniel P. Berrangé <berrange@xxxxxxxxxx>, Paolo Bonzini <pbonzini@xxxxxxxxxx>, Coiby Xu <Coiby.Xu@xxxxxxxxx>, Paul Durrant <paul@xxxxxxx>, Ronnie Sahlberg <ronniesahlberg@xxxxxxxxx>, Eduardo Habkost <eduardo@xxxxxxxxxxx>, Juan Quintela <quintela@xxxxxxxxxx>, "Michael S. Tsirkin" <mst@xxxxxxxxxx>, Stefano Garzarella <sgarzare@xxxxxxxxxx>, Anthony Perard <anthony.perard@xxxxxxxxxx>, Kevin Wolf <kwolf@xxxxxxxxxx>, "Richard W.M. Jones" <rjones@xxxxxxxxxx>, Richard Henderson <richard.henderson@xxxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxxx, qemu-block@xxxxxxxxxx, "Dr. David Alan Gilbert" <dgilbert@xxxxxxxxxx>, Philippe Mathieu-Daudé <philmd@xxxxxxxxxx>, Peter Lieven <pl@xxxxxxx>, eesposit@xxxxxxxxxx, Aarushi Mehta <mehta.aaru20@xxxxxxxxx>, Stefan Weil <sw@xxxxxxxxxxx>, Xie Yongji <xieyongji@xxxxxxxxxxxxx>, David Woodhouse <dwmw2@xxxxxxxxxxxxx>, David Woodhouse <dwmw@xxxxxxxxxxxx>
- Delivery-date: Wed, 19 Apr 2023 17:28:59 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
There is no need to suspend activity between aio_disable_external() and
aio_enable_external(), which is mainly used for the block layer's drain
operation.
This is part of ongoing work to remove the aio_disable_external() API.
Reviewed-by: David Woodhouse <dwmw@xxxxxxxxxxxx>
Signed-off-by: Stefan Hajnoczi <stefanha@xxxxxxxxxx>
---
hw/i386/kvm/xen_xenstore.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/hw/i386/kvm/xen_xenstore.c b/hw/i386/kvm/xen_xenstore.c
index 900679af8a..6e81bc8791 100644
--- a/hw/i386/kvm/xen_xenstore.c
+++ b/hw/i386/kvm/xen_xenstore.c
@@ -133,7 +133,7 @@ static void xen_xenstore_realize(DeviceState *dev, Error
**errp)
error_setg(errp, "Xenstore evtchn port init failed");
return;
}
- aio_set_fd_handler(qemu_get_aio_context(), xen_be_evtchn_fd(s->eh), true,
+ aio_set_fd_handler(qemu_get_aio_context(), xen_be_evtchn_fd(s->eh), false,
xen_xenstore_event, NULL, NULL, NULL, s);
s->impl = xs_impl_create(xen_domid);
--
2.39.2
|