Re: [Qemu-block] [PATCH v4 00/20] block: transactionless incremental backup series

[John Snow]

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