[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-block] [PATCH 2/5] coroutine: abort if we try to
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [Qemu-block] [PATCH 2/5] coroutine: abort if we try to enter coroutine scheduled for another ctx |
Date: |
Mon, 20 Nov 2017 11:28:26 +0000 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Sun, Nov 19, 2017 at 09:46:43PM -0500, Jeff Cody wrote:
> diff --git a/include/qemu/coroutine_int.h b/include/qemu/coroutine_int.h
> index cb98892..931cdc9 100644
> --- a/include/qemu/coroutine_int.h
> +++ b/include/qemu/coroutine_int.h
> @@ -53,6 +53,9 @@ struct Coroutine {
>
> /* Only used when the coroutine has yielded. */
> AioContext *ctx;
> +
> + int scheduled;
s/int/bool/
> diff --git a/util/qemu-coroutine.c b/util/qemu-coroutine.c
> index d6095c1..2edab63 100644
> --- a/util/qemu-coroutine.c
> +++ b/util/qemu-coroutine.c
> @@ -109,6 +109,15 @@ void qemu_aio_coroutine_enter(AioContext *ctx, Coroutine
> *co)
>
> trace_qemu_aio_coroutine_enter(ctx, self, co, co->entry_arg);
>
> + /* if the Coroutine has already been scheduled, entering it again will
> + * cause us to enter it twice, potentially even after the coroutine has
> + * been deleted */
> + if (co->scheduled == 1) {
> + fprintf(stderr, "Cannot enter a co-routine that has already "
> + "been scheduled to run in a different AioContext\n");
This error message is too specific, the AioContext doesn't have to be
different from the current one:
block/blkdebug.c: aio_co_schedule(qemu_get_current_aio_context(),
qemu_coroutine_self());
If something calls qemu_aio_coroutine_enter() on the coroutine it might
be from the same AioContext - but still an error condition worth failing
loudly on.
I suggest simplifying the error message:
fprintf(stderr, "Cannot enter a co-routine that has already "
"been scheduled\n");
signature.asc
Description: PGP signature
- [Qemu-devel] [PATCH 0/5] Fix segfault in blockjob race condition, Jeff Cody, 2017/11/19
- [Qemu-devel] [PATCH 2/5] coroutine: abort if we try to enter coroutine scheduled for another ctx, Jeff Cody, 2017/11/19
- Re: [Qemu-devel] [Qemu-block] [PATCH 2/5] coroutine: abort if we try to enter coroutine scheduled for another ctx,
Stefan Hajnoczi <=
- [Qemu-devel] [PATCH 4/5] qemu-iotests: add option in common.qemu for mismatch only, Jeff Cody, 2017/11/19
- [Qemu-devel] [PATCH 3/5] coroutines: abort if we try to enter a still-sleeping coroutine, Jeff Cody, 2017/11/19
- Re: [Qemu-devel] [PATCH 3/5] coroutines: abort if we try to enter a still-sleeping coroutine, Paolo Bonzini, 2017/11/20