Re: [Qemu-devel] [PATCH 3/4] nbd/server: implement dirty bitmap export

2018-03-28 Thread Vladimir Sementsov-Ogievskiy
22.03.2018 18:26, Vladimir Sementsov-Ogievskiy wrote: 21.03.2018 19:57, Eric Blake wrote: On 03/21/2018 07:19 AM, Vladimir Sementsov-Ogievskiy wrote: Signed-off-by: Vladimir Sementsov-Ogievskiy Rather sparse on the details in the commit message; I had to read the patch to even learn what th

Re: [Qemu-devel] [PATCH 3/4] nbd/server: implement dirty bitmap export

2018-03-22 Thread Vladimir Sementsov-Ogievskiy
21.03.2018 19:57, Eric Blake wrote: On 03/21/2018 07:19 AM, Vladimir Sementsov-Ogievskiy wrote: Signed-off-by: Vladimir Sementsov-Ogievskiy [...] + +#define NBD_STATE_DIRTY 1 Does this belong better in nbd.h, alongside NBD_STATE_HOLE?  (And defining it as (1 << 0) might be nicer, too

Re: [Qemu-devel] [PATCH 3/4] nbd/server: implement dirty bitmap export

2018-03-22 Thread Vladimir Sementsov-Ogievskiy
21.03.2018 19:57, Eric Blake wrote: On 03/21/2018 07:19 AM, Vladimir Sementsov-Ogievskiy wrote: Signed-off-by: Vladimir Sementsov-Ogievskiy Rather sparse on the details in the commit message; I had to read the patch to even learn what the new namespace is. oh, yes, sorry :( @@ -92,6 +101

Re: [Qemu-devel] [PATCH 3/4] nbd/server: implement dirty bitmap export

2018-03-21 Thread Eric Blake
On 03/21/2018 07:19 AM, Vladimir Sementsov-Ogievskiy wrote: Signed-off-by: Vladimir Sementsov-Ogievskiy Rather sparse on the details in the commit message; I had to read the patch to even learn what the new namespace is. --- include/block/nbd.h | 2 + nbd/server.c| 207 +++

[Qemu-devel] [PATCH 3/4] nbd/server: implement dirty bitmap export

2018-03-21 Thread Vladimir Sementsov-Ogievskiy
Signed-off-by: Vladimir Sementsov-Ogievskiy --- include/block/nbd.h | 2 + nbd/server.c| 207 ++-- 2 files changed, 203 insertions(+), 6 deletions(-) diff --git a/include/block/nbd.h b/include/block/nbd.h index fcdcd54502..f0b459283f 1006