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

2014-07-11 Thread Peter Zijlstra
On Thu, Jul 10, 2014 at 04:16:23PM -0700, j...@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: > > > > > ... >

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

2014-07-10 Thread 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 (kernel/sched/rt.c) > > > > R: Juri Lelli (kerne

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

2014-07-10 Thread 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 (kernel/sched/rt.c) > > > R: Juri Lelli (kernel/sched/deadline.c) > > > > Maybe a better syntax might be something lik

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

2014-07-10 Thread Peter Zijlstra
On Wed, Jul 09, 2014 at 06:23:22AM -0700, Joe Perches wrote: > > SCHEDULER: > > ... > > R: Steven Rostedt (kernel/sched/rt.c) > > R: Juri Lelli (kernel/sched/deadline.c) > > Maybe a better syntax might be something like: > R:Steven Rostedt > F: kernel/sched/rt.c > > where

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

2014-07-09 Thread 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

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

2014-07-09 Thread 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 an

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

2014-07-09 Thread Joe Perches
On Wed, 2014-07-09 at 10:00 -0700, j...@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 line

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

2014-07-09 Thread 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 an

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

2014-07-09 Thread 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 al

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

2014-07-09 Thread 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 an

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

2014-07-08 Thread 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 wil

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

2014-07-08 Thread 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 a

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

2014-07-08 Thread 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

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

2014-07-08 Thread 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,

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

2014-07-08 Thread 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

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

2014-07-08 Thread 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

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

2014-07-08 Thread 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/MAINT

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

2014-07-08 Thread Peter Zijlstra
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/t

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

2014-07-07 Thread Paul E. McKenney
From: "Paul E. McKenney" 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) Sign