Re: [PATCH V2 01/45] MAINTAINERS: Add reviewer for CPR

[Peter Xu <peterx@redhat.com>]

On Fri, Feb 14, 2025 at 06:13:43AM -0800, Steve Sistare wrote:
> CPR is integrated with live migration, and has the same maintainers.
> But, add a CPR section to add a reviewer.
> 
> Signed-off-by: Steve Sistare <steven.sistare@oracle.com>
> ---
>  MAINTAINERS | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 3848d37..2f9a6da 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2951,6 +2951,17 @@ F: include/qemu/co-shared-resource.h
>  T: git https://gitlab.com/jsnow/qemu.git jobs
>  T: git https://gitlab.com/vsementsov/qemu.git block
>  
> +CheckPoint and Restart (CPR)
> +M: Peter Xu <peterx@redhat.com>
> +M: Fabiano Rosas <farosas@suse.de>
> +R: Steve Sistare <steven.sistare@oracle.com>
> +S: Supported
> +F: hw/vfio/cpr*
> +F: include/migration/cpr.h
> +F: migration/cpr*
> +F: tests/qtest/migration/cpr*
> +F: docs/devel/migration/CPR.rst

All above files are covered by either migration or vfio.

If the plan is to have CPR being part of existing subsystems, IMHO we could
drop the M: entries here but keep R: only.  Or, make one M: entry for
yourself.

With that, IIUC anyone using get_maintainers.pl will always get the right
people to copy: it goes to VFIO if it's under the 1st entry (hw/vfio/cpr*),
or it goes to migration if it's the rest four entries.  Meanwhile, if any
of above is touched you'll get copied too.

> +
>  Compute Express Link
>  M: Jonathan Cameron <jonathan.cameron@huawei.com>
>  R: Fan Ni <fan.ni@samsung.com>
> -- 
> 1.8.3.1
> 

-- 
Peter Xu