Jaderné noviny – 7. 6. 2018: Statistiky vývojového cyklu 4.17

24. 6. 2018 | David Kolibáč | Jaderné noviny | 2299×

Stav vydání jádra. Citáty týdne: Linus Torvalds a Dan Carpenter. Statistiky vývojového cyklu 4.17.

Stav vydání jádra

Kernel release status. Jonathan Corbet. 6. června 2018

Současné jádro je 4.17 vydané 3. června. Linus v oznámení poznamenal, že prozatím odolal pokušení změnit hlavní číslo verze: „Ne, nenazval jsem to 5.0, i když počty objektů v gitu se nacházely ve správné konstelaci. Stane se tak v ne _příliš_ vzdálené budoucnosti a skripty pro vydání na kernel.org prý na to jsou připraveny, ale zatím mi nepřišlo, že by k tomu byl skutečný důvod.“

Stěžejní vlastnosti tohoto vydání zahrnují vylepšené odhadování zátěže v plánovači CPU, surové sledovací body BPF, podporu lazytime v souborovém systému XFS, úplnou podporu protokolu TLS přímo v jádře, vyvolávání histogramů při trasování, omezení dopadů zatím posledních variant Spectre a samozřejmě odstranění podpory osmi procesorových architektur, jimž se nedostávalo pozornosti.

Stabilní aktualizace: 4.16.14, 4.14.48 a 4.9.106 byly vydány 5. června. Následovaly aktualizace 4.9.107 a 4.4.136 v době článku revidované a 7. června vydané.

Citáty týdne

Quotes of the week. Jonathan Corbet. 6. června 2018

„Argumentace standardy“ by se v oboru programování měla brát za potenciální logický klam. Standardy mají své místo, ale mají i svá úskalí.

—Linus Torvalds

Jako jaderný vývojář se někdy můžeš cítit, jakoby všichni čekali zrovna na tebe. Není tomu tak. Není kam spěchat. Tvůj e-mail si přečteme, odpovíme na něj a pak se z našich hlav vypaří jako pára nad hrncem. Nikdy není kam spěchat a nikdy ti nedojde práce, kterou bys mohl dělat.

—Dan Carpenter

Statistiky vývojového cyklu 4.17

Statistics from the 4.17 kernel development cycle. Jonathan Corbet. 31. května 2018

Zdá se, že jádro 4.17 je na nejlepší cestě k vydání 3. června, nedojde-li na poslední chvíli k nečekanému překvapení. Přišel tedy čas jako obvykle se podívat na některé statistiky tohoto cyklu. Ačkoliv 4.17 je v zásadě obvyklý cyklus, v jednom významném směru vybočuje: je to všehovšudy třetí vydání jádra, které je menší (ve smyslu počtu řádek kódu) než jeho předchůdce.

Jádro 4.17, počítáno po vydání 4.17-rc7, přineslo 13 453 neslučovacích sad změn od 1 696 vývojářů. Z těchto vývojářů v tomto cyklu 256 přispělo do jádra vůbec poprvé – to je nejméně nováčků od vydání 4.8 (tehdy to bylo 237). Počet sad změn je téměř shodný s vydáním 4.16 (které jich mělo 13 630), ale vývojářů od předchozího cyklu z tehdejších 1 774 ubylo.

Tito vývojáři přidali 690 tisíc řádek kódu, ale 869 tisíc jich odebrali, což činí celkový úbytek téměř 180 tisíc řádek. Hlavní příčinou tohoto úbytku je samozřejmě odstranění osmi nepoužívaných architektur. Stojí za zmínku, že navzdory odebrání takového množství kódu je jádro 4.17 o kousek větší než vydání 4.15.

Nejaktivnějšími vývojáři tentokrát byli:

Nejaktivnější vývojáři cyklu 4.17

Podle počtu sad změn Kuninori Morimoto 245 1,8 % Kirill Tkhai 160 1,2 % Arnd Bergmann 148 1,1 % Chris Wilson 147 1,1 % Colin Ian King 133 1,0 % Alexandre Belloni 124 0,9 % Rex Zhu 122 0,9 % Dominik Brodowski 119 0,9 % Christian König 119 0,9 % Mauro Carvalho Chehab 106 0,8 % Ajay Singh 102 0,8 % Ville Syrjälä 100 0,7 % Arnaldo Carvalho de Melo 99 0,7 % Geert Uytterhoeven 94 0,7 % Hans de Goede 86 0,6 % Masahiro Jamada 83 0,6 % Eric Dumazet 77 0,6 % Gustavo A. R. Silva 72 0,5 % Fabio Estevam 72 0,5 % Linus Walleij 71 0,5 %

Podle počtu změněných řádek Arnd Bergmann 315103 22,9 % Jesper Nilsson 100033 7,3 % Greg Kroah-Hartman 81362 5,9 % Feifei Xu 52509 3,8 % David Howells 40705 3,0 % Tom St Denis 32968 2,4 % James Hogan 31998 2,3 % Anirudh Venkataramanan 18937 1,4 % Kuninori Morimoto 16175 1,2 % Corentin Labbe 15265 1,1 % John Crispin 13188 1,0 % Jasunari Takiguči 12983 0,9 % Gilad Ben-Yossef 12426 0,9 % Greentime Hu 11690 0,8 % Rex Zhu 11458 0,8 % Erik Schmauss 10980 0,8 % Jacopo Mondi 10842 0,8 % Harry Wentland 10198 0,7 % Simon Horman 9179 0,7 % Eric Biggers 8626 0,6 %

Kuninori Morimoto přispěl 245 patchi, z toho byly skoro všechny věnovány přejmenovávání v subsystému audio ovladačů ALSA. Kirill Thkai zapracoval na větším využívání paralelismu v síťovém subsystému. Arnd Bergmann odstranil většinu kódu starých architektur a pročistil toho více napříč jádrem, zabýval se také problémem roku 2038. Chris Wilson udělal velký kus práce na grafickém ovladači Intel i915. Colin Ian King přispěl skupinou úklidových patchů a oprav překlepů.

Sloupci „Podle počtu změněných řádek“ dominuje Bergmann a Jesper Nilsson (který odstranil architekturu Cris). Greg Kroah-Hartman odstranil kus kódu ze staging (a to včetně úctyhodného kódu ovladačů infračerveného rozhraní IRDA), Feifei Xu přidal další definice AMD GPU a David Howells odstranil architekturu mn10300 a přispěl svým dílem práce na souborových systémech.

Práce v cyklu 4.17 podporovalo 241 známých společností, z toho nejaktivnější byly:

Nejaktivnější zaměstnavatelé cyklu 4.17

Podle počtu sad změn Intel 1392 10,3 % (žádný) 977 7,3 % Red Hat 870 6,5 % (neznámý) 756 5,6 % AMD 754 5,6 % IBM 564 4,2 % Renesas Electronics 559 4,2 % Linaro 527 3,9 % Google 448 3,3 % Mellanox 405 3,0 % SUSE 400 3,0 % Bootlin 330 2,5 % Samsung 268 2,0 % Oracle 267 2,0 % Huawei Technologies 244 1,8 % Odin 232 1,7 % ARM 222 1,7 % (konzultanti) 201 1,5 % Canonical 188 1,4 % Code Aurora Forum 181 1,3 %

Podle počtu změněných řádek Linaro 338103 24,6 % AMD 138729 10,1 % Axis Communications 100396 7,3 % Intel 84613 6,2 % Linux Foundation 81678 5,9 % Red Hat 71152 5,2 % Renesas Electronics 42565 3,1 % (žádný) 35960 2,6 % Imagination Technologies 32000 2,3 % IBM 25841 1,9 % ARM 23906 1,7 % (neznámý) 22646 1,6 % Google 21390 1,6 % BayLibre 20931 1,5 % Mellanox 19081 1,4 % Bootlin 16256 1,2 % (konzultanti) 15353 1,1 % Sony 14029 1,0 % Fon 13188 1,0 % Samsung 12823 0,9 %

Jako obvykle je v tomto výčtu jen málo překvapení.

Štítek Reviewed-by vznikl za účelem ukázat na ty, kteří revidují kód před začleněním do jádra. V praxi se ale používá jen sporadicky, a tak není dobrým ukazatelem, kdo se vlastně na kód dívá. Přesto může stát za to se na něj podívat (a lidé si stěžují, když ho přeskočíme). Následuje tedy seznam nejvíce označovaných recenzentů a vedle něj výčet neautorských podpisů (což jsou také ukazatele revidování patchů):

Nejaktivnější recenzenti cyklu 4.17

Štítky Reviewed-by („revidoval“) Alex Deucher 213 4,5 % Rob Herring 192 4,1 % Tony Cheng 123 2,6 % Geert Uytterhoeven 108 2,3 % Andrew Morton 102 2,2 % Andy Shevchenko 94 2,0 % Christian König 83 1,8 % Chris Wilson 69 1,5 % Daniel Vetter 64 1,4 % Laurent Pinchart 57 1,2 % Sebastian Reichel 56 1,2 % Harry Wentland 56 1,2 % Johannes Thumshirn 55 1,2 % Hannes Reinecke 55 1,2 % Christoph Hellwig 53 1,1 % Guenter Roeck 51 1,1 % Simon Horman 48 1,0 % Darrick J. Wong 45 1,0 % David Štěrba 43 0,9 % Ido Schimmel 42 0,9 %

Neautorské podpisy David S. Miller 1378 10,9 % Greg Kroah-Hartman 876 7,0 % Alex Deucher 640 5,1 % Mark Brown 537 4,3 % Mauro Carvalho Chehab 390 3,1 % Andrew Morton 335 2,7 % Ingo Molnar 318 2,5 % Arnaldo Carvalho de Melo 213 1,7 % Michael Ellerman 210 1,7 % Herbert Xu 209 1,7 % Jens Axboe 201 1,6 % Martin K. Petersen 198 1,6 % Kalle Valo 155 1,2 % Thomas Gleixner 153 1,2 % David Štěrba 151 1,2 % Jason Gunthorpe 134 1,1 % Jeff Kirsher 133 1,1 % Simon Horman 128 1,0 % Doug Ledford 123 1,0 % Shawn Guo 121 1,0 %

Na rozdíl od mnoha správců přidává Alex Deucher štítek Reviewed-by mnoha patchům, které aplikuje ve svém repozitáři, tudíž se objevuje v obou seznamech. Rob Herring revidoval širokou škálu patchů vesměs mířících na asociace zařízení–strom a související problematiku; tyto patche obvykle aplikoval někdo jiný. Geert Uytterhoeven reviduje patche od docela hodně autorů, ale obvykle není tím správcem, který by je aplikoval.

Andrew Morton se dívá na mnohem více kódu, než ukazují čísla. Donedávna jeho činnost nebyla vyjádřena štítky Reviewed-by: byl to právě jeden v roce 2008, 14 v roce 2009, po jednom v letech 2012 a 2015. Změnilo se to až letos v lednu, kdy začal tyto štítky přidávat ke mnoha patchům, které aplikuje ve svém vlastním stromě – jde o součást rozsáhlejšího úsilí, aby byly revidovány všechny patche týkající se správy paměti. Morton chápe, co to znamená doopravdy revidovat patch, takže každý z těchto štítků bezpochyby ukazuje na skutečně odvedenou práci.

Tony Cheng je zajímavý a dost možná odlišný případ. Jde o zaměstnance AMD a podle všeho jsou jeho štítky Reviewed-by bez výjimky přiřazeny patchům dalších vývojářů z AMD, přičemž samotné recenze se nevyskytují ve veřejných e-mailových konferencích. Štítky Reviewed-by přidává i ke svým vlastním patchům, které bývají malé a bývá jich poměrně málo (příklad). Ostatní vývojáři se často na štítky Reviewed-by od lidí ze stejné firmy jako autor patche dívají podezřívavě, zvláště když recenze probíhají za zavřenými dveřmi. Pravdou je, že interní recenze mohou patřit mezi ty nejdůkladnější a nejnáročnější – nebo také mohou být pouze symbolické. Tak jako tak ale přidávání štítků Reviewed-by k vlastním patchům není běžná praxe.

Sloupec věnovaný podpisům pochopitelně ukazuje, kteří správci přijímají nejvíce patchů. Nezaručuje, že správce všechny tyto patche před aplikací zkontroloval, ale správci by měli zajistit, že to někdo udělal. Každopádně podepsání mnoha patchů ukazuje na nějakou odvedenou práci.

Zdá se, že štítky Reviewed-by se používají stále častěji. Časem by to mohlo pomoci vyjasnit, kolik recenzních prací v jádře probíhá. Zatím jsou ale data nekonzistentní a mají chyby. Je to lepší než nic, ale k úplnému obrázku to má daleko. Nakonec mají všechny tyto metriky uvedené v tomto článku daleko k dokonalosti, aby říkaly, kdo se opravdu podílí na chodu vývoje jádra.

Z těchto čísel je zřejmá jedna věc: v jádře je nadále rušno – je to jeden z rušnějších projektů v oblasti vývoje softwaru na světě. Nezdá se, že by se v dohledné době měl vývoj zpomalit.

       

Komentáře