[PATCH RFC] MAINTAINERS: add TRACE EVENT LIBRARY section

[PATCH RFC] MAINTAINERS: add TRACE EVENT LIBRARY section

[Lukas Bulwahn]

The git history shows that the files under ./tools/lib/traceevent/ are
being developed and maintained by Tzetomir Stoyanov and Steven Rostedt
and are discussed on the linux-trace-devel list.

Add a suitable section in MAINTAINERS for patches to reach them.

This was identified with a small script that finds all files only
belonging to "THE REST" according to the current MAINTAINERS file, and I
acted upon its output.

Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
Ceco, Steven, I added the information based on what I could see from an
outsider view. Please change and more files to the entry if needed.

applies cleanly on current master and next-20200131

Ceco, congrats becoming a kernel maintainer :)

 MAINTAINERS | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 1f77fb8cdde3..17eb358c3fda 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -16903,6 +16903,13 @@ T:	git git://git.infradead.org/users/jjs/linux-tpmdd.git
 S:	Maintained
 F:	drivers/char/tpm/
 
+TRACE EVENT LIBRARY
+M:	Tzvetomir Stoyanov <tz.stoyanov@gmail.com>
+M:	Steven Rostedt <rostedt@goodmis.org>
+L:	linux-trace-devel@vger.kernel.org
+S:	Maintained
+F:	tools/lib/traceevent/
+
 TRACING
 M:	Steven Rostedt <rostedt@goodmis.org>
 M:	Ingo Molnar <mingo@redhat.com>
-- 
2.17.1

Re: [PATCH RFC] MAINTAINERS: add TRACE EVENT LIBRARY section

[Steven Rostedt]

On Sat,  1 Feb 2020 17:19:31 +0100
Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:

> The git history shows that the files under ./tools/lib/traceevent/ are
> being developed and maintained by Tzetomir Stoyanov and Steven Rostedt
> and are discussed on the linux-trace-devel list.
> 
> Add a suitable section in MAINTAINERS for patches to reach them.
> 
> This was identified with a small script that finds all files only
> belonging to "THE REST" according to the current MAINTAINERS file, and I
> acted upon its output.

Thanks Lukas!

Arnaldo, would you like to take this?

Acked-by: Steven Rostedt (VMware) <rostedt@goodmis.org>

-- Steve

> 
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---
> Ceco, Steven, I added the information based on what I could see from an
> outsider view. Please change and more files to the entry if needed.
> 
> applies cleanly on current master and next-20200131
> 
> Ceco, congrats becoming a kernel maintainer :)
> 
>  MAINTAINERS | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1f77fb8cdde3..17eb358c3fda 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -16903,6 +16903,13 @@ T:	git git://git.infradead.org/users/jjs/linux-tpmdd.git
>  S:	Maintained
>  F:	drivers/char/tpm/
>  
> +TRACE EVENT LIBRARY
> +M:	Tzvetomir Stoyanov <tz.stoyanov@gmail.com>
> +M:	Steven Rostedt <rostedt@goodmis.org>
> +L:	linux-trace-devel@vger.kernel.org
> +S:	Maintained
> +F:	tools/lib/traceevent/
> +
>  TRACING
>  M:	Steven Rostedt <rostedt@goodmis.org>
>  M:	Ingo Molnar <mingo@redhat.com>

Re: [PATCH RFC] MAINTAINERS: add TRACE EVENT LIBRARY section

[Arnaldo Carvalho de Melo]

Em Sat, Feb 01, 2020 at 01:14:53PM -0500, Steven Rostedt escreveu:
> On Sat,  1 Feb 2020 17:19:31 +0100
> Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
> 
> > The git history shows that the files under ./tools/lib/traceevent/ are
> > being developed and maintained by Tzetomir Stoyanov and Steven Rostedt
> > and are discussed on the linux-trace-devel list.
> > 
> > Add a suitable section in MAINTAINERS for patches to reach them.
> > 
> > This was identified with a small script that finds all files only
> > belonging to "THE REST" according to the current MAINTAINERS file, and I
> > acted upon its output.
> 
> Thanks Lukas!
> 
> Arnaldo, would you like to take this?
> 
> Acked-by: Steven Rostedt (VMware) <rostedt@goodmis.org>

Sure
 
> -- Steve
> 
> > 
> > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> > ---
> > Ceco, Steven, I added the information based on what I could see from an
> > outsider view. Please change and more files to the entry if needed.
> > 
> > applies cleanly on current master and next-20200131
> > 
> > Ceco, congrats becoming a kernel maintainer :)
> > 
> >  MAINTAINERS | 7 +++++++
> >  1 file changed, 7 insertions(+)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 1f77fb8cdde3..17eb358c3fda 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -16903,6 +16903,13 @@ T:	git git://git.infradead.org/users/jjs/linux-tpmdd.git
> >  S:	Maintained
> >  F:	drivers/char/tpm/
> >  
> > +TRACE EVENT LIBRARY
> > +M:	Tzvetomir Stoyanov <tz.stoyanov@gmail.com>
> > +M:	Steven Rostedt <rostedt@goodmis.org>
> > +L:	linux-trace-devel@vger.kernel.org
> > +S:	Maintained
> > +F:	tools/lib/traceevent/
> > +
> >  TRACING
> >  M:	Steven Rostedt <rostedt@goodmis.org>
> >  M:	Ingo Molnar <mingo@redhat.com>
> 

-- 

- Arnaldo

Re: [PATCH RFC] MAINTAINERS: add TRACE EVENT LIBRARY section

[Tzvetomir Stoyanov]

On Sat, Feb 1, 2020 at 6:19 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
>
> The git history shows that the files under ./tools/lib/traceevent/ are
> being developed and maintained by Tzetomir Stoyanov and Steven Rostedt
> and are discussed on the linux-trace-devel list.
>
> Add a suitable section in MAINTAINERS for patches to reach them.
>
> This was identified with a small script that finds all files only
> belonging to "THE REST" according to the current MAINTAINERS file, and I
> acted upon its output.
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---
> Ceco, Steven, I added the information based on what I could see from an
> outsider view. Please change and more files to the entry if needed.
>
> applies cleanly on current master and next-20200131
>
> Ceco, congrats becoming a kernel maintainer :)
>
Thanks, Lukas!

>  MAINTAINERS | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1f77fb8cdde3..17eb358c3fda 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -16903,6 +16903,13 @@ T:     git git://git.infradead.org/users/jjs/linux-tpmdd.git
>  S:     Maintained
>  F:     drivers/char/tpm/
>
> +TRACE EVENT LIBRARY
> +M:     Tzvetomir Stoyanov <tz.stoyanov@gmail.com>
> +M:     Steven Rostedt <rostedt@goodmis.org>
> +L:     linux-trace-devel@vger.kernel.org
> +S:     Maintained
> +F:     tools/lib/traceevent/
> +
>  TRACING
>  M:     Steven Rostedt <rostedt@goodmis.org>
>  M:     Ingo Molnar <mingo@redhat.com>
> --
> 2.17.1
>


-- 
Tzvetomir (Ceco) Stoyanov
VMware Open Source Technology Center

Re: [PATCH RFC] MAINTAINERS: add TRACE EVENT LIBRARY section

[Andy Shevchenko]

On Sat, Feb 1, 2020 at 6:21 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
>
> The git history shows that the files under ./tools/lib/traceevent/ are
> being developed and maintained by Tzetomir Stoyanov and Steven Rostedt
> and are discussed on the linux-trace-devel list.
>
> Add a suitable section in MAINTAINERS for patches to reach them.
>
> This was identified with a small script that finds all files only
> belonging to "THE REST" according to the current MAINTAINERS file, and I
> acted upon its output.

> +TRACE EVENT LIBRARY
> +M:     Tzvetomir Stoyanov <tz.stoyanov@gmail.com>
> +M:     Steven Rostedt <rostedt@goodmis.org>
> +L:     linux-trace-devel@vger.kernel.org
> +S:     Maintained
> +F:     tools/lib/traceevent/

Don't forget to run early mentioned scripts (in some other threads).

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH RFC] MAINTAINERS: add TRACE EVENT LIBRARY section

[Lukas Bulwahn]

On Mon, 3 Feb 2020, Andy Shevchenko wrote:

> On Sat, Feb 1, 2020 at 6:21 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
> >
> > The git history shows that the files under ./tools/lib/traceevent/ are
> > being developed and maintained by Tzetomir Stoyanov and Steven Rostedt
> > and are discussed on the linux-trace-devel list.
> >
> > Add a suitable section in MAINTAINERS for patches to reach them.
> >
> > This was identified with a small script that finds all files only
> > belonging to "THE REST" according to the current MAINTAINERS file, and I
> > acted upon its output.
> 
> > +TRACE EVENT LIBRARY
> > +M:     Tzvetomir Stoyanov <tz.stoyanov@gmail.com>
> > +M:     Steven Rostedt <rostedt@goodmis.org>
> > +L:     linux-trace-devel@vger.kernel.org
> > +S:     Maintained
> > +F:     tools/lib/traceevent/
> 
> Don't forget to run early mentioned scripts (in some other threads).
>

Andy, I did run on next-20200203:

$ ./scripts/checkpatch.pl -f MAINTAINERS

WARNING: MAINTAINERS entries use one tab after TYPE:
#14607: FILE: MAINTAINERS:14607:
+M:     Micah Morton <mortonm@chromium.org>

WARNING: MAINTAINERS entries use one tab after TYPE:
#14608: FILE: MAINTAINERS:14608:
+S:     Supported

WARNING: MAINTAINERS entries use one tab after TYPE:
#14609: FILE: MAINTAINERS:14609:
+F:     security/safesetid/

WARNING: MAINTAINERS entries use one tab after TYPE:
#14610: FILE: MAINTAINERS:14610:
+F:     Documentation/admin-guide/LSM/SafeSetID.rst

total: 0 errors, 4 warnings, 18577 lines checked


That issue in MAINTAINERS has a pending patch since 2019-12-07, with three 
attempts of asking to be picked up by now:

- https://lore.kernel.org/lkml/20191207182751.14249-1-lukas.bulwahn@gmail.com/
- https://lore.kernel.org/lkml/20200116185844.11201-1-lukas.bulwahn@gmail.com/
- https://lore.kernel.org/lkml/20200204040434.7173-1-lukas.bulwahn@gmail.com/

It is not related to this patch in MAINTAINERS here.


I also ran $ perl ./scripts/parse-maintainers.pl and checked the generated 
diff for this entry, but there was no reordering required; a one-element 
list of F: entries is difficult to get unsorted ;)

I am not adding any mess (ordering issues) to MAINTAINERS with this patch, 
other than what is already there, but cleaning that up is completely other 
story.

Lukas

Re: [PATCH RFC] MAINTAINERS: add TRACE EVENT LIBRARY section

[Andy Shevchenko]

On Tue, Feb 4, 2020 at 6:42 AM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
>
>
>
> On Mon, 3 Feb 2020, Andy Shevchenko wrote:
>
> > On Sat, Feb 1, 2020 at 6:21 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
> > >
> > > The git history shows that the files under ./tools/lib/traceevent/ are
> > > being developed and maintained by Tzetomir Stoyanov and Steven Rostedt
> > > and are discussed on the linux-trace-devel list.
> > >
> > > Add a suitable section in MAINTAINERS for patches to reach them.
> > >
> > > This was identified with a small script that finds all files only
> > > belonging to "THE REST" according to the current MAINTAINERS file, and I
> > > acted upon its output.
> >
> > > +TRACE EVENT LIBRARY
> > > +M:     Tzvetomir Stoyanov <tz.stoyanov@gmail.com>
> > > +M:     Steven Rostedt <rostedt@goodmis.org>
> > > +L:     linux-trace-devel@vger.kernel.org
> > > +S:     Maintained
> > > +F:     tools/lib/traceevent/
> >
> > Don't forget to run early mentioned scripts (in some other threads).
> >
>
> Andy, I did run on next-20200203:

Awesome, thanks!

> $ ./scripts/checkpatch.pl -f MAINTAINERS
>
> WARNING: MAINTAINERS entries use one tab after TYPE:
> #14607: FILE: MAINTAINERS:14607:
> +M:     Micah Morton <mortonm@chromium.org>
>
> WARNING: MAINTAINERS entries use one tab after TYPE:
> #14608: FILE: MAINTAINERS:14608:
> +S:     Supported
>
> WARNING: MAINTAINERS entries use one tab after TYPE:
> #14609: FILE: MAINTAINERS:14609:
> +F:     security/safesetid/
>
> WARNING: MAINTAINERS entries use one tab after TYPE:
> #14610: FILE: MAINTAINERS:14610:
> +F:     Documentation/admin-guide/LSM/SafeSetID.rst
>
> total: 0 errors, 4 warnings, 18577 lines checked
>
>
> That issue in MAINTAINERS has a pending patch since 2019-12-07, with three
> attempts of asking to be picked up by now:
>
> - https://lore.kernel.org/lkml/20191207182751.14249-1-lukas.bulwahn@gmail.com/
> - https://lore.kernel.org/lkml/20200116185844.11201-1-lukas.bulwahn@gmail.com/
> - https://lore.kernel.org/lkml/20200204040434.7173-1-lukas.bulwahn@gmail.com/
>
> It is not related to this patch in MAINTAINERS here.
>
>
> I also ran $ perl ./scripts/parse-maintainers.pl and checked the generated
> diff for this entry, but there was no reordering required; a one-element
> list of F: entries is difficult to get unsorted ;)
>
> I am not adding any mess (ordering issues) to MAINTAINERS with this patch,
> other than what is already there, but cleaning that up is completely other
> story.

-- 
With Best Regards,
Andy Shevchenko

Re: [PATCH RFC] MAINTAINERS: add TRACE EVENT LIBRARY section

[Lukas Bulwahn]

On Sat, Feb 1, 2020 at 8:02 PM Arnaldo Carvalho de Melo
<arnaldo.melo@gmail.com> wrote:
>
> Em Sat, Feb 01, 2020 at 01:14:53PM -0500, Steven Rostedt escreveu:
> > On Sat,  1 Feb 2020 17:19:31 +0100
> > Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
> >
> > > The git history shows that the files under ./tools/lib/traceevent/ are
> > > being developed and maintained by Tzetomir Stoyanov and Steven Rostedt
> > > and are discussed on the linux-trace-devel list.
> > >
> > > Add a suitable section in MAINTAINERS for patches to reach them.
> > >
> > > This was identified with a small script that finds all files only
> > > belonging to "THE REST" according to the current MAINTAINERS file, and I
> > > acted upon its output.
> >
> > Thanks Lukas!
> >
> > Arnaldo, would you like to take this?
> >
> > Acked-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
>
> Sure
>

Arnaldo, I think you never picked this patch and it did not make it
into mainline. I checked that the patch still applies to the current
master, so maybe you can pick it now. It is a minor non-urgent patch
and cannot break any builds :)

Thanks,

Lukas