From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51798) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XoTaF-0000zf-4b for qemu-devel@nongnu.org; Wed, 12 Nov 2014 03:47:29 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XoTa8-00084e-Uc for qemu-devel@nongnu.org; Wed, 12 Nov 2014 03:47:23 -0500 Received: from mx1.redhat.com ([209.132.183.28]:34098) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XoTa8-00084X-Lf for qemu-devel@nongnu.org; Wed, 12 Nov 2014 03:47:16 -0500 Message-ID: <54631E8D.7080807@redhat.com> Date: Wed, 12 Nov 2014 09:47:09 +0100 From: Max Reitz MIME-Version: 1.0 References: <1415627159-15941-1-git-send-email-mreitz@redhat.com> <1415627159-15941-13-git-send-email-mreitz@redhat.com> <54624FF3.2080509@redhat.com> In-Reply-To: <54624FF3.2080509@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 12/21] qcow2: Allow creation with refcount order != 4 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake , qemu-devel@nongnu.org Cc: Kevin Wolf , Peter Lieven , Stefan Hajnoczi On 2014-11-11 at 19:05, Eric Blake wrote: > 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 >> --- >> 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. Good point, but I can't find any JSON object which contains a "lazy-refcounts" (or "lazy_refcounts") field and which seems to be related to image creation (if "lazy-refcounts" is somewhere not relating to options for opening an existing image, "refcount-width" should be there, too). I guess there probably is no way of directly creating an image over QMP yet at all. >> 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. That was actually unintentional. You seem fine with it, though, so I'll remove all trailing whitespace in all the test outputs I'm touching in v2. Max > Reviewed-by: Eric Blake