From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:45401) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YZ33H-0004rS-RZ for qemu-devel@nongnu.org; Fri, 20 Mar 2015 15:57:52 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YZ33G-0001JO-TS for qemu-devel@nongnu.org; Fri, 20 Mar 2015 15:57:51 -0400 Message-ID: <550C7BB8.2050902@redhat.com> Date: Fri, 20 Mar 2015 15:57:44 -0400 From: John Snow MIME-Version: 1.0 References: <1426879023-18151-1-git-send-email-jsnow@redhat.com> <550C7A65.6050405@redhat.com> In-Reply-To: <550C7A65.6050405@redhat.com> Content-Type: text/plain; charset=iso-8859-15; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v4 00/20] block: transactionless incremental backup series List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Max Reitz , qemu-block@nongnu.org Cc: kwolf@redhat.com, famz@redhat.com, qemu-devel@nongnu.org, armbru@redhat.com, vsementsov@parallels.com, stefanha@redhat.com On 03/20/2015 03:52 PM, Max Reitz wrote: > On 2015-03-20 at 15:16, John Snow wrote: >> I've run out of cheeky jokes for my cover letters. >> >> This patchset enables the in-memory part of the incremental backup >> feature, without transactional support. >> >> Support for transactions was separated into a separate series which >> is also now available on-list. Getting this portion of the series >> committed will help stabilize work on bitmap persistence and bitmap >> migration. >> >> Thanks to Fam Zheng for the original versions of this patchset, >> And thanks to Max for reviewing 2,395 versions of it since. >> >> === >> v4: >> === >> >> 04: Some in-line documentation for block_dirty_bitmap_lookup >> Changed behavior with respect to aio_context >> (always acquire, release if pbs == null, give to user otherwise) >> 10: Removed vestigial (currently nop) bdrv_enable_dirty_bitmap call >> Kept R-B. >> 16: Added some comments to test_check_boundary_bits. >> Kept R-B. >> 17: Folded in refactor from "incremental transactions v1" (Poor Kitty) >> 18: Pulled forward from "incremental transactions v1" >> Kept R-B from that series. >> 19: Folded in refactor from "incremental transactions v1" >> Added offset assertions into wait_incremental >> 20: Removed error tolerance from wait_until_completed, as >> these patches no longer make use of that feature. > > Thanks for squashing the iotest refactoring code into 17, 19, and 20! > > Max It wound up being easier than I thought it would be at first, and it keeps the second series smaller, so we can focus on how badly I named everything. --js