[RFC] kernel/ review rules

[RFC] kernel/ review rules

[Bradley Morgan]

(CCed some people I talk to loads, I don't think I should CC every single
kernel/ maintainer, that would be fun otherwise, if you feel others should
be CCed, please CC them)

Hey everyone, it's been about God knows how long since I've started
reviewing kernel/, here are my current stats:
    
Reviewed-by     
13
Tested-by          
7
Reported-by        
1

Pretty good, ngl, I'm happy to review these patches, and I'm happy to
review the slop patches, and the people's patches, and anything with
[PATCH] in the damn title.

My very kind slop tool generated these stats for me.. 

So I've had some controversy about, like, how I do these reviews, some
people say how I review without context, or other stuff, like I'm wrong.

And I would like to suggest a fix to this, stuff.

So, when I do reviews from now on, if I haven't reviewed the patch
super vigourisly, e.g: for nits, or for like little things, instead of
adding my R-B, I'll add this instead:

Acked-by: Bradley Morgan <include@grrlz.net>

Thanks for letting me review kernel/ patches!



Thanks!

Re: [RFC] kernel/ review rules

[Andrew Morton]

On Fri, 21 Aug 2026 01:20:47 +0100 Bradley Morgan <include@grrlz.net> wrote:

> So, when I do reviews from now on, if I haven't reviewed the patch
> super vigourisly, e.g: for nits, or for like little things, instead of
> adding my R-B, I'll add this instead:
> 
> Acked-by: Bradley Morgan <include@grrlz.net>

That's conventional.

Here's my interpretation:

Acked-by:			"cool changelog!"
Reviewed-by: + nothing		"cool changelog and I scrolled through it"
Reviewed-by: + commentary:	"I reviewed it!"