| Message ID | 20210713123356.6924-14-sj38.park@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | Introduce Data Access MONitor (DAMON) | expand |
On Tue, 2021-07-13 at 12:33 +0000, SeongJae Park wrote: > From: SeongJae Park <sjpark@amazon.de> > > This commit updates MAINTAINERS file for DAMON related files. > > Signed-off-by: SeongJae Park <sjpark@amazon.de> > Reviewed-by: Markus Boehme <markubo@amazon.de> > --- > 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@amazon.de> > +L: linux-mm@kvack.org > +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@amazon.de> L: linux-mm@kvack.org 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/
From: SeongJae Park <sjpark@amazon.de> Hello Joe, On Tue, 13 Jul 2021 09:49:53 -0700 Joe Perches <joe@perches.com> wrote: > On Tue, 2021-07-13 at 12:33 +0000, SeongJae Park wrote: > > From: SeongJae Park <sjpark@amazon.de> > > > > This commit updates MAINTAINERS file for DAMON related files. > > > > Signed-off-by: SeongJae Park <sjpark@amazon.de> > > Reviewed-by: Markus Boehme <markubo@amazon.de> > > --- > > 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@amazon.de> > > +L: linux-mm@kvack.org > > +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@amazon.de> > L: linux-mm@kvack.org > 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/ There is no subdirectory under the paths. I will update those as you recommended in the next spin. Thanks, SeongJae Park
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@amazon.de> +L: linux-mm@kvack.org +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/* + DAVICOM FAST ETHERNET (DMFE) NETWORK DRIVER L: netdev@vger.kernel.org S: Orphan