[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 12/21] qcow2: Allow creation with refcount order
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH 12/21] qcow2: Allow creation with refcount order != 4 |
Date: |
Tue, 11 Nov 2014 11:05:39 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 |
On 11/10/2014 06:45 AM, Max Reitz wrote:
> Add a creation option to qcow2 for setting the refcount order of images
> to be created, and respect that option's value.
>
> This breaks some test outputs, fix them.
>
> Signed-off-by: Max Reitz <address@hidden>
> ---
> block/qcow2.c | 20 ++++++++
> include/block/block_int.h | 1 +
> tests/qemu-iotests/049.out | 112
> ++++++++++++++++++++++-----------------------
> tests/qemu-iotests/079.out | 18 ++++----
> tests/qemu-iotests/082.out | 41 ++++++++++++++---
> tests/qemu-iotests/085.out | 38 +++++++--------
> 6 files changed, 139 insertions(+), 91 deletions(-)
Is there any .json file that needs to be modified to allow this option
to be set via QMP? I guess that can be a followup.
> qemu-img create -f qcow2 TEST_DIR/t.qcow2 -- 1kilobyte
> -qemu-img: Invalid image size specified! You may use k, M, G, T, P or E
> suffixes for
> +qemu-img: Invalid image size specified! You may use k, M, G, T, P or E
> suffixes for
> qemu-img: kilobytes, megabytes, gigabytes, terabytes, petabytes and exabytes.
Nice that you are also getting rid of trailing whitespace.
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH 13/21] block: Add opaque value to the amend CB, (continued)
- [Qemu-devel] [PATCH 14/21] qcow2: Use error_report() in qcow2_amend_options(), Max Reitz, 2014/11/10
- [Qemu-devel] [PATCH 15/21] qcow2: Use abort() instead of assert(false), Max Reitz, 2014/11/10
- [Qemu-devel] [PATCH 16/21] qcow2: Split upgrade/downgrade paths for amend, Max Reitz, 2014/11/10
- [Qemu-devel] [PATCH 12/21] qcow2: Allow creation with refcount order != 4, Max Reitz, 2014/11/10
- Re: [Qemu-devel] [PATCH 12/21] qcow2: Allow creation with refcount order != 4,
Eric Blake <=
- [Qemu-devel] [PATCH 17/21] qcow2: Use intermediate helper CB for amend, Max Reitz, 2014/11/10
- [Qemu-devel] [PATCH 18/21] qcow2: Add function for refcount order amendment, Max Reitz, 2014/11/10
- Re: [Qemu-devel] [PATCH 18/21] qcow2: Add function for refcount order amendment, Eric Blake, 2014/11/12