From: John Snow <jsnow@redhat.com>
To: Vladimir Sementsov-Ogievskiy <vsementsov@virtuozzo.com>,
qemu-devel@nongnu.org
Cc: kwolf@redhat.com, famz@redhat.com, qemu-block@nongnu.org,
mreitz@redhat.com, stefanha@redhat.com, den@openvz.org,
pbonzini@redhat.com
Subject: Re: [Qemu-devel] [PATCH v4 RFC 00/17] qcow2: persistent dirty bitmaps
Date: Fri, 26 Feb 2016 16:41:54 -0500 [thread overview]
Message-ID: <56D0C6A2.70305@redhat.com> (raw)
In-Reply-To: <1455722949-17060-1-git-send-email-vsementsov@virtuozzo.com>
Do you have this mirrored in a git repo so I can browse it more
effectively? I can't figure out what all the prerequisite patches are,
so it will be convenient to just have a repo in that case for the RFC.
On 02/17/2016 10:28 AM, Vladimir Sementsov-Ogievskiy wrote:
> This series add persistent dirty bitmaps feature to qcow2.
> Specification is in docs/spec/qcow2.txt (not merged yet, see
> [PATCH v10] spec: add qcow2 bitmaps extension specification)
>
> This series are based on Fam's
> [PATCH v2 00/13] Dirty bitmap changes for migration/persistence work
> (meta bitmaps not used, and only hbitmap_deserialize_finish from
> serialization)
>
> This also needs some preparation patches, most of them are in my
> bitmap-migration series. I've not listed them here to keep things
> simpler, this is RFC any way.
>
> v4:
>
> Previous version was posted more than five months ago, so I will not
> carefully list all changes.
>
> What should be noted:
> - some changes are made to sutisfy last version of specification
> - removed staff, related to possibility of saving bitmaps for one
> disk in the other qcow2.
> - to make bitmap store/load zero-copy, I've moved load/store code to
> HBitmap - this is new patch 01.
> so, bdrv_dirty_bitmap_serialize_part and friends are not needed,
> only hbitmap_deserialize_finish, to repair bitmap consistency after
> loading its last level.
> - two patches added about AUTO and EXTRA_DATA_COMPATIBLE flags
> - some fixes made after John's comments on v3
>
> Vladimir Sementsov-Ogievskiy (17):
> hbitmap: load/store
> qcow2: Bitmaps extension: structs and consts
> qcow2-dirty-bitmap: read dirty bitmap directory
> qcow2-dirty-bitmap: add qcow2_bitmap_load()
> qcow2-dirty-bitmap: add qcow2_bitmap_store()
> qcow2: add dirty bitmaps extension
> qcow2-dirty-bitmap: add qcow2_bitmap_load_check()
> block: store persistent dirty bitmaps
> block: add bdrv_load_dirty_bitmap()
> qcow2-dirty-bitmap: add autoclear bit
> qemu: command line option for dirty bitmaps
> qcow2-dirty-bitmap: add IN_USE flag
> qcow2-dirty-bitmaps: disallow stroing bitmap to other bs
> iotests: add VM.test_launcn()
> iotests: test internal persistent dirty bitmap
> qcow2-dirty-bitmap: add AUTO flag
> qcow2-dirty-bitmap: add EXTRA_DATA_COMPATIBLE flag
>
> block.c | 3 +
> block/Makefile.objs | 2 +-
> block/dirty-bitmap.c | 101 +++++
> block/qcow2-dirty-bitmap.c | 839 ++++++++++++++++++++++++++++++++++++++++++
> block/qcow2.c | 105 +++++-
> block/qcow2.h | 59 +++
> blockdev.c | 36 ++
> include/block/block_int.h | 9 +
> include/block/dirty-bitmap.h | 21 ++
> include/qemu/hbitmap.h | 12 +
> include/sysemu/blockdev.h | 1 +
> include/sysemu/sysemu.h | 1 +
> qemu-options.hx | 35 ++
> tests/qemu-iotests/160 | 112 ++++++
> tests/qemu-iotests/160.out | 21 ++
> tests/qemu-iotests/group | 1 +
> tests/qemu-iotests/iotests.py | 25 ++
> util/hbitmap.c | 182 +++++++++
> vl.c | 78 ++++
> 19 files changed, 1640 insertions(+), 3 deletions(-)
> create mode 100644 block/qcow2-dirty-bitmap.c
> create mode 100755 tests/qemu-iotests/160
> create mode 100644 tests/qemu-iotests/160.out
>
next prev parent reply other threads:[~2016-02-26 21:42 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-17 15:28 [Qemu-devel] [PATCH v4 RFC 00/17] qcow2: persistent dirty bitmaps Vladimir Sementsov-Ogievskiy
2016-02-17 15:28 ` [Qemu-devel] [PATCH 01/17] hbitmap: load/store Vladimir Sementsov-Ogievskiy
2016-02-17 15:28 ` [Qemu-devel] [PATCH 02/17] qcow2: Bitmaps extension: structs and consts Vladimir Sementsov-Ogievskiy
2016-02-17 15:28 ` [Qemu-devel] [PATCH 03/17] qcow2-dirty-bitmap: read dirty bitmap directory Vladimir Sementsov-Ogievskiy
2016-02-17 15:28 ` [Qemu-devel] [PATCH 04/17] qcow2-dirty-bitmap: add qcow2_bitmap_load() Vladimir Sementsov-Ogievskiy
2016-02-17 15:28 ` [Qemu-devel] [PATCH 05/17] qcow2-dirty-bitmap: add qcow2_bitmap_store() Vladimir Sementsov-Ogievskiy
2016-02-17 15:28 ` [Qemu-devel] [PATCH 06/17] qcow2: add dirty bitmaps extension Vladimir Sementsov-Ogievskiy
2016-02-17 15:28 ` [Qemu-devel] [PATCH 07/17] qcow2-dirty-bitmap: add qcow2_bitmap_load_check() Vladimir Sementsov-Ogievskiy
2016-02-17 15:29 ` [Qemu-devel] [PATCH 08/17] block: store persistent dirty bitmaps Vladimir Sementsov-Ogievskiy
2016-02-17 15:29 ` [Qemu-devel] [PATCH 09/17] block: add bdrv_load_dirty_bitmap() Vladimir Sementsov-Ogievskiy
2016-02-17 15:29 ` [Qemu-devel] [PATCH 10/17] qcow2-dirty-bitmap: add autoclear bit Vladimir Sementsov-Ogievskiy
2016-02-17 15:29 ` [Qemu-devel] [PATCH 11/17] qemu: command line option for dirty bitmaps Vladimir Sementsov-Ogievskiy
2016-02-17 15:29 ` [Qemu-devel] [PATCH 12/17] qcow2-dirty-bitmap: add IN_USE flag Vladimir Sementsov-Ogievskiy
2016-02-17 15:29 ` [Qemu-devel] [PATCH 13/17] qcow2-dirty-bitmaps: disallow stroing bitmap to other bs Vladimir Sementsov-Ogievskiy
2016-02-17 15:29 ` [Qemu-devel] [PATCH 14/17] iotests: add VM.test_launcn() Vladimir Sementsov-Ogievskiy
2016-02-17 15:29 ` [Qemu-devel] [PATCH 15/17] iotests: test internal persistent dirty bitmap Vladimir Sementsov-Ogievskiy
2016-02-17 15:29 ` [Qemu-devel] [PATCH 16/17] qcow2-dirty-bitmap: add AUTO flag Vladimir Sementsov-Ogievskiy
2016-02-17 15:29 ` [Qemu-devel] [PATCH 17/17] qcow2-dirty-bitmap: add EXTRA_DATA_COMPATIBLE flag Vladimir Sementsov-Ogievskiy
2016-02-26 21:41 ` John Snow [this message]
2016-02-29 15:49 ` [Qemu-devel] [PATCH v4 RFC 00/17] qcow2: persistent dirty bitmaps Vladimir Sementsov-Ogievskiy
2016-03-03 9:10 ` Denis V. Lunev
2016-03-04 18:00 ` John Snow
2016-03-05 12:26 ` Vladimir Sementsov-Ogievskiy
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56D0C6A2.70305@redhat.com \
--to=jsnow@redhat.com \
--cc=den@openvz.org \
--cc=famz@redhat.com \
--cc=kwolf@redhat.com \
--cc=mreitz@redhat.com \
--cc=pbonzini@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
--cc=vsementsov@virtuozzo.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).