[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v4 09/16] qemu-img: Use BlockBackend as far as p

From: Eric Blake
Subject: Re: [Qemu-devel] [PATCH v4 09/16] qemu-img: Use BlockBackend as far as possible
Date: Thu, 05 Feb 2015 14:03:10 -0700
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0

On 02/05/2015 11:58 AM, Max Reitz wrote:
> Although qemu-img already creates BlockBackends, it does not do accesses
> to the images through them. This patch converts all of the bdrv_* calls
> for which this is currently possible to blk_* calls. Most of the
> remaining calls will probably stay bdrv_* calls because they really do
> operate on the BDS level instead of the BB level.
> Signed-off-by: Max Reitz <address@hidden>
> ---
>  qemu-img.c | 82 
> ++++++++++++++++++++++++++++++--------------------------------
>  1 file changed, 39 insertions(+), 43 deletions(-)
> diff --git a/qemu-img.c b/qemu-img.c
> index 26fb3a1..77fd4e4 100644
> --- a/qemu-img.c
> +++ b/qemu-img.c
> @@ -1010,19 +1010,19 @@ static int64_t sectors_to_process(int64_t total, 
> int64_t from)
>   * Returns 0 in case sectors are filled with 0, 1 if sectors contain non-zero
>   * data and negative value on error.
>   *
> - * @param bs:  Driver used for accessing file
> + * @param blk:  BlockBackend for the image
>   * @param sect_num: Number of first sector to check
>   * @param sect_count: Number of sectors to check

As long as you are touching this, is it worth compressing the double
space into one for consistency with the other lines?

That's minor, and it DOES look a lot nicer without the conversions from
bytes to sectors.

Reviewed-by: Eric Blake <address@hidden>

Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

reply via email to

[Prev in Thread] Current Thread [Next in Thread]