[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH 1/2] qcow2: Prefer byte-based calls into bs->fil
From: |
Alberto Garcia |
Subject: |
Re: [Qemu-block] [PATCH 1/2] qcow2: Prefer byte-based calls into bs->file |
Date: |
Wed, 21 Feb 2018 10:42:00 +0100 |
User-agent: |
Notmuch/0.18.2 (http://notmuchmail.org) Emacs/24.4.1 (i586-pc-linux-gnu) |
On Tue 20 Feb 2018 11:24:58 PM CET, Eric Blake wrote:
> diff --git a/block/qcow2-refcount.c b/block/qcow2-refcount.c
> index d46b69d7f34..3fefeb3dc50 100644
> --- a/block/qcow2-refcount.c
> +++ b/block/qcow2-refcount.c
> @@ -2310,8 +2310,8 @@ write_refblocks:
> on_disk_refblock = (void *)((char *) *refcount_table +
> refblock_index * s->cluster_size);
>
> - ret = bdrv_write(bs->file, refblock_offset / BDRV_SECTOR_SIZE,
> - on_disk_refblock, s->cluster_sectors);
> + ret = bdrv_pwrite(bs->file, refblock_offset,
> + on_disk_refblock, s->cluster_size);
It looks like the second line is not properly indented. I think you can
also move on_disk_refblock to the previous line.
Otherwise the patch looks good.
Reviewed-by: Alberto Garcia <address@hidden>
Berto