Re: [Ksummit-discuss] Allowing something Change-Id (or something like it) in kernel commits

Dan Carpenter
2019-08-28T08:57:52Z

On Tue, Aug 27, 2019 at 08:25:05AM -0700, Paul E. McKenney wrote:
> Someone then reviews the patch and reports a bug. I update the
> commit with a fix. Now what I do is this:
>
> [ paulmck: Update based on blablabla feedback from So And So. ]

It sort of bothers me that there isn't a proper tag for these so they're
are not searchable. Last week I fixed a bug and a reviewer noticed that
there was a similar issue earlier in the function so I fixed that as
well and sent a v2 but he only gets Reviewed-by credit. Reviewed-by is
the lamest sort of credit.

It should be that reviewers get credit for finding bugs in patches (no
credit for complaining about checkpatch issues, that is its own reward).

regards,
dan carpenter