On 10/12/2017 09:53 AM, Vladimir Sementsov-Ogievskiy wrote:
> Hi all. Here is a continuation of my "new backup architecture" series,
> The very first part, but it is meaningful itself.
> 
> v2: rebase on master
> 01: add test and fix bug

Nice, glad you caught it.

Reviewed-by: John Snow <js...@redhat.com>

> 02: move commit about NONE mode to this patch, add John's r-b (hope,
>      you are OK with this tiny change)

ACK

> 03: tiny refactor (John)

Reviewed-by: John Snow <js...@redhat.com>

> 04: improve commit message, add John's r-b
> 05: add John's r-b
> 
> Vladimir Sementsov-Ogievskiy (5):
>   hbitmap: add next_zero function
>   backup: move from done_bitmap to copy_bitmap
>   backup: init copy_bitmap from sync_bitmap for incremental
>   backup: simplify non-dirty bits progress processing
>   backup: use copy_bitmap in incremental backup
> 
>  include/block/dirty-bitmap.h |   1 +
>  include/qemu/hbitmap.h       |   8 +++
>  block/backup.c               | 118 
> ++++++++++++++++++++++++-------------------
>  block/dirty-bitmap.c         |   5 ++
>  tests/test-hbitmap.c         |  61 ++++++++++++++++++++++
>  util/hbitmap.c               |  39 ++++++++++++++
>  6 files changed, 180 insertions(+), 52 deletions(-)
> 

I'm not sure if this still qualifies for RC, but since it's my fault it
languished on the list, I'd still be willing to send the PR and suffer
Peter's wrath.

I apologize for the unexpected leave.

--John

Reply via email to