Re: [PATCH RFC 1/2] MAINTAINERS: Add "R:" designated-reviewers tag

[josh@joshtriplett.org]

On Mon, Jun 02, 2014 at 10:22:58AM -0700, Joe Perches wrote:
> On Mon, 2014-06-02 at 10:00 -0700, Paul E. McKenney wrote:
> > 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.
> 
> I'm not sure of the value of this.
> 
> Why not just mark the actual reviewers as maintainers?

As discussed in the kernel summit discussion, being a regular patch
reviewer isn't the same thing as being *the* maintainer.

> The lack of reviewers problem is entirely distinct from
> the proposed solution.

Not arguing with that; this change alone won't produce more reviewers,
but it does provide a means of tracking regular reviewers.

> I believe active reviewers will generally subscribe to a
> subsystem specific mailing list.

Not every driver or subsystem is large enough to have its own mailing
list, and many would prefer to keep their discussion on LKML.  Also,
mailing lists introduce diffusion of responsibility.

> Perhaps it'd be better to get a "linux-rcu@vger.kernel.org"
> mailing list going.

Perhaps.

> > diff --git a/MAINTAINERS b/MAINTAINERS
> []
> > @@ -73,6 +73,8 @@ Descriptions of section entries:
> >  	L: Mailing list that is relevant to this area
> >  	W: Web-page with status/info
> >  	Q: Patchwork web based patch tracking system site
> > +	R: Designated reviewer, who should be CCed on patches,
> > +	   format: FullName <address@domain>
> >  	T: SCM tree type and location.
> >  	   Type is one of: git, hg, quilt, stgit, topgit
> >  	S: Status, one of the following:
> 
> If this is actually done, I suggest putting this
> "R" entry description immediately after the "M" entry.

Yeah, it does seem like these entries are not quite alphabetized, so
putting this right after M makes sense.

- Josh Triplett