[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 06/19] block: remove bdrv_is_allocated_above/
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH v3 06/19] block: remove bdrv_is_allocated_above/bdrv_co_is_allocated_above distinction |
Date: |
Mon, 29 Jul 2013 15:34:07 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
Am 25.07.2013 um 16:23 hat Paolo Bonzini geschrieben:
> Now that bdrv_is_allocated detects coroutine context, the two can
> use the same code.
>
> Reviewed-by: Eric Blake <address@hidden>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> block.c | 46 ++++------------------------------------------
> block/commit.c | 6 +++---
> block/mirror.c | 4 ++--
> block/stream.c | 4 ++--
> include/block/block.h | 4 ----
> 5 files changed, 11 insertions(+), 53 deletions(-)
>
> diff --git a/block.c b/block.c
> index dd4c570..1ee1d93 100644
> --- a/block.c
> +++ b/block.c
> @@ -3058,10 +3058,10 @@ int bdrv_is_allocated(BlockDriverState *bs, int64_t
> sector_num, int nb_sectors,
> * allocated/unallocated state.
> *
> */
> -int coroutine_fn bdrv_co_is_allocated_above(BlockDriverState *top,
> - BlockDriverState *base,
> - int64_t sector_num,
> - int nb_sectors, int *pnum)
> +int coroutine_fn bdrv_is_allocated_above(BlockDriverState *top,
> + BlockDriverState *base,
> + int64_t sector_num,
> + int nb_sectors, int *pnum)
This is no longer a coroutine_fn.
However, if this was the only reason for making bdrv_is_allocated() a
hybrid function, it may be easier to simply let the only caller
(img_compare in qemu-img) run in a coroutine and drop the synchronous
version entirely, keeping only a coroutine_fn.
Kevin
- Re: [Qemu-devel] [PATCH v3 04/19] block: update bs->total_sectors on writes, (continued)
[Qemu-devel] [PATCH v3 03/19] cow: do not call bdrv_co_is_allocated, Paolo Bonzini, 2013/07/25
[Qemu-devel] [PATCH v3 05/19] block: make bdrv_co_is_allocated static, Paolo Bonzini, 2013/07/25
[Qemu-devel] [PATCH v3 06/19] block: remove bdrv_is_allocated_above/bdrv_co_is_allocated_above distinction, Paolo Bonzini, 2013/07/25
- Re: [Qemu-devel] [PATCH v3 06/19] block: remove bdrv_is_allocated_above/bdrv_co_is_allocated_above distinction,
Kevin Wolf <=
[Qemu-devel] [PATCH v3 07/19] block: expect errors from bdrv_co_is_allocated, Paolo Bonzini, 2013/07/25
[Qemu-devel] [PATCH v3 09/19] block: make bdrv_has_zero_init return false for copy-on-write-images, Paolo Bonzini, 2013/07/25
[Qemu-devel] [PATCH v3 08/19] qemu-img: always probe the input image for allocated sectors, Paolo Bonzini, 2013/07/25
[Qemu-devel] [PATCH v3 10/19] block: introduce bdrv_get_block_status API, Paolo Bonzini, 2013/07/25
[Qemu-devel] [PATCH v3 12/19] block: return get_block_status data and flags for formats, Paolo Bonzini, 2013/07/25