[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 01/26] block: remove incorrect coroutine_fn annotations
From: |
Eric Blake |
Subject: |
Re: [PATCH v2 01/26] block: remove incorrect coroutine_fn annotations |
Date: |
Tue, 10 May 2022 10:03:51 -0500 |
User-agent: |
NeoMutt/20220429-35-ca2e7f |
On Mon, May 09, 2022 at 12:29:54PM +0200, Paolo Bonzini wrote:
> This is incorrect because blk_pwritev_part() is called by
> blk_pwrite_zeroes() and blk_pwrite(), neither of which has to be called
> from a coroutine.
>
> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
> ---
> block/block-backend.c | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
Reviewed-by: Eric Blake <eblake@redhat.com>
>
> diff --git a/block/block-backend.c b/block/block-backend.c
> index e0e1aff4b1..fedf2eca83 100644
> --- a/block/block-backend.c
> +++ b/block/block-backend.c
> @@ -1391,10 +1391,10 @@ int coroutine_fn blk_co_pwritev(BlockBackend *blk,
> int64_t offset,
> return blk_co_pwritev_part(blk, offset, bytes, qiov, 0, flags);
> }
>
> -static int coroutine_fn blk_pwritev_part(BlockBackend *blk, int64_t offset,
> - int64_t bytes,
> - QEMUIOVector *qiov, size_t
> qiov_offset,
> - BdrvRequestFlags flags)
> +static int blk_pwritev_part(BlockBackend *blk, int64_t offset,
> + int64_t bytes,
> + QEMUIOVector *qiov, size_t qiov_offset,
> + BdrvRequestFlags flags)
> {
> int ret;
>
> --
> 2.35.1
>
>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
- [PATCH v2 00/26] block: fix coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 03/26] nbd: remove incorrect coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 02/26] qcow2: remove incorrect coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 01/26] block: remove incorrect coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- Re: [PATCH v2 01/26] block: remove incorrect coroutine_fn annotations,
Eric Blake <=
- [PATCH v2 05/26] blkdebug: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 06/26] blkverify: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 04/26] coroutine: remove incorrect coroutine_fn annotations, Paolo Bonzini, 2022/05/09
- [PATCH v2 07/26] block: add missing coroutine_fn annotations, Paolo Bonzini, 2022/05/09