|
|
Subscribe / Log in / New account

Who wrote 2.6.37

By Jonathan Corbet
December 30, 2010
The 2.6.37 development cycle is coming to a close; that must mean that it is time to look at the development statistics for this release. 2.6.37 has been a more active cycle than most, with 11,220 non-merge changesets added (as of 2.6.37-rc7). Only three cycles have seen more changes: 2.6.25 (12,243 changesets), 2.6.29 (11,678), and 2.6.30 (11,989). The relatively slow period since 2.6.33 appears to have come to an end.

The 2.6.36 kernel was unique in that it was actually smaller than its predecessor. 2.6.37 does not continue that trend; some 1,140,000 lines of code were added, and 641,000 lines were removed, for a net growth of 494,000 lines. Most notably, perhaps: the 2.6.37 kernel includes patches from 1,250 developers, the highest ever. The development community, clearly, is active and growing.

The most active contributors this time around were:

Most active 2.6.37 developers
By changesets
Chris Wilson2111.9%
Greg Kroah-Hartman1711.5%
Eric Dumazet1661.5%
Johannes Berg1491.3%
Thomas Gleixner1471.3%
Paul Mundt1401.2%
Mauro Carvalho Chehab1221.1%
Joe Perches1071.0%
Avi Kivity1050.9%
Mark Brown1000.9%
Hans Verkuil1000.9%
Namhyung Kim1000.9%
Dan Carpenter990.9%
Christoph Hellwig930.8%
Jean Delvare880.8%
Axel Lin880.8%
Daniel Vetter870.8%
Vasiliy Kulikov860.8%
Arnd Bergmann860.8%
Julia Lawall840.7%
By changed lines
Henry Ptasinski1433039.6%
Greg Kroah-Hartman1008616.7%
Vipin Mehta923986.2%
Luis R. Rodriguez716364.8%
Mark Brown536903.6%
David Cross481983.2%
Dmitry Kravkov471983.1%
Larry Finger403782.7%
Krishna Gudipati387122.6%
Stefan Richter294342.0%
Stephen Hemminger285041.9%
Rasesh Mody223351.5%
Prashant P. Shah150861.0%
Michael Chan141180.9%
Christian Lamparter135300.9%
Liam Girdwood133240.9%
William Hubbs129710.9%
Vinod Koul129440.9%
Marek Belisko115310.8%
Al Cho100970.7%

These lists feature a combination of old and new names. Chris Wilson got to the top of the by-changesets list by virtue of his work with the Intel graphics drivers. Greg Kroah-Hartman worked on the USB and TTY subsystems, but the bulk of his changes were made to the staging tree, and the new brcm80211 driver in particular. Eric Dumazet has been active all over the networking layer, Johannes Berg has been busy with wireless networking, and Thomas Gleixner rewrote the generic interrupt handling code (among many other things).

Looking at lines changed: Henry Ptasinski arrived at the top of the list through the addition of the brcm80211 driver which, as can be seen, is not a small piece of code. Vipin Mehta added the ath6kl driver to the staging tree, Luis Rodriguez worked on the mac80211 layer and the ath9k driver, and Mark Brown, as always, did a massive amount of work within the ALSA sound layer. It's interesting to note that five of the top ten in this column were mainly involved with the staging tree. One of the others (Krishna Gudipati) submitted a single "driver cleanup patch" which drew Linus's ire at the time - calling a nearly 40,000-line patch a "cleanup" seemed like a bit much.

A minimum of 193 employers supported work on the 2.6.37 kernel; the top supporters were:

Most active 2.6.37 employers
By changesets
(None)186416.6%
Red Hat126511.3%
(Unknown)9468.4%
Intel7466.6%
Novell6445.7%
IBM4474.0%
Oracle2722.4%
Texas Instruments2502.2%
Nokia2352.1%
Renesas Technology2101.9%
Samsung2051.8%
Broadcom1721.5%
Societe Française de Radiotelephone1661.5%
AMD1631.5%
(Consultant)1551.4%
Pengutronix1541.4%
(Academia)1461.3%
Wolfson Micro1291.1%
Google1281.1%
Fujitsu1271.1%
By lines changed
Broadcom24674916.4%
(None)18883012.6%
Atheros16569311.0%
Novell1274768.5%
Wolfson Micro1001966.7%
Brocade640124.3%
(Unknown)593254.0%
Red Hat590003.9%
Intel538933.6%
Cypress Semiconductor482413.2%
Vyatta307202.0%
Texas Instruments228741.5%
Samsung167711.1%
IBM165671.1%
Nokia147311.0%
Renesas Technology144881.0%
(Consultant)135920.9%
Slimlogic Ltd133240.9%
ST Ericsson130560.9%
Chelsio107700.7%

Looking at the by-changesets column, the situation looks mostly like business as usual. Red Hat remains, by far, the largest corporate contributor to the kernel. On the lines-changed side, instead, Red Hat had to settle for eighth place behind companies which, for the most part, have contributed a lot of driver code.

It has been some time since we looked at the reviewing and testing of code. In the 2.6.37 development cycle, the developers with the most Reviewed-by tags were:

Most active 2.6.37 reviewers
Ingo Molnar8612.6%
Christoph Hellwig375.4%
Mike Christie294.3%
Michael Chan263.8%
Josh Triplett223.2%
Daniel Vetter213.1%
Chuck Lever213.1%
H. Peter Anvin202.9%
Konrad Rzeszutek Wilk192.8%
Alex Elder172.5%
Luciano Coelho162.3%
Suresh Jayaraman162.3%
Swen Schillig162.3%
Michal Marek152.2%
Jeff Layton152.2%
Sam Ravnborg152.2%
Wu Fengguang121.8%
Francisco Jerez121.8%
KOSAKI Motohiro121.8%
Christoph Lameter111.6%

As always, there is only so much that can be learned from these numbers; the bulk of all patch reviews do not lead to the addition of a Reviewed-by tag. On the other hand, there has been some real social pressure to credit users who test patches and report bugs:

Most credited 2.6.37 reporters and testers
Reported-by credits
Stephen Rothwell244.2%
Randy Dunlap213.7%
Linus Torvalds193.3%
Ingo Molnar152.6%
Guennadi Liakhovetski101.8%
Jonathan Cameron71.2%
David Brownell71.2%
Sitsofe Wheeler61.1%
Andrew Morton61.1%
Dan Rosenberg61.1%
Jiri Slaby50.9%
Ben Greear50.9%
Eric Dumazet40.7%
Daniel Vetter40.7%
David S. Miller40.7%
Robin Holt40.7%
Andi Kleen40.7%
Dan Carpenter40.7%
Sachin Sant40.7%
Dr. David Alan Gilbert40.7%
Tested-by: credits
Wolfram Sang154.1%
Luciano Coelho113.0%
Kevin Hilman102.7%
Jeff Pieper102.7%
Will Deacon92.5%
Caglar Akyuz82.2%
Michael Williamson82.2%
Emil Tantilov71.9%
Randy Dunlap61.6%
Stephen Ko61.6%
Ben Greear61.6%
Eric Benard51.4%
Tuomas Katila51.4%
Maxim Levitsky51.4%
Ingo Molnar41.1%
Juuso Oikarinen30.8%
Jason Wessel30.8%
Kuninori Morimoto30.8%
Rabin Vincent30.8%
Ben Gardiner30.8%

All told, there were 568 Reported-by and 366 Tested-by tag lines found in patches merged for 2.6.37. We are, it seems, slowly getting better at recognizing the people who are doing this crucially important work.

In summary, the kernel development process continues to look healthy. We have a great deal of activity from an increasing number of developers, while, it seems, keeping a reasonable lid on the number of regressions introduced. Whether the high patch rate will continue into 2.6.38 remains to be seen; as of this writing, there are just under 5,000 changes in linux-next. Unless the subsystem maintainers put more work into linux-next in the near future, the next development cycle could be relatively slow.

Index entries for this article
KernelReleases/2.6.37


to post comments

Who wrote 2.6.37

Posted Dec 30, 2010 20:41 UTC (Thu) by mtaht (subscriber, #11087) [Link]

I would also enjoy a ( http://code.google.com/p/gource/ ) gource video and musical rendition of the latest commits, showing the attentions and focuses of the developers, reporters, and testers, much like this one: http://www.thealphablenders.com/2010/10/new-zealand-open-......

Could some of the output of your lovely, yet tabular, data crunching feed into that?

Lies, damned lies, and statistics

Posted Dec 31, 2010 10:35 UTC (Fri) by patrick_g (subscriber, #44470) [Link] (2 responses)

I really would like to know why there is such discrepancy between the LWN kernel stats and Remword kernel stats. In this article you wrote: "there were 568 Reported-by and 366 Tested-by tag lines found in patches merged for 2.6.37".
In this page (generated Dec. 30) I can see 458 Reported-by tags and in this page I can see 289 Tested-by tags.

The gap is huge! All these numbers are based on the same Git tree and should be the same.

Lies, damned lies, and statistics

Posted Dec 31, 2010 23:15 UTC (Fri) by geofft (subscriber, #59789) [Link] (1 responses)

Intriguing, I get a bunch of different numbers:

opus:/home/geofft/linux-2.6 geofft$ git log v2.6.36..v2.6.37-rc7 | grep Reviewed-by | wc -l
686
opus:/home/geofft/linux-2.6 geofft$ git log --first-parent v2.6.36..v2.6.37-rc7 | grep Reviewed-by | wc -l
17

Lies, damned lies, and statistics

Posted Jan 6, 2011 14:35 UTC (Thu) by SEJeff (guest, #51588) [Link]

Just because I'm OCD it, try grep -c instead of grep ... | wc -l

Changes by kernel area

Posted Dec 31, 2010 11:21 UTC (Fri) by epa (subscriber, #39769) [Link] (3 responses)

I see that by lines changed, Broadcom is the top contributor. But I suspect their work is only in device drivers. Can the contributions be broken down by area somehow - perhaps something as crude as top-level directory within the source tree?

Changes by kernel area

Posted Dec 31, 2010 14:09 UTC (Fri) by dwmw2 (subscriber, #2063) [Link] (2 responses)

I see that by lines changed, Broadcom is the top contributor. But I suspect their work is only in device drivers.
And device drivers that have been accused of being an ocular carcinogen, at that. A decent driver would probably have been a lot fewer lines; especially if it'd added to the existing b43 driver infrastructure instead of reinventing a bunch of stuff for itself.

These statistics should probably exclude the drivers/staging directory of the kernel. In fact, I think it would be better if Linus' tree didn't include the staging crap at all, and it was only in an external repository.

Changes by kernel area

Posted Jan 2, 2011 13:16 UTC (Sun) by wsa (guest, #52415) [Link]

+1 (again) for skipping staging in the stats

Changes by kernel area

Posted Mar 15, 2011 23:05 UTC (Tue) by josh (subscriber, #17465) [Link]

Excluding staging would exclude both the original contributors of the drivers and those who help clean them up. Excluding the latter seems uncalled for. Excluding the former denigrates the attempt to contribute at all. That a driver needs work does not make it worthless as a contribution.

As long as staging lives in the main kernel tree (which has made sense so far), the statistics should continue to count it.

Who wrote 2.6.37

Posted Jan 8, 2011 20:56 UTC (Sat) by maks (guest, #32426) [Link]

Please would it be possibled to add Developer summary for the `Acked-by:' tag next to Reviewed-by stats. (:

Thank you.


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