Re: [PATCH v33 13/13] MAINTAINERS: Update for DAMON

From: Joe Perches
Date: Tue Jul 13 2021 - 12:50:11 EST


On Tue, 2021-07-13 at 12:33 +0000, SeongJae Park wrote:
> From: SeongJae Park <sjpark@xxxxxxxxx>
>
> This commit updates MAINTAINERS file for DAMON related files.
>
> Signed-off-by: SeongJae Park <sjpark@xxxxxxxxx>
> Reviewed-by: Markus Boehme <markubo@xxxxxxxxx>
> ---
>  MAINTAINERS | 12 ++++++++++++
>  1 file changed, 12 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a61f4f3b78a9..d492ae8b77de 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -5114,6 +5114,18 @@ F: net/ax25/ax25_out.c
>  F: net/ax25/ax25_timer.c
>  F: net/ax25/sysctl_net_ax25.c
>  
>
> +DATA ACCESS MONITOR
> +M: SeongJae Park <sjpark@xxxxxxxxx>
> +L: linux-mm@xxxxxxxxx
> +S: Maintained
> +F: Documentation/admin-guide/mm/damon/*
> +F: Documentation/vm/damon/*
> +F: include/linux/damon.h
> +F: include/trace/events/damon.h
> +F: mm/damon/*
> +F: tools/damon/*
> +F: tools/testing/selftests/damon/*

Using "F: <path>/*" means that you are maintaining only
files in that specific path and not any subdirectory of that path.

Are there subdirectories in these uses of <path>/*
that you are avoiding?

If not, the proper use is just a trailing forward slash like

F: <path>/

I believe this should be:

DATA ACCESS MONITOR
M: SeongJae Park <sjpark@xxxxxxxxx>
L: linux-mm@xxxxxxxxx
S: Maintained
F: Documentation/admin-guide/mm/damon/
F: Documentation/vm/damon/
F: include/linux/damon.h
F: include/trace/events/damon.h
F: mm/damon/
F: tools/damon/
F: tools/testing/selftests/damon/