[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [Qemu-devel] [PATCH 1/2] sheepdog: cleanup repeated exp
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-block] [Qemu-devel] [PATCH 1/2] sheepdog: cleanup repeated expression |
Date: |
Tue, 22 May 2018 17:25:45 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 |
On 05/22/2018 05:10 PM, Paolo Bonzini wrote:
> The expression "SD_INODE_SIZE - sizeof(inode.data_vdi_id)" already has a macro
> defined for the same value (though with a nicer definition using offsetof).
> Replace it.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> block/sheepdog.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/block/sheepdog.c b/block/sheepdog.c
> index 4237132419..23cf5a8430 100644
> --- a/block/sheepdog.c
> +++ b/block/sheepdog.c
> @@ -2337,7 +2337,7 @@ static int sd_truncate(BlockDriverState *bs, int64_t
> offset,
> }
>
> /* we don't need to update entire object */
> - datalen = SD_INODE_SIZE - sizeof(s->inode.data_vdi_id);
> + datalen = SD_INODE_HEADER_SIZE;
> s->inode.vdi_size = offset;
> ret = write_object(fd, s->bs, (char *)&s->inode,
> vid_to_vdi_oid(s->inode.vdi_id), s->inode.nr_copies,
> @@ -2705,7 +2705,7 @@ static int sd_snapshot_create(BlockDriverState *bs,
> QEMUSnapshotInfo *sn_info)
> */
> strncpy(s->inode.tag, sn_info->name, sizeof(s->inode.tag));
> /* we don't need to update entire object */
> - datalen = SD_INODE_SIZE - sizeof(s->inode.data_vdi_id);
> + datalen = SD_INODE_HEADER_SIZE;
> inode = g_malloc(datalen);
>
> /* refresh inode. */
> @@ -2991,7 +2991,7 @@ static int sd_snapshot_list(BlockDriverState *bs,
> QEMUSnapshotInfo **psn_tab)
> /* we don't need to read entire object */
> ret = read_object(fd, s->bs, (char *)&inode,
> vid_to_vdi_oid(vid),
> - 0, SD_INODE_SIZE - sizeof(inode.data_vdi_id), 0,
> + 0, SD_INODE_HEADER_SIZE, 0,
> s->cache_flags);
>
> if (ret) {
>