From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35683) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YLFSo-0004la-Fm for qemu-devel@nongnu.org; Tue, 10 Feb 2015 13:23:11 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YLFSh-0001IS-ST for qemu-devel@nongnu.org; Tue, 10 Feb 2015 13:23:10 -0500 Received: from mx1.redhat.com ([209.132.183.28]:58289) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YLFSh-0001IL-HB for qemu-devel@nongnu.org; Tue, 10 Feb 2015 13:23:03 -0500 Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id t1AIN3H7015029 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=FAIL) for ; Tue, 10 Feb 2015 13:23:03 -0500 Message-ID: <54DA4C86.9000207@redhat.com> Date: Tue, 10 Feb 2015 11:23:02 -0700 From: Eric Blake MIME-Version: 1.0 References: <1423509950-7468-1-git-send-email-mreitz@redhat.com> In-Reply-To: <1423509950-7468-1-git-send-email-mreitz@redhat.com> Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="8scvisf2EiCitcV3TqOWOGPhe7l06nspX" Subject: Re: [Qemu-devel] [PATCH v3 00/12] qcow2: Add new overlap check functions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Max Reitz , qemu-devel@nongnu.org Cc: Kevin Wolf , Stefan Hajnoczi This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --8scvisf2EiCitcV3TqOWOGPhe7l06nspX Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 02/09/2015 12:25 PM, Max Reitz wrote: > As has been requested, this series adds new overlap check functions to > the qcow2 code. My local branch is called "qcow2-improved-overlap-v1", > but I am not so sure whether it is actually an improvement; that is lef= t > for you to decide, dear reviewers. >=20 > v3: > - Patch 1: > - s/few memory/little memory/ [Eric] > - s/2014/2015/ [Eric] > - s/of a or a/of or a/ [Eric] > - Use nb_clusters_minus_one instead of nb_clusters, because > nb_clusters =3D=3D 0 is invalid [Eric] > - Drop cached_windows_index [Stefan] > - Reset current_nb_clusters to 1 when a new range starts in the > algorithm which rebuilds the fragment list from the bitmap > representation > - Patch 3: Added a TODO comment that the user should be able to overrid= e > the default cache size limit > - Patch 8: As the patch "qcow2: Buffer L1 table in snapshot refcount > update" has been dropped, this patch was changed to no longer rely on= > it > - Patch 11: Replaced the size limit for a snapshot's L1 table by > QCOW_MAX_L1_SIZE (was: INT_MAX / sizeof(uint64_t)) [Eric] On this one, it was actually Stefan that spotted it. --=20 Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org --8scvisf2EiCitcV3TqOWOGPhe7l06nspX Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 Comment: Public key at http://people.redhat.com/eblake/eblake.gpg Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQEcBAEBCAAGBQJU2kyGAAoJEKeha0olJ0NqZAwIAIS16CDIVgVbubg4WNq7vHwT GPl4y9D3PrRknMlYpP4tWzAQ6g5oJ/xlgFmjRGnUtRsfYiigidKm4FnS40jYh9x5 N6XkjDnCU0uNgorfBuzJp4l/d+lqA9Pn2OLD+ovkP8DGwsC/oJR7gNuCLRPlLGHZ b+qjexXYb0TFQlfPx62m848LcT5iTy2/pmdZtQv+IFPXVHxfuMjc8knS1hUADoGT YXSze0XpD7JRTiHKpVG87ExBL6li8X3MgBR5jwhSMsfEDVZFj/dLHycqhjNQ07Y4 NoOIA6BbX1HYABG5j6BfseKCl2NBivZx1t9DVwdM9xSbQpdQeq3APsTbGFh4zL8= =DMkl -----END PGP SIGNATURE----- --8scvisf2EiCitcV3TqOWOGPhe7l06nspX--