Re: [Qemu-devel] [PATCH 12/13] vmdk: Convert to bdrv_co_pwrite_zeroes()

[Kevin Wolf <kwolf@redhat.com>]

Am 25.05.2016 um 00:25 hat Eric Blake geschrieben:
> Another step on our continuing quest to switch to byte-based
> interfaces.
> 
> Signed-off-by: Eric Blake <eblake@redhat.com>
> ---
>  block/vmdk.c | 13 ++++++-------
>  1 file changed, 6 insertions(+), 7 deletions(-)
> 
> diff --git a/block/vmdk.c b/block/vmdk.c
> index 8494d63..284d7a0 100644
> --- a/block/vmdk.c
> +++ b/block/vmdk.c
> @@ -1704,15 +1704,14 @@ static int vmdk_write_compressed(BlockDriverState *bs,
>      }
>  }
> 
> -static int coroutine_fn vmdk_co_write_zeroes(BlockDriverState *bs,
> -                                             int64_t sector_num,
> -                                             int nb_sectors,
> -                                             BdrvRequestFlags flags)
> +static int coroutine_fn vmdk_co_pwrite_zeroes(BlockDriverState *bs,
> +                                              int64_t offset,
> +                                              int count,
> +                                              BdrvRequestFlags flags)
>  {
>      int ret;
>      BDRVVmdkState *s = bs->opaque;
> -    uint64_t offset = sector_num * BDRV_SECTOR_SIZE;
> -    uint64_t bytes = nb_sectors * BDRV_SECTOR_SIZE;
> +    uint64_t bytes = count;

That's an unnecessary variable again. Whether you decide to change it or
not:

Reviewed-by: Kevin Wolf <kwolf@redhat.com>