]> git.proxmox.com Git - qemu.git/commit - trace-events
coroutine: stop using AioContext in CoQueue
authorStefan Hajnoczi <stefanha@redhat.com>
Fri, 17 May 2013 13:51:26 +0000 (15:51 +0200)
committerStefan Hajnoczi <stefanha@redhat.com>
Fri, 24 May 2013 14:17:56 +0000 (16:17 +0200)
commit02ffb504485f0920cfc75a0982a602f824a9a4f4
tree036ffed1e7ca8bed9042bcaf19f439b1065dbf01
parentb84c4586234b26ccc875595713f6f4491e5b3385
coroutine: stop using AioContext in CoQueue

qemu_co_queue_next(&queue) arranges that the next queued coroutine is
run at a later point in time.  This deferred restart is useful because
the caller may not want to transfer control yet.

This behavior was implemented using QEMUBH in the past, which meant that
CoQueue (and hence CoMutex and CoRwlock) had a dependency on the
AioContext event loop.  This hidden dependency causes trouble when we
move to a world with multiple event loops - now qemu_co_queue_next()
needs to know which event loop to schedule the QEMUBH in.

After pondering how to stash AioContext I realized the best solution is
to not use AioContext at all.  This patch implements the deferred
restart behavior purely in terms of coroutines and no longer uses
QEMUBH.

Here is how it works:

Each Coroutine has a wakeup queue that starts out empty.  When
qemu_co_queue_next() is called, the next coroutine is added to our
wakeup queue.  The wakeup queue is processed when we yield or terminate.

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
include/block/coroutine_int.h
qemu-coroutine-lock.c
qemu-coroutine.c
trace-events