[Ksummit-discuss] [TOPIC] Encouraging more reviewers

[Paul Walmsley]

On Sat, 24 May 2014, James Bottomley wrote:

> I'm sure there are many other things people could suggest.

What's needed is to bring quality reviewers up to the same level of 
recognition and control as maintainers.

Ideally, maintainers would recognize quality reviewers, and list them in 
the MAINTAINERS file - perhaps with an "R:" tag?  Maintainers would be 
expected to designate at least one quality reviewer, but ideally more, for 
a given subsystem.  

Then we should require every patch to have at least one "Reviewed-by:", 
aside from the maintainer's "Signed-off-by:" before being merged.  This 
"Reviewed-by:" could come from the maintainer, but ideally would come from 
a quality reviewer.

Patch submitters would need to get their patches reviewed by at least one 
of the recognized reviewers before expecting it to be merged.

Part of the goal here would also be to convert quality reviewers into 
co-maintainers over time, so maintainership duties can be spread among a 
larger group of people.


- Paul