On Thu, 2021-04-22 at 16:36 +0100, Mark Brown wrote:
> On Thu, Apr 22, 2021 at 08:28:00AM -0700, James Bottomley wrote:
> > On Thu, 2021-04-22 at 08:32 -0400, Martin K. Petersen wrote:
> > > Another metric that may be worth capturing is how many Fixes:
> > > tags refer to patches authored by this submitter.
> >  
> > Or perhaps invert it: no bug fix without a Fixes: tag.  Some of the
> > human handlers of robot based finders, like Dan's smatch, do go
> > back and figure out where the bug came from, but if we encourage
> > the rule that if you're fixing a bug you must identify the origin
> > and explain the bug it may help weed out some bogus fixes.
> 
> Script that use git blame to generate a commit to put in the Fixes:
> tag incoming...

Any system can be gamed, but I was thinking fixes helps ground the
reviewer in where the bug was introduced.  I also wasn't thinking fixes
alone, but fixes: *and* explanation of the bug.

James