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

From: Joe Perches
Date: Thu Jul 10 2014 - 06:00:27 EST


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@xxxxxxxxxxx> (kernel/sched/rt.c)
> > > R: Juri Lelli <jundri.lelli@xxxxxxxxx> (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.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/