diff options
author | Paolo Bonzini <pbonzini@redhat.com> | 2012-09-24 15:11:48 +0200 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2012-10-30 09:30:54 +0100 |
commit | 22bfa75eafc21522afbb265091faa9cc0649e9fb (patch) | |
tree | 43aca4008b8b68b98c997469e0b3ef8d9be8331c /async.c | |
parent | 4c8d0d27676778febad3802a95218d5ceaca171e (diff) | |
download | qemu-22bfa75eafc21522afbb265091faa9cc0649e9fb.tar.gz qemu-22bfa75eafc21522afbb265091faa9cc0649e9fb.tar.bz2 qemu-22bfa75eafc21522afbb265091faa9cc0649e9fb.zip |
aio: clean up now-unused functions
Some cleanups can now be made, now that the main loop does not anymore need
hooks into the bottom half code.
Reviewed-by: Anthony Liguori <aliguori@us.ibm.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'async.c')
-rw-r--r-- | async.c | 23 |
1 files changed, 7 insertions, 16 deletions
@@ -117,16 +117,20 @@ void qemu_bh_delete(QEMUBH *bh) bh->deleted = 1; } -void aio_bh_update_timeout(AioContext *ctx, uint32_t *timeout) +static gboolean +aio_ctx_prepare(GSource *source, gint *timeout) { + AioContext *ctx = (AioContext *) source; QEMUBH *bh; + bool scheduled = false; for (bh = ctx->first_bh; bh; bh = bh->next) { if (!bh->deleted && bh->scheduled) { + scheduled = true; if (bh->idle) { /* idle bottom halves will be polled at least * every 10ms */ - *timeout = MIN(10, *timeout); + *timeout = 10; } else { /* non-idle bottom halves will be executed * immediately */ @@ -135,21 +139,8 @@ void aio_bh_update_timeout(AioContext *ctx, uint32_t *timeout) } } } -} - -static gboolean -aio_ctx_prepare(GSource *source, gint *timeout) -{ - AioContext *ctx = (AioContext *) source; - uint32_t wait = -1; - aio_bh_update_timeout(ctx, &wait); - - if (wait != -1) { - *timeout = MIN(*timeout, wait); - return wait == 0; - } - return false; + return scheduled; } static gboolean |