On Mon, Sep 25, 2023 at 03:55:08PM +0200, Markus Armbruster wrote: > Peter Xu <[email protected]> writes: > > > It's not obvious to many that RDMA migration is in Odd Fixes stage for a > > long time. Add an explicit sub entry for it (besides migration, which > > already covers the rdma files) to be clear on that, meanwhile add Zhijian > > as Reviewer, so Zhijian can see the patches and review when he still has > > the bandwidth. > > > > Cc: Daniel P. Berrangé <[email protected]> > > Cc: Juan Quintela <[email protected]> > > Cc: Markus Armbruster <[email protected]> > > Cc: Zhijian Li (Fujitsu) <[email protected]> > > Cc: Fabiano Rosas <[email protected]> > > Signed-off-by: Peter Xu <[email protected]> > > --- > > MAINTAINERS | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index 355b1960ce..f6b21da753 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -3217,6 +3217,11 @@ F: qapi/migration.json > > F: tests/migration/ > > F: util/userfaultfd.c > > > > +RDMA Migration > > +R: Li Zhijian <[email protected]> > > +S: Odd Fixes > > +F: migration/rdma* > > + > > Migration dirty limit and dirty page rate > > M: Hyman Huang <[email protected]> > > S: Maintained > > Hmm... > > $ scripts/get_maintainer.pl --status -f migration/rdma.cLi Zhijian > <[email protected]> (reviewer:RDMA Migration) > Juan Quintela <[email protected]> (maintainer:Migration) > Peter Xu <[email protected]> (reviewer:Migration) > Leonardo Bras <[email protected]> (reviewer:Migration) > [email protected] (open list:All patches CC here) > Odd Fixes > Maintained > > Contradictory status. Do we care?
Not a problem to me, but if you or anyone thinks that's better, please feel free to post another patch to replace this one with Zhijian's ACK attached. > > To avoid, try something like the appended patch. With that one: > > $ scripts/get_maintainer.pl --status -f migration/rdma.c > Juan Quintela <[email protected]> (odd fixer:RDMA Migration) > Li Zhijian <[email protected]> (reviewer:RDMA Migration) > Peter Xu <[email protected]> (reviewer:RDMA Migration) > Leonardo Bras <[email protected]> (reviewer:RDMA Migration) > [email protected] (open list:All patches CC here) > Odd Fixes > > > diff --git a/MAINTAINERS b/MAINTAINERS > index 355b1960ce..3e80857eab 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -3216,6 +3216,15 @@ F: docs/devel/migration.rst > F: qapi/migration.json > F: tests/migration/ > F: util/userfaultfd.c > +X: migration/rdma* > + > +RDMA Migration > +M: Juan Quintela <[email protected]> > +R: Li Zhijian <[email protected]> > +R: Peter Xu <[email protected]> > +R: Leonardo Bras <[email protected]> > +S: Odd Fixes > +F: migration/rdma* It's slightly a pity we need to copy the lines, and I think this may not be the only place where we have >1 entries pointing to the same file. > > Migration dirty limit and dirty page rate > M: Hyman Huang <[email protected]> > Thanks, -- Peter Xu
