> -----Original Message-----
> From: Lukas Straub <lukasstra...@web.de>
> Sent: Thursday, November 14, 2019 12:36 AM
> To: qemu-devel <qemu-devel@nongnu.org>
> Cc: Zhang, Chen <chen.zh...@intel.com>; Jason Wang
> <jasow...@redhat.com>; Wen Congyang <wencongya...@huawei.com>;
> Xie Changlong <xiechanglon...@gmail.com>; Kevin Wolf
> <kw...@redhat.com>; Max Reitz <mre...@redhat.com>; qemu-block
> <qemu-bl...@nongnu.org>
> Subject: Re: [PATCH v7 0/4] colo: Add support for continuous replication
>
> On Fri, 25 Oct 2019 19:06:31 +0200
> Lukas Straub <lukasstra...@web.de> wrote:
>
> > Hello Everyone,
> > These Patches add support for continuous replication to colo. This
> > means that after the Primary fails and the Secondary did a failover,
> > the Secondary can then become Primary and resume replication to a new
> Secondary.
> >
> > Regards,
> > Lukas Straub
> >
> > v7:
> > - clarify meaning of ip's in documentation and note that active and hidden
> > images just need to be created once
> > - reverted removal of bdrv_is_root_node(top_bs) in replication and
> adjusted
> > the top-id= parameter in documentation acordingly
> >
> > v6:
> > - documented the position= and insert= options
> > - renamed replication test
> > - clarified documentation by using different ip's for primary and
> > secondary
> > - added Reviewed-by tags
> >
> > v5:
> > - change syntax for the position= parameter
> > - fix spelling mistake
> >
> > v4:
> > - fix checkpatch.pl warnings
> >
> > v3:
> > - add test for replication changes
> > - check if the filter to be inserted before/behind belongs to the
> > same interface
> > - fix the error message for the position= parameter
> > - rename term "after" -> "behind" and variable "insert_before" ->
> "insert_before_flag"
> > - document the quorum node on the secondary side
> > - simplify quorum parameters in documentation
> > - remove trailing spaces in documentation
> > - clarify the testing procedure in documentation
> >
> > v2:
> > - fix email formating
> > - fix checkpatch.pl warnings
> > - fix patchew error
> > - clearer commit messages
> >
> >
> > Lukas Straub (4):
> > block/replication.c: Ignore requests after failover
> > tests/test-replication.c: Add test for for secondary node continuing
> > replication
> > net/filter.c: Add Options to insert filters anywhere in the filter
> > list
> > colo: Update Documentation for continuous replication
> >
> > block/replication.c | 35 +++++-
> > docs/COLO-FT.txt | 224 +++++++++++++++++++++++++++----------
> > docs/block-replication.txt | 28 +++--
> > include/net/filter.h | 2 +
> > net/filter.c | 92 ++++++++++++++-
> > qemu-options.hx | 31 ++++-
> > tests/test-replication.c | 52 +++++++++
> > 7 files changed, 389 insertions(+), 75 deletions(-)
> >
>
> Hello Everyone,
> So I guess this is ready for merging or will that have to wait until the 4.2
> release is done?
Due to Qemu 4.2 release schedule, after soft feature freeze(Oct29) the master
branch does not accept feature changes.
But I don't know if sub-maintainer(block or net) can queue this series first
then merge it after 4.2 release?
Thanks
Zhang Chen
>
> Regards,
> Lukas Straub
