Re: [PATCH] MAINTAINERS: Add entry for rdma migration

[Markus Armbruster]

Peter Xu <peterx@redhat.com> 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é <berrange@redhat.com>
> Cc: Juan Quintela <quintela@redhat.com>
> Cc: Markus Armbruster <armbru@redhat.com>
> Cc: Zhijian Li (Fujitsu) <lizhijian@fujitsu.com>
> Cc: Fabiano Rosas <farosas@suse.de>
> Signed-off-by: Peter Xu <peterx@redhat.com>
> ---
>  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 <lizhijian@fujitsu.com>
> +S: Odd Fixes
> +F: migration/rdma*
> +
>  Migration dirty limit and dirty page rate
>  M: Hyman Huang <yong.huang@smartx.com>
>  S: Maintained

Hmm...

    $ scripts/get_maintainer.pl --status -f migration/rdma.cLi Zhijian 
<lizhijian@fujitsu.com> (reviewer:RDMA Migration)
    Juan Quintela <quintela@redhat.com> (maintainer:Migration)
    Peter Xu <peterx@redhat.com> (reviewer:Migration)
    Leonardo Bras <leobras@redhat.com> (reviewer:Migration)
    qemu-devel@nongnu.org (open list:All patches CC here)
    Odd Fixes
    Maintained

Contradictory status.  Do we care?

To avoid, try something like the appended patch.  With that one:

    $ scripts/get_maintainer.pl --status -f migration/rdma.c
    Juan Quintela <quintela@redhat.com> (odd fixer:RDMA Migration)
    Li Zhijian <lizhijian@fujitsu.com> (reviewer:RDMA Migration)
    Peter Xu <peterx@redhat.com> (reviewer:RDMA Migration)
    Leonardo Bras <leobras@redhat.com> (reviewer:RDMA Migration)
    qemu-devel@nongnu.org (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 <quintela@redhat.com>
+R: Li Zhijian <lizhijian@fujitsu.com>
+R: Peter Xu <peterx@redhat.com>
+R: Leonardo Bras <leobras@redhat.com>
+S: Odd Fixes
+F: migration/rdma*
 
 Migration dirty limit and dirty page rate
 M: Hyman Huang <yong.huang@smartx.com>