|
|
Subscribe / Log in / New account

Who wrote - and approved - 2.6.22

The 2.6.22 kernel is getting closer to its final state with its official release likely to happen near the end of this month. Patches are still being added to the mainline repository, but things have stabilized enough that it makes sense to take a look at where the code came from this time around. Accordingly, your editor has fixed up his scripts and cranked through the changesets added in this kernel development cycle.

As of this writing, just over 6,000 changesets have been accepted for 2.6.22. Those patches were contributed by 885 different developers, added 494,000 lines, and deleted 241,000 other lines (without counting renames, which would otherwise increase both numbers by about 60,000 lines). That makes 2.6.22 a large change relative to its immediate predecessors:

ReleaseDevelopersChangesets Lines
added
Lines
removed
2.6.207414983286,000 160,000
2.6.218425349343,000 199,000
2.6.22-rc4+8856093 494,000241,000

Here's the top contributors of those changes:

Most active 2.6.22 developers
By changesets
David S. Miller1753.0%
Kristian Høgsberg1091.9%
Stephen Hemminger861.5%
Arnaldo Carvalho de Melo821.4%
Andrew Morton791.3%
Stefan Richter791.3%
Christoph Lameter771.3%
Patrick McHardy761.3%
Jean Delvare751.3%
Dmitry Torokhov701.2%
Stephen Rothwell681.2%
Paul Mundt661.1%
David Brownell651.1%
Jeff Dike631.1%
Alan Cox601.0%
Andi Kleen591.0%
Antonino Daplas581.0%
Adrian Bunk581.0%
Tejun Heo571.0%
Russell King571.0%
By changed lines
Bryan Wu7759412.9%
David Howells233103.9%
Marcelo Tosatti223513.7%
Patrick McHardy217463.6%
Jiri Benc183283.0%
Hans Verkuil136832.3%
David S. Miller135952.3%
Roland Dreier122472.0%
Artem B. Bityutskiy120652.0%
Kristian Høgsberg111531.9%
Robert P. J. Day75541.3%
Christoph Lameter73781.2%
Andrew Victor66381.1%
Mike Frysinger63131.0%
David Brownell60331.0%
Michael Chan58511.0%
Andi Kleen54310.9%
David Gibson53210.9%
Nobuhiro Iwamatsu52960.9%
Mark Fasheh49210.8%

Bryan Wu makes it to the top of the list of contributors (by lines changed) by virtue of being the person to contribute support for the Blackfin architecture. David Howells contributed the AF_RXRPC and AFS filesystem work; Marcelo Tosatti wrote the OLPC "Libertas" wireless driver, and Jiri Benc's name appears on the mac80211 stack.

When broken down by employer, the (approximate, as always) numbers come out like this:

Most active 2.6.22 employers
By changesets
(Unknown)176630.2%
Red Hat72012.3%
IBM60110.3%
Novell4117.0%
(None)2454.2%
Intel2033.5%
Oracle1272.2%
(Consultant)1192.0%
Linux Foundation1162.0%
Google1111.9%
SGI931.6%
Nokia831.4%
Freescale801.4%
Astaro761.3%
XenSource561.0%
MontaVista561.0%
Qumranet550.9%
HP530.9%
QLogic520.9%
Analog Devices490.8%
By lines changed
(Unknown)13016421.6%
Red Hat10462717.4%
Analog Devices8456114.0%
Novell413666.9%
IBM336295.6%
Astaro220653.7%
(None)200973.3%
(Consultant)154032.6%
Linutronix135852.3%
Intel122882.0%
Cisco122802.0%
Oracle104821.7%
Freescale101161.7%
SGI86391.4%
Nokia73281.2%
SANPeople70451.2%
Broadcom59521.0%
MontaVista58101.0%
Linux Foundation57461.0%
Atmel52200.9%

One thing which jumps out here is that the amount of code contributed by developers known to be working on their own time has dropped; 2.6.22 will be one of the most corporate kernels yet.

Looking at the developers who put Signed-off-by lines onto patches yields some interesting results. If one tabulates all 12,678 signoffs in 2.6.22, the results look like this:

Developers with the most signoffs (total 12678)
Andrew Morton141511.2%
Linus Torvalds129910.2%
David S. Miller8146.4%
Paul Mackerras3813.0%
Jeff Garzik3442.7%
Andi Kleen2522.0%
Greg Kroah-Hartman2361.9%
Mauro Carvalho Chehab2361.9%
Stefan Richter2101.7%
Russell King1891.5%
James Bottomley1761.4%
Jaroslav Kysela1451.1%
Takashi Iwai1311.0%
Len Brown1261.0%
Kristian Høgsberg1261.0%
Patrick McHardy1170.9%
Jean Delvare1100.9%
Roland Dreier1090.9%
Antonino Daplas1060.8%
Dmitry Torokhov1050.8%

All authors must sign off on their code. Additionally, any maintainer who passes a patch up toward the mainline adds a signoff indicating that he or she believes the code is legitimate and suitable for inclusion. If one excludes signoffs by the author of each patch, the remaining 7,000 signoffs are (almost) all by people through whom the code has passed (a few of them are by additional authors of the patch). Those adding non-author signoffs can thus be thought of as the gatekeepers through whom each patch must pass. Non-author signoffs break down like this:

Non-author signoffs (total 7028)
Andrew Morton133619.0%
Linus Torvalds127918.2%
David S. Miller6409.1%
Paul Mackerras3715.3%
Jeff Garzik3224.6%
Greg Kroah-Hartman2223.2%
Mauro Carvalho Chehab2163.1%
Andi Kleen1932.7%
James Bottomley1632.3%
Jaroslav Kysela1422.0%
Russell King1321.9%
Stefan Richter1311.9%
Len Brown1151.6%
John W. Linville851.2%
Roland Dreier851.2%
Takashi Iwai791.1%
Martin Schwidefsky540.8%
David Woodhouse530.8%
Ralf Baechle480.7%
Antonino Daplas480.7%

In summary, 80% of the patches merged into the mainline kernel passed through the twenty developers listed above. One can take another step, and look at the number of non-author signoffs by employer:

Non-author signoffs by employer
Google133819.0%
Linux Foundation128118.2%
Red Hat124617.7%
Novell70010.0%
(Unknown)6609.4%
IBM5537.9%
(None)2934.2%
Intel1932.7%
SteelEye1632.3%
Cisco851.2%
MIPS Technologies480.7%
Nokia420.6%
Astaro410.6%
Analog Devices350.5%
QLogic350.5%
Cendio320.5%
SGI280.4%
NetApp280.4%
(Consultant)230.3%
Oracle220.3%

The bottom line: while Linux kernel development is a highly distributed activity, the work of several hundred developers is channeled through a surprisingly small number of individuals, and an even smaller number of companies on its way into the mainline.

Index entries for this article
KernelDevelopment model/Contributor statistics
KernelReleases/2.6.22


(Log in to post comments)

Who wrote - and approved - 2.6.22

Posted Jun 14, 2007 2:45 UTC (Thu) by pr1268 (subscriber, #24648) [Link]

A positive perspective I see is that the 'corporate' bias in the recent kernels (and a list of the very companies supporting this development) as a sign that Linux is indeed a valid, viable, and stable alternative to proprietary operating system (and related) software.

Either that, or (as I've stated before) the companies who throw resources1 at Linux are the same companies who have the most to gain from its continued development.

1Perhaps "throw resources" sounds a little harsh, but I mean it in the nicest sense of the term.

Who wrote - and approved - 2.6.22

Posted Jun 14, 2007 8:37 UTC (Thu) by xav (guest, #18536) [Link]

I think having a lot of "corporate" reviewers is a good thing. Doing code
review isn't always the most interesting thing to do, just developing is
far more fun. So having a lot of people payed by corps to do the tedious
work of reviewing code, while independant developers continue to write
their crappy drivers is IMHO a very good equilibrium point.

Xav

Who wrote - and approved - 2.6.22

Posted Jun 14, 2007 20:48 UTC (Thu) by elanthis (guest, #6227) [Link]

Silly question, but how do 20% of patches get into mainline without passing through Linux (or even Andrew) ?

Is it just that Linus doesn't sign-off when he merges patches, so the sign-offs from him are those he reviewed personally versus those reviewed by some other "trusted lieutenant" not on that top 20 list?

Who wrote - and approved - 2.6.22

Posted Jun 15, 2007 0:23 UTC (Fri) by aegl (guest, #37581) [Link]

Yes. When Linus does a "pull" from a subsystem maintainer's tree, none of the changesets get his sign-off.

Who wrote - and approved - 2.6.22

Posted Jun 15, 2007 16:25 UTC (Fri) by aegl (guest, #37581) [Link]

It isn't a sure bet that a change-set with a sign-off from Linus has actually been reviewed by him either ... when Andrew Morton sends a "patch-bomb" with several hundred e-mail messages moving code from -mm to Linus, chances are that Linus does not sit down and read each and every one of them. But they will all get a "Signed-off-by" from Linus because the GIT tool that applies patches from a mailbox adds a sign-off to each commit that is applied that way.

If Andrew started using a GIT tree to transfer to Linus, then many of those "Signed-off-by: Linus" commits would just end the trail-of-blame with Andrew's sign-off.

This is sort of documented in Documentation/SubmittingPatches: "The Signed-off-by: tag indicates that the signer was involved in the development of the patch, or that he/she was in the patch's delivery path.". "In the patches delivery path" does not necessarily mean "Reviewed by".

So some care is needed when interpreting this data since some quite large parts are affected by the mechanisms used to move changes around.

Who wrote - and approved - 2.6.22

Posted Jun 16, 2007 13:05 UTC (Sat) by hingo (guest, #14792) [Link]

Surely Andrew Morton uses GIT to transfer code to Linus?

If the opposite was true, then we would have just uncovered a major scoop, that Linus doesn't really do anything at all anymore, just sits around taking credit for not coding anything, not approving anything.

Who wrote - and approved - 2.6.22

Posted Jun 16, 2007 18:02 UTC (Sat) by dlang (guest, #313) [Link]

the facts are in between these two.

as I understand it, Andrew doesn't normally use git to send patches to Linus, but at the same time signed off by indicates that the person is vouching for the patch. I don't think that it's done automaticaly by the git scripts (remember, they are used by projects that don't do 'signed off by' lines)

Linus has said many times that his job is more to be a gatekeeper then a coder nowdays, but he still writes and modifies code.

David Lang

Who wrote - and approved - 2.6.22

Posted Jun 17, 2007 9:25 UTC (Sun) by hingo (guest, #14792) [Link]

Ok. What I was really wondering was that Andrew Signs off 0.8% more patches than Linus, so mathematically it's impossible to claim that some script by Linux automatically signs off everything Andrew has signed?

Who wrote - and approved - 2.6.22

Posted Jun 15, 2007 21:23 UTC (Fri) by jkm (guest, #14176) [Link]

It's fairly misleading to see Google listed at the top of non-author signoffs by employer, considering only two of the signoffs came from someone other than Andrew.

Who wrote - and approved - 2.6.22

Posted Jun 20, 2007 21:53 UTC (Wed) by roelofs (guest, #2599) [Link]

It's fairly misleading to see Google listed at the top of non-author signoffs by employer, considering only two of the signoffs came from someone other than Andrew.

It's hardly any different for Linus/Linux Foundation, is it?

Keep in mind that there is at least a vague correlation between number of signoffs performed and (paid) effort expended. Red Hat just spreads theirs around a bit more. ;-)

Greg


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