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

From: Dave Chinner
Date: Mon Jun 02 2014 - 19:19:57 EST

On Mon, Jun 02, 2014 at 12:17:46PM -0700, Joe Perches wrote:
> On Mon, 2014-06-02 at 12:09 -0700, josh@xxxxxxxxxxxxxxxx wrote:
> > On Mon, Jun 02, 2014 at 12:05:17PM -0700, Joe Perches wrote:
> > > On Mon, 2014-06-02 at 11:55 -0700, josh@xxxxxxxxxxxxxxxx wrote:
> > > > this should go along with a change to
> > > > to add those folks to the CC list.
> > >
> > > Something like this:
> >
> > Yes, exactly. Given an appropriate commit message,
> > Reviewed-by: Josh Triplett <josh@xxxxxxxxxxxxxxxx>
> That's the sort of patch where reviewing is
> pretty useless.
> What it needs is testing, not reviewing.
> I tested it for all of 10 seconds.

>From Documentation/SubmittingPatches:

" (c) While there may be things that could be improved with this
submission, I believe that it is, at this time, (1) a
worthwhile modification to the kernel, and (2) free of known
issues which would argue against its inclusion.

A Reviewed-by tag is a statement of opinion that the patch is an
appropriate modification of the kernel without any remaining serious
technical issues."

So, for someone to say they have reviewed the code and are able to
say it is free of known issues and has no remaining technical
issues, they would have had to apply, compile and test the patch,

i.e. Reviewed-by implies both Acked-by, Tested-by and that the code
is technically sound.

Anyone using Reviewed-by without having actually applied and tested
the patch is mis-using the tag - they should be using Acked-by: if
all they have done is read the code in their mail program....


Dave Chinner
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at