[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 07/15] block: introduce QEMU_IN_COROUTINE macro
From: |
Kevin Wolf |
Subject: |
Re: [PATCH v5 07/15] block: introduce QEMU_IN_COROUTINE macro |
Date: |
Wed, 23 Nov 2022 17:49:37 +0100 |
Am 23.11.2022 um 12:42 hat Emanuele Giuseppe Esposito geschrieben:
> This macro will be used to mark all coroutine_fn functions.
> Right now, it will be used for the newly introduced coroutine_fn, since
> we know the callers.
>
> As a TODO, in the future we might want to add this macro to all
> corotuine_fn functions, to be sure that they are only called in
s/corotuine_fn/coroutine_fn/
> coroutines context.
>
> Signed-off-by: Emanuele Giuseppe Esposito <eesposit@redhat.com>
I already asked about other opinions on this in patch 1.
These assertions are runtime checks and I don't feel they are the right
tool to verify coroutine_fn consistency. Asserting in tricky places
makes sense to me, especially as long as we can't rely on static
analysis, but adding it everywhere feels over the top to me.
Kevin
- [PATCH v5 00/15] Still more coroutine and various fixes in block layer, Emanuele Giuseppe Esposito, 2022/11/23
- [PATCH v5 03/15] nbd/server.c: add missing coroutine_fn annotations, Emanuele Giuseppe Esposito, 2022/11/23
- [PATCH v5 04/15] block-backend: replace bdrv_*_above with blk_*_above, Emanuele Giuseppe Esposito, 2022/11/23
- [PATCH v5 02/15] block-copy: add missing coroutine_fn annotations, Emanuele Giuseppe Esposito, 2022/11/23
- [PATCH v5 07/15] block: introduce QEMU_IN_COROUTINE macro, Emanuele Giuseppe Esposito, 2022/11/23
- Re: [PATCH v5 07/15] block: introduce QEMU_IN_COROUTINE macro,
Kevin Wolf <=
- [PATCH v5 10/15] block-coroutine-wrapper.py: introduce generated_co_wrapper_simple, Emanuele Giuseppe Esposito, 2022/11/23
- [PATCH v5 06/15] block: avoid duplicating filename string in bdrv_create, Emanuele Giuseppe Esposito, 2022/11/23
- [PATCH v5 05/15] block/vmdk: add missing coroutine_fn annotations, Emanuele Giuseppe Esposito, 2022/11/23
- [PATCH v5 08/15] block: distinguish between bdrv_create running in coroutine and not, Emanuele Giuseppe Esposito, 2022/11/23
- [PATCH v5 09/15] block: bdrv_create_file is a coroutine_fn, Emanuele Giuseppe Esposito, 2022/11/23