|
|
Subscribe / Log in / New account

5.3 Kernel development cycle statistics

By Jonathan Corbet
September 9, 2019
It's that time of the development cycle again: work on the 5.3 kernel is winding down with an expected final release date of September 15. Read on for LWN's traditional look at where the code in 5.3 came from in this relatively busy development cycle.

As of this writing, 14,435 non-merge changesets have been pulled into the mainline repository for 5.3; these changes were contributed by 1,846 developers. That makes 5.3 one of the busier cycles in recent times:

ReleaseChangesetsDevelopers
4.1514,866 1,801
4.1613,630 1,805
4.1713,541 1,713
4.1813,283 1,728
4.1914,043 1,752
4.2013,884 1,749
5.012,808 1,757
5.113,034 1,737
5.214,024 1,783
5.314,435 1,846

The traffic in 5.3 thus did not set a new record; indeed, 4.9, with 16,214 changesets merged, seems likely to hold that record for a while yet. The 1,846 developers contributing during this cycle is a new high, though, suggesting that the kernel community is still growing; 256 of those developers were first-time contributors.

Those 1,846 developers deleted an impressive 389,000 lines of code from the kernel during this cycle. They also added 973,000 lines, though, so the kernel grew by 584,000 lines of code. The most active developers this time around were:

Most active 5.3 developers
By changesets
Kuninori Morimoto2711.9%
Christoph Hellwig2621.8%
Mauro Carvalho Chehab2051.4%
Nishka Dasgupta1621.1%
Chris Wilson1601.1%
Greg Kroah-Hartman1451.0%
Yue Haibing1370.9%
Masahiro Yamada1300.9%
Gustavo A. R. Silva1280.9%
Geert Uytterhoeven1200.8%
Takashi Sakamoto1130.8%
Wolfram Sang1010.7%
Arnaldo Carvalho de Melo990.7%
Colin Ian King980.7%
Arnd Bergmann960.7%
Adrian Hunter960.7%
Russell King940.7%
David Howells920.6%
Hawking Zhang900.6%
Maxime Ripard840.6%
By changed lines
Hawking Zhang36437131.2%
Arnd Bergmann849937.3%
Mauro Carvalho Chehab340412.9%
Harry Wentland335682.9%
Jason Gunthorpe224211.9%
Chris Wilson173821.5%
Richard Fitzgerald157911.4%
Linus Walleij133921.1%
Greg Kroah-Hartman117131.0%
Bernard Metzler109630.9%
Christoph Hellwig108440.9%
Pawel Laszczak77020.7%
Hannes Reinecke72550.6%
Andrii Nakryiko71430.6%
Maxime Ripard67090.6%
David Ahern65570.6%
Daniele Ceraolo Spurio57810.5%
Darrick J. Wong57760.5%
Kuninori Morimoto47170.4%
Thomas Zimmermann47060.4%

The biggest contributor of changesets this time around was Kuninori Morimoto, who did a lot of work updating code in the sound subsystem. Christoph Hellwig made significant improvements across the core kernel and the block subsystem, Mauro Carvalho Chehab converted a vast number of documentation files to the restructured text format, Nishka Dasgupta fixed a large number of Coccinelle warnings, and Chris Wilson worked extensively on the Intel i915 graphics driver.

Hawking Zhang was responsible for nearly one-third of the changed lines in 5.3; as is so often the case, the new code is yet another set of register definitions for the amdgpu graphics driver. Arnd Bergmann removed the old ISDN subsystem and, as usual, made improvements across the kernel tree. Harry Wentland also contributed AMD graphics driver changes, and Jason Gunthorpe removed some obsolete RDMA code.

The top testers and reviewers this time around were:

Test and review credits in 5.3
Tested-by
Andrew Bowers11512.0%
Arnaldo Carvalho de Melo272.8%
Leo Yan242.5%
Tony Lindgren242.5%
Tim Schumacher192.0%
Aaron Brown181.9%
Andre Przywara181.9%
Keerthy171.8%
Nick Desaulniers161.7%
Hans de Goede161.7%
Shijith Thotton161.7%
Reviewed-by
Alex Deucher1933.3%
Rob Herring1672.8%
Chris Wilson1151.9%
Christoph Hellwig1081.8%
Hawking Zhang1001.7%
Simon Horman931.6%
Darrick J. Wong911.5%
Andrew Lunn901.5%
Tvrtko Ursulin791.3%
Geert Uytterhoeven781.3%
Maarten Lankhorst771.3%

The biggest collector of Tested-by credits is Andrew Bowers, a developer who has never contributed a patch to the kernel, but who seemingly does extensive testing of changes to various Intel drivers. Review credits are more evenly spread out, though still concentrated on subsystems that require reviews before merging. 4,824 Patches in 5.3 (33% of the total) carried Reviewed-by tags; that is higher than the long-term average but slightly less than the number seen in 5.2.

The Reported-by tags, applied to give credit to the reporter of a problem, are worth a quick look as well. There were 687 patches with such tags merged for 5.3, with the top reporters being:

Reported-by credits in 5.3
Hulk Robot8910.7%
kbuild test robot627.5%
Stephen Rothwell374.4%
Dan Carpenter242.9%
syzbot182.2%
Randy Dunlap111.3%
Geert Uytterhoeven91.1%
Guenter Roeck91.1%
kernel test robot91.1%
Arnd Bergmann81.0%

Of the top five bug reporters, three are automated testing systems, one is the linux-next maintainer (linux-next being the first point of integration testing for many patches), and one is a developer who aggressively looks for (and fixes) bugs with a static-analysis tool. In other words, it is fair to say that the testing mechanisms that have slowly been built up around the kernel process are having an effect, catching bugs before they can affect users.

That said, it's worth noting that 2,026 changes in 5.3 — 14% of the total — carry Fixes: tags, implying that they are fixes for bugs that did make it into the mainline. Many fixes undoubtedly are merged without the addition of a Fixes: tag, so the actual percentage of bug fixes is certain to be significantly higher than that. Clearly, there is plenty of scope for improving our testing infrastructure and procedures still.

Finally, 233 companies (that we were able to identify) supported work on the 5.3 kernel. The most active of these companies were:

Most active 5.3 employers
By changesets
Intel154510.7%
AMD9666.7%
(Unknown)9636.7%
Red Hat9256.4%
(None)8656.0%
Google6384.4%
Renesas Electronics6384.4%
Linaro5483.8%
IBM4603.2%
Mellanox4403.0%
(Consultant)4102.8%
Huawei Technologies3722.6%
SUSE3372.3%
Samsung3112.2%
NXP Semiconductors3092.1%
Linux Foundation2912.0%
ARM2801.9%
Facebook1881.3%
Oracle1701.2%
BayLibre1701.2%
By lines changed
AMD43253737.0%
Linaro1159269.9%
Intel769506.6%
Mellanox467324.0%
Samsung378693.2%
Google316662.7%
(Unknown)313102.7%
Red Hat310892.7%
IBM294492.5%
(None)278032.4%
SUSE189281.6%
Cirrus Logic165091.4%
(Consultant)141921.2%
Facebook137231.2%
Linux Foundation134051.1%
ARM127551.1%
Pengutronix123981.1%
NXP Semiconductors122021.0%
Renesas Electronics113451.0%
Huawei Technologies101650.9%

As usual, there are few surprises to be found in the above table. The list of the most active companies does not tend to vary much from one development cycle to the next.

Overall, the picture that emerges is of a development community that continues to integrate massive numbers of changes on a predictable schedule, and which continues to grow slowly. For all its faults, the kernel development community is clearly doing some things right.

Index entries for this article
KernelReleases/5.3


to post comments

5.3 Kernel development cycle statistics

Posted Sep 10, 2019 11:54 UTC (Tue) by pm215 (subscriber, #98099) [Link]

So will the 5.4 round of these statistics merge "Red Hat" and "IBM" in the "employers" rankings? That should give them a fighting chance of taking the top spot :-)

5.3 Kernel development cycle statistics

Posted Jun 19, 2020 11:04 UTC (Fri) by vegard (subscriber, #52330) [Link] (2 responses)

Dmitry Vyukov commented these statistics on Twitter:

https://twitter.com/dvyukov/status/1171169404995997696

I get a similar result as him:

$ git log --oneline --grep "Reported-.*syzbot" v5.2..v5.3 | wc -l
103

Is there any chance the article will be corrected? (Or to see a comment explaining the difference?)

Syzbot stats

Posted Jun 19, 2020 12:55 UTC (Fri) by corbet (editor, #1) [Link] (1 responses)

The discrepancy had to do with how syzbot was being credited - each bug had a different email address associated with it, and so looked like it was coming from a different "person". Special handling for syzbot was added to gitdm after Dmitry complained, and an explanation of what was going on was included with the 5.4 statistics.

Syzbot stats

Posted Jun 19, 2020 12:57 UTC (Fri) by vegard (subscriber, #52330) [Link]

Thanks!


Copyright © 2019, Eklektix, Inc.
This article may be redistributed under the terms of the Creative Commons CC BY-SA 4.0 license
Comments and public postings are copyrighted by their creators.
Linux is a registered trademark of Linus Torvalds