Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-08-27 Thread Vladimir Sementsov-Ogievskiy
I've made this similar to snapshots, but now I think... Why should we maintain dirty bitmap directory in ram? This only gives us saving of one extra bdrv_read of header on loading bitmap, but is doesn't matter in comparison with reading the whole bitmap to memory. Also, the bitmap should not

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-08-26 Thread Stefan Hajnoczi
On Fri, Aug 14, 2015 at 08:14:46PM +0300, Vladimir Sementsov-Ogievskiy wrote: On 10.06.2015 17:30, Stefan Hajnoczi wrote: On Mon, Jun 08, 2015 at 06:21:20PM +0300, Vladimir Sementsov-Ogievskiy wrote: +ret = bdrv_pread(bs-file, bm-l1_table_offset, l1_table, + bm-l1_size

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-08-26 Thread Vladimir Sementsov-Ogievskiy
On 13.06.2015 00:55, John Snow wrote: On 06/08/2015 11:21 AM, Vladimir Sementsov-Ogievskiy wrote: From: Vladimir Sementsov-Ogievskiy vsement...@parallels.com Adds dirty-bitmaps feature to qcow2 format as specified in docs/specs/qcow2.txt Signed-off-by: Vladimir Sementsov-Ogievskiy

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-08-26 Thread Vladimir Sementsov-Ogievskiy
On 26.08.2015 16:15, Vladimir Sementsov-Ogievskiy wrote: On 13.06.2015 00:55, John Snow wrote: On 06/08/2015 11:21 AM, Vladimir Sementsov-Ogievskiy wrote: From: Vladimir Sementsov-Ogievskiy vsement...@parallels.com Adds dirty-bitmaps feature to qcow2 format as specified in

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-08-14 Thread Vladimir Sementsov-Ogievskiy
On 10.06.2015 17:30, Stefan Hajnoczi wrote: On Mon, Jun 08, 2015 at 06:21:20PM +0300, Vladimir Sementsov-Ogievskiy wrote: +ret = bdrv_pread(bs-file, bm-l1_table_offset, l1_table, + bm-l1_size * sizeof(uint64_t)); +if (ret 0) { +goto fail; +} + +

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-24 Thread Stefan Hajnoczi
On Tue, Jun 23, 2015 at 01:57:55PM -0400, John Snow wrote: On 06/15/2015 10:42 AM, Stefan Hajnoczi wrote: On Fri, Jun 12, 2015 at 03:02:33PM -0400, John Snow wrote: On 06/10/2015 10:30 AM, Stefan Hajnoczi wrote: On Mon, Jun 08, 2015 at 06:21:20PM +0300, Vladimir Sementsov-Ogievskiy

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-23 Thread John Snow
On 06/15/2015 10:42 AM, Stefan Hajnoczi wrote: On Fri, Jun 12, 2015 at 03:02:33PM -0400, John Snow wrote: On 06/10/2015 10:30 AM, Stefan Hajnoczi wrote: On Mon, Jun 08, 2015 at 06:21:20PM +0300, Vladimir Sementsov-Ogievskiy wrote: I noticed a corner case, it's probably not a problem in

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-15 Thread Vladimir Sementsov-Ogievskiy
On 12.06.2015 02:04, John Snow wrote: On 06/08/2015 11:21 AM, Vladimir Sementsov-Ogievskiy wrote: From: Vladimir Sementsov-Ogievskiy vsement...@parallels.com Adds dirty-bitmaps feature to qcow2 format as specified in docs/specs/qcow2.txt Signed-off-by: Vladimir Sementsov-Ogievskiy

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-15 Thread Stefan Hajnoczi
On Fri, Jun 12, 2015 at 03:02:33PM -0400, John Snow wrote: On 06/10/2015 10:30 AM, Stefan Hajnoczi wrote: On Mon, Jun 08, 2015 at 06:21:20PM +0300, Vladimir Sementsov-Ogievskiy wrote: I noticed a corner case, it's probably not a problem in practice: Since the dirty bitmap is

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-15 Thread John Snow
On 06/15/2015 10:05 AM, Vladimir Sementsov-Ogievskiy wrote: On 12.06.2015 02:04, John Snow wrote: On 06/08/2015 11:21 AM, Vladimir Sementsov-Ogievskiy wrote: From: Vladimir Sementsov-Ogievskiy vsement...@parallels.com Adds dirty-bitmaps feature to qcow2 format as specified in

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-12 Thread John Snow
On 06/10/2015 10:30 AM, Stefan Hajnoczi wrote: On Mon, Jun 08, 2015 at 06:21:20PM +0300, Vladimir Sementsov-Ogievskiy wrote: I noticed a corner case, it's probably not a problem in practice: Since the dirty bitmap is stored with the help of a BlockDriverState (and its bs-file), it's

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-12 Thread John Snow
On 06/08/2015 11:21 AM, Vladimir Sementsov-Ogievskiy wrote: From: Vladimir Sementsov-Ogievskiy vsement...@parallels.com Adds dirty-bitmaps feature to qcow2 format as specified in docs/specs/qcow2.txt Signed-off-by: Vladimir Sementsov-Ogievskiy vsement...@parallels.com Signed-off-by:

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-11 Thread John Snow
On 06/08/2015 11:21 AM, Vladimir Sementsov-Ogievskiy wrote: From: Vladimir Sementsov-Ogievskiy vsement...@parallels.com Adds dirty-bitmaps feature to qcow2 format as specified in docs/specs/qcow2.txt Signed-off-by: Vladimir Sementsov-Ogievskiy vsement...@parallels.com Signed-off-by:

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-10 Thread Stefan Hajnoczi
On Mon, Jun 08, 2015 at 06:21:20PM +0300, Vladimir Sementsov-Ogievskiy wrote: I noticed a corner case, it's probably not a problem in practice: Since the dirty bitmap is stored with the help of a BlockDriverState (and its bs-file), it's possible that writing the bitmap will cause bits in the

Re: [Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-09 Thread Stefan Hajnoczi
On Mon, Jun 08, 2015 at 06:21:20PM +0300, Vladimir Sementsov-Ogievskiy wrote: I haven't fully reviewed this patch yet but here are initial comments. +int qcow2_read_dirty_bitmaps(BlockDriverState *bs) +{ +BDRVQcowState *s = bs-opaque; +QCowDirtyBitmapHeader h; +QCowDirtyBitmap

[Qemu-devel] [PATCH 2/8] qcow2: add dirty-bitmaps feature

2015-06-08 Thread Vladimir Sementsov-Ogievskiy
From: Vladimir Sementsov-Ogievskiy vsement...@parallels.com Adds dirty-bitmaps feature to qcow2 format as specified in docs/specs/qcow2.txt Signed-off-by: Vladimir Sementsov-Ogievskiy vsement...@parallels.com Signed-off-by: Vladimir Sementsov-Ogievskiy vsement...@virtuozzo.com ---