Messages in this thread | | | Subject | Re: [PATCH tip/core/rcu 2/3] rcu: Add designated reviewers for RCU | From | Joe Perches <> | Date | Thu, 10 Jul 2014 03:00:12 -0700 |
| |
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.
| |