|
|
Subscribe / Log in / New account

Statistics from the 4.18 development cycle

By Jonathan Corbet
July 24, 2018
The 4.18-rc6 kernel prepatch came out on July 22, right on schedule. That is a sign that this development cycle is approaching its conclusion, so the time has come for a look at some statistics for how things went this time around. It was another fairly ordinary release cycle for the most part, but with a couple of distinctive features.

As of 4.18-rc6, 12,879 non-merge changesets had found their way into the mainline repository. This work was contributed by 1,668 developers who added 553,000 lines of code and removed 652,000 lines, for a net reduction of 99,000 lines. This will be the fourth time in the project's history that a release is smaller than its predecessor — and the first time that this has happened for two releases in a row. Of those 1,668 developers, 226 made their first contribution to the kernel this time around; that is the smallest number of first-time contributors since 4.5 was released in March 2016.

More generally, the number of first-time contributors to each release since 3.0 looks like this:

[First-time contributors]

While the number of new contributors varies a bit over time, it has remained consistently between 200 and 300 for each development cycle for a long time. New contributors are important to the health of any development project, so it is good that the kernel continues to attract developers over time.

The most active developers for 4.18 were:

Most active 4.18 developers
By changesets
Christoph Hellwig2181.7%
Sergio Paracuellos2031.6%
Ben Skeggs1621.3%
Mauro Carvalho Chehab1591.2%
Colin Ian King1371.1%
Geert Uytterhoeven1120.9%
Chris Wilson1110.9%
Christian Lütke‑Stetzkamp1090.8%
Arnaldo Carvalho de Melo1080.8%
Arnd Bergmann1060.8%
Ajay Singh1060.8%
Fabio Estevam940.7%
David Ahern870.7%
Neil Brown830.6%
Masahiro Yamada810.6%
Darrick J. Wong770.6%
Hans de Goede750.6%
Quytelda Kahja750.6%
Jakub Kicinski690.5%
Wolfram Sang680.5%
By changed lines
Greg Kroah-Hartman20727420.1%
Sakari Ailus16808516.3%
Eric Biggers320623.1%
Ben Skeggs173681.7%
Ondrej Mosnacek157871.5%
Christoph Hellwig105531.0%
Srinivas Kandagatla99841.0%
Ian Kent78340.8%
Alexandre Belloni68010.7%
Martin KaFai Lau65180.6%
John Fastabend64790.6%
Oleksandr Andrushchenko62030.6%
Steven Eckhoff59930.6%
Felix Kuehling58860.6%
Mathieu Desnoyers56260.5%
Dave Chinner55880.5%
Kai Chieh Chuang55840.5%
Manivannan Sadhasivam53110.5%
Christian Lütke‑Stetzkamp52720.5%
Niklas Söderlund51120.5%

Christoph Hellwig ended up at the top of the per-changesets list with work throughout the block, virtual filesystem, and driver subsystems, including the since-reverted AIO polling interface. Sergio Paracuellos made a number of improvements to a couple of staging drivers, Ben Skeggs did a great deal of work on the Nouveau driver as usual, Mauro Carvalho Chehab's work was mostly in the media subsystem (of which he is the maintainer), and Colin Ian King continued his work fixing spelling errors and similar issues throughout the tree.

In the lines-changed column, Greg Kroah-Hartman removed the Lustre filesystem and the ncpfs filesystem as well. Sakari Ailus removed the atomisp driver (which was also in staging), Eric Biggers did a bunch of cryptography-related work (removing a bunch of code in the process), and Ondrej Mosnacek added some new optimized crypto algorithm implementations.

The developers working on 4.18 were supported by 233 companies that we were able to identify. The most active employers this time around were:

Most active 4.18 employers
By changesets
Intel12189.5%
(None)10087.8%
Red Hat9657.5%
(Unknown)7185.6%
AMD5874.6%
IBM5534.3%
Linaro4853.8%
Renesas Electronics4433.4%
Google3803.0%
SUSE3712.9%
Samsung3562.8%
(Consultant)3352.6%
Mellanox2812.2%
Huawei Technologies2662.1%
Oracle2552.0%
Facebook2261.8%
Orbital Critical Systems2031.6%
Bootlin1841.4%
Code Aurora Forum1831.4%
Canonical1761.4%
By lines changed
Intel22912122.2%
Linux Foundation20838220.2%
Red Hat580575.6%
Google495404.8%
AMD350063.4%
(None)313713.0%
Linaro298452.9%
(Unknown)269532.6%
IBM248162.4%
Renesas Electronics235682.3%
Bootlin209722.0%
Code Aurora Forum196341.9%
Facebook173911.7%
Samsung171851.7%
(Academia)167861.6%
(Consultant)137901.3%
Mellanox133531.3%
MediaTek121351.2%
SUSE103091.0%
Oracle91050.9%

If a developer applies a Signed-off-by tag to a patch that they are not the author of, it usually means that said developer was the maintainer who applied the patch and set it on the path to mainline inclusion. Looking at non-author signoffs (and associated employers) for 4.18 yields a table that looks like this:

Non-author signoffs in 4.18
By developer
David S. Miller130410.7%
Greg Kroah-Hartman11179.2%
Alex Deucher4773.9%
Mark Brown3623.0%
Mauro Carvalho Chehab3462.9%
Martin K. Petersen2912.4%
Daniel Borkmann2612.2%
Kalle Valo2612.2%
Michael Ellerman2351.9%
Simon Horman1831.5%
Andrew Morton1731.4%
Jens Axboe1711.4%
Jonathan Cameron1691.4%
Ingo Molnar1621.3%
David Sterba1591.3%
Rafael J. Wysocki1411.2%
Thomas Gleixner1391.1%
Alexei Starovoitov1271.0%
Linus Walleij1251.0%
Hans Verkuil1211.0%
By employer
Red Hat224218.5%
Linux Foundation11359.4%
Linaro9597.9%
Intel9287.6%
AMD5724.7%
Samsung4894.0%
Google4413.6%
IBM4393.6%
SUSE4023.3%
Oracle3913.2%
Huawei Technologies3803.1%
Facebook3402.8%
Code Aurora Forum3162.6%
(None)3052.5%
Mellanox2712.2%
Renesas Electronics2702.2%
ARM2041.7%
Bootlin1691.4%
(Unknown)1581.3%
linutronix1531.3%

It can be instructive to compare these numbers to those that were published for 2.6.24 in 2008. Many of the names in the left column were the same, though the ordering has changed — Andrew Morton had 1,679 non-author signoffs in 2.6.24. Many of the employer names are the same as well. But, in 2008, just over half of the non-author signoffs were made by developers working for two companies: Red Hat and the Linux Foundation. In 2018, those two organizations retain the top positions in the table, but one has to look at the top six companies to get up to 50% of the total. The process has been slow, but the concentration of maintainers has been dispersing over time.

Finally, with regard to test and review credits, the numbers are:

Test and review credits in 4.18
Tested-by
Andrew Bowers577.7%
Nicholas Piggin435.8%
Marek Szyprowski344.6%
Arnaldo Carvalho de Melo212.8%
Aaron Brown152.0%
Angelo Dureghello141.9%
Mathieu Malaterre141.9%
Randy Dunlap131.7%
Ard Biesheuvel131.7%
Dmitry Osipenko121.6%
Vijaya Kumar K121.6%
Xiongfeng Wang121.6%
Tomasz Nowicki121.6%
Nguyen Viet Dung111.5%
Jarkko Sakkinen81.1%
Song Liu81.1%
Geert Uytterhoeven70.9%
Reviewed-by
Alex Deucher1583.2%
Rob Herring1533.1%
Geert Uytterhoeven1152.3%
Christian König1042.1%
Darrick J. Wong1032.1%
Christoph Hellwig992.0%
David Sterba951.9%
Neil Brown901.8%
Laurent Pinchart871.7%
Simon Horman841.7%
Tony Cheng701.4%
Andrew Morton611.2%
Hawking Zhang551.1%
Hannes Reinecke511.0%
Brian Foster511.0%
Chris Wilson460.9%
Mika Kuoppala460.9%

The 4.18 kernel appears to be on track for an August 5 release, assuming no severe last-minute problems turn up. Once again, it will be the work of a huge community of developers, all of whom have managed to come together to improve this common resource. For all its faults, the kernel development community continues to function like a well-tuned machine, producing and integrating code at a pace that few other projects can match.

Index entries for this article
KernelReleases/4.18


to post comments

Statistics from the 4.18 development cycle

Posted Jul 25, 2018 9:30 UTC (Wed) by dvyukov (guest, #57055) [Link] (6 responses)

Hi Jonathan,

You missed syzbot in test credits:

$ git log v4.17..HEAD | grep "tested-by.*syzbot"
Reported-and-tested-by: syzbot+4c20b3866171ce8441d2@syzkaller.appspotmail.com
Reported-and-tested-by: syzbot+bf9253040425feb155ad@syzkaller.appspotmail.com
Reported-and-tested-by: syzbot+7d6d31d3bc702f566ce3@syzkaller.appspotmail.com
Reported-and-tested-by: syzbot+0a725420475916460f12@syzkaller.appspotmail.com
Reported-and-tested-by: syzbot+83699adeb2d13579c31e@syzkaller.appspotmail.com
Reported-and-tested-by: syzbot+41a1b341571f0952badb@syzkaller.appspotmail.com
Reported-and-tested-by: syzbot+d154ec99402c6f628887@syzkaller.appspotmail.com

Syzbot

Posted Jul 25, 2018 14:01 UTC (Wed) by corbet (editor, #1) [Link] (5 responses)

The problem there, of course, is that each tag has a different email address in it, so each looks like a different tester.

Syzbot

Posted Jul 26, 2018 7:27 UTC (Thu) by dvyukov (guest, #57055) [Link]

There is no problem in Computer Science that can't be solved with another pipe through sed :)

Syzbot

Posted Jul 27, 2018 9:31 UTC (Fri) by andy_shev (subscriber, #75870) [Link] (3 responses)

Perhaps poor parser, + is a special symbol in the e-mail addresses, so, everything after + and before @ should be dropped. Voilà — it's the same address!

Syzbot

Posted Jul 27, 2018 9:42 UTC (Fri) by gevaerts (subscriber, #21521) [Link] (2 responses)

+ is not a special symbol in email addresses.

Syzbot

Posted Jul 27, 2018 20:25 UTC (Fri) by flussence (guest, #85566) [Link] (1 responses)

You're technically correct, but nobody speaks Þe Olde RFC822 on the present internet. The addr+suffix convention is already widespread enough that several MTAs understand it.

+ suffix

Posted Jul 27, 2018 20:57 UTC (Fri) by corbet (editor, #1) [Link]

MTA's understand it, and ...some developers take advantage of that to use the + suffix to indicate the employer who sponsored the work, so gitdm can't just trim it. A special case would have to be made for syzbot in particular; I may get there but it's not the highest priority at the moment.

Statistics from the 4.18 development cycle

Posted Jul 25, 2018 9:36 UTC (Wed) by liam (guest, #84133) [Link]

[...]including the since-reverted AIO polling interface.


Speaking of, I hope Christoph makes further attempts to get it passed the Doom of AIO. This being part of one of the last conspicuous features missing in the kernel.


Copyright © 2018, 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