mirror of
https://github.com/qemu/qemu.git
synced 2024-11-27 13:53:45 +08:00
event-notifier: Add "is_external" parameter
All callers pass "false" keeping the old semantics. The windows implementation doesn't distinguish the flag yet. On posix, it is passed down to the underlying aio context. Signed-off-by: Fam Zheng <famz@redhat.com> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
parent
bcd82a968f
commit
54e18d35e4
@ -407,7 +407,7 @@ static int init_event_notifier(EmulatedState *card)
|
||||
DPRINTF(card, 2, "event notifier creation failed\n");
|
||||
return -1;
|
||||
}
|
||||
event_notifier_set_handler(&card->notifier, card_event_handler);
|
||||
event_notifier_set_handler(&card->notifier, false, card_event_handler);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
@ -1775,10 +1775,10 @@ void virtio_queue_set_guest_notifier_fd_handler(VirtQueue *vq, bool assign,
|
||||
bool with_irqfd)
|
||||
{
|
||||
if (assign && !with_irqfd) {
|
||||
event_notifier_set_handler(&vq->guest_notifier,
|
||||
event_notifier_set_handler(&vq->guest_notifier, false,
|
||||
virtio_queue_guest_notifier_read);
|
||||
} else {
|
||||
event_notifier_set_handler(&vq->guest_notifier, NULL);
|
||||
event_notifier_set_handler(&vq->guest_notifier, false, NULL);
|
||||
}
|
||||
if (!assign) {
|
||||
/* Test and clear notifier before closing it,
|
||||
@ -1829,10 +1829,10 @@ void virtio_queue_set_host_notifier_fd_handler(VirtQueue *vq, bool assign,
|
||||
bool set_handler)
|
||||
{
|
||||
if (assign && set_handler) {
|
||||
event_notifier_set_handler(&vq->host_notifier,
|
||||
event_notifier_set_handler(&vq->host_notifier, false,
|
||||
virtio_queue_host_notifier_read);
|
||||
} else {
|
||||
event_notifier_set_handler(&vq->host_notifier, NULL);
|
||||
event_notifier_set_handler(&vq->host_notifier, false, NULL);
|
||||
}
|
||||
if (!assign) {
|
||||
/* Test and clear notifier before after disabling event,
|
||||
|
@ -34,7 +34,9 @@ int event_notifier_init(EventNotifier *, int active);
|
||||
void event_notifier_cleanup(EventNotifier *);
|
||||
int event_notifier_set(EventNotifier *);
|
||||
int event_notifier_test_and_clear(EventNotifier *);
|
||||
int event_notifier_set_handler(EventNotifier *, EventNotifierHandler *);
|
||||
int event_notifier_set_handler(EventNotifier *,
|
||||
bool is_external,
|
||||
EventNotifierHandler *);
|
||||
|
||||
#ifdef CONFIG_POSIX
|
||||
void event_notifier_init_fd(EventNotifier *, int fd);
|
||||
|
@ -9,3 +9,13 @@ void qemu_set_fd_handler(int fd,
|
||||
{
|
||||
abort();
|
||||
}
|
||||
|
||||
void aio_set_fd_handler(AioContext *ctx,
|
||||
int fd,
|
||||
bool is_external,
|
||||
IOHandler *io_read,
|
||||
IOHandler *io_write,
|
||||
void *opaque)
|
||||
{
|
||||
abort();
|
||||
}
|
||||
|
@ -88,7 +88,7 @@ HvSintRoute *kvm_hv_sint_route_create(uint32_t vcpu_id, uint32_t sint,
|
||||
goto err_sint_set_notifier;
|
||||
}
|
||||
|
||||
event_notifier_set_handler(&sint_route->sint_ack_notifier,
|
||||
event_notifier_set_handler(&sint_route->sint_ack_notifier, false,
|
||||
kvm_hv_sint_ack_handler);
|
||||
|
||||
gsi = kvm_irqchip_add_hv_sint_route(kvm_state, vcpu_id, sint);
|
||||
@ -112,7 +112,7 @@ HvSintRoute *kvm_hv_sint_route_create(uint32_t vcpu_id, uint32_t sint,
|
||||
err_irqfd:
|
||||
kvm_irqchip_release_virq(kvm_state, gsi);
|
||||
err_gsi:
|
||||
event_notifier_set_handler(&sint_route->sint_ack_notifier, NULL);
|
||||
event_notifier_set_handler(&sint_route->sint_ack_notifier, false, NULL);
|
||||
event_notifier_cleanup(&sint_route->sint_ack_notifier);
|
||||
err_sint_set_notifier:
|
||||
event_notifier_cleanup(&sint_route->sint_set_notifier);
|
||||
@ -128,7 +128,7 @@ void kvm_hv_sint_route_destroy(HvSintRoute *sint_route)
|
||||
&sint_route->sint_set_notifier,
|
||||
sint_route->gsi);
|
||||
kvm_irqchip_release_virq(kvm_state, sint_route->gsi);
|
||||
event_notifier_set_handler(&sint_route->sint_ack_notifier, NULL);
|
||||
event_notifier_set_handler(&sint_route->sint_ack_notifier, false, NULL);
|
||||
event_notifier_cleanup(&sint_route->sint_ack_notifier);
|
||||
event_notifier_cleanup(&sint_route->sint_set_notifier);
|
||||
g_free(sint_route);
|
||||
|
@ -91,9 +91,11 @@ int event_notifier_get_fd(const EventNotifier *e)
|
||||
}
|
||||
|
||||
int event_notifier_set_handler(EventNotifier *e,
|
||||
bool is_external,
|
||||
EventNotifierHandler *handler)
|
||||
{
|
||||
qemu_set_fd_handler(e->rfd, (IOHandler *)handler, NULL, e);
|
||||
aio_set_fd_handler(iohandler_get_aio_context(), e->rfd, is_external,
|
||||
(IOHandler *)handler, NULL, e);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
@ -33,6 +33,7 @@ HANDLE event_notifier_get_handle(EventNotifier *e)
|
||||
}
|
||||
|
||||
int event_notifier_set_handler(EventNotifier *e,
|
||||
bool is_external,
|
||||
EventNotifierHandler *handler)
|
||||
{
|
||||
if (handler) {
|
||||
|
Loading…
Reference in New Issue
Block a user