[PATCH tip/core/rcu 0/3] Maintainership changes for 3.17

[PATCH tip/core/rcu 0/3] Maintainership changes for 3.17

[Paul E. McKenney]

Hello!

This series updates RCU maintainership as follows:

1.	Add "R:" designated-reviewers tag as discussed on ksummit-discuss
	email list.

2.	Add designated reviewers for RCU.

3.	Update RCU maintainership.

							Thanx, Paul

------------------------------------------------------------------------

 b/MAINTAINERS |   14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

[PATCH tip/core/rcu 1/3] MAINTAINERS: Add "R:" designated-reviewers tag

[Paul E. McKenney]

From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>

A ksummit-discuss email thread looked at the difficulty recruiting
and retaining reviewers.  Paul Walmsley also noted the need for patch
submitters to know who the key reviewers are and suggested adding an
"R:" tag to the MAINTAINERS file to record this information on a
per-subsystem basis.  This commit does just that, and a subsequent
commit tags the designated reviewer for the RCU-related subsystems.

http://lists.linuxfoundation.org/pipermail/ksummit-discuss/2014-May/000830.html

Suggested-by: Paul Walmsley <paul@pwsan.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Reviewed-by: Josh Triplett <josh@joshtriplett.org>
Acked-by: Steven Rostedt <rostedt@goodmis.org>
---
 MAINTAINERS | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 134483f206e4..1814075c0e74 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -70,6 +70,8 @@ Descriptions of section entries:
 
 	P: Person (obsolete)
 	M: Mail patches to: FullName <address@domain>
+	R: Designated reviewer: FullName <address@domain>
+	   These reviewers should be CCed on patches.
 	L: Mailing list that is relevant to this area
 	W: Web-page with status/info
 	Q: Patchwork web based patch tracking system site
-- 
1.8.1.5

[PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Paul E. McKenney]

From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>

Adding Steven Rostedt, Mathieu Desnoyers, and Lai Jiangshan as designated
RCU reviewers based on recent emails:

o	https://lkml.org/lkml/2014/6/2/578 (Steven)
o	https://lkml.org/lkml/2014/6/2/621 (Mathieu)
o	https://lkml.org/lkml/2014/6/3/897 (Lai)

Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Reviewed-by: Josh Triplett <josh@joshtriplett.org>
---
 MAINTAINERS | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 1814075c0e74..92f2bf91eec1 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7405,6 +7405,9 @@ F:	kernel/rcu/torture.c
 
 RCUTORTURE TEST FRAMEWORK
 M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
+R:	Steven Rostedt <rostedt@goodmis.org>
+R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
+R:	Lai Jiangshan <laijs@cn.fujitsu.com>
 L:	linux-kernel@vger.kernel.org
 S:	Supported
 T:	git git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git
@@ -7429,6 +7432,9 @@ F:	net/rds/
 READ-COPY UPDATE (RCU)
 M:	Dipankar Sarma <dipankar@in.ibm.com>
 M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
+R:	Steven Rostedt <rostedt@goodmis.org>
+R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
+R:	Lai Jiangshan <laijs@cn.fujitsu.com>
 L:	linux-kernel@vger.kernel.org
 W:	http://www.rdrop.com/users/paulmck/RCU/
 S:	Supported
@@ -8208,6 +8214,8 @@ F:	mm/sl?b.c
 SLEEPABLE READ-COPY UPDATE (SRCU)
 M:	Lai Jiangshan <laijs@cn.fujitsu.com>
 M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
+R:	Steven Rostedt <rostedt@goodmis.org>
+R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
 L:	linux-kernel@vger.kernel.org
 W:	http://www.rdrop.com/users/paulmck/RCU/
 S:	Supported
-- 
1.8.1.5

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Peter Zijlstra]

[-- Attachment #1: Type: text/plain, Size: 1496 bytes --]

On Mon, Jul 07, 2014 at 03:42:03PM -0700, Paul E. McKenney wrote:
>  MAINTAINERS | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 1814075c0e74..92f2bf91eec1 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7405,6 +7405,9 @@ F:	kernel/rcu/torture.c
>  
>  RCUTORTURE TEST FRAMEWORK
>  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> +R:	Steven Rostedt <rostedt@goodmis.org>
> +R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> +R:	Lai Jiangshan <laijs@cn.fujitsu.com>
>  L:	linux-kernel@vger.kernel.org
>  S:	Supported
>  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git

OK,.. so if we're going to do this can we talk about the syntax here.

I would like to also use this for SCHED and PERF, but since both are
rather big I would like to be able to subdivide things. Now I could of
course go create many MAINTAINER sections, but that feels artificial.

So I would like to be able to write:

SCHEDULER:
...
R:   Steven Rostedt <rostedt@goodmis.org> (kernel/sched/rt.c)
R:   Juri Lelli <juri.lelli@gmail.com>    (kernel/sched/deadline.c)

or something like that. And here I've already done concessions in that
the above is machine readable. Ideally I'd be able to say things like:
'for sched and numa related things' add 'rik and mel'.

Hmm, maybe we can look at the patch subject tag like: sched/numa or
sched/rt or sched/deadline etc.. instead of the files touched.

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Joe Perches]

On Tue, 2014-07-08 at 10:09 +0200, Peter Zijlstra wrote:
> On Mon, Jul 07, 2014 at 03:42:03PM -0700, Paul E. McKenney wrote:
> >  MAINTAINERS | 8 ++++++++
> >  1 file changed, 8 insertions(+)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 1814075c0e74..92f2bf91eec1 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -7405,6 +7405,9 @@ F:	kernel/rcu/torture.c
> >  
> >  RCUTORTURE TEST FRAMEWORK
> >  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> > +R:	Steven Rostedt <rostedt@goodmis.org>
> > +R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> > +R:	Lai Jiangshan <laijs@cn.fujitsu.com>
> >  L:	linux-kernel@vger.kernel.org
> >  S:	Supported
> >  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git
> 
> OK,.. so if we're going to do this can we talk about the syntax here.
> 
> I would like to also use this for SCHED and PERF, but since both are
> rather big I would like to be able to subdivide things. Now I could of
> course go create many MAINTAINER sections, but that feels artificial.
> 
> So I would like to be able to write:
> 
> SCHEDULER:
> ...
> R:   Steven Rostedt <rostedt@goodmis.org> (kernel/sched/rt.c)
> R:   Juri Lelli <juri.lelli@gmail.com>    (kernel/sched/deadline.c)
> 
> or something like that. And here I've already done concessions in that
> the above is machine readable. Ideally I'd be able to say things like:
> 'for sched and numa related things' add 'rik and mel'.
> 
> Hmm, maybe we can look at the patch subject tag like: sched/numa or
> sched/rt or sched/deadline etc.. instead of the files touched.

I think it's best to use separate sections.

The parsing logic in get_maintainers is already ugly enough.

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Paul E. McKenney]

On Tue, Jul 08, 2014 at 11:06:24AM -0700, Joe Perches wrote:
> On Tue, 2014-07-08 at 10:09 +0200, Peter Zijlstra wrote:
> > On Mon, Jul 07, 2014 at 03:42:03PM -0700, Paul E. McKenney wrote:
> > >  MAINTAINERS | 8 ++++++++
> > >  1 file changed, 8 insertions(+)
> > > 
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 1814075c0e74..92f2bf91eec1 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -7405,6 +7405,9 @@ F:	kernel/rcu/torture.c
> > >  
> > >  RCUTORTURE TEST FRAMEWORK
> > >  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> > > +R:	Steven Rostedt <rostedt@goodmis.org>
> > > +R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> > > +R:	Lai Jiangshan <laijs@cn.fujitsu.com>
> > >  L:	linux-kernel@vger.kernel.org
> > >  S:	Supported
> > >  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git
> > 
> > OK,.. so if we're going to do this can we talk about the syntax here.
> > 
> > I would like to also use this for SCHED and PERF, but since both are
> > rather big I would like to be able to subdivide things. Now I could of
> > course go create many MAINTAINER sections, but that feels artificial.
> > 
> > So I would like to be able to write:
> > 
> > SCHEDULER:
> > ...
> > R:   Steven Rostedt <rostedt@goodmis.org> (kernel/sched/rt.c)
> > R:   Juri Lelli <juri.lelli@gmail.com>    (kernel/sched/deadline.c)
> > 
> > or something like that. And here I've already done concessions in that
> > the above is machine readable. Ideally I'd be able to say things like:
> > 'for sched and numa related things' add 'rik and mel'.
> > 
> > Hmm, maybe we can look at the patch subject tag like: sched/numa or
> > sched/rt or sched/deadline etc.. instead of the files touched.
> 
> I think it's best to use separate sections.
> 
> The parsing logic in get_maintainers is already ugly enough.

OK, here is a basic change.  Hey, I figure that if you weren't desperately
in need of entertainment, you would not have asked me to hack a perl script!

							Thanx, Paul

------------------------------------------------------------------------

scripts: Teach get_maintainer.pl about the new "R:" tag

We can now designate reviewers in the MAINTAINERS file with the new
"R:" tag, so this commit teaches get_maintainers.pl to add their
email addresses.

Reported-by: Joe Perches <joe@perches.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 41987885bd31..03785ce7b5b6 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -536,7 +536,7 @@ sub range_has_maintainer {
 	if ($line =~ m/^(\C):\s*(.*)/) {
 	    my $type = $1;
 	    my $value = $2;
-	    if ($type eq 'M') {
+	    if ($type eq 'M' || $type eq 'R') {
 		return 1;
 	    }
 	}
@@ -1047,7 +1047,7 @@ sub add_categories {
 			}
 		    }
 		}
-	    } elsif ($ptype eq "M") {
+	    } elsif ($ptype eq "M" || $ptype eq "R") {
 		my ($name, $address) = parse_email($pvalue);
 		if ($name eq "") {
 		    if ($i > 0) {
@@ -1062,7 +1062,11 @@ sub add_categories {
 		}
 		if ($email_maintainer) {
 		    my $role = get_maintainer_role($i);
-		    push_email_addresses($pvalue, $role);
+		    if ($ptype eq "M") {
+		      push_email_addresses($pvalue, $role);
+		    } else {
+		      push_email_addresses($pvalue, "designated reviewer");
+		    }
 		}
 	    } elsif ($ptype eq "T") {
 		push(@scm, $pvalue);

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Joe Perches]

On Tue, 2014-07-08 at 13:41 -0700, Paul E. McKenney wrote:
> On Tue, Jul 08, 2014 at 11:06:24AM -0700, Joe Perches wrote:
> > On Tue, 2014-07-08 at 10:09 +0200, Peter Zijlstra wrote:
> > > On Mon, Jul 07, 2014 at 03:42:03PM -0700, Paul E. McKenney wrote:
> > > >  MAINTAINERS | 8 ++++++++
> > > >  1 file changed, 8 insertions(+)
> > > > 
> > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > index 1814075c0e74..92f2bf91eec1 100644
> > > > --- a/MAINTAINERS
> > > > +++ b/MAINTAINERS
> > > > @@ -7405,6 +7405,9 @@ F:	kernel/rcu/torture.c
> > > >  
> > > >  RCUTORTURE TEST FRAMEWORK
> > > >  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> > > > +R:	Steven Rostedt <rostedt@goodmis.org>
> > > > +R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> > > > +R:	Lai Jiangshan <laijs@cn.fujitsu.com>
> > > >  L:	linux-kernel@vger.kernel.org
> > > >  S:	Supported
> > > >  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git
> > > 
> > > OK,.. so if we're going to do this can we talk about the syntax here.
> > > 
> > > I would like to also use this for SCHED and PERF, but since both are
> > > rather big I would like to be able to subdivide things. Now I could of
> > > course go create many MAINTAINER sections, but that feels artificial.
> > > 
> > > So I would like to be able to write:
> > > 
> > > SCHEDULER:
> > > ...
> > > R:   Steven Rostedt <rostedt@goodmis.org> (kernel/sched/rt.c)
> > > R:   Juri Lelli <juri.lelli@gmail.com>    (kernel/sched/deadline.c)
> > > 
> > > or something like that. And here I've already done concessions in that
> > > the above is machine readable. Ideally I'd be able to say things like:
> > > 'for sched and numa related things' add 'rik and mel'.
> > > 
> > > Hmm, maybe we can look at the patch subject tag like: sched/numa or
> > > sched/rt or sched/deadline etc.. instead of the files touched.
> > 
> > I think it's best to use separate sections.
> > 
> > The parsing logic in get_maintainers is already ugly enough.
> 
> OK, here is a basic change.  Hey, I figure that if you weren't desperately
> in need of entertainment, you would not have asked me to hack a perl script!
> 
> 							Thanx, Paul
> 
> ------------------------------------------------------------------------
> 
> scripts: Teach get_maintainer.pl about the new "R:" tag
> 
> We can now designate reviewers in the MAINTAINERS file with the new
> "R:" tag, so this commit teaches get_maintainers.pl to add their
> email addresses.
> 
> Reported-by: Joe Perches <joe@perches.com>
> Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> 
> diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
> index 41987885bd31..03785ce7b5b6 100755
> --- a/scripts/get_maintainer.pl
> +++ b/scripts/get_maintainer.pl
> @@ -536,7 +536,7 @@ sub range_has_maintainer {
>  	if ($line =~ m/^(\C):\s*(.*)/) {
>  	    my $type = $1;
>  	    my $value = $2;
> -	    if ($type eq 'M') {
> +	    if ($type eq 'M' || $type eq 'R') {
>  		return 1;
>  	    }
>  	}
> @@ -1047,7 +1047,7 @@ sub add_categories {
>  			}
>  		    }
>  		}..
> -	    } elsif ($ptype eq "M") {
> +	    } elsif ($ptype eq "M" || $ptype eq "R") {
>  		my ($name, $address) = parse_email($pvalue);
>  		if ($name eq "") {
>  		    if ($i > 0) {
> @@ -1062,7 +1062,11 @@ sub add_categories {
>  		}
>  		if ($email_maintainer) {
>  		    my $role = get_maintainer_role($i);
> -		    push_email_addresses($pvalue, $role);
> +		    if ($ptype eq "M") {
> +		      push_email_addresses($pvalue, $role);
> +		    } else {
> +		      push_email_addresses($pvalue, "designated reviewer");
> +		    }
>  		}
>  	    } elsif ($ptype eq "T") {
>  		push(@scm, $pvalue);
> 

Paul, I already sent you a better version over a month ago.

https://lkml.org/lkml/2014/6/2/585

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Paul E. McKenney]

On Tue, Jul 08, 2014 at 02:01:04PM -0700, Joe Perches wrote:
> On Tue, 2014-07-08 at 13:41 -0700, Paul E. McKenney wrote:
> > On Tue, Jul 08, 2014 at 11:06:24AM -0700, Joe Perches wrote:
> > > On Tue, 2014-07-08 at 10:09 +0200, Peter Zijlstra wrote:
> > > > On Mon, Jul 07, 2014 at 03:42:03PM -0700, Paul E. McKenney wrote:
> > > > >  MAINTAINERS | 8 ++++++++
> > > > >  1 file changed, 8 insertions(+)
> > > > > 
> > > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > > index 1814075c0e74..92f2bf91eec1 100644
> > > > > --- a/MAINTAINERS
> > > > > +++ b/MAINTAINERS
> > > > > @@ -7405,6 +7405,9 @@ F:	kernel/rcu/torture.c
> > > > >  
> > > > >  RCUTORTURE TEST FRAMEWORK
> > > > >  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> > > > > +R:	Steven Rostedt <rostedt@goodmis.org>
> > > > > +R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> > > > > +R:	Lai Jiangshan <laijs@cn.fujitsu.com>
> > > > >  L:	linux-kernel@vger.kernel.org
> > > > >  S:	Supported
> > > > >  T:	git git://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git
> > > > 
> > > > OK,.. so if we're going to do this can we talk about the syntax here.
> > > > 
> > > > I would like to also use this for SCHED and PERF, but since both are
> > > > rather big I would like to be able to subdivide things. Now I could of
> > > > course go create many MAINTAINER sections, but that feels artificial.
> > > > 
> > > > So I would like to be able to write:
> > > > 
> > > > SCHEDULER:
> > > > ...
> > > > R:   Steven Rostedt <rostedt@goodmis.org> (kernel/sched/rt.c)
> > > > R:   Juri Lelli <juri.lelli@gmail.com>    (kernel/sched/deadline.c)
> > > > 
> > > > or something like that. And here I've already done concessions in that
> > > > the above is machine readable. Ideally I'd be able to say things like:
> > > > 'for sched and numa related things' add 'rik and mel'.
> > > > 
> > > > Hmm, maybe we can look at the patch subject tag like: sched/numa or
> > > > sched/rt or sched/deadline etc.. instead of the files touched.
> > > 
> > > I think it's best to use separate sections.
> > > 
> > > The parsing logic in get_maintainers is already ugly enough.
> > 
> > OK, here is a basic change.  Hey, I figure that if you weren't desperately
> > in need of entertainment, you would not have asked me to hack a perl script!
> > 
> > 							Thanx, Paul
> > 
> > ------------------------------------------------------------------------
> > 
> > scripts: Teach get_maintainer.pl about the new "R:" tag

[ . . . ]

> Paul, I already sent you a better version over a month ago.

That would explain why I had already forgotten about it.  ;-)

> https://lkml.org/lkml/2014/6/2/585

Unless you tell me otherwise, I assume that this means that you are
OK with my adding your Signed-off-by to that patch.  (The lack thereof
being another reason I would not have queued it.)  Please see below for
the proposed commit log and Signed-off-by.

If you don't tell me otherwise, I will submit this to the next merge
window.

								Thanx, Paul

------------------------------------------------------------------------

scripts: Teach get_maintainer.pl about the new "R:" tag

We can now designate reviewers in the MAINTAINERS file with the new
"R:" tag, so this commit teaches get_maintainers.pl to add their
email addresses.

Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>

diff --git a/scripts/get_maintainer.pl b/scripts/get_maintainer.pl
index 41987885bd31..d7016279ec2b 100755
--- a/scripts/get_maintainer.pl
+++ b/scripts/get_maintainer.pl
@@ -21,6 +21,7 @@ my $lk_path = "./";
 my $email = 1;
 my $email_usename = 1;
 my $email_maintainer = 1;
+my $email_reviewer = 1;
 my $email_list = 1;
 my $email_subscriber_list = 0;
 my $email_git_penguin_chiefs = 0;
@@ -202,6 +203,7 @@ if (!GetOptions(
 		'remove-duplicates!' => \$email_remove_duplicates,
 		'mailmap!' => \$email_use_mailmap,
 		'm!' => \$email_maintainer,
+		'r!' => \$email_reviewer,
 		'n!' => \$email_usename,
 		'l!' => \$email_list,
 		's!' => \$email_subscriber_list,
@@ -260,7 +262,8 @@ if ($sections) {
 }
 
 if ($email &&
-    ($email_maintainer + $email_list + $email_subscriber_list +
+    ($email_maintainer + $email_reviewer +
+     $email_list + $email_subscriber_list +
      $email_git + $email_git_penguin_chiefs + $email_git_blame) == 0) {
     die "$P: Please select at least 1 email option\n";
 }
@@ -750,6 +753,7 @@ MAINTAINER field selection options:
     --hg-since => hg history to use (default: $email_hg_since)
     --interactive => display a menu (mostly useful if used with the --git option)
     --m => include maintainer(s) if any
+    --r => include reviewer(s) if any
     --n => include name 'Full Name <addr\@domain.tld>'
     --l => include list(s) if any
     --s => include subscriber only list(s) if any
@@ -1064,6 +1068,22 @@ sub add_categories {
 		    my $role = get_maintainer_role($i);
 		    push_email_addresses($pvalue, $role);
 		}
+	    } elsif ($ptype eq "R") {
+		my ($name, $address) = parse_email($pvalue);
+		if ($name eq "") {
+		    if ($i > 0) {
+			my $tv = $typevalue[$i - 1];
+			if ($tv =~ m/^(\C):\s*(.*)/) {
+			    if ($1 eq "P") {
+				$name = $2;
+				$pvalue = format_email($name, $address, $email_usename);
+			    }
+			}
+		    }
+		}
+		if ($email_reviewer) {
+		    push_email_addresses($pvalue, 'reviewer');
+		}
 	    } elsif ($ptype eq "T") {
 		push(@scm, $pvalue);
 	    } elsif ($ptype eq "W") {

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Joe Perches]

On Tue, 2014-07-08 at 14:52 -0700, Paul E. McKenney wrote:
> On Tue, Jul 08, 2014 at 02:01:04PM -0700, Joe Perches wrote:
> > On Tue, 2014-07-08 at 13:41 -0700, Paul E. McKenney wrote:
[]
> > > scripts: Teach get_maintainer.pl about the new "R:" tag
> 
> [ . . . ]
> 
> > Paul, I already sent you a better version over a month ago.
> 
> That would explain why I had already forgotten about it.  ;-)
> 
> > https://lkml.org/lkml/2014/6/2/585

Too bad because you added your own "tested-by"
in a reply. :p

> Unless you tell me otherwise, I assume that this means that you are
> OK with my adding your Signed-off-by to that patch.  (The lack thereof
> being another reason I would not have queued it.)  Please see below for
> the proposed commit log and Signed-off-by.
> 
> If you don't tell me otherwise, I will submit this to the next merge
> window.

Swell.

I still think the concept is pretty useless and it's
just a duplication of "M:", which isn't anything other
than a list of who should be sent patches.

MAINTAINERS describes the "M:" letter as just:

	M: Mail patches to: FullName <address@domain>

It doesn't describe anything like a pull hierarchy
or even describe actual maintainership.

But if "R:" is going to go in, then get_maintainers
should be updated at the same time.

btw: most "M:" entries aren't actual maintainers.

Maybe those non-maintainer "M:" entries should be
converted to the "R:" category and "M:" should
only be used to describe for the pull hierarchy.

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Paul E. McKenney]

On Tue, Jul 08, 2014 at 03:05:16PM -0700, Joe Perches wrote:
> On Tue, 2014-07-08 at 14:52 -0700, Paul E. McKenney wrote:
> > On Tue, Jul 08, 2014 at 02:01:04PM -0700, Joe Perches wrote:
> > > On Tue, 2014-07-08 at 13:41 -0700, Paul E. McKenney wrote:
> []
> > > > scripts: Teach get_maintainer.pl about the new "R:" tag
> > 
> > [ . . . ]
> > 
> > > Paul, I already sent you a better version over a month ago.
> > 
> > That would explain why I had already forgotten about it.  ;-)
> > 
> > > https://lkml.org/lkml/2014/6/2/585
> 
> Too bad because you added your own "tested-by"
> in a reply. :p

That was because I was for whatever reason thinking that Someone Else
was going to be queueing these.  As it stands, I have a Signed-off-by,
which in my case implies Tested-by.

> > Unless you tell me otherwise, I assume that this means that you are
> > OK with my adding your Signed-off-by to that patch.  (The lack thereof
> > being another reason I would not have queued it.)  Please see below for
> > the proposed commit log and Signed-off-by.
> > 
> > If you don't tell me otherwise, I will submit this to the next merge
> > window.
> 
> Swell.
> 
> I still think the concept is pretty useless and it's
> just a duplication of "M:", which isn't anything other
> than a list of who should be sent patches.
> 
> MAINTAINERS describes the "M:" letter as just:
> 
> 	M: Mail patches to: FullName <address@domain>
> 
> It doesn't describe anything like a pull hierarchy
> or even describe actual maintainership.
> 
> But if "R:" is going to go in, then get_maintainers
> should be updated at the same time.
> 
> btw: most "M:" entries aren't actual maintainers.
> 
> Maybe those non-maintainer "M:" entries should be
> converted to the "R:" category and "M:" should
> only be used to describe for the pull hierarchy.

It will be interesting to see how things go.  There did seem to be
some people who were comfortable being listed as RCU reviewers, but
not as RCU maintainers.  Perhaps the same thing happens elsewhere.

							Thanx, Paul

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Joe Perches]

On Tue, 2014-07-08 at 15:25 -0700, Paul E. McKenney wrote:
> On Tue, Jul 08, 2014 at 03:05:16PM -0700, Joe Perches wrote:
[]
> > I still think the concept is pretty useless and it's
> > just a duplication of "M:", which isn't anything other
> > than a list of who should be sent patches.
[]
> It will be interesting to see how things go.

Yes, it might work out fine and maybe might cause some
other beneficial changes.

> There did seem to be
> some people who were comfortable being listed as RCU reviewers, but
> not as RCU maintainers.  Perhaps the same thing happens elsewhere.

Maybe so.

I wrote a script to find which maintainer addresses that
haven't signed or authored a commit in the last 2 years.

I got ~250 entries.  That's about 1/4 of all maintainer
email addresses.

Maybe a dozen of these are false positives though because
some maintainers prefer to receive email at one address
but ack from another.

These are all lowercase for better matching.

abhijit mahajan <abhijit.mahajan@avagotech.com>
achim leubner <achim_leubner@adaptec.com>
adam radford <linuxraid@lsi.com>
al cho <acho@novell.com>
ali akcaagac <aliakc@web.de>
allan stephens <allan.stephens@windriver.com>
aloisio almeida jr <aloisio.almeida@openbossa.org>
andreas dilger <adilger.kernel@dilger.ca>
andreas koensgen <ajk@comnets.uni-bremen.de>
andrew hendry <andrew.hendry@gmail.com>
andrew veliath <andrewtv@usa.net>
andries brouwer <aeb@cwi.nl>
andy henroid <andrew.d.henroid@intel.com>
anil ravindranath <anil_ravindranath@pmc-sierra.com>
anil s keshavamurthy <anil.s.keshavamurthy@intel.com>
antoine jacquet <royale@zerezo.com>
antonino daplas <adaplas@gmail.com>
armin schindler <mac@melware.de>
artur paszkiewicz <artur.paszkiewicz@intel.com>
"arvind r." <arvino55@gmail.com>
ashley lai <ashley@ashleylai.com>
aurelien jacquiot <a-jacquiot@ti.com>
balbir singh <bsingharora@gmail.com>
bill metzenthen <billm@melbpc.org.au>
bradley grove <linuxdrivers@attotech.com>
brett rudley <brudley@broadcom.com>
brian johnson <brijohn@gmail.com>
bruce chang <brucechang@via.com.tw>
bryan huntsman <bryanh@codeaurora.org>
ceph-devel@vger.kernel.org
cesar miquel <miquel@df.uba.ar>
cezary jackiewicz <cezary.jackiewicz@gmail.com>
chaoming li <chaoming_li@realsil.com.cn>
chen liqin <liqin.linux@gmail.com>
chien yen <chien.yen@oracle.com>
chirag kantharia <chirag.kantharia@hp.com>
chris brannon <chris@the-brannons.com>
christine caulfield <ccaulfie@redhat.com>
christopher harrer <charrer@alacritech.com>
"christopher li" <sparse@chrisli.org>
christoph raisch <raisch@de.ibm.com>
cliff whickman <cpw@sgi.com>
colin leroy <colin@colino.net>
corentin labbe <clabbe.montjoie@gmail.com>
daniel drake <dsd@gentoo.org>
daniele venzano <venza@brownhat.org>
daniel oliveira nascimento <don@syst.com.br>
daniel ribeiro <drwyrm@gmail.com>
dario ballabio <ballabio_dario@emc.com>
david kershner <david.kershner@unisys.com>
david rowe <david@rowetel.com>
david safford <safford@us.ibm.com>
david täht <d@teklibre.com>
deepak saxena <dsaxena@plexity.net>
dirk eibach <eibach@gdsys.de>
dirk opfer <dirk@opfer-online.de>
dmitry tarnyagin <dmitry.tarnyagin@lockless.no>
dominik brodowski <linux@dominikbrodowski.net>
doug gilbert <dgilbert@interlog.com>
doug thompson <dougthompson@xmission.com>
doug warzecha <douglas_warzecha@dell.com>
duncan sands <duncan.sands@free.fr>
"ed l. cashin" <ecashin@coraid.com>
egor martovetsky <egor@pasemi.com>
eric biederman <ebiederm@xmission.com>
eric piel <eric.piel@tremplin-utc.net>
erik andren <erik.andren@gmail.com>
faisal latif <faisal.latif@intel.com>
ferenc bakonyi <fero@drama.obuda.kando.hu>
florian schilhabel <florian.c.schilhabel@googlemail.com>.
forest bond <forest@alittletooquiet.net>
frank seidel <frank@f-seidel.de>
fujita tomonori <fujita.tomonori@lab.ntt.co.jp>
gary zambrano <zambrano@broadcom.com>
gilles muller <gilles.muller@lip6.fr>
gleb natapov <gleb@kernel.org>
guillaume ligneul <guillaume.ligneul@gmail.com>
guo-fu tseng <cooldavid@cooldavid.org>
gustavo padovan <gustavo@padovan.org>
haavard skinnemoen <hskinnemoen@gmail.com>
hal rosenstock <hal.rosenstock@gmail.com>
hansjoerg lipp <hjlipp@web.de>
hans ulli kroll <ulli.kroll@googlemail.com>
harald welte <haraldwelte@viatech.com>
harald welte <laforge@openezx.org>
henk vergonet <henk.vergonet@gmail.com>
henrique de moraes holschuh <ibm-acpi@hmh.eng.br>
herton ronaldo krzesinski <herton@canonical.com>
heungjun kim <riverful.kim@samsung.com>
hideaki yoshifuji <yoshfuji@linux-ipv6.org>
hoang-nam nguyen <hnguyen@de.ibm.com>
hubert feurstein <hubert.feurstein@contec.at>
"hung hing lun, mike" <hlhung3i@gmail.com>
ian campbell <ijc+devicetree@hellion.org.uk>
ian molton <spyro@f2s.com>
ion badulescu <ionut@badula.org>
ishizaki kou <kou.ishizaki@toshiba.co.jp>
ivan kokshaysky <ink@jurassic.park.msu.ru>
jakub schmidtke <sjakub@gmail.com>
"james e.j. bottomley" <james.bottomley@hansenpartnership.com>
"james e.j. bottomley" <jbottomley@parallels.com>
"james e.j. bottomley" <jejb@parisc-linux.org>
james morris <jmorris@namei.org>
"james r. van zandt" <jrv@vanzandt.mv.com>
jamie lenehan <lenehan@twibble.org>
jan-benedict glaw <jbglaw@lug-owl.de>
jan dumon <j.dumon@option.com>
jan harkes <jaharkes@cs.cmu.edu>
jan-simon moeller <jansimon.moeller@gmx.de>
jarkko lavinen <jarkko.lavinen@nokia.com>
jarod wilson <jarod@wilsonet.com>
jason uhlenkott <juhlenko@akamai.com>
jaya kumar <jayakumar.alsa@gmail.com>
jaya kumar <jayalk@intworks.biz>
jay cliburn <jcliburn@gmail.com>
jean-paul roubelat <jpr@f6fbb.org>
jeff dike <jdike@addtoit.com>
jens osterkamp <jens@de.ibm.com>
jes sorensen <jes@trained-monkey.org>
jim paris <jim@jtan.com>
jiri kosina <trivial@kernel.org>
jiri slaby <jirislaby@gmail.com>
joerg reuter <jreuter@yaina.de>
joern engel <joern@lazybastard.org>
johan hedberg <johan.hedberg@gmail.com>
john mccutchan <john@johnmccutchan.com>
john ronciak <john.ronciak@intel.com>
"john w. linville" <linville@tuxdriver.com>
jonathan buzzard <jonathan@buzzard.org.uk>
"jon d. mason" <jdmason@kudzu.us>
jon nettleton <jon.nettleton@gmail.com>
josh triplett <josh@freedesktop.org>
joshua morris <josh.h.morris@us.ibm.com>
joshua thompson <funaho@jurai.org>
juanjo ciarlante <jjciarla@raiz.uncu.edu.ar>
"juergen e. fischer" <fischer@norbit.de>
juergen stuber <starblue@users.sourceforge.net>
juerg haefliger <juergh@gmail.com>
kentaro takeda <takedakn@nttdata.co.jp>
kevin curtis <kevin.curtis@farsite.co.uk>
kevin hilman <khilman@kernel.org>
kirk reiser <kirk@reisers.ca>
koichi yasutake <yasutake.koichi@jp.panasonic.com>
kristoffer glembo <kristoffer@gaisler.com>
kurt garloff <garloff@suse.de>
leandro costantino <lcostantino@gmail.com>
len brown <lenb@kernel.org>
lennert buytenhek <kernel@wantstofly.org>
liam girdwood <lgirdwood@gmail.com>
liam girdwood <lrg@slimlogic.co.uk>
lior dotan <liodot@gmail.com>
"luis r. rodriguez" <mcgrof@gmail.com>
maik broemme <mbroemme@plusserver.de>
manohar vanga <manohar.vanga@gmail.com>
marcelo henrique cerri <mhcerri@linux.vnet.ibm.com>
marc hulsman <m.hulsman@tudelft.nl>
"mark f. brown" <mark.brown314@gmail.com>
martin mares <mj@ucw.cz>
matthew wilcox <matthew@wil.cx>
matt mackall <mpm@selenic.com>
maxim krasnyansky <maxk@qti.qualcomm.com>
michael hanselmann <linux-kernel@hansmi.ch>
michael petchkovsky <mkpetch@internode.on.net>
michael reed <mdr@sgi.com>
michal januszewski <spock@gentoo.org>
michal ostrowski <mostrows@earthlink.net>
miguel ojeda sandonis <miguel.ojeda.sandonis@gmail.com>
mikael starvik <starvik@axis.com>
mike marciniszyn <infinipath@intel.com>
mike rapoport <mike@compulab.co.il>
miloslav trmac <mitr@volny.cz>
miroslav zagorac <zaga@fly.cc.fer.hr>
mitchell blank jr <mitch@sfgoth.com>
muli ben-yehuda <muli@il.ibm.com>
nagalakshmi nandigama <nagalakshmi.nandigama@avagotech.com>
neela syam kolli <megaraidlinux@lsi.com>
nelson castillo <arhuaco@freaks-unidos.net>
nicholas a. bellinger <nab@linux-iscsi.org>
nick piggin <npiggin@kernel.dk>
olav kongas <ok@artecdesign.ee>
olivier lorin <o.lorin@laposte.net>
pantelis antoniou <pantelis.antoniou@gmail.com>
pat gefre <pfg@sgi.com>
paul mackerras <paulus@au.ibm.com>
pau oliva fora <pof@eslack.org>
peter chubb <pleb@gelato.unsw.edu.au>
peter feuerer <peter@piie.net>
philip blundell <philb@gnu.org>
philip kelleher <pjk1939@linux.vnet.ibm.com>
philip moltmann <moltmann@vmware.com>
praveen krishnamoorthy <praveen.krishnamoorthy@avagotech.com>
rakesh iyer <riyer@nvidia.com>
ranganathan desikan <ravi@jetztechnologies.com>
remi denis-courmont <courmisch@gmail.com>
riccardo facchetti <fizban@tin.it>
richard l maliszewski <richard.l.maliszewski@intel.com>
"richard russon (flatcap)" <ldm@flatcap.org>
rik faith <faith@cs.unc.edu>
robert gerlach <khnz@gmx.de>
robert love <rlove@rlove.org>
robert love <rml@tech9.net>
rob herring <robh+dt@kernel.org>
roland dreier <roland@kernel.org>
roland mcgrath <roland@redhat.com>
ron mercer <ron.mercer@qlogic.com>
ron minnich <rminnich@sandia.gov>
samuel chessman <chessman@tux.org>
santosh raspatur <santosh@chelsio.com>
scott murray <scott@spiteful.org>
sergey lapin <slapin@ossfans.org>
shane wang <shane.wang@intel.com>
simon arlott <cxacru@fire.lp0.eu>
simon kelley <simon@thekelleys.org.uk>
sorbica shieh <sorbica@icplus.com.tw>
srinivas kandagatla <srinivas.kandagatla@gmail.com>
stefan eletzhofer <stefan.eletzhofer@eletztrick.de>
stefano brivio <stefano.brivio@polimi.it>
steve french <sfrench@samba.org>
steve sakoman <sakoman@gmail.com>
steve underwood <steveu@coppice.org>
steve wise <swise@chelsio.com>
subbu seetharaman <subbu.seetharaman@emulex.com>
sudarsana kalluru <sudarsana.kalluru@qlogic.com>
sujith thomas <sujith.thomas@intel.com>
teddy wang <teddy.wang@siliconmotion.com.cn>
terje bergström <tbergstrom@nvidia.com>
thibaut varene <t-bone@parisc-linux.org>
thomas dahlmann <dahlmann.thomas@arcor.de>
thomas winischhofer <thomas@winischhofer.net>
"tigran a. aivazian" <tigran@aivazian.fsnet.co.uk>
tigran aivazian <tigran@aivazian.fsnet.co.uk>
till harbaum <till@harbaum.org>
tim small <tim@buttersideup.com>
tim waugh <tim@cyberelk.net>
tony finch <dot@dotat.at>
tony olech <tony.olech@elandigitalsystems.com>
torben mathiasen <device@lanana.org>
ulrich kunitz <kune@deine-taler.de>
vegard nossum <vegardno@ifi.uio.no>
ville syrjala <syrjala@sci.fi>
vishwas a deshpande <vishwas.a.deshpande@ti.com>
vitaly bordug <vbordug@ru.mvista.com>
vitaly bordug <vitb@kernel.crashing.org>
wensong zhang <wensong@linux-vs.org>
willem riede <osst@riede.org>
willy tarreau <willy@meta-x.org>
"wong hoi sing, edison" <hswong3i@gmail.com>
xavier deguillard <xdeguillard@vmware.com>
yokota hiroshi <yokota@netlab.is.tsukuba.ac.jp>
zhang wei <zw@zh-kernel.org>
zwane mwaikambo <zwanem@gmail.com>

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Paul E. McKenney]

On Tue, Jul 08, 2014 at 09:09:25PM -0700, Joe Perches wrote:
> On Tue, 2014-07-08 at 15:25 -0700, Paul E. McKenney wrote:
> > On Tue, Jul 08, 2014 at 03:05:16PM -0700, Joe Perches wrote:
> []
> > > I still think the concept is pretty useless and it's
> > > just a duplication of "M:", which isn't anything other
> > > than a list of who should be sent patches.
> []
> > It will be interesting to see how things go.
> 
> Yes, it might work out fine and maybe might cause some
> other beneficial changes.
> 
> > There did seem to be
> > some people who were comfortable being listed as RCU reviewers, but
> > not as RCU maintainers.  Perhaps the same thing happens elsewhere.
> 
> Maybe so.
> 
> I wrote a script to find which maintainer addresses that
> haven't signed or authored a commit in the last 2 years.
> 
> I got ~250 entries.  That's about 1/4 of all maintainer
> email addresses.
> 
> Maybe a dozen of these are false positives though because
> some maintainers prefer to receive email at one address
> but ack from another.
> 
> These are all lowercase for better matching.

I am not too surprised.  There is currently no penalty for keeping a
stale maintainer listing, and possibly severe social penalties for
wrongly removing one.  ;-)

							Thanx, Paul

> abhijit mahajan <abhijit.mahajan@avagotech.com>
> achim leubner <achim_leubner@adaptec.com>
> adam radford <linuxraid@lsi.com>
> al cho <acho@novell.com>
> ali akcaagac <aliakc@web.de>
> allan stephens <allan.stephens@windriver.com>
> aloisio almeida jr <aloisio.almeida@openbossa.org>
> andreas dilger <adilger.kernel@dilger.ca>
> andreas koensgen <ajk@comnets.uni-bremen.de>
> andrew hendry <andrew.hendry@gmail.com>
> andrew veliath <andrewtv@usa.net>
> andries brouwer <aeb@cwi.nl>
> andy henroid <andrew.d.henroid@intel.com>
> anil ravindranath <anil_ravindranath@pmc-sierra.com>
> anil s keshavamurthy <anil.s.keshavamurthy@intel.com>
> antoine jacquet <royale@zerezo.com>
> antonino daplas <adaplas@gmail.com>
> armin schindler <mac@melware.de>
> artur paszkiewicz <artur.paszkiewicz@intel.com>
> "arvind r." <arvino55@gmail.com>
> ashley lai <ashley@ashleylai.com>
> aurelien jacquiot <a-jacquiot@ti.com>
> balbir singh <bsingharora@gmail.com>
> bill metzenthen <billm@melbpc.org.au>
> bradley grove <linuxdrivers@attotech.com>
> brett rudley <brudley@broadcom.com>
> brian johnson <brijohn@gmail.com>
> bruce chang <brucechang@via.com.tw>
> bryan huntsman <bryanh@codeaurora.org>
> ceph-devel@vger.kernel.org
> cesar miquel <miquel@df.uba.ar>
> cezary jackiewicz <cezary.jackiewicz@gmail.com>
> chaoming li <chaoming_li@realsil.com.cn>
> chen liqin <liqin.linux@gmail.com>
> chien yen <chien.yen@oracle.com>
> chirag kantharia <chirag.kantharia@hp.com>
> chris brannon <chris@the-brannons.com>
> christine caulfield <ccaulfie@redhat.com>
> christopher harrer <charrer@alacritech.com>
> "christopher li" <sparse@chrisli.org>
> christoph raisch <raisch@de.ibm.com>
> cliff whickman <cpw@sgi.com>
> colin leroy <colin@colino.net>
> corentin labbe <clabbe.montjoie@gmail.com>
> daniel drake <dsd@gentoo.org>
> daniele venzano <venza@brownhat.org>
> daniel oliveira nascimento <don@syst.com.br>
> daniel ribeiro <drwyrm@gmail.com>
> dario ballabio <ballabio_dario@emc.com>
> david kershner <david.kershner@unisys.com>
> david rowe <david@rowetel.com>
> david safford <safford@us.ibm.com>
> david täht <d@teklibre.com>
> deepak saxena <dsaxena@plexity.net>
> dirk eibach <eibach@gdsys.de>
> dirk opfer <dirk@opfer-online.de>
> dmitry tarnyagin <dmitry.tarnyagin@lockless.no>
> dominik brodowski <linux@dominikbrodowski.net>
> doug gilbert <dgilbert@interlog.com>
> doug thompson <dougthompson@xmission.com>
> doug warzecha <douglas_warzecha@dell.com>
> duncan sands <duncan.sands@free.fr>
> "ed l. cashin" <ecashin@coraid.com>
> egor martovetsky <egor@pasemi.com>
> eric biederman <ebiederm@xmission.com>
> eric piel <eric.piel@tremplin-utc.net>
> erik andren <erik.andren@gmail.com>
> faisal latif <faisal.latif@intel.com>
> ferenc bakonyi <fero@drama.obuda.kando.hu>
> florian schilhabel <florian.c.schilhabel@googlemail.com>.
> forest bond <forest@alittletooquiet.net>
> frank seidel <frank@f-seidel.de>
> fujita tomonori <fujita.tomonori@lab.ntt.co.jp>
> gary zambrano <zambrano@broadcom.com>
> gilles muller <gilles.muller@lip6.fr>
> gleb natapov <gleb@kernel.org>
> guillaume ligneul <guillaume.ligneul@gmail.com>
> guo-fu tseng <cooldavid@cooldavid.org>
> gustavo padovan <gustavo@padovan.org>
> haavard skinnemoen <hskinnemoen@gmail.com>
> hal rosenstock <hal.rosenstock@gmail.com>
> hansjoerg lipp <hjlipp@web.de>
> hans ulli kroll <ulli.kroll@googlemail.com>
> harald welte <haraldwelte@viatech.com>
> harald welte <laforge@openezx.org>
> henk vergonet <henk.vergonet@gmail.com>
> henrique de moraes holschuh <ibm-acpi@hmh.eng.br>
> herton ronaldo krzesinski <herton@canonical.com>
> heungjun kim <riverful.kim@samsung.com>
> hideaki yoshifuji <yoshfuji@linux-ipv6.org>
> hoang-nam nguyen <hnguyen@de.ibm.com>
> hubert feurstein <hubert.feurstein@contec.at>
> "hung hing lun, mike" <hlhung3i@gmail.com>
> ian campbell <ijc+devicetree@hellion.org.uk>
> ian molton <spyro@f2s.com>
> ion badulescu <ionut@badula.org>
> ishizaki kou <kou.ishizaki@toshiba.co.jp>
> ivan kokshaysky <ink@jurassic.park.msu.ru>
> jakub schmidtke <sjakub@gmail.com>
> "james e.j. bottomley" <james.bottomley@hansenpartnership.com>
> "james e.j. bottomley" <jbottomley@parallels.com>
> "james e.j. bottomley" <jejb@parisc-linux.org>
> james morris <jmorris@namei.org>
> "james r. van zandt" <jrv@vanzandt.mv.com>
> jamie lenehan <lenehan@twibble.org>
> jan-benedict glaw <jbglaw@lug-owl.de>
> jan dumon <j.dumon@option.com>
> jan harkes <jaharkes@cs.cmu.edu>
> jan-simon moeller <jansimon.moeller@gmx.de>
> jarkko lavinen <jarkko.lavinen@nokia.com>
> jarod wilson <jarod@wilsonet.com>
> jason uhlenkott <juhlenko@akamai.com>
> jaya kumar <jayakumar.alsa@gmail.com>
> jaya kumar <jayalk@intworks.biz>
> jay cliburn <jcliburn@gmail.com>
> jean-paul roubelat <jpr@f6fbb.org>
> jeff dike <jdike@addtoit.com>
> jens osterkamp <jens@de.ibm.com>
> jes sorensen <jes@trained-monkey.org>
> jim paris <jim@jtan.com>
> jiri kosina <trivial@kernel.org>
> jiri slaby <jirislaby@gmail.com>
> joerg reuter <jreuter@yaina.de>
> joern engel <joern@lazybastard.org>
> johan hedberg <johan.hedberg@gmail.com>
> john mccutchan <john@johnmccutchan.com>
> john ronciak <john.ronciak@intel.com>
> "john w. linville" <linville@tuxdriver.com>
> jonathan buzzard <jonathan@buzzard.org.uk>
> "jon d. mason" <jdmason@kudzu.us>
> jon nettleton <jon.nettleton@gmail.com>
> josh triplett <josh@freedesktop.org>
> joshua morris <josh.h.morris@us.ibm.com>
> joshua thompson <funaho@jurai.org>
> juanjo ciarlante <jjciarla@raiz.uncu.edu.ar>
> "juergen e. fischer" <fischer@norbit.de>
> juergen stuber <starblue@users.sourceforge.net>
> juerg haefliger <juergh@gmail.com>
> kentaro takeda <takedakn@nttdata.co.jp>
> kevin curtis <kevin.curtis@farsite.co.uk>
> kevin hilman <khilman@kernel.org>
> kirk reiser <kirk@reisers.ca>
> koichi yasutake <yasutake.koichi@jp.panasonic.com>
> kristoffer glembo <kristoffer@gaisler.com>
> kurt garloff <garloff@suse.de>
> leandro costantino <lcostantino@gmail.com>
> len brown <lenb@kernel.org>
> lennert buytenhek <kernel@wantstofly.org>
> liam girdwood <lgirdwood@gmail.com>
> liam girdwood <lrg@slimlogic.co.uk>
> lior dotan <liodot@gmail.com>
> "luis r. rodriguez" <mcgrof@gmail.com>
> maik broemme <mbroemme@plusserver.de>
> manohar vanga <manohar.vanga@gmail.com>
> marcelo henrique cerri <mhcerri@linux.vnet.ibm.com>
> marc hulsman <m.hulsman@tudelft.nl>
> "mark f. brown" <mark.brown314@gmail.com>
> martin mares <mj@ucw.cz>
> matthew wilcox <matthew@wil.cx>
> matt mackall <mpm@selenic.com>
> maxim krasnyansky <maxk@qti.qualcomm.com>
> michael hanselmann <linux-kernel@hansmi.ch>
> michael petchkovsky <mkpetch@internode.on.net>
> michael reed <mdr@sgi.com>
> michal januszewski <spock@gentoo.org>
> michal ostrowski <mostrows@earthlink.net>
> miguel ojeda sandonis <miguel.ojeda.sandonis@gmail.com>
> mikael starvik <starvik@axis.com>
> mike marciniszyn <infinipath@intel.com>
> mike rapoport <mike@compulab.co.il>
> miloslav trmac <mitr@volny.cz>
> miroslav zagorac <zaga@fly.cc.fer.hr>
> mitchell blank jr <mitch@sfgoth.com>
> muli ben-yehuda <muli@il.ibm.com>
> nagalakshmi nandigama <nagalakshmi.nandigama@avagotech.com>
> neela syam kolli <megaraidlinux@lsi.com>
> nelson castillo <arhuaco@freaks-unidos.net>
> nicholas a. bellinger <nab@linux-iscsi.org>
> nick piggin <npiggin@kernel.dk>
> olav kongas <ok@artecdesign.ee>
> olivier lorin <o.lorin@laposte.net>
> pantelis antoniou <pantelis.antoniou@gmail.com>
> pat gefre <pfg@sgi.com>
> paul mackerras <paulus@au.ibm.com>
> pau oliva fora <pof@eslack.org>
> peter chubb <pleb@gelato.unsw.edu.au>
> peter feuerer <peter@piie.net>
> philip blundell <philb@gnu.org>
> philip kelleher <pjk1939@linux.vnet.ibm.com>
> philip moltmann <moltmann@vmware.com>
> praveen krishnamoorthy <praveen.krishnamoorthy@avagotech.com>
> rakesh iyer <riyer@nvidia.com>
> ranganathan desikan <ravi@jetztechnologies.com>
> remi denis-courmont <courmisch@gmail.com>
> riccardo facchetti <fizban@tin.it>
> richard l maliszewski <richard.l.maliszewski@intel.com>
> "richard russon (flatcap)" <ldm@flatcap.org>
> rik faith <faith@cs.unc.edu>
> robert gerlach <khnz@gmx.de>
> robert love <rlove@rlove.org>
> robert love <rml@tech9.net>
> rob herring <robh+dt@kernel.org>
> roland dreier <roland@kernel.org>
> roland mcgrath <roland@redhat.com>
> ron mercer <ron.mercer@qlogic.com>
> ron minnich <rminnich@sandia.gov>
> samuel chessman <chessman@tux.org>
> santosh raspatur <santosh@chelsio.com>
> scott murray <scott@spiteful.org>
> sergey lapin <slapin@ossfans.org>
> shane wang <shane.wang@intel.com>
> simon arlott <cxacru@fire.lp0.eu>
> simon kelley <simon@thekelleys.org.uk>
> sorbica shieh <sorbica@icplus.com.tw>
> srinivas kandagatla <srinivas.kandagatla@gmail.com>
> stefan eletzhofer <stefan.eletzhofer@eletztrick.de>
> stefano brivio <stefano.brivio@polimi.it>
> steve french <sfrench@samba.org>
> steve sakoman <sakoman@gmail.com>
> steve underwood <steveu@coppice.org>
> steve wise <swise@chelsio.com>
> subbu seetharaman <subbu.seetharaman@emulex.com>
> sudarsana kalluru <sudarsana.kalluru@qlogic.com>
> sujith thomas <sujith.thomas@intel.com>
> teddy wang <teddy.wang@siliconmotion.com.cn>
> terje bergström <tbergstrom@nvidia.com>
> thibaut varene <t-bone@parisc-linux.org>
> thomas dahlmann <dahlmann.thomas@arcor.de>
> thomas winischhofer <thomas@winischhofer.net>
> "tigran a. aivazian" <tigran@aivazian.fsnet.co.uk>
> tigran aivazian <tigran@aivazian.fsnet.co.uk>
> till harbaum <till@harbaum.org>
> tim small <tim@buttersideup.com>
> tim waugh <tim@cyberelk.net>
> tony finch <dot@dotat.at>
> tony olech <tony.olech@elandigitalsystems.com>
> torben mathiasen <device@lanana.org>
> ulrich kunitz <kune@deine-taler.de>
> vegard nossum <vegardno@ifi.uio.no>
> ville syrjala <syrjala@sci.fi>
> vishwas a deshpande <vishwas.a.deshpande@ti.com>
> vitaly bordug <vbordug@ru.mvista.com>
> vitaly bordug <vitb@kernel.crashing.org>
> wensong zhang <wensong@linux-vs.org>
> willem riede <osst@riede.org>
> willy tarreau <willy@meta-x.org>
> "wong hoi sing, edison" <hswong3i@gmail.com>
> xavier deguillard <xdeguillard@vmware.com>
> yokota hiroshi <yokota@netlab.is.tsukuba.ac.jp>
> zhang wei <zw@zh-kernel.org>
> zwane mwaikambo <zwanem@gmail.com>
> 
> 

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Joe Perches]

On Wed, 2014-07-09 at 04:14 -0700, Paul E. McKenney wrote:
> On Tue, Jul 08, 2014 at 09:09:25PM -0700, Joe Perches wrote:
[]
> > I wrote a script to find which maintainer addresses that
> > haven't signed or authored a commit in the last 2 years.
> > 
> > I got ~250 entries.  That's about 1/4 of all maintainer
> > email addresses.
> > 
> > Maybe a dozen of these are false positives though because
> > some maintainers prefer to receive email at one address
> > but ack from another.
> > 
> > These are all lowercase for better matching.
> 
> I am not too surprised.  There is currently no penalty for keeping a
> stale maintainer listing, and possibly severe social penalties for
> wrongly removing one.  ;-)

I guess I'm not that social.

Most of these were the initial code submitters that
haven't had to change anything in quite awhile.

My suggestion is to change the below "M:" entries to "R:"
sometime after "R:" is upstream.

Maybe also change the sections that have only these names to
"S:	Odd Fixes" or "S:	Orphan" or add a new
"S:	Unmaintained but Reviewed" section type.

This might encourage people to take over maintainership of
old code, help people figure out who better to send a patch
to and maybe who's more likely to apply it.

Also reviewer ack's aren't an indication that an upstream
maintainer will pick it up and push it forward.

To Peter's point:

On Tue, 2014-07-08 at 10:09 +0200, Peter Zijlstra wrote:
> So I would like to be able to write:
> 
> SCHEDULER:
> ...
> R:   Steven Rostedt <rostedt@goodmis.org> (kernel/sched/rt.c)
> R:   Juri Lelli <jundri.lelli@gmail.com>    (kernel/sched/deadline.c)

Maybe a better syntax might be something like:
R:	Steven Rostedt
	F:	kernel/sched/rt.c

where optional F:/X: lines override the default
assumption of all F:/X: from the section.

Anyone have any comment on these inactive maintainers
that should be converted to "R:" reviewers?

> > abhijit mahajan <abhijit.mahajan@avagotech.com>
> > achim leubner <achim_leubner@adaptec.com>
> > adam radford <linuxraid@lsi.com>
> > al cho <acho@novell.com>
> > ali akcaagac <aliakc@web.de>
> > allan stephens <allan.stephens@windriver.com>
> > aloisio almeida jr <aloisio.almeida@openbossa.org>
> > andreas dilger <adilger.kernel@dilger.ca>
> > andreas koensgen <ajk@comnets.uni-bremen.de>
> > andrew hendry <andrew.hendry@gmail.com>onl
> > andrew veliath <andrewtv@usa.net>
> > andries brouwer <aeb@cwi.nl>
> > andy henroid <andrew.d.henroid@intel.com>
> > anil ravindranath <anil_ravindranath@pmc-sierra.com>
> > anil s keshavamurthy <anil.s.keshavamurthy@intel.com>
> > antoine jacquet <royale@zerezo.com>
> > antonino daplas <adaplas@gmail.com>
> > armin schindler <mac@melware.de>
> > artur paszkiewicz <artur.paszkiewicz@intel.com>
> > "arvind r." <arvino55@gmail.com>
> > ashley lai <ashley@ashleylai.com>
> > aurelien jacquiot <a-jacquiot@ti.com>
> > balbir singh <bsingharora@gmail.com>
> > bill metzenthen <billm@melbpc.org.au>
> > bradley grove <linuxdrivers@attotech.com>
> > brett rudley <brudley@broadcom.com>
> > brian johnson <brijohn@gmail.com>
> > bruce chang <brucechang@via.com.tw>
> > bryan huntsman <bryanh@codeaurora.org>
> > ceph-devel@vger.kernel.org
> > cesar miquel <miquel@df.uba.ar>
> > cezary jackiewicz <cezary.jackiewicz@gmail.com>
> > chaoming li <chaoming_li@realsil.com.cn>
> > chen liqin <liqin.linux@gmail.com>
> > chien yen <chien.yen@oracle.com>
> > chirag kantharia <chirag.kantharia@hp.com>
> > chris brannon <chris@the-brannons.com>
> > christine caulfield <ccaulfie@redhat.com>
> > christopher harrer <charrer@alacritech.com>
> > "christopher li" <sparse@chrisli.org>
> > christoph raisch <raisch@de.ibm.com>
> > cliff whickman <cpw@sgi.com>
> > colin leroy <colin@colino.net>
> > corentin labbe <clabbe.montjoie@gmail.com>
> > daniel drake <dsd@gentoo.org>
> > daniele venzano <venza@brownhat.org>
> > daniel oliveira nascimento <don@syst.com.br>
> > daniel ribeiro <drwyrm@gmail.com>
> > dario ballabio <ballabio_dario@emc.com>
> > david kershner <david.kershner@unisys.com>
> > david rowe <david@rowetel.com>
> > david safford <safford@us.ibm.com>
> > david täht <d@teklibre.com>
> > deepak saxena <dsaxena@plexity.net>
> > dirk eibach <eibach@gdsys.de>
> > dirk opfer <dirk@opfer-online.de>
> > dmitry tarnyagin <dmitry.tarnyagin@lockless.no>
> > dominik brodowski <linux@dominikbrodowski.net>
> > doug gilbert <dgilbert@interlog.com>
> > doug thompson <dougthompson@xmission.com>
> > doug warzecha <douglas_warzecha@dell.com>
> > duncan sands <duncan.sands@free.fr>
> > "ed l. cashin" <ecashin@coraid.com>
> > egor martovetsky <egor@pasemi.com>
> > eric biederman <ebiederm@xmission.com>
> > eric piel <eric.piel@tremplin-utc.net>
> > erik andren <erik.andren@gmail.com>
> > faisal latif <faisal.latif@intel.com>
> > ferenc bakonyi <fero@drama.obuda.kando.hu>
> > florian schilhabel <florian.c.schilhabel@googlemail.com>.
> > forest bond <forest@alittletooquiet.net>
> > frank seidel <frank@f-seidel.de>
> > fujita tomonori <fujita.tomonori@lab.ntt.co.jp>
> > gary zambrano <zambrano@broadcom.com>
> > gilles muller <gilles.muller@lip6.fr>
> > gleb natapov <gleb@kernel.org>
> > guillaume ligneul <guillaume.ligneul@gmail.com>
> > guo-fu tseng <cooldavid@cooldavid.org>
> > gustavo padovan <gustavo@padovan.org>
> > haavard skinnemoen <hskinnemoen@gmail.com>
> > hal rosenstock <hal.rosenstock@gmail.com>
> > hansjoerg lipp <hjlipp@web.de>
> > hans ulli kroll <ulli.kroll@googlemail.com>
> > harald welte <haraldwelte@viatech.com>
> > harald welte <laforge@openezx.org>
> > henk vergonet <henk.vergonet@gmail.com>
> > henrique de moraes holschuh <ibm-acpi@hmh.eng.br>
> > herton ronaldo krzesinski <herton@canonical.com>
> > heungjun kim <riverful.kim@samsung.com>
> > hideaki yoshifuji <yoshfuji@linux-ipv6.org>
> > hoang-nam nguyen <hnguyen@de.ibm.com>
> > hubert feurstein <hubert.feurstein@contec.at>
> > "hung hing lun, mike" <hlhung3i@gmail.com>
> > ian campbell <ijc+devicetree@hellion.org.uk>
> > ian molton <spyro@f2s.com>
> > ion badulescu <ionut@badula.org>
> > ishizaki kou <kou.ishizaki@toshiba.co.jp>
> > ivan kokshaysky <ink@jurassic.park.msu.ru>
> > jakub schmidtke <sjakub@gmail.com>
> > "james e.j. bottomley" <james.bottomley@hansenpartnership.com>
> > "james e.j. bottomley" <jbottomley@parallels.com>
> > "james e.j. bottomley" <jejb@parisc-linux.org>
> > james morris <jmorris@namei.org>
> > "james r. van zandt" <jrv@vanzandt.mv.com>
> > jamie lenehan <lenehan@twibble.org>
> > jan-benedict glaw <jbglaw@lug-owl.de>
> > jan dumon <j.dumon@option.com>
> > jan harkes <jaharkes@cs.cmu.edu>
> > jan-simon moeller <jansimon.moeller@gmx.de>
> > jarkko lavinen <jarkko.lavinen@nokia.com>
> > jarod wilson <jarod@wilsonet.com>
> > jason uhlenkott <juhlenko@akamai.com>
> > jaya kumar <jayakumar.alsa@gmail.com>
> > jaya kumar <jayalk@intworks.biz>
> > jay cliburn <jcliburn@gmail.com>
> > jean-paul roubelat <jpr@f6fbb.org>
> > jeff dike <jdike@addtoit.com>
> > jens osterkamp <jens@de.ibm.com>
> > jes sorensen <jes@trained-monkey.org>
> > jim paris <jim@jtan.com>
> > jiri kosina <trivial@kernel.org>
> > jiri slaby <jirislaby@gmail.com>
> > joerg reuter <jreuter@yaina.de>
> > joern engel <joern@lazybastard.org>
> > johan hedberg <johan.hedberg@gmail.com>
> > john mccutchan <john@johnmccutchan.com>
> > john ronciak <john.ronciak@intel.com>
> > "john w. linville" <linville@tuxdriver.com>
> > jonathan buzzard <jonathan@buzzard.org.uk>
> > "jon d. mason" <jdmason@kudzu.us>
> > jon nettleton <jon.nettleton@gmail.com>
> > josh triplett <josh@freedesktop.org>
> > joshua morris <josh.h.morris@us.ibm.com>
> > joshua thompson <funaho@jurai.org>
> > juanjo ciarlante <jjciarla@raiz.uncu.edu.ar>
> > "juergen e. fischer" <fischer@norbit.de>
> > juergen stuber <starblue@users.sourceforge.net>
> > juerg haefliger <juergh@gmail.com>
> > kentaro takeda <takedakn@nttdata.co.jp>
> > kevin curtis <kevin.curtis@farsite.co.uk>
> > kevin hilman <khilman@kernel.org>
> > kirk reiser <kirk@reisers.ca>
> > koichi yasutake <yasutake.koichi@jp.panasonic.com>
> > kristoffer glembo <kristoffer@gaisler.com>
> > kurt garloff <garloff@suse.de>
> > leandro costantino <lcostantino@gmail.com>
> > len brown <lenb@kernel.org>
> > lennert buytenhek <kernel@wantstofly.org>
> > liam girdwood <lgirdwood@gmail.com>
> > liam girdwood <lrg@slimlogic.co.uk>
> > lior dotan <liodot@gmail.com>
> > "luis r. rodriguez" <mcgrof@gmail.com>
> > maik broemme <mbroemme@plusserver.de>
> > manohar vanga <manohar.vanga@gmail.com>
> > marcelo henrique cerri <mhcerri@linux.vnet.ibm.com>
> > marc hulsman <m.hulsman@tudelft.nl>
> > "mark f. brown" <mark.brown314@gmail.com>
> > martin mares <mj@ucw.cz>
> > matthew wilcox <matthew@wil.cx>
> > matt mackall <mpm@selenic.com>
> > maxim krasnyansky <maxk@qti.qualcomm.com>
> > michael hanselmann <linux-kernel@hansmi.ch>
> > michael petchkovsky <mkpetch@internode.on.net>
> > michael reed <mdr@sgi.com>
> > michal januszewski <spock@gentoo.org>
> > michal ostrowski <mostrows@earthlink.net>
> > miguel ojeda sandonis <miguel.ojeda.sandonis@gmail.com>
> > mikael starvik <starvik@axis.com>
> > mike marciniszyn <infinipath@intel.com>
> > mike rapoport <mike@compulab.co.il>
> > miloslav trmac <mitr@volny.cz>
> > miroslav zagorac <zaga@fly.cc.fer.hr>
> > mitchell blank jr <mitch@sfgoth.com>
> > muli ben-yehuda <muli@il.ibm.com>
> > nagalakshmi nandigama <nagalakshmi.nandigama@avagotech.com>
> > neela syam kolli <megaraidlinux@lsi.com>
> > nelson castillo <arhuaco@freaks-unidos.net>
> > nicholas a. bellinger <nab@linux-iscsi.org>
> > nick piggin <npiggin@kernel.dk>
> > olav kongas <ok@artecdesign.ee>
> > olivier lorin <o.lorin@laposte.net>
> > pantelis antoniou <pantelis.antoniou@gmail.com>
> > pat gefre <pfg@sgi.com>
> > paul mackerras <paulus@au.ibm.com>
> > pau oliva fora <pof@eslack.org>
> > peter chubb <pleb@gelato.unsw.edu.au>
> > peter feuerer <peter@piie.net>
> > philip blundell <philb@gnu.org>
> > philip kelleher <pjk1939@linux.vnet.ibm.com>
> > philip moltmann <moltmann@vmware.com>
> > praveen krishnamoorthy <praveen.krishnamoorthy@avagotech.com>
> > rakesh iyer <riyer@nvidia.com>
> > ranganathan desikan <ravi@jetztechnologies.com>
> > remi denis-courmont <courmisch@gmail.com>
> > riccardo facchetti <fizban@tin.it>
> > richard l maliszewski <richard.l.maliszewski@intel.com>
> > "richard russon (flatcap)" <ldm@flatcap.org>
> > rik faith <faith@cs.unc.edu>
> > robert gerlach <khnz@gmx.de>
> > robert love <rlove@rlove.org>
> > robert love <rml@tech9.net>
> > rob herring <robh+dt@kernel.org>
> > roland dreier <roland@kernel.org>
> > roland mcgrath <roland@redhat.com>
> > ron mercer <ron.mercer@qlogic.com>
> > ron minnich <rminnich@sandia.gov>
> > samuel chessman <chessman@tux.org>
> > santosh raspatur <santosh@chelsio.com>
> > scott murray <scott@spiteful.org>
> > sergey lapin <slapin@ossfans.org>
> > shane wang <shane.wang@intel.com>
> > simon arlott <cxacru@fire.lp0.eu>
> > simon kelley <simon@thekelleys.org.uk>
> > sorbica shieh <sorbica@icplus.com.tw>
> > srinivas kandagatla <srinivas.kandagatla@gmail.com>
> > stefan eletzhofer <stefan.eletzhofer@eletztrick.de>
> > stefano brivio <stefano.brivio@polimi.it>
> > steve french <sfrench@samba.org>
> > steve sakoman <sakoman@gmail.com>
> > steve underwood <steveu@coppice.org>
> > steve wise <swise@chelsio.com>
> > subbu seetharaman <subbu.seetharaman@emulex.com>
> > sudarsana kalluru <sudarsana.kalluru@qlogic.com>
> > sujith thomas <sujith.thomas@intel.com>
> > teddy wang <teddy.wang@siliconmotion.com.cn>
> > terje bergström <tbergstrom@nvidia.com>
> > thibaut varene <t-bone@parisc-linux.org>
> > thomas dahlmann <dahlmann.thomas@arcor.de>
> > thomas winischhofer <thomas@winischhofer.net>
> > "tigran a. aivazian" <tigran@aivazian.fsnet.co.uk>
> > tigran aivazian <tigran@aivazian.fsnet.co.uk>
> > till harbaum <till@harbaum.org>
> > tim small <tim@buttersideup.com>
> > tim waugh <tim@cyberelk.net>
> > tony finch <dot@dotat.at>
> > tony olech <tony.olech@elandigitalsystems.com>
> > torben mathiasen <device@lanana.org>
> > ulrich kunitz <kune@deine-taler.de>
> > vegard nossum <vegardno@ifi.uio.no>
> > ville syrjala <syrjala@sci.fi>
> > vishwas a deshpande <vishwas.a.deshpande@ti.com>
> > vitaly bordug <vbordug@ru.mvista.com>
> > vitaly bordug <vitb@kernel.crashing.org>
> > wensong zhang <wensong@linux-vs.org>
> > willem riede <osst@riede.org>
> > willy tarreau <willy@meta-x.org>
> > "wong hoi sing, edison" <hswong3i@gmail.com>
> > xavier deguillard <xdeguillard@vmware.com>
> > yokota hiroshi <yokota@netlab.is.tsukuba.ac.jp>
> > zhang wei <zw@zh-kernel.org>
> > zwane mwaikambo <zwanem@gmail.com>

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Peter Zijlstra]

On Wed, Jul 09, 2014 at 06:23:22AM -0700, Joe Perches wrote:

> > SCHEDULER:
> > ...
> > R:   Steven Rostedt <rostedt@goodmis.org> (kernel/sched/rt.c)
> > R:   Juri Lelli <jundri.lelli@gmail.com>    (kernel/sched/deadline.c)
> 
> Maybe a better syntax might be something like:
> R:	Steven Rostedt
> 	F:	kernel/sched/rt.c
> 
> where optional F:/X: lines override the default
> assumption of all F:/X: from the section.

Would RF: make sense? Instead of the indenting.

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Joe Perches]

On Thu, 2014-07-10 at 11:39 +0200, Peter Zijlstra wrote:
> On Wed, Jul 09, 2014 at 06:23:22AM -0700, Joe Perches wrote:
> > > SCHEDULER:
> > > ...
> > > R:   Steven Rostedt <rostedt@goodmis.org> (kernel/sched/rt.c)
> > > R:   Juri Lelli <jundri.lelli@gmail.com>    (kernel/sched/deadline.c)
> > 
> > Maybe a better syntax might be something like:
> > R:	Steven Rostedt
> > 	F:	kernel/sched/rt.c
> > 
> > where optional F:/X: lines override the default
> > assumption of all F:/X: from the section.
> 
> Would RF: make sense? Instead of the indenting.

Maybe.

As a preface:

I doubt the need for associating a subset of the files
patterns for a subsystem with a particular reviewer.

If a reviewer is interested enough in a subsystem to
volunteer to read patches then that reviewer likely won't
be overburdened by getting a few more emailed patches
that may be outside a scope of interest.

To answer the question:

If it's done, there needs to be some mechanism to
associate a particular reviewer and the files.

Ordering of "letter:" types in MAINTAINERS is currently
unstructured.

Given a desire to have more than one reviewer interested
in different files, any subset use will either impose some
ordering requirement on the file list or some association
with the reviewer and the reviewer's file patterns of interest.

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[josh]

On Thu, Jul 10, 2014 at 03:00:12AM -0700, Joe Perches wrote:
> On Thu, 2014-07-10 at 11:39 +0200, Peter Zijlstra wrote:
> > On Wed, Jul 09, 2014 at 06:23:22AM -0700, Joe Perches wrote:
> > > > SCHEDULER:
> > > > ...
> > > > R:   Steven Rostedt <rostedt@goodmis.org> (kernel/sched/rt.c)
> > > > R:   Juri Lelli <jundri.lelli@gmail.com>    (kernel/sched/deadline.c)
> > > 
> > > Maybe a better syntax might be something like:
> > > R:	Steven Rostedt
> > > 	F:	kernel/sched/rt.c
> > > 
> > > where optional F:/X: lines override the default
> > > assumption of all F:/X: from the section.
> > 
> > Would RF: make sense? Instead of the indenting.
> 
> Maybe.
> 
> As a preface:
> 
> I doubt the need for associating a subset of the files
> patterns for a subsystem with a particular reviewer.
> 
> If a reviewer is interested enough in a subsystem to
> volunteer to read patches then that reviewer likely won't
> be overburdened by getting a few more emailed patches
> that may be outside a scope of interest.

I agree.  And if a subset of files needs a separate set of maintainers
or reviewers, it doesn't seem excessive to split it into a separate
MAINTAINERS entry.  For instance, if you want kernel/sched/rt.c to have
an additional set of maintainers/reviewers, just add a MAINTAINERS entry
for "SCHEDULER - REALTIME" with an appropriate "F:" line.

- Josh Triplett

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Peter Zijlstra]

On Thu, Jul 10, 2014 at 04:16:23PM -0700, josh@joshtriplett.org wrote:
> On Thu, Jul 10, 2014 at 03:00:12AM -0700, Joe Perches wrote:
> > On Thu, 2014-07-10 at 11:39 +0200, Peter Zijlstra wrote:
> > > On Wed, Jul 09, 2014 at 06:23:22AM -0700, Joe Perches wrote:
> > > > > SCHEDULER:
> > > > > ...
> > > > > R:   Steven Rostedt <rostedt@goodmis.org> (kernel/sched/rt.c)
> > > > > R:   Juri Lelli <jundri.lelli@gmail.com>    (kernel/sched/deadline.c)
> > > > 
> > > > Maybe a better syntax might be something like:
> > > > R:	Steven Rostedt
> > > > 	F:	kernel/sched/rt.c
> > > > 
> > > > where optional F:/X: lines override the default
> > > > assumption of all F:/X: from the section.
> > > 
> > > Would RF: make sense? Instead of the indenting.
> > 
> > Maybe.
> > 
> > As a preface:
> > 
> > I doubt the need for associating a subset of the files
> > patterns for a subsystem with a particular reviewer.
> > 
> > If a reviewer is interested enough in a subsystem to
> > volunteer to read patches then that reviewer likely won't
> > be overburdened by getting a few more emailed patches
> > that may be outside a scope of interest.
> 
> I agree.  And if a subset of files needs a separate set of maintainers
> or reviewers, it doesn't seem excessive to split it into a separate
> MAINTAINERS entry.  For instance, if you want kernel/sched/rt.c to have
> an additional set of maintainers/reviewers, just add a MAINTAINERS entry
> for "SCHEDULER - REALTIME" with an appropriate "F:" line.

Ideally I'd want semantic boundaries, but given this all needs to be
robot parsed that's going to be a massive pain.

I guess I'm just not going to use it then. Too much hassle.

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[josh]

On Tue, Jul 08, 2014 at 09:09:25PM -0700, Joe Perches wrote:
> On Tue, 2014-07-08 at 15:25 -0700, Paul E. McKenney wrote:
> > On Tue, Jul 08, 2014 at 03:05:16PM -0700, Joe Perches wrote:
> []
> > > I still think the concept is pretty useless and it's
> > > just a duplication of "M:", which isn't anything other
> > > than a list of who should be sent patches.
> []
> > It will be interesting to see how things go.
> 
> Yes, it might work out fine and maybe might cause some
> other beneficial changes.
> 
> > There did seem to be
> > some people who were comfortable being listed as RCU reviewers, but
> > not as RCU maintainers.  Perhaps the same thing happens elsewhere.
> 
> Maybe so.
> 
> I wrote a script to find which maintainer addresses that
> haven't signed or authored a commit in the last 2 years.

Did you count Acked-by or Reviewed-by lines as maintainer activity as
well?

I do see a few cases of stale email addresses in there, for active
developers who don't use the listed email address.  Myself included (I
don't use @freedesktop.org anymore, though it still works).  I also see
a few lists or role addresses: ceph-devel and trivial@kernel.org seem
unlikely to author commits.

> I got ~250 entries.  That's about 1/4 of all maintainer
> email addresses.
> 
> Maybe a dozen of these are false positives though because
> some maintainers prefer to receive email at one address
> but ack from another.
> 
> These are all lowercase for better matching.
> 
> abhijit mahajan <abhijit.mahajan@avagotech.com>
> achim leubner <achim_leubner@adaptec.com>
> adam radford <linuxraid@lsi.com>
> al cho <acho@novell.com>
> ali akcaagac <aliakc@web.de>
> allan stephens <allan.stephens@windriver.com>
> aloisio almeida jr <aloisio.almeida@openbossa.org>
> andreas dilger <adilger.kernel@dilger.ca>
> andreas koensgen <ajk@comnets.uni-bremen.de>
> andrew hendry <andrew.hendry@gmail.com>
> andrew veliath <andrewtv@usa.net>
> andries brouwer <aeb@cwi.nl>
> andy henroid <andrew.d.henroid@intel.com>
> anil ravindranath <anil_ravindranath@pmc-sierra.com>
> anil s keshavamurthy <anil.s.keshavamurthy@intel.com>
> antoine jacquet <royale@zerezo.com>
> antonino daplas <adaplas@gmail.com>
> armin schindler <mac@melware.de>
> artur paszkiewicz <artur.paszkiewicz@intel.com>
> "arvind r." <arvino55@gmail.com>
> ashley lai <ashley@ashleylai.com>
> aurelien jacquiot <a-jacquiot@ti.com>
> balbir singh <bsingharora@gmail.com>
> bill metzenthen <billm@melbpc.org.au>
> bradley grove <linuxdrivers@attotech.com>
> brett rudley <brudley@broadcom.com>
> brian johnson <brijohn@gmail.com>
> bruce chang <brucechang@via.com.tw>
> bryan huntsman <bryanh@codeaurora.org>
> ceph-devel@vger.kernel.org
> cesar miquel <miquel@df.uba.ar>
> cezary jackiewicz <cezary.jackiewicz@gmail.com>
> chaoming li <chaoming_li@realsil.com.cn>
> chen liqin <liqin.linux@gmail.com>
> chien yen <chien.yen@oracle.com>
> chirag kantharia <chirag.kantharia@hp.com>
> chris brannon <chris@the-brannons.com>
> christine caulfield <ccaulfie@redhat.com>
> christopher harrer <charrer@alacritech.com>
> "christopher li" <sparse@chrisli.org>
> christoph raisch <raisch@de.ibm.com>
> cliff whickman <cpw@sgi.com>
> colin leroy <colin@colino.net>
> corentin labbe <clabbe.montjoie@gmail.com>
> daniel drake <dsd@gentoo.org>
> daniele venzano <venza@brownhat.org>
> daniel oliveira nascimento <don@syst.com.br>
> daniel ribeiro <drwyrm@gmail.com>
> dario ballabio <ballabio_dario@emc.com>
> david kershner <david.kershner@unisys.com>
> david rowe <david@rowetel.com>
> david safford <safford@us.ibm.com>
> david täht <d@teklibre.com>
> deepak saxena <dsaxena@plexity.net>
> dirk eibach <eibach@gdsys.de>
> dirk opfer <dirk@opfer-online.de>
> dmitry tarnyagin <dmitry.tarnyagin@lockless.no>
> dominik brodowski <linux@dominikbrodowski.net>
> doug gilbert <dgilbert@interlog.com>
> doug thompson <dougthompson@xmission.com>
> doug warzecha <douglas_warzecha@dell.com>
> duncan sands <duncan.sands@free.fr>
> "ed l. cashin" <ecashin@coraid.com>
> egor martovetsky <egor@pasemi.com>
> eric biederman <ebiederm@xmission.com>
> eric piel <eric.piel@tremplin-utc.net>
> erik andren <erik.andren@gmail.com>
> faisal latif <faisal.latif@intel.com>
> ferenc bakonyi <fero@drama.obuda.kando.hu>
> florian schilhabel <florian.c.schilhabel@googlemail.com>.
> forest bond <forest@alittletooquiet.net>
> frank seidel <frank@f-seidel.de>
> fujita tomonori <fujita.tomonori@lab.ntt.co.jp>
> gary zambrano <zambrano@broadcom.com>
> gilles muller <gilles.muller@lip6.fr>
> gleb natapov <gleb@kernel.org>
> guillaume ligneul <guillaume.ligneul@gmail.com>
> guo-fu tseng <cooldavid@cooldavid.org>
> gustavo padovan <gustavo@padovan.org>
> haavard skinnemoen <hskinnemoen@gmail.com>
> hal rosenstock <hal.rosenstock@gmail.com>
> hansjoerg lipp <hjlipp@web.de>
> hans ulli kroll <ulli.kroll@googlemail.com>
> harald welte <haraldwelte@viatech.com>
> harald welte <laforge@openezx.org>
> henk vergonet <henk.vergonet@gmail.com>
> henrique de moraes holschuh <ibm-acpi@hmh.eng.br>
> herton ronaldo krzesinski <herton@canonical.com>
> heungjun kim <riverful.kim@samsung.com>
> hideaki yoshifuji <yoshfuji@linux-ipv6.org>
> hoang-nam nguyen <hnguyen@de.ibm.com>
> hubert feurstein <hubert.feurstein@contec.at>
> "hung hing lun, mike" <hlhung3i@gmail.com>
> ian campbell <ijc+devicetree@hellion.org.uk>
> ian molton <spyro@f2s.com>
> ion badulescu <ionut@badula.org>
> ishizaki kou <kou.ishizaki@toshiba.co.jp>
> ivan kokshaysky <ink@jurassic.park.msu.ru>
> jakub schmidtke <sjakub@gmail.com>
> "james e.j. bottomley" <james.bottomley@hansenpartnership.com>
> "james e.j. bottomley" <jbottomley@parallels.com>
> "james e.j. bottomley" <jejb@parisc-linux.org>
> james morris <jmorris@namei.org>
> "james r. van zandt" <jrv@vanzandt.mv.com>
> jamie lenehan <lenehan@twibble.org>
> jan-benedict glaw <jbglaw@lug-owl.de>
> jan dumon <j.dumon@option.com>
> jan harkes <jaharkes@cs.cmu.edu>
> jan-simon moeller <jansimon.moeller@gmx.de>
> jarkko lavinen <jarkko.lavinen@nokia.com>
> jarod wilson <jarod@wilsonet.com>
> jason uhlenkott <juhlenko@akamai.com>
> jaya kumar <jayakumar.alsa@gmail.com>
> jaya kumar <jayalk@intworks.biz>
> jay cliburn <jcliburn@gmail.com>
> jean-paul roubelat <jpr@f6fbb.org>
> jeff dike <jdike@addtoit.com>
> jens osterkamp <jens@de.ibm.com>
> jes sorensen <jes@trained-monkey.org>
> jim paris <jim@jtan.com>
> jiri kosina <trivial@kernel.org>
> jiri slaby <jirislaby@gmail.com>
> joerg reuter <jreuter@yaina.de>
> joern engel <joern@lazybastard.org>
> johan hedberg <johan.hedberg@gmail.com>
> john mccutchan <john@johnmccutchan.com>
> john ronciak <john.ronciak@intel.com>
> "john w. linville" <linville@tuxdriver.com>
> jonathan buzzard <jonathan@buzzard.org.uk>
> "jon d. mason" <jdmason@kudzu.us>
> jon nettleton <jon.nettleton@gmail.com>
> josh triplett <josh@freedesktop.org>
> joshua morris <josh.h.morris@us.ibm.com>
> joshua thompson <funaho@jurai.org>
> juanjo ciarlante <jjciarla@raiz.uncu.edu.ar>
> "juergen e. fischer" <fischer@norbit.de>
> juergen stuber <starblue@users.sourceforge.net>
> juerg haefliger <juergh@gmail.com>
> kentaro takeda <takedakn@nttdata.co.jp>
> kevin curtis <kevin.curtis@farsite.co.uk>
> kevin hilman <khilman@kernel.org>
> kirk reiser <kirk@reisers.ca>
> koichi yasutake <yasutake.koichi@jp.panasonic.com>
> kristoffer glembo <kristoffer@gaisler.com>
> kurt garloff <garloff@suse.de>
> leandro costantino <lcostantino@gmail.com>
> len brown <lenb@kernel.org>
> lennert buytenhek <kernel@wantstofly.org>
> liam girdwood <lgirdwood@gmail.com>
> liam girdwood <lrg@slimlogic.co.uk>
> lior dotan <liodot@gmail.com>
> "luis r. rodriguez" <mcgrof@gmail.com>
> maik broemme <mbroemme@plusserver.de>
> manohar vanga <manohar.vanga@gmail.com>
> marcelo henrique cerri <mhcerri@linux.vnet.ibm.com>
> marc hulsman <m.hulsman@tudelft.nl>
> "mark f. brown" <mark.brown314@gmail.com>
> martin mares <mj@ucw.cz>
> matthew wilcox <matthew@wil.cx>
> matt mackall <mpm@selenic.com>
> maxim krasnyansky <maxk@qti.qualcomm.com>
> michael hanselmann <linux-kernel@hansmi.ch>
> michael petchkovsky <mkpetch@internode.on.net>
> michael reed <mdr@sgi.com>
> michal januszewski <spock@gentoo.org>
> michal ostrowski <mostrows@earthlink.net>
> miguel ojeda sandonis <miguel.ojeda.sandonis@gmail.com>
> mikael starvik <starvik@axis.com>
> mike marciniszyn <infinipath@intel.com>
> mike rapoport <mike@compulab.co.il>
> miloslav trmac <mitr@volny.cz>
> miroslav zagorac <zaga@fly.cc.fer.hr>
> mitchell blank jr <mitch@sfgoth.com>
> muli ben-yehuda <muli@il.ibm.com>
> nagalakshmi nandigama <nagalakshmi.nandigama@avagotech.com>
> neela syam kolli <megaraidlinux@lsi.com>
> nelson castillo <arhuaco@freaks-unidos.net>
> nicholas a. bellinger <nab@linux-iscsi.org>
> nick piggin <npiggin@kernel.dk>
> olav kongas <ok@artecdesign.ee>
> olivier lorin <o.lorin@laposte.net>
> pantelis antoniou <pantelis.antoniou@gmail.com>
> pat gefre <pfg@sgi.com>
> paul mackerras <paulus@au.ibm.com>
> pau oliva fora <pof@eslack.org>
> peter chubb <pleb@gelato.unsw.edu.au>
> peter feuerer <peter@piie.net>
> philip blundell <philb@gnu.org>
> philip kelleher <pjk1939@linux.vnet.ibm.com>
> philip moltmann <moltmann@vmware.com>
> praveen krishnamoorthy <praveen.krishnamoorthy@avagotech.com>
> rakesh iyer <riyer@nvidia.com>
> ranganathan desikan <ravi@jetztechnologies.com>
> remi denis-courmont <courmisch@gmail.com>
> riccardo facchetti <fizban@tin.it>
> richard l maliszewski <richard.l.maliszewski@intel.com>
> "richard russon (flatcap)" <ldm@flatcap.org>
> rik faith <faith@cs.unc.edu>
> robert gerlach <khnz@gmx.de>
> robert love <rlove@rlove.org>
> robert love <rml@tech9.net>
> rob herring <robh+dt@kernel.org>
> roland dreier <roland@kernel.org>
> roland mcgrath <roland@redhat.com>
> ron mercer <ron.mercer@qlogic.com>
> ron minnich <rminnich@sandia.gov>
> samuel chessman <chessman@tux.org>
> santosh raspatur <santosh@chelsio.com>
> scott murray <scott@spiteful.org>
> sergey lapin <slapin@ossfans.org>
> shane wang <shane.wang@intel.com>
> simon arlott <cxacru@fire.lp0.eu>
> simon kelley <simon@thekelleys.org.uk>
> sorbica shieh <sorbica@icplus.com.tw>
> srinivas kandagatla <srinivas.kandagatla@gmail.com>
> stefan eletzhofer <stefan.eletzhofer@eletztrick.de>
> stefano brivio <stefano.brivio@polimi.it>
> steve french <sfrench@samba.org>
> steve sakoman <sakoman@gmail.com>
> steve underwood <steveu@coppice.org>
> steve wise <swise@chelsio.com>
> subbu seetharaman <subbu.seetharaman@emulex.com>
> sudarsana kalluru <sudarsana.kalluru@qlogic.com>
> sujith thomas <sujith.thomas@intel.com>
> teddy wang <teddy.wang@siliconmotion.com.cn>
> terje bergström <tbergstrom@nvidia.com>
> thibaut varene <t-bone@parisc-linux.org>
> thomas dahlmann <dahlmann.thomas@arcor.de>
> thomas winischhofer <thomas@winischhofer.net>
> "tigran a. aivazian" <tigran@aivazian.fsnet.co.uk>
> tigran aivazian <tigran@aivazian.fsnet.co.uk>
> till harbaum <till@harbaum.org>
> tim small <tim@buttersideup.com>
> tim waugh <tim@cyberelk.net>
> tony finch <dot@dotat.at>
> tony olech <tony.olech@elandigitalsystems.com>
> torben mathiasen <device@lanana.org>
> ulrich kunitz <kune@deine-taler.de>
> vegard nossum <vegardno@ifi.uio.no>
> ville syrjala <syrjala@sci.fi>
> vishwas a deshpande <vishwas.a.deshpande@ti.com>
> vitaly bordug <vbordug@ru.mvista.com>
> vitaly bordug <vitb@kernel.crashing.org>
> wensong zhang <wensong@linux-vs.org>
> willem riede <osst@riede.org>
> willy tarreau <willy@meta-x.org>
> "wong hoi sing, edison" <hswong3i@gmail.com>
> xavier deguillard <xdeguillard@vmware.com>
> yokota hiroshi <yokota@netlab.is.tsukuba.ac.jp>
> zhang wei <zw@zh-kernel.org>
> zwane mwaikambo <zwanem@gmail.com>
> 
> 

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Joe Perches]

On Wed, 2014-07-09 at 10:00 -0700, josh@joshtriplett.org wrote:
> On Tue, Jul 08, 2014 at 09:09:25PM -0700, Joe Perches wrote:
[]
> > I wrote a script to find which maintainer addresses that
> > haven't signed or authored a commit in the last 2 years.
> 
> Did you count Acked-by or Reviewed-by lines as maintainer activity as
> well?

Yes.

Anything other than a cc: line counted as activity.

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Frederic Weisbecker]

On Tue, Jul 08, 2014 at 09:09:25PM -0700, Joe Perches wrote:
> On Tue, 2014-07-08 at 15:25 -0700, Paul E. McKenney wrote:
> > On Tue, Jul 08, 2014 at 03:05:16PM -0700, Joe Perches wrote:
> []
> > > I still think the concept is pretty useless and it's
> > > just a duplication of "M:", which isn't anything other
> > > than a list of who should be sent patches.
> []
> > It will be interesting to see how things go.
> 
> Yes, it might work out fine and maybe might cause some
> other beneficial changes.
> 
> > There did seem to be
> > some people who were comfortable being listed as RCU reviewers, but
> > not as RCU maintainers.  Perhaps the same thing happens elsewhere.
> 
> Maybe so.
> 
> I wrote a script to find which maintainer addresses that
> haven't signed or authored a commit in the last 2 years.
> 
> I got ~250 entries.  That's about 1/4 of all maintainer
> email addresses.
> 
> Maybe a dozen of these are false positives though because
> some maintainers prefer to receive email at one address
> but ack from another.
> 
> These are all lowercase for better matching.

Aren't we all falling in the typical pessimism trap, evil of the century!
Why do we always want to consider the lack of activity from a maintainer as a bad symptom?

What if it were actually a positive sign resulting of a finished, completed, and perfect
clean bug-free subsystem?

Once we accept that, all we need is support from the MAINTAINERS file:

diff --git a/MAINTAINERS b/MAINTAINERS
index 970c4a0..5da75dd 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -76,6 +76,9 @@ Descriptions of section entries:
 	T: SCM tree type and location.
 	   Type is one of: git, hg, quilt, stgit, topgit
 	S: Status, one of the following:
+          Finished:	The code is finished. The maintainer has fixed all
+			the bugs and adressed all missing features. No further
+			patch is needed.
 	   Supported:	Someone is actually paid to look after this.
 	   Maintained:	Someone actually looks after it.
 	   Odd Fixes:	It has a maintainer but they don't have time to do

Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU

[Joe Perches]

On Wed, 2014-07-09 at 19:42 +0200, Frederic Weisbecker wrote:
> On Tue, Jul 08, 2014 at 09:09:25PM -0700, Joe Perches wrote:
> > On Tue, 2014-07-08 at 15:25 -0700, Paul E. McKenney wrote:
> > > On Tue, Jul 08, 2014 at 03:05:16PM -0700, Joe Perches wrote:
> > []
> > > > I still think the concept is pretty useless and it's
> > > > just a duplication of "M:", which isn't anything other
> > > > than a list of who should be sent patches.
> > []
> > > It will be interesting to see how things go.
> > 
> > Yes, it might work out fine and maybe might cause some
> > other beneficial changes.
> > 
> > > There did seem to be
> > > some people who were comfortable being listed as RCU reviewers, but
> > > not as RCU maintainers.  Perhaps the same thing happens elsewhere.
> > 
> > Maybe so.
> > 
> > I wrote a script to find which maintainer addresses that
> > haven't signed or authored a commit in the last 2 years.
> > 
> > I got ~250 entries.  That's about 1/4 of all maintainer
> > email addresses.
> > 
> > Maybe a dozen of these are false positives though because
> > some maintainers prefer to receive email at one address
> > but ack from another.
> > 
> > These are all lowercase for better matching.
> 
> Aren't we all falling in the typical pessimism trap, evil of the century!
> Why do we always want to consider the lack of activity from a maintainer as a bad symptom?

> What if it were actually a positive sign resulting of a finished, completed, and perfect
> clean bug-free subsystem?

You mean "free of known defects".

> diff --git a/MAINTAINERS b/MAINTAINERS
[]
> @@ -76,6 +76,9 @@ Descriptions of section entries:
>  	T: SCM tree type and location.
>  	   Type is one of: git, hg, quilt, stgit, topgit
>  	S: Status, one of the following:
> +          Finished:	The code is finished. The maintainer has fixed all
> +			the bugs and adressed all missing features. No further
> +			patch is needed.

Yeah, some'um like that.

But very few people are capable of writing bug-free,
"finished" code akin to Donald Knuth.

Maybe finding a bug in a section marked like that
should be worth something to the patch submitter.

Maybe:
	virtual credit << bugs found

Most of those entries were for the people that
submitted the original code.  Most of those too
are no longer active in linux development.

So either put them in the CREDITS file and remove
them altogether from MAINTAINERS or maybe mark them
as reviewers instead of maintainers.

btw:  Here's a slightly cleaned up version of the
      script I used

$ cat find_active_maintainers.sh
#!/bin/bash

git log --since=2-years-ago | tr [:upper:] [:lower:] > git.commit.log
git grep -E "^M:" MAINTAINERS | tr [:upper:] [:lower:] | cut -d: -f3- | \
    sort | uniq | sed  -r 's/^\s+//' > maintainer.addresses
cat maintainer.addresses |
while read entry ; do
    email=$(echo $entry | sed -r -e 's/^[^<]*<//' -e 's/>.*//')
    echo "---> $entry	email --> $email"
    grep -P "^\s*(?:author:|[\w\-]+by:).*\b$email\b" git.commit.log | \
	sed -r -e 's/^\s+//' -e 's/>.*/>/' -e 's/\s+/ /g' | \
	sort | uniq -c
done

[PATCH tip/core/rcu 3/3] rcu: Update RCU maintainership

[Paul E. McKenney]

From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>

Drop Dipankar Sarma at his request (https://lkml.org/lkml/2014/6/2/628),
add Josh Triplett based on long-term review, contributions, and
agreement to take on this role (https://lkml.org/lkml/2014/6/2/554).

Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
---
 MAINTAINERS | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 92f2bf91eec1..922661d17083 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -7405,6 +7405,7 @@ F:	kernel/rcu/torture.c
 
 RCUTORTURE TEST FRAMEWORK
 M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
+M:	Josh Triplett <josh@joshtriplett.org>
 R:	Steven Rostedt <rostedt@goodmis.org>
 R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
 R:	Lai Jiangshan <laijs@cn.fujitsu.com>
@@ -7430,8 +7431,8 @@ S:	Supported
 F:	net/rds/
 
 READ-COPY UPDATE (RCU)
-M:	Dipankar Sarma <dipankar@in.ibm.com>
 M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
+M:	Josh Triplett <josh@joshtriplett.org>
 R:	Steven Rostedt <rostedt@goodmis.org>
 R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
 R:	Lai Jiangshan <laijs@cn.fujitsu.com>
@@ -8214,6 +8215,7 @@ F:	mm/sl?b.c
 SLEEPABLE READ-COPY UPDATE (SRCU)
 M:	Lai Jiangshan <laijs@cn.fujitsu.com>
 M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
+M:	Josh Triplett <josh@joshtriplett.org>
 R:	Steven Rostedt <rostedt@goodmis.org>
 R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
 L:	linux-kernel@vger.kernel.org
-- 
1.8.1.5

Re: [PATCH tip/core/rcu 3/3] rcu: Update RCU maintainership

[Josh Triplett]

On Mon, Jul 07, 2014 at 03:42:04PM -0700, Paul E. McKenney wrote:
> From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> 
> Drop Dipankar Sarma at his request (https://lkml.org/lkml/2014/6/2/628),
> add Josh Triplett based on long-term review, contributions, and
> agreement to take on this role (https://lkml.org/lkml/2014/6/2/554).
> 
> Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>

Reviewed-by: Josh Triplett <josh@joshtriplett.org>

Does it make sense to keep separate entries for each of these, at this
point, or should we just merge all of them into the main "READ-COPY
UPDATE (RCU)" entry?

>  MAINTAINERS | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 92f2bf91eec1..922661d17083 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -7405,6 +7405,7 @@ F:	kernel/rcu/torture.c
>  
>  RCUTORTURE TEST FRAMEWORK
>  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> +M:	Josh Triplett <josh@joshtriplett.org>
>  R:	Steven Rostedt <rostedt@goodmis.org>
>  R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
>  R:	Lai Jiangshan <laijs@cn.fujitsu.com>
> @@ -7430,8 +7431,8 @@ S:	Supported
>  F:	net/rds/
>  
>  READ-COPY UPDATE (RCU)
> -M:	Dipankar Sarma <dipankar@in.ibm.com>
>  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> +M:	Josh Triplett <josh@joshtriplett.org>
>  R:	Steven Rostedt <rostedt@goodmis.org>
>  R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
>  R:	Lai Jiangshan <laijs@cn.fujitsu.com>
> @@ -8214,6 +8215,7 @@ F:	mm/sl?b.c
>  SLEEPABLE READ-COPY UPDATE (SRCU)
>  M:	Lai Jiangshan <laijs@cn.fujitsu.com>
>  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> +M:	Josh Triplett <josh@joshtriplett.org>
>  R:	Steven Rostedt <rostedt@goodmis.org>
>  R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
>  L:	linux-kernel@vger.kernel.org
> -- 
> 1.8.1.5
> 

Re: [PATCH tip/core/rcu 3/3] rcu: Update RCU maintainership

[Paul E. McKenney]

On Tue, Jul 08, 2014 at 10:58:55AM -0700, Josh Triplett wrote:
> On Mon, Jul 07, 2014 at 03:42:04PM -0700, Paul E. McKenney wrote:
> > From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> > 
> > Drop Dipankar Sarma at his request (https://lkml.org/lkml/2014/6/2/628),
> > add Josh Triplett based on long-term review, contributions, and
> > agreement to take on this role (https://lkml.org/lkml/2014/6/2/554).
> > 
> > Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
> 
> Reviewed-by: Josh Triplett <josh@joshtriplett.org>

Got it, thank you!

> Does it make sense to keep separate entries for each of these, at this
> point, or should we just merge all of them into the main "READ-COPY
> UPDATE (RCU)" entry?

Good question!  Let's see how we feel about it now, and then again a
year from now.

							Thanx, Paul

> >  MAINTAINERS | 4 +++-
> >  1 file changed, 3 insertions(+), 1 deletion(-)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 92f2bf91eec1..922661d17083 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -7405,6 +7405,7 @@ F:	kernel/rcu/torture.c
> >  
> >  RCUTORTURE TEST FRAMEWORK
> >  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> > +M:	Josh Triplett <josh@joshtriplett.org>
> >  R:	Steven Rostedt <rostedt@goodmis.org>
> >  R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> >  R:	Lai Jiangshan <laijs@cn.fujitsu.com>
> > @@ -7430,8 +7431,8 @@ S:	Supported
> >  F:	net/rds/
> >  
> >  READ-COPY UPDATE (RCU)
> > -M:	Dipankar Sarma <dipankar@in.ibm.com>
> >  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> > +M:	Josh Triplett <josh@joshtriplett.org>
> >  R:	Steven Rostedt <rostedt@goodmis.org>
> >  R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> >  R:	Lai Jiangshan <laijs@cn.fujitsu.com>
> > @@ -8214,6 +8215,7 @@ F:	mm/sl?b.c
> >  SLEEPABLE READ-COPY UPDATE (SRCU)
> >  M:	Lai Jiangshan <laijs@cn.fujitsu.com>
> >  M:	"Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
> > +M:	Josh Triplett <josh@joshtriplett.org>
> >  R:	Steven Rostedt <rostedt@goodmis.org>
> >  R:	Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> >  L:	linux-kernel@vger.kernel.org
> > -- 
> > 1.8.1.5
> > 
> 

Re: [PATCH tip/core/rcu 0/3] Maintainership changes for 3.17

[Joe Perches]

On Mon, 2014-07-07 at 15:41 -0700, Paul E. McKenney wrote:
> This series updates RCU maintainership as follows:
> 
> 1.	Add "R:" designated-reviewers tag as discussed on ksummit-discuss
> 	email list.
> 
> 2.	Add designated reviewers for RCU.
> 
> 3.	Update RCU maintainership.

Missing
4. update get_maintainers...