[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [PATCH v2 1/2] block/vhdx: Remove redundant IEC binar
From: |
Laurent Vivier |
Subject: |
Re: [Qemu-trivial] [PATCH v2 1/2] block/vhdx: Remove redundant IEC binary prefixes definition |
Date: |
Thu, 11 Apr 2019 21:54:20 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.8.0 |
On 27/03/2019 10:56, Stefano Garzarella wrote:
> IEC binary prefixes are already defined in "qemu/units.h",
> so we can remove redundant definitions in "block/vhdx.h".
>
> Signed-off-by: Stefano Garzarella <address@hidden>
> ---
> block/vhdx.c | 3 ++-
> block/vhdx.h | 6 +-----
> 2 files changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/block/vhdx.c b/block/vhdx.c
> index b785aef4b7..7cd1fc3731 100644
> --- a/block/vhdx.c
> +++ b/block/vhdx.c
> @@ -1889,7 +1889,8 @@ static int coroutine_fn
> vhdx_co_create(BlockdevCreateOptions *opts,
> return -EINVAL;
> }
> if (block_size > VHDX_BLOCK_SIZE_MAX) {
> - error_setg(errp, "Block size must not exceed %d",
> VHDX_BLOCK_SIZE_MAX);
> + error_setg(errp, "Block size must not exceed %" PRId64,
> + VHDX_BLOCK_SIZE_MAX);
> return -EINVAL;
> }
>
> diff --git a/block/vhdx.h b/block/vhdx.h
> index 1bfb4e4f73..bf72090c8f 100644
> --- a/block/vhdx.h
> +++ b/block/vhdx.h
> @@ -17,11 +17,7 @@
>
> #ifndef BLOCK_VHDX_H
> #define BLOCK_VHDX_H
> -
> -#define KiB (1 * 1024)
> -#define MiB (KiB * 1024)
> -#define GiB (MiB * 1024)
> -#define TiB ((uint64_t) GiB * 1024)
> +#include "qemu/units.h"
>
> #define DEFAULT_LOG_SIZE 1048576 /* 1MiB */
> /* Structures and fields present in the VHDX file */
>
Applied to my trivial-patches branch.
Thanks,
Laurent
- Re: [Qemu-trivial] [PATCH v2 1/2] block/vhdx: Remove redundant IEC binary prefixes definition,
Laurent Vivier <=