Xen Project 4.8 Acknowledgements: Difference between revisions
Lars.kurth (talk | contribs) |
Lars.kurth (talk | contribs) |
||
(23 intermediate revisions by 2 users not shown) | |||
Line 1: | Line 1: | ||
Note that the Acknowledgements have been created based on 4.8.0-rc8, which was the final RC for Xen 4.8.0. |
|||
== Overview == |
== Overview == |
||
Line 8: | Line 8: | ||
{|class="zebra" |
{|class="zebra" |
||
!style="width: |
!style="width: 170px;"|Attribute |
||
!style="width: 170px;"|[[Xen_Project_4.7_Acknowledgements|4.7 Hypervisor Total]] <sup>[ 1 ]</sup> |
!style="width: 170px;"|[[Xen_Project_4.7_Acknowledgements|4.7 Hypervisor Total]] <sup>[ 1 ]</sup> |
||
!style="width: 170px; color: blue;"|4.8 Hypervisor Total <sup>[ 1 ]</sup> |
!style="width: 170px; color: blue;"|4.8 Hypervisor Total <sup>[ 1 ]</sup> |
||
Line 22: | Line 22: | ||
! |
! |
||
|- |
|- |
||
!colspan="6"|Development Velocity |
|||
|'''Changes/Month''' |
|||
|225 |
|||
|style="color: blue;"|'''235''' |
|||
| |
|||
| |
|||
! |
|||
|- |
|||
|'''Changes/Dev Month <sup>[ 6 ]</sup>''' |
|||
|291 |
|||
|style="color: blue;"|'''305''' |
|||
| |
|||
| |
|||
! |
|||
|- |
|- |
||
|'''Changesets''' |
|'''Changesets''' |
||
Line 41: | Line 29: | ||
|967 |
|967 |
||
|278 |
|278 |
||
! |
!321 |
||
|- |
|- |
||
|'''Series (=Features)''' <sup>[ 5 ]</sup> |
|'''Series (=Features)''' <sup>[ 5 ]</sup> |
||
|1025 |
|1025 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''575''' |
||
| |
| |
||
| |
| |
||
! |
! |
||
|- |
|||
|'''Average Size of Feature''' |
|||
|1.84 |
|||
|style="color: blue;"|'''2.17''' <span style="color:green">(+15%)</span> |
|||
| |
|||
| |
|||
! |
|||
|- |
|||
|'''Changes / Day''' |
|||
|7.5 |
|||
|style="color: blue;"|'''7.7''' <span style="color:green">(+3%)</span> |
|||
| |
|||
| |
|||
! |
|||
|- |
|||
|'''Changes / Dev Day''' <sup>[ 6 ]</sup> |
|||
|9.6 |
|||
|style="color: blue;"|'''10.0''' <span style="color:green">(+4%)</span> |
|||
| |
|||
| |
|||
! |
|||
|- |
|||
|'''Features / Day''' <sup>[ 5 ]</sup> |
|||
|4.0 |
|||
|style="color: blue;"|'''3.6''' <span style="color:red">(-11%)</span> |
|||
| |
|||
| |
|||
! |
|||
|- |
|||
!colspan="6"|Code Review Statistics |
|||
|- |
|- |
||
|'''Review Comments''' <sup>[ 5 ]</sup> |
|'''Review Comments''' <sup>[ 5 ]</sup> |
||
|11628 |
|11628 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''6950''' |
||
| |
|||
| |
|||
! |
|||
|- |
|||
|'''Review Comments / Day''' <sup>[ 5 ]</sup> |
|||
|45.6 |
|||
|style="color: blue;"|'''43.2''' <span style="color:red">(-6%)</span> |
|||
| |
| |
||
| |
| |
||
Line 59: | Line 84: | ||
|'''Comments / Series''' <sup>[ 5 ]</sup> |
|'''Comments / Series''' <sup>[ 5 ]</sup> |
||
|11.3 |
|11.3 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''12.1''' <span style="color:green">(+7%)</span> |
||
| |
| |
||
| |
| |
||
! |
! |
||
|- |
|||
!colspan="6"|Code Churn |
|||
|- |
|- |
||
|'''Lines Added''' |
|'''Lines Added''' |
||
Line 69: | Line 96: | ||
|70833 |
|70833 |
||
|8066 |
|8066 |
||
!18421 |
|||
!TBD |
|||
|- |
|- |
||
|'''Lines Removed''' |
|'''Lines Removed''' |
||
Line 76: | Line 103: | ||
|59502 |
|59502 |
||
|2823 |
|2823 |
||
!13775 |
|||
!TBD |
|||
|- |
|||
!colspan="6"|Engagement |
|||
|- |
|- |
||
|'''Individuals''' |
|'''Individuals''' |
||
Line 83: | Line 112: | ||
|68 |
|68 |
||
|5 |
|5 |
||
! |
!59 |
||
|- |
|- |
||
|'''Employers''' |
|'''Employers''' |
||
Line 90: | Line 119: | ||
|33 |
|33 |
||
|4 |
|4 |
||
! |
!37 |
||
|} |
|} |
||
Line 177: | Line 206: | ||
| Kai Huang || 24 || 22 || 1 |
| Kai Huang || 24 || 22 || 1 |
||
|- |
|- |
||
| Konrad Rzeszutek |
| Konrad Rzeszutek Wilk || 2219 || 1064 || 50 |
||
|- |
|- |
||
| Kyle Huey || 100 || 6 || 2 |
| Kyle Huey || 100 || 6 || 2 |
||
Line 189: | Line 218: | ||
| Luwei Kang || 60 || 1 || 2 |
| Luwei Kang || 60 || 1 || 2 |
||
|- |
|- |
||
| Marek Marczykowski- |
| Marek Marczykowski-Górecki || 45 || 1 || 3 |
||
|- |
|- |
||
| Meng Xu || 13 || 6 || 4 |
| Meng Xu || 13 || 6 || 4 |
||
Line 239: | Line 268: | ||
| Zhi Wang || 321 || 4 || 1 |
| Zhi Wang || 321 || 4 || 1 |
||
|} |
|} |
||
<br> |
|||
== 4.8 Hypervisor Related Code Contributions <sup>[ 3 ]</sup> == |
== 4.8 Hypervisor Related Code Contributions <sup>[ 3 ]</sup> == |
||
Line 273: | Line 304: | ||
!style="min-width: 110px;"|Lines Removed {{Sortable}} |
!style="min-width: 110px;"|Lines Removed {{Sortable}} |
||
!style="min-width: 110px;"|Changesets {{Sortable}} |
!style="min-width: 110px;"|Changesets {{Sortable}} |
||
|- |
|||
| Andrey Grodzovsky || 2 || 4 || 1 |
|||
|- |
|||
| Anthony PERARD || 6 || 7 || 2 |
|||
|- |
|||
| Arnd Bergmann || 3 || 3 || 1 |
|||
|- |
|||
| Bryan Drewery || 1 || 1 || 1 |
|||
|- |
|||
| Ben Chalmers || 88 || 25 || 5 |
|||
|- |
|||
| Bhaktipriya Shridhar || 3 || 25 || 2 |
|||
|- |
|||
| Bob Liu || 96 || 92 || 4 |
|||
|- |
|||
| Manuel Bouyer || 57 || 57 || 2 |
|||
|- |
|||
| Bruce Rogers || 1 || 1 || 1 |
|||
|- |
|||
| Cao Jin || 2 || 3 || 1 |
|||
|- |
|||
| Christoph Hellwig || 0 || 1 || 1 |
|||
|- |
|||
| Christophe Ricard || 19 || 17 || 4 |
|||
|- |
|||
| Dan Williams || 1 || 2 || 1 |
|||
|- |
|||
| David Buches || 9 || 1 || 1 |
|||
|- |
|||
| David S. Miller || 2 || 107 || 3 |
|||
|- |
|||
| David Vrabel || 297 || 411 || 6 |
|||
|- |
|||
| Dmitry Torokhov || 0 || 0 || 1 |
|||
|- |
|||
| Dongli Zhang || 4 || 4 || 2 |
|||
|- |
|||
| Emil Condrea || 661 || 583 || 13 |
|||
|- |
|||
| Eric Blake || 5 || 4 || 2 |
|||
|- |
|||
| Eytan Heidingsfeld || 5 || 0 || 1 |
|||
|- |
|||
| Fabian Frederick || 1 || 1 || 1 |
|||
|- |
|||
| Fam Zheng || 2 || 6 || 2 |
|||
|- |
|||
| Filipe Manco || 30 || 16 || 1 |
|||
|- |
|||
| Gerd Hoffmann || 3 || 0 || 1 |
|||
|- |
|||
| Jan Beulich || 108 || 151 || 21 |
|||
|- |
|||
| Jaromír Dolecek || 14 || 11 || 1 |
|||
|- |
|||
| Jens Axboe || 0 || 0 || 1 |
|||
|- |
|||
| John Baldwin || 2 || 0 || 1 |
|||
|- |
|||
| Jung-uk Kim || 0 || 0 || 1 |
|||
|- |
|||
| Joerg Wunsch || 4 || 1 || 1 |
|||
|- |
|||
| John Snow || 0 || 2 || 1 |
|||
|- |
|||
| Juergen Gross || 616 || 371 || 21 |
|||
|- |
|||
| Kamil Rytarowski || 72 || 5 || 1 |
|||
|- |
|||
| KarimAllah Ahmed || 11 || 84 || 1 |
|||
|- |
|||
| Kevin Wolf || 1 || 1 || 1 |
|||
|- |
|||
| Konstantin Belousov || 8 || 0 || 1 |
|||
|- |
|||
| Robert Elz || 9 || 1 || 1 |
|||
|- |
|||
| Krzysztof Kozlowski || 13 || 13 || 1 |
|||
|- |
|||
| Linus Torvalds || 0 || 0 || 16 |
|||
|- |
|||
| Marc-Andre Lureau || 38 || 41 || 6 |
|||
|- |
|||
| Markus Armbruster || 12 || 14 || 3 |
|||
|- |
|||
| Maxime Villard || 327 || 384 || 12 |
|||
|- |
|||
| Masanobu SAITOH || 71 || 70 || 2 |
|||
|- |
|||
| Muhammad Falak R Wani || 3 || 3 || 1 |
|||
|- |
|||
| Nonaka Kimihiro || 8 || 2 || 1 |
|||
|- |
|||
| Olaf Hering || 30 || 1 || 2 |
|||
|- |
|||
| Owen Smith || 2251 || 3552 || 22 |
|||
|- |
|||
| Ryota Ozaki || 2 || 3 || 1 |
|||
|- |
|||
| Paul Durrant || 11274 || 7362 || 127 |
|||
|- |
|||
| Paul Gortmaker || 1 || 1 || 1 |
|||
|- |
|||
| Paulina Szubarczyk || 162 || 5 || 1 |
|||
|- |
|||
| Peter Maydell || 0 || 0 || 3 |
|||
|- |
|||
| Radim Krčmář || 6 || 0 || 1 |
|||
|- |
|||
| Ross Lagerwall || 43 || 25 || 2 |
|||
|- |
|||
| Roger Pau Monné || 1599 || 87 || 8 |
|||
|- |
|||
| Sebastian Andrzej Siewior || 12 || 22 || 1 |
|||
|- |
|||
| Shannon Zhao || 379 || 165 || 9 |
|||
|- |
|||
| Stefano Stabellini || 2 || 2 || 1 |
|||
|- |
|||
| Thomas Huth || 1 || 1 || 1 |
|||
|- |
|||
| Vitaly Kuznetsov || 34 || 11 || 7 |
|||
|- |
|||
| Vlastimil Babka || 2 || 2 || 1 |
|||
|- |
|||
| Wei Liu || 6 || 0 || 1 |
|||
|- |
|||
| Wei Yongjun || 2 || 11 || 1 |
|||
|} |
|} |
||
<br> |
|||
{{TODOLeft|Insert data}} |
|||
== 4.8 Hypervisor Reviewers <sup> [ 5 ] </sup> == |
|||
This table is sortable : please click on the {{Sortable}} icon besides the column header, to sort by a specific column. Note that in the table below, we '''only''' count review comments by community members other than comments by the author of a patch. We do '''not''' count replies and clarifications by the author of a patch that are made in response to review comments by others. |
|||
{|class="zebra sortable" |
|||
!style="min-width: 200px;"|Name {{Sortable}} |
|||
!style="min-width: 110px;"|Review Comments {{Sortable}} |
|||
|- |
|||
| 105 others (less than 10 review comments each) |
|||
| 300 |
|||
|- |
|||
| Andrew Cooper |
|||
| 651 |
|||
|- |
|||
| Andrii Anisov |
|||
| 21 |
|||
|- |
|||
| Anshul Makkar |
|||
| 16 |
|||
|- |
|||
| Anthony Perard |
|||
| 42 |
|||
|- |
|||
| Bob Liu |
|||
| 20 |
|||
|- |
|||
| Boris Ostrovsky |
|||
| 269 |
|||
|- |
|||
| Corneliu Zuzu |
|||
| 88 |
|||
|- |
|||
| Daniel De Graaf |
|||
| 35 |
|||
|- |
|||
| Daniel Kiper |
|||
| 82 |
|||
|- |
|||
| Dario Faggioli |
|||
| 124 |
|||
|- |
|||
| David Vrabel |
|||
| 77 |
|||
|- |
|||
| Dirk Behme |
|||
| 50 |
|||
|- |
|||
| Doug Goldstein |
|||
| 35 |
|||
|- |
|||
| Feng Wu |
|||
| 65 |
|||
|- |
|||
| George Dunlap |
|||
| 215 |
|||
|- |
|||
| Greg Kurz |
|||
| 10 |
|||
|- |
|||
| Ian Jackson |
|||
| 204 |
|||
|- |
|||
| Jan Beulich |
|||
| 1310 |
|||
|- |
|||
| Joao Martins |
|||
| 26 |
|||
|- |
|||
| Juergen Gross |
|||
| 159 |
|||
|- |
|||
| Julien Grall |
|||
| 644 |
|||
|- |
|||
| Kevin Tian |
|||
| 85 |
|||
|- |
|||
| Konrad Wilk |
|||
| 394 |
|||
|- |
|||
| Kyle Huey |
|||
| 27 |
|||
|- |
|||
| Lars Kurth |
|||
| 40 |
|||
|- |
|||
| Luis R. Rodriguez |
|||
| 22 |
|||
|- |
|||
| Luwei Kang |
|||
| 10 |
|||
|- |
|||
| Meng Xu |
|||
| 11 |
|||
|- |
|||
| Michael Turquette |
|||
| 11 |
|||
|- |
|||
| Mihai Donțu |
|||
| 16 |
|||
|- |
|||
| Olaf Hering |
|||
| 19 |
|||
|- |
|||
| Oleksandr Andrushchenko |
|||
| 48 |
|||
|- |
|||
| Paul Durrant |
|||
| 64 |
|||
|- |
|||
| Paulina Szubarczyk |
|||
| 14 |
|||
|- |
|||
| Peng Fan |
|||
| 14 |
|||
|- |
|||
| Quan Xu |
|||
| 31 |
|||
|- |
|||
| Razvan Cojocaru |
|||
| 97 |
|||
|- |
|||
| Roger Pau Monné |
|||
| 91 |
|||
|- |
|||
| Ross Lagerwall |
|||
| 36 |
|||
|- |
|||
| Samuel Thibault |
|||
| 77 |
|||
|- |
|||
| Sergej Proskurin |
|||
| 125 |
|||
|- |
|||
| Shanker Donthineni |
|||
| 39 |
|||
|- |
|||
| Shannon Zhao |
|||
| 74 |
|||
|- |
|||
| Stefano Stabellini |
|||
| 271 |
|||
|- |
|||
| Tamas Lengyel |
|||
| 157 |
|||
|- |
|||
| Tim Deegan |
|||
| 40 |
|||
|- |
|||
| Wei Liu |
|||
| 643 |
|||
|- |
|||
| Yi Y Sun |
|||
| 20 |
|||
|- |
|||
| Yu Zhang |
|||
| 31 |
|||
|} |
|||
== Footnotes == |
== Footnotes == |
||
Line 284: | Line 600: | ||
</span><br> |
</span><br> |
||
<span id="Foot5"><sup>[ 5 ]</sup> This data was extracted from the [https://kibana.bitergia.com/xen Xen Project Code Review Dashboard]</span><br> |
<span id="Foot5"><sup>[ 5 ]</sup> This data was extracted from the [https://kibana.bitergia.com/xen Xen Project Code Review Dashboard]</span><br> |
||
<span id="Foot6"><sup>[ |
<span id="Foot6"><sup>[ 6 ]</sup> These are the days/months during which we were not in code freeze</span><br> |
||
[[Category:Xen]] |
[[Category:Xen]] |
Latest revision as of 11:27, 5 June 2017
Note that the Acknowledgements have been created based on 4.8.0-rc8, which was the final RC for Xen 4.8.0.
Overview
As we have made significant changes to the xen source tree structure, we will count acknowledgements in line with RFC Xen 4.6 Acknowledgements.
The following table compares Xen 4.7 with Xen 4.8. We also cover changes to Xen only components in Linux, FreeBSD, NetBSD and QEMU in the "Drivers and Devices" column, which are primarily made by Xen Project developers. Note that the number is not part of the 4.8 Hypervisor Total.
Attribute | 4.7 Hypervisor Total [ 1 ] | 4.8 Hypervisor Total [ 1 ] | 4.8 Hypervisor Core[ 2 ] | 4.8 Hypervisor Other [ 3 ] | Drivers and Devices (other projects)[ 4 ] |
---|---|---|---|---|---|
Release Cycle | 255 days 197 dev window/days[ 6 ] 8.4 months |
161 days 124 dev window/days[ 6 ] 5.3 months |
|||
Development Velocity | |||||
Changesets | 1887 | 1245 | 967 | 278 | 321 |
Series (=Features) [ 5 ] | 1025 | 575 | |||
Average Size of Feature | 1.84 | 2.17 (+15%) | |||
Changes / Day | 7.5 | 7.7 (+3%) | |||
Changes / Dev Day [ 6 ] | 9.6 | 10.0 (+4%) | |||
Features / Day [ 5 ] | 4.0 | 3.6 (-11%) | |||
Code Review Statistics | |||||
Review Comments [ 5 ] | 11628 | 6950 | |||
Review Comments / Day [ 5 ] | 45.6 | 43.2 (-6%) | |||
Comments / Series [ 5 ] | 11.3 | 12.1 (+7%) | |||
Code Churn | |||||
Lines Added | 106606 | 78899 | 70833 | 8066 | 18421 |
Lines Removed | 37160 | 62325 | 59502 | 2823 | 13775 |
Engagement | |||||
Individuals | 102 | 68 | 68 | 5 | 59 |
Employers | 36 | 34 | 33 | 4 | 37 |
4.8 Hypervisor Only Code Contributions [ 2 ]
Breakdown by Individual
This table is sortable : please click on the icon besides the column header, to sort by a specific column.
4.8 Hypervisor Related Code Contributions [ 3 ]
Breakdown by Individual
Note that the table below does not cover contributions to Linux, NetBSD, FreeBSD, QEMU, libvirt and other related projects. It lists contributors to mini-os (formerly part of xen.git), osstest (our test code and test cases), raisin (a deployment and test tool) and livepatch-build-tools (tools to builld livepatch payloads).
This table is sortable : please click on the icon besides the column header, to sort by a specific column.
Name | Lines Added | Lines Removed | Changesets |
---|---|---|---|
Ian Jackson | 3452 | 745 | 182 |
Juergen Gross | 3778 | 1792 | 58 |
Konrad Rzeszutek Wilk | 24 | 6 | 7 |
Ross Lagerwall | 99 | 28 | 5 |
Wei Liu | 713 | 252 | 28 |
Code Contributions to Xen Related Drivers and Devices in Linux, NetBSD, FreeBSD [ 4 ]
Breakdown by Individual
This table is sortable : please click on the icon besides the column header, to sort by a specific column.
4.8 Hypervisor Reviewers [ 5 ]
This table is sortable : please click on the icon besides the column header, to sort by a specific column. Note that in the table below, we only count review comments by community members other than comments by the author of a patch. We do not count replies and clarifications by the author of a patch that are made in response to review comments by others.
Footnotes
[ 1 ] Includes the following repos: xen, mini-os, osstest, raisin, livepatch-build-tools
[ 2 ] Includes the following repos: xen
[ 3 ] Includes the following repos: mini-os, osstest, raisin, livepatch-build-tools
[ 4 ] Includes commits to the following Xen specific directories of the following upstream projects: linux: arch/*/xen/ arch/*/include/asm/xen/ drivers/xen/ drivers/*/*xen* include/xen/ arch/*/pci/*xen* ; netbsd: sys/arch/xen; freebsd: sys/dev/xen/ sys/x86/xen sys/xen/; qemu: xen* hw/i386/xen hw/xenpv hw/*/xen* hw/xen include/*/xen*, xen windows pv drivers
[ 5 ] This data was extracted from the Xen Project Code Review Dashboard
[ 6 ] These are the days/months during which we were not in code freeze