diff options
author | Paolo Bonzini <pbonzini@redhat.com> | 2014-07-09 11:53:01 +0200 |
---|---|---|
committer | Stefan Hajnoczi <stefanha@redhat.com> | 2014-08-29 10:46:58 +0100 |
commit | 845ca10dd089b4e48f0a79bad005fb30eb77584e (patch) | |
tree | 18d8041935c716ea293399136f769dafdbd8614d /aio-posix.c | |
parent | 3cbbe9fd1feaf3264f745fccb0bf5f62c583078f (diff) | |
download | qemu-845ca10dd089b4e48f0a79bad005fb30eb77584e.zip qemu-845ca10dd089b4e48f0a79bad005fb30eb77584e.tar.gz qemu-845ca10dd089b4e48f0a79bad005fb30eb77584e.tar.bz2 |
AioContext: take bottom halves into account when computing aio_poll timeout
Right now, QEMU invokes aio_bh_poll before the "poll" phase
of aio_poll. It is simpler to do it afterwards and skip the
"poll" phase altogether when the OS-dependent parts of AioContext
are invoked from GSource. This way, AioContext behaves more
similarly when used as a GSource vs. when used as stand-alone.
As a start, take bottom halves into account when computing the
poll timeout. If a bottom half is ready, do a non-blocking
poll. As a side effect, this makes idle bottom halves work
with aio_poll; an improvement, but not really an important
one since they are deprecated.
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Diffstat (limited to 'aio-posix.c')
-rw-r--r-- | aio-posix.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/aio-posix.c b/aio-posix.c index 2eada2e..55706f8 100644 --- a/aio-posix.c +++ b/aio-posix.c @@ -249,7 +249,7 @@ bool aio_poll(AioContext *ctx, bool blocking) /* wait until next event */ ret = qemu_poll_ns((GPollFD *)ctx->pollfds->data, ctx->pollfds->len, - blocking ? timerlistgroup_deadline_ns(&ctx->tlg) : 0); + blocking ? aio_compute_timeout(ctx) : 0); /* if we have any readable fds, dispatch event */ if (ret > 0) { |