lkml.org 
[lkml]   [2014]   [Jun]   [2]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH RFC 1/2] MAINTAINERS: Add "R:" designated-reviewers tag
On Mon, 02 Jun 2014 11:44:29 -0700
Joe Perches <joe@perches.com> wrote:

> On Mon, 2014-06-02 at 11:16 -0700, Paul E. McKenney wrote:
> > But there have been people who have found serious issues in RCU patches
> > who I would not trust as full maintainers. The ability to find defects
> > is valuable in and of itself, and should be recognized as such, even
> > when not accompanied by the rest of the maintainership package.
>
> Maybe, but odd-lot reviewers are most likely going to find
> these same defects regardless of any "R" designation in
> MAINTAINERS.
>

Actually, I'm thinking the R: tag is a good idea and we should have
people ask to be added to MAINTAINERS if they want to review certain
subsystems. Grant you, it should be people that the maintainers trust.
I can think of several people I would like to be added as R: in tracing.

The point is, when patches go out, it is easy to see who the Cc list
should be. And perhaps this will get patches reviewed more. Maybe
maintainers of other subsystems should ask to have the R: tag added for
something they don't maintain but want to help out in.

I may add myself to the x86, scheduler, time keeping and perhaps even
RCU, as I like to read those patches. I'm not at the level of
maintaining those subsystems, but I feel comfortable enough to review
any changes there.

-- Steve


\
 
 \ /
  Last update: 2014-06-02 22:21    [W:0.106 / U:1.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site