[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 02/26] qcow2: remove incorrect coroutine_fn annotations
From: |
Eric Blake |
Subject: |
Re: [PATCH 02/26] qcow2: remove incorrect coroutine_fn annotations |
Date: |
Tue, 19 Apr 2022 13:07:19 -0500 |
User-agent: |
NeoMutt/20211029-35-db88c3 |
On Fri, Apr 15, 2022 at 03:18:36PM +0200, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
Again, a sentence on why this is correct would be helpful.
> block/qcow2-refcount.c | 4 ++--
> block/qcow2.h | 4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c
> index b91499410c..b6f90b2702 100644
> --- a/block/qcow2-refcount.c
> +++ b/block/qcow2-refcount.c
> @@ -1206,7 +1206,7 @@ void qcow2_free_any_cluster(BlockDriverState *bs,
> uint64_t l2_entry,
> }
> }
>
> -int coroutine_fn qcow2_write_caches(BlockDriverState *bs)
> +int qcow2_write_caches(BlockDriverState *bs)
> {
> BDRVQcow2State *s = bs->opaque;
> int ret;
> @@ -1226,7 +1226,7 @@ int coroutine_fn qcow2_write_caches(BlockDriverState
> *bs)
> return 0;
> }
>
> -int coroutine_fn qcow2_flush_caches(BlockDriverState *bs)
> +int qcow2_flush_caches(BlockDriverState *bs)
> {
> int ret = qcow2_write_caches(bs);
Both of these eventually hit qcow2_cache_write, which is not marked
coroutine, so these should not be either.
Reviewed-by: Eric Blake <eblake@redhat.com>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
[PATCH 06/26] blkverify: add missing coroutine_fn annotations, Paolo Bonzini, 2022/04/15
[PATCH 07/26] block: add missing coroutine_fn annotations, Paolo Bonzini, 2022/04/15
[PATCH 05/26] blkdebug: add missing coroutine_fn annotations, Paolo Bonzini, 2022/04/15