From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52413) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XbxEU-00076E-Ew for qemu-devel@nongnu.org; Wed, 08 Oct 2014 15:49:16 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XbxEO-0002o4-AS for qemu-devel@nongnu.org; Wed, 08 Oct 2014 15:49:10 -0400 Received: from mx1.redhat.com ([209.132.183.28]:14657) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XbxEO-0002nz-1l for qemu-devel@nongnu.org; Wed, 08 Oct 2014 15:49:04 -0400 Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id s98Jn20o010697 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Wed, 8 Oct 2014 15:49:03 -0400 Message-ID: <5435952B.7050508@redhat.com> Date: Wed, 08 Oct 2014 21:48:59 +0200 From: Max Reitz MIME-Version: 1.0 References: <1408117628-17512-1-git-send-email-mreitz@redhat.com> In-Reply-To: <1408117628-17512-1-git-send-email-mreitz@redhat.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v3 0/7] block/qcow2: Improve zero cluster expansion List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , Stefan Hajnoczi On 15.08.2014 17:47, Max Reitz wrote: > The main purpose of this series is to add a progress report to > qemu-img amend. This is achieved by adding a callback function to > bdrv_amend_options() - the reasons for this choice are explained in > patch 1. > > While adapting qcow2's expand_zero_clusters_in_l1() accordingly, I > noticed a way to simplify it and get rid of the rather ugly bitmap used > there (patch 6). > > This series depends on v2 of my "qemu-img: Allow source cache mode > specification" series. > > > Changes from v2: > - Patch 1: Fixed misspelling in the commit message [Eric] > > > git-backport-diff against v2: > > Key: > [----] : patches are identical > [####] : number of functional differences between upstream/downstream p= atch > [down] : patch is downstream-only > The flags [FC] indicate (F)unctional and (C)ontextual differences, resp= ectively > > 001/7:[----] [--] 'block: Add status callback to bdrv_amend_options()' > 002/7:[----] [--] 'qemu-img: Add progress output for amend' > 003/7:[----] [--] 'qemu-img: Fix insignificant memleak' > 004/7:[----] [--] 'block/qcow2: Implement status CB for amend' > 005/7:[----] [--] 'block/qcow2: Make get_refcount() global' > 006/7:[----] [--] 'block/qcow2: Simplify shared L2 handling in amend' > 007/7:[----] [--] 'iotests: Expand test 061' > > > Max Reitz (7): > block: Add status callback to bdrv_amend_options() > qemu-img: Add progress output for amend > qemu-img: Fix insignificant memleak > block/qcow2: Implement status CB for amend > block/qcow2: Make get_refcount() global > block/qcow2: Simplify shared L2 handling in amend > iotests: Expand test 061 > > block.c | 5 +- > block/qcow2-cluster.c | 115 ++++++++++++++++++++++--------------= --------- > block/qcow2-refcount.c | 26 +++++----- > block/qcow2.c | 10 ++-- > block/qcow2.h | 5 +- > include/block/block.h | 8 +++- > include/block/block_int.h | 3 +- > qemu-img-cmds.hx | 4 +- > qemu-img.c | 29 ++++++++++-- > qemu-img.texi | 2 +- > tests/qemu-iotests/061 | 25 ++++++++++ > tests/qemu-iotests/061.out | 30 ++++++++++++ > tests/qemu-iotests/group | 2 +- > 13 files changed, 174 insertions(+), 90 deletions(-) Ping. All patches have been reviewed by Eric and Beno=C3=AEt. Max