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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140602145020.400d4e0c@gandalf.local.home>
Date:	Mon, 2 Jun 2014 14:50:20 -0400
From:	Steven Rostedt <rostedt@...dmis.org>
To:	Joe Perches <joe@...ches.com>
Cc:	paulmck@...ux.vnet.ibm.com, josh@...htriplett.org,
	linux-kernel@...r.kernel.org, mingo@...nel.org,
	laijs@...fujitsu.com, dipankar@...ibm.com,
	akpm@...ux-foundation.org, mathieu.desnoyers@...icios.com,
	niv@...ibm.com, tglx@...utronix.de, peterz@...radead.org,
	dhowells@...hat.com, edumazet@...gle.com, dvhart@...ux.intel.com,
	fweisbec@...il.com, oleg@...hat.com, sbw@....edu
Subject: Re: [PATCH RFC 1/2] MAINTAINERS: Add "R:" designated-reviewers tag

On Mon, 02 Jun 2014 11:44:29 -0700
Joe Perches <joe@...ches.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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/