This series implements an active and synchronous mirroring mode. You can read the cover letter of v4 here (I don’t like to copy-paste that because people who reviewed previous versions know it already and this saves them from having to look out for potential changes):
http://lists.nongnu.org/archive/html/qemu-block/2018-04/msg00185.html v5: - Patch 3: Rebase conflict with Kevin’s job series (kept R-bs because it’s just a change in removed code (and a trivial one on top)) - Patch 6: Removed completely unused BdrvChildList typedef [Berto] (kept R-bs again because this isn’t even a functional change...) - Patch 11: Added, because jobs are no longer supposed to have direct access to their progress info, but we need some way of just adding an offset to the progress end value in the next patch - Patch 12: - s/2.12/3.0/ [Eric] - Use job_progress_*() functions (kept R-b because I think those to be very obvious changes) - Patch 13: s/2.13/3.0/ (kept R-b because come on) - Patch 14: - s/2017/2018/ (hey, why not) - qmp_to_opts() has been moved to VM (kept R-bs because I consider those to be trivial or obvious changes, respectively) git-backport-diff to v4: Key: [----] : patches are identical [####] : number of functional differences between upstream/downstream patch [down] : patch is downstream-only The flags [FC] indicate (F)unctional and (C)ontextual differences, respectively 001/14:[----] [--] 'block/mirror: Pull out mirror_perform()' 002/14:[----] [--] 'block/mirror: Convert to coroutines' 003/14:[0002] [FC] 'block/mirror: Use CoQueue to wait on in-flight ops' 004/14:[----] [-C] 'block/mirror: Wait for in-flight op conflicts' 005/14:[----] [-C] 'block/mirror: Use source as a BdrvChild' 006/14:[0002] [FC] 'block: Generalize should_update_child() rule' 007/14:[----] [--] 'hbitmap: Add @advance param to hbitmap_iter_next()' 008/14:[----] [--] 'test-hbitmap: Add non-advancing iter_next tests' 009/14:[----] [--] 'block/dirty-bitmap: Add bdrv_dirty_iter_next_area' 010/14:[----] [-C] 'block/mirror: Add MirrorBDSOpaque' 011/14:[down] 'job: Add job_progress_increase_remaining()' 012/14:[0006] [FC] 'block/mirror: Add active mirroring' 013/14:[0004] [FC] 'block/mirror: Add copy mode QAPI interface' 014/14:[0006] [FC] 'iotests: Add test for active mirroring' Max Reitz (14): block/mirror: Pull out mirror_perform() block/mirror: Convert to coroutines block/mirror: Use CoQueue to wait on in-flight ops block/mirror: Wait for in-flight op conflicts block/mirror: Use source as a BdrvChild block: Generalize should_update_child() rule hbitmap: Add @advance param to hbitmap_iter_next() test-hbitmap: Add non-advancing iter_next tests block/dirty-bitmap: Add bdrv_dirty_iter_next_area block/mirror: Add MirrorBDSOpaque job: Add job_progress_increase_remaining() block/mirror: Add active mirroring block/mirror: Add copy mode QAPI interface iotests: Add test for active mirroring qapi/block-core.json | 29 +- include/block/block_int.h | 4 +- include/block/dirty-bitmap.h | 2 + include/qemu/hbitmap.h | 5 +- include/qemu/job.h | 15 + block.c | 44 ++- block/backup.c | 2 +- block/dirty-bitmap.c | 57 +++- block/mirror.c | 605 ++++++++++++++++++++++++++++------- blockdev.c | 9 +- job.c | 5 + tests/test-hbitmap.c | 38 ++- util/hbitmap.c | 10 +- tests/qemu-iotests/151 | 120 +++++++ tests/qemu-iotests/151.out | 5 + tests/qemu-iotests/group | 1 + 16 files changed, 799 insertions(+), 152 deletions(-) create mode 100755 tests/qemu-iotests/151 create mode 100644 tests/qemu-iotests/151.out -- 2.17.1
