* [Qemu-devel] ping Re: [PATCH 0/2] replication: drop extra sync
[not found] <20180919124343.28206-1-vsementsov@virtuozzo.com>
@ 2018-10-11 15:40 ` Vladimir Sementsov-Ogievskiy
2018-10-31 9:30 ` [Qemu-devel] ping2 " Vladimir Sementsov-Ogievskiy
1 sibling, 0 replies; 3+ messages in thread
From: Vladimir Sementsov-Ogievskiy @ 2018-10-11 15:40 UTC (permalink / raw)
To: qemu-devel, qemu-block
Cc: xiechanglong.d, wencongyang2, mreitz, kwolf, jcody, Denis Lunev, famz
ping
19.09.2018 15:43, Vladimir Sementsov-Ogievskiy wrote:
> Hi all!
>
> We finally implemented fleecing scheme, create a test case for it (222)
> and even made it safe (f8d59dfb40).
>
> However, fleecing is used inside block-replication long ago, and to make
> the scheme safe it uses direct interface to backup-job copy requests
> synchronization, which can be dropped now.
>
> Vladimir Sementsov-Ogievskiy (2):
> block/replication: drop extra synchronization
> block/backup: drop unused synchronization interface
>
> include/block/block_backup.h | 13 ------------
> block/backup.c | 38 +++++++-----------------------------
> block/replication.c | 24 +----------------------
> 3 files changed, 8 insertions(+), 67 deletions(-)
>
--
Best regards,
Vladimir
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Qemu-devel] ping2 Re: [PATCH 0/2] replication: drop extra sync
[not found] <20180919124343.28206-1-vsementsov@virtuozzo.com>
2018-10-11 15:40 ` [Qemu-devel] ping Re: [PATCH 0/2] replication: drop extra sync Vladimir Sementsov-Ogievskiy
@ 2018-10-31 9:30 ` Vladimir Sementsov-Ogievskiy
2018-11-06 17:17 ` Kevin Wolf
1 sibling, 1 reply; 3+ messages in thread
From: Vladimir Sementsov-Ogievskiy @ 2018-10-31 9:30 UTC (permalink / raw)
To: qemu-devel, qemu-block
Cc: xiechanglong.d, wencongyang2, mreitz, kwolf, jcody, Denis Lunev, famz
ping2
Hi, it's a first step to backup refactoring and improving. Just get rid
of this extra and unnatural synchronization.
19.09.2018 15:43, Vladimir Sementsov-Ogievskiy wrote:
> Hi all!
>
> We finally implemented fleecing scheme, create a test case for it (222)
> and even made it safe (f8d59dfb40).
>
> However, fleecing is used inside block-replication long ago, and to make
> the scheme safe it uses direct interface to backup-job copy requests
> synchronization, which can be dropped now.
>
> Vladimir Sementsov-Ogievskiy (2):
> block/replication: drop extra synchronization
> block/backup: drop unused synchronization interface
>
> include/block/block_backup.h | 13 ------------
> block/backup.c | 38 +++++++-----------------------------
> block/replication.c | 24 +----------------------
> 3 files changed, 8 insertions(+), 67 deletions(-)
>
--
Best regards,
Vladimir
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Qemu-devel] ping2 Re: [PATCH 0/2] replication: drop extra sync
2018-10-31 9:30 ` [Qemu-devel] ping2 " Vladimir Sementsov-Ogievskiy
@ 2018-11-06 17:17 ` Kevin Wolf
0 siblings, 0 replies; 3+ messages in thread
From: Kevin Wolf @ 2018-11-06 17:17 UTC (permalink / raw)
To: Vladimir Sementsov-Ogievskiy
Cc: qemu-devel, qemu-block, xiechanglong.d, wencongyang2, mreitz,
jcody, Denis Lunev, famz
Am 31.10.2018 um 10:30 hat Vladimir Sementsov-Ogievskiy geschrieben:
> ping2
>
> Hi, it's a first step to backup refactoring and improving. Just get rid
> of this extra and unnatural synchronization.
The replication maintainers don't seem to have an opinion about this, so
thanks, applied to block-next (for 3.2).
Kevin
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-11-06 17:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <20180919124343.28206-1-vsementsov@virtuozzo.com>
2018-10-11 15:40 ` [Qemu-devel] ping Re: [PATCH 0/2] replication: drop extra sync Vladimir Sementsov-Ogievskiy
2018-10-31 9:30 ` [Qemu-devel] ping2 " Vladimir Sementsov-Ogievskiy
2018-11-06 17:17 ` Kevin Wolf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.