coroutine: abort if we try to schedule or enter a pending coroutine
commit6133b39f3c36623425a6ede9e89d93175fde15cd
authorJeff Cody <jcody@redhat.com>
Sat, 18 Nov 2017 03:27:09 +0000 (17 22:27 -0500)
committerJeff Cody <jcody@redhat.com>
Tue, 21 Nov 2017 16:58:07 +0000 (21 11:58 -0500)
tree6ea888f4baa110f6e1dbf434e2ff25d275091a1f
parent4afeffc8572f40d8844b946a30c00b10da4442b1
coroutine: abort if we try to schedule or enter a pending coroutine

The previous patch fixed a race condition, in which there were
coroutines being executing doubly, or after coroutine deletion.

We can detect common scenarios when this happens, and print an error
message and abort before we corrupt memory / data, or segfault.

This patch will abort if an attempt to enter a coroutine is made while
it is currently pending execution, either in a specific AioContext bh,
or pending execution via a timer.  It will also abort if a coroutine
is scheduled, before a prior scheduled run has occurred.

We cannot rely on the existing co->caller check for recursive re-entry
to catch this, as the coroutine may run and exit with
COROUTINE_TERMINATE before the scheduled coroutine executes.

(This is the scenario that was occurring and fixed in the previous
patch).

This patch also re-orders the Coroutine struct elements in an attempt to
optimize caching.

Signed-off-by: Jeff Cody <jcody@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
include/qemu/coroutine_int.h
util/async.c
util/qemu-coroutine-sleep.c
util/qemu-coroutine.c