LWN.net Logo

Tracking of testers and bug reporters - a status report

By Jonathan Corbet
November 11, 2008
A recurring topic at kernel summits is proper recognition for users who report bugs and test fixes. These people help the development process considerably, but they are far less visible than the developers who are creating those bugs in the first place. Since we would like to have more testers and reporters, it makes sense to reward them in whatever way we can. One of the strongest currencies we hold is credit for work done. So it stands to reason that crediting those who help the development process is in the interest of everybody involved.

One mechanism developed for this purpose is a set of tags applied to patches before they are merged into the mainline. When a patch fixes a bug, the user(s) who reported that bug should be credited through the addition of a Reported-by: tag. Similarly, testers are credited with the Tested-by: tag. As it happens, some developers have adopted the habit of using Reported-and-tested-by: as a way of saving valuable newlines in the common case where a user fills both roles.

There is a certain warm feeling that comes with having one's name stored in a changelog entry in the kernel source repository. But the amount of visibility which comes from this event is relatively small. So your editor decided to hack up his git data mining utility to track these tags. Without further ado, here are the top problem reporters and patch testers for the 2.6.27 development cycle:

Most credited 2.6.27 testers
Reported-by credits
Adrian Bunk4321.0%
Robert P. J. Day125.9%
Eric Sesterhenn52.4%
Andrew Morton42.0%
Alexey Dobriyan42.0%
Denys Fedoryshchenko42.0%
Yinghai Lu31.5%
David S. Miller31.5%
Vegard Nossum31.5%
Stephen Rothwell31.5%
Juha Leppanen31.5%
Russell King21.0%
Andi Kleen21.0%
Ingo Molnar21.0%
Benjamin Herrenschmidt21.0%
Daniel J Blueman21.0%
Daniel Exner21.0%
Manuel Lauss21.0%
Atsushi Nemoto21.0%
Mikael Pettersson21.0%
Tested-by: credits
Ingo Molnar74.6%
Andrew Savchenko63.9%
Rene Herman42.6%
Mariusz Kozlowski32.0%
Alexey Dobriyan32.0%
Tino Keitel32.0%
Robert Jarzmik32.0%
KOSAKI Motohiro21.3%
Benjamin Herrenschmidt21.3%
Larry Finger21.3%
Kenji Kaneshige21.3%
Jack Howarth21.3%
Gerald Schaefer21.3%
Dennis Jansen21.3%
Daniel J Blueman21.3%
Daniel Exner21.3%
Steven Noonan21.3%
Rus21.3%
Lawrence Greenfield21.3%
Mark Langsdorf21.3%

All told, there were a total of 205 Reported-by: and 153 Tested-by: credits entered during the 2.6.27 kernel cycle. This is arguably a reasonable start for a new tag, but it seems clear that a lot of problem reporters are not, yet, being credited in this manner. Your editor became curious to see just who is taking the time to credit these people; they, too, deserve some credit. A bit more script hacking yielded these tables:

Developers giving credits in 2.6.27
Reported-by credits
Adrian Bunk4421.5%
Linus Torvalds125.9%
Ingo Molnar83.9%
Andrew Morton73.4%
Peter Zijlstra73.4%
Bartlomiej Zolnierkiewicz62.9%
Yinghai Lu52.4%
Jarek Poplawski52.4%
Jiri Kosina52.4%
Hugh Dickins42.0%
FUJITA Tomonori42.0%
Paul Mundt42.0%
Vegard Nossum31.5%
Russell King31.5%
Jeremy Fitzhardinge31.5%
Roland McGrath31.5%
Haavard Skinnemoen31.5%
Dmitry Torokhov31.5%
David Woodhouse31.5%
Oleg Nesterov31.5%
Tested-by: credits
Pekka Enberg74.6%
Linus Torvalds74.6%
Takashi Iwai53.3%
Bartlomiej Zolnierkiewicz53.3%
Peter Zijlstra42.6%
Rafael J. Wysocki42.6%
Yinghai Lu42.6%
Hugh Dickins42.6%
Alan Stern42.6%
Eric Miao42.6%
Thomas Gleixner32.0%
Lennert Buytenhek32.0%
Alex Chiang32.0%
Krzysztof Helt32.0%
Stefan Richter32.0%
Andy Whitcroft32.0%
KOSAKI Motohiro21.3%
Dennis Jansen21.3%
Andrew Morton21.3%
David S. Miller21.3%

The end result: Adrian Bunk gave over 20% of the total bug reporting credits - to himself. Beyond that, a number of the core developers are taking at least some time to credit those who report bugs and test patches. But, in the end, the 10,628 changesets merged for 2.6.27 probably contained quite a few more patches which could have carried such tags. If the reporting and testing tags are to become truly useful and significant, they will have to be more universally used.

While your editor was at it, he also collected statistics for Reviewed-by: tags. These tags differ in that they are offered by the reviewer, who thereby states that a reasonably thorough review has been done and the code has not been found seriously wanting. Code review is perennially in short supply in just about any free software project, so, again, proper credit for reviewers seems like more than just a good idea. Here's the top 2.6.27 credited reviewers:

Developers with the most reviews (total 123)
Ingo Molnar2318.7%
Paul Jackson129.8%
Peter Zijlstra118.9%
Christoph Lameter108.1%
Aneesh Kumar K.V75.7%
KOSAKI Motohiro64.9%
Paul E. McKenney64.9%
Jeff Moyer54.1%
Robert P. J. Day43.3%
Nadia Derbey32.4%
Paul E. McKenney32.4%
Mingming Cao21.6%
Michael Buesch21.6%
Li Zefan21.6%
Matthew Wilcox21.6%
Ingo Oeser21.6%
Badari Pulavarty21.6%

If these numbers are to be believed, only 123 reviews were performed over the 2.6.27 development cycle. Even the most cynical observer is likely to agree that a bit more reviewing than that is going on. Most reviewers do not offer the associated tag, so their contribution goes unrecorded. In particular, Andrew Morton, who seems to review almost every patch which appears, should be at the top of the above list.

Clearly, the task of ensuring proper credit for testers, bug reporters, and reviewers is still in its initial stages. But one has to start somewhere; this is more information than we had before. Hopefully, over time, the habit of crediting those who help with the development process will become more widespread. And that, with luck, will encourage more testing and bug reporting and, as a result, a better kernel.


(Log in to post comments)

Tracking of testers and bug reporters - a status report

Posted Nov 13, 2008 7:00 UTC (Thu) by masuel (guest, #28661) [Link]

Developers with the most reviews (total 123)
...
Paul E. McKenney 6 4.9%
...
Paul E. McKenney 3 2.4%
...

A bit script tweaking still needed here :)

Tracking of testers and bug reporters - a status report

Posted Nov 13, 2008 16:40 UTC (Thu) by PaulMcKenney (subscriber, #9624) [Link]

Bartender! Give me a double!!!

Doubles

Posted Nov 13, 2008 16:58 UTC (Thu) by corbet (editor, #1) [Link]

It seems we already have a double:

    Reviewed-by: "Paul E. McKenney" <paulmck@us.ibm.com>
    Reviewed-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>

The script has been updated to not get confused when somebody (double-)quotes his name. Sorry for the confusion...

Looks like something I would do...

Posted Nov 13, 2008 18:33 UTC (Thu) by PaulMcKenney (subscriber, #9624) [Link]

I guess we can only be thankful that I thus far haven't mistakenly given my old Sequent or SRI email addresses... :-)

Tracking of testers and bug reporters - a status report

Posted Nov 13, 2008 9:57 UTC (Thu) by wfranzini (subscriber, #6946) [Link]

Nice report, however it only tracks reporters for closed bugs.

Is it possible to extract the same informations also from a different
source: http://bugzilla.kernel.org?

It would be nice if the data in the git repository can be a subset,
because referred only to closed bugs, of the information stored by
bugzilla.

Tracking of testers and bug reporters - a status report

Posted Nov 14, 2008 12:17 UTC (Fri) by pointwood (guest, #2814) [Link]

Very nice and thanks for the report. By shining some light on this info, we could hope more people would start using those tags.

Tracking of testers and bug reporters - a status report

Posted Nov 16, 2008 14:21 UTC (Sun) by JohnNilsson (guest, #41242) [Link]

How about using this as an end credit. Just as games and movies list the
pepole at the end the kernel could do the same when it panics ;-P

Copyright © 2008, Eklektix, Inc.
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds