Difference between revisions of "Xen Project 4.11 Acknowledgements"
Lars.kurth (talk | contribs) (Created page with "{{TODOLeft|Populate with data before the release}} == Overview == As we have made significant changes to the xen source tree structure, we will count acknowledgements in lin...") |
Lars.kurth (talk | contribs) (→Overview) |
||
(5 intermediate revisions by the same user not shown) | |||
Line 1: | Line 1: | ||
− | {{TODOLeft| |
+ | {{TODOLeft|The data is based on staging, snapshot June 18, 2018.}} |
== Overview == |
== Overview == |
||
Line 5: | Line 5: | ||
As we have made significant changes to the xen source tree structure, we will count acknowledgements in line with [http://lists.xenproject.org/archives/html/xen-devel/2015-10/msg01063.html RFC Xen 4.6 Acknowledgements]. |
As we have made significant changes to the xen source tree structure, we will count acknowledgements in line with [http://lists.xenproject.org/archives/html/xen-devel/2015-10/msg01063.html RFC Xen 4.6 Acknowledgements]. |
||
− | The following table compares Xen 4. |
+ | The following table compares Xen 4.10 with Xen 4.11. 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.11 Hypervisor Total. |
+ | |||
+ | <span style="color:red"><b>Note:</b></span> The data below shows a significant reduction in public code review activity. This figure is substantially skewed by a number of large and complex security related code series that were developed in private under security embargoes. In other words, the reduction signifies that we are doing fewer public reviews to satisfy requirements that allow project members to fix issues under security embargoes. |
||
{|class="zebra" |
{|class="zebra" |
||
Line 17: | Line 19: | ||
|'''Release Cycle''' |
|'''Release Cycle''' |
||
|174 days<br>111 dev window/days <sup>[ 6 ] </sup><br>5.9 months |
|174 days<br>111 dev window/days <sup>[ 6 ] </sup><br>5.9 months |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''201 days <br>129 dev window/days <sup>[ 6 ]</sup><br>6.8 months''' |
| |
| |
||
| |
| |
||
Line 26: | Line 28: | ||
|'''Changesets''' |
|'''Changesets''' |
||
|1250 |
|1250 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''1793''' <span style="color:green">(+30.3%)</span> |
|TODO |
|TODO |
||
|TODO |
|TODO |
||
Line 33: | Line 35: | ||
|'''Series (=Features)''' <sup>[ 5 ]</sup> |
|'''Series (=Features)''' <sup>[ 5 ]</sup> |
||
|525 |
|525 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''676''' |
| |
| |
||
| |
| |
||
Line 40: | Line 42: | ||
|'''Average Size of Feature''' |
|'''Average Size of Feature''' |
||
|2.3 |
|2.3 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''2.7''' |
| |
| |
||
| |
| |
||
Line 47: | Line 49: | ||
|'''Changes / Day''' |
|'''Changes / Day''' |
||
|7.2 |
|7.2 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''8.9''' <span style="color:green">(+19.5%)</span> |
| |
| |
||
| |
| |
||
Line 54: | Line 56: | ||
|'''Changes / Dev Day''' <sup>[ 6 ]</sup> |
|'''Changes / Dev Day''' <sup>[ 6 ]</sup> |
||
|11.2 |
|11.2 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''12.9''' <span style="color:green">(+12.7%)</span> |
| |
| |
||
| |
| |
||
Line 61: | Line 63: | ||
|'''Features / Day''' <sup>[ 5 ]</sup> |
|'''Features / Day''' <sup>[ 5 ]</sup> |
||
|3.0 |
|3.0 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''3.4''' |
| |
| |
||
| |
| |
||
Line 70: | Line 72: | ||
|'''Review Comments''' <sup>[ 5 ]</sup> |
|'''Review Comments''' <sup>[ 5 ]</sup> |
||
|6527 |
|6527 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''4525''' |
| |
| |
||
| |
| |
||
Line 77: | Line 79: | ||
|'''Review Comments / Day''' <sup>[ 5 ]</sup> |
|'''Review Comments / Day''' <sup>[ 5 ]</sup> |
||
|37.8 |
|37.8 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''22.5''' <span style="color:red">(-67.8%)</span> |
| |
| |
||
| |
| |
||
Line 84: | Line 86: | ||
|'''Review Comments / Dev Day''' <sup>[ 6 ]</sup> |
|'''Review Comments / Dev Day''' <sup>[ 6 ]</sup> |
||
|59.2 |
|59.2 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''32.6''' <span style="color:red">(-81.9%)</span> |
| |
| |
||
| |
| |
||
Line 91: | Line 93: | ||
|'''Comments / Series''' <sup>[ 5 ]</sup> |
|'''Comments / Series''' <sup>[ 5 ]</sup> |
||
|12.5 |
|12.5 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''6.7''' <span style="color:red">(-87.0%%)</span> |
| |
| |
||
| |
| |
||
Line 100: | Line 102: | ||
|'''Lines Added''' |
|'''Lines Added''' |
||
|68199 |
|68199 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''65300''' |
|TODO |
|TODO |
||
|TODO |
|TODO |
||
Line 107: | Line 109: | ||
|'''Lines Removed''' |
|'''Lines Removed''' |
||
|25777 |
|25777 |
||
− | |style="color: blue;"|''' |
+ | |style="color: blue;"|'''21560''' |
|TODO |
|TODO |
||
|TODO |
|TODO |
||
Line 163: | Line 165: | ||
{|class="zebra sortable" |
{|class="zebra sortable" |
||
!style="min-width: 200px;"|Name {{Sortable}} |
!style="min-width: 200px;"|Name {{Sortable}} |
||
− | !style="min-width: 110px;"|Review Comments {{Sortable}} |
+ | !style="min-width: 110px;"|Review Comments (excludes replies to own series) {{Sortable}} |
+ | |- |
||
+ | | 240 others (less than 10 review comments each) || 555 |
||
+ | |- |
||
+ | | Jan Beulich || 1,177 |
||
+ | |- |
||
+ | | Andrew Cooper || 618 |
||
+ | |- |
||
+ | | Julien Grall || 554 |
||
+ | |- |
||
+ | | Roger Pau Monne || 395 |
||
+ | |- |
||
+ | | Wei Liu || 376 |
||
+ | |- |
||
+ | | Stefano Stabellini || 354 |
||
+ | |- |
||
+ | | Juergen Gross || 282 |
||
+ | |- |
||
+ | | Boris Ostrovsky || 226 |
||
+ | |- |
||
+ | | Oleksandr Andrushchenko || 215 |
||
+ | |- |
||
+ | | George Dunlap || 190 |
||
+ | |- |
||
+ | | Paul Durrant || 142 |
||
+ | |- |
||
+ | | Andre Przywara || 100 |
||
+ | |- |
||
+ | | Ian Jackson || 136 |
||
+ | |- |
||
+ | | Razvan Cojocaru || 68 |
||
+ | |- |
||
+ | | Kevin Tian || 66 |
||
+ | |- |
||
+ | | Konrad Wilk || 56 |
||
+ | |- |
||
+ | | Tamas K Lengyel || 51 |
||
+ | |- |
||
+ | | Dario Faggioli || 50 |
||
+ | |- |
||
+ | | Mirela Simonovic || 50 |
||
+ | |- |
||
+ | | Daniel Kiper || 45 |
||
+ | |- |
||
+ | | Olaf Hering || 45 |
||
+ | |- |
||
+ | | Chao Gao || 41 |
||
+ | |- |
||
+ | | Alexandru Isaila || 39 |
||
+ | |- |
||
+ | | Anthony Perard || 39 |
||
+ | |- |
||
+ | | Lars Kurth || 38 |
||
+ | |- |
||
+ | | Volodymyr Babchuk || 37 |
||
+ | |- |
||
+ | | Doug Goldstein || 34 |
||
+ | |- |
||
+ | | Paul Semel || 29 |
||
+ | |- |
||
+ | | Manish Jagg || 28 |
||
+ | |- |
||
+ | | Anthony Liguori || 26 |
||
+ | |- |
||
+ | | Michael Tsirkin || 26 |
||
+ | |- |
||
+ | | Takashi Iwai || 25 |
||
+ | |- |
||
+ | | Tim Deegan || 23 |
||
+ | |- |
||
+ | | Simon Gaiser || 22 |
||
+ | |- |
||
+ | | Daniel Vetter || 21 |
||
+ | |- |
||
+ | | Ross Lagerwall || 21 |
||
+ | |- |
||
+ | | Marek Marczykowski || 19 |
||
+ | |- |
||
+ | | Daniel Berrange || 18 |
||
+ | |- |
||
+ | | Sky Liu || 15 |
||
+ | |- |
||
+ | | Govinda Tatti || 15 |
||
+ | |- |
||
+ | | Alexey Gerasimenk || 15 |
||
+ | |- |
||
+ | | Oleksandr Grytsov || 14 |
||
+ | |- |
||
+ | | Igor Druzhinin || 14 |
||
+ | |- |
||
+ | | Maran Wilson || 14 |
||
+ | |- |
||
+ | | Marc Zyngier || 14 |
||
+ | |- |
||
+ | | Ingo Molnar || 14 |
||
+ | |- |
||
+ | | Christian Lindig || 13 |
||
+ | |- |
||
+ | | Brian Woods || 12 |
||
+ | |- |
||
+ | | Sergey Dyasli || 12 |
||
+ | |- |
||
+ | | Andrii Anisov || 11 |
||
+ | |- |
||
+ | | Dmitry Torokhov || 11 |
||
+ | |- |
||
+ | | Dongwon Kim || 11 |
||
+ | |- |
||
+ | | Natarajan Janakarajan || 11 |
||
+ | |- |
||
+ | | homas Garnier || 11 |
||
+ | |- |
||
+ | | Volodymyr Babchuk || 11 |
||
+ | |- |
||
+ | | Zhenzhong Duan || 11 |
||
+ | |- |
||
+ | | Arnd Bergmann || 10 |
||
+ | |- |
||
+ | | David Woodhouse || 10 |
||
+ | |- |
||
+ | | Eric Blak || 10 |
||
|- |
|- |
||
+ | | Zhouyang Jia || 10 |
||
|} |
|} |
||
<br> |
<br> |
Revision as of 11:03, 18 June 2018
Contents
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.10 with Xen 4.11. 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.11 Hypervisor Total.
Note: The data below shows a significant reduction in public code review activity. This figure is substantially skewed by a number of large and complex security related code series that were developed in private under security embargoes. In other words, the reduction signifies that we are doing fewer public reviews to satisfy requirements that allow project members to fix issues under security embargoes.
Attribute | 4.10 Hypervisor Total [ 1 ] | 4.11 Hypervisor Total [ 1 ] | 4.11 Hypervisor Core [ 2 ] | 4.11 Hypervisor Other [ 3 ] | Drivers and Devices (other projects) [ 4 ] |
---|---|---|---|---|---|
Release Cycle | 174 days 111 dev window/days [ 6 ] 5.9 months |
201 days 129 dev window/days [ 6 ] 6.8 months |
|||
Development Velocity | |||||
Changesets | 1250 | 1793 (+30.3%) | TODO | TODO | TODO |
Series (=Features) [ 5 ] | 525 | 676 | |||
Average Size of Feature | 2.3 | 2.7 | |||
Changes / Day | 7.2 | 8.9 (+19.5%) | |||
Changes / Dev Day [ 6 ] | 11.2 | 12.9 (+12.7%) | |||
Features / Day [ 5 ] | 3.0 | 3.4 | |||
Code Review Statistics | |||||
Review Comments [ 5 ] | 6527 | 4525 | |||
Review Comments / Day [ 5 ] | 37.8 | 22.5 (-67.8%) | |||
Review Comments / Dev Day [ 6 ] | 59.2 | 32.6 (-81.9%) | |||
Comments / Series [ 5 ] | 12.5 | 6.7 (-87.0%%) | |||
Code Churn | |||||
Lines Added | 68199 | 65300 | TODO | TODO | TODO |
Lines Removed | 25777 | 21560 | TODO | TODO | TODO |
Engagement | |||||
Individuals | 80 | TODO (TODO%) | |||
Employers [ 7 ] | 31 | TODO (TODO%) |
4.11 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.
Name | Lines Added | Lines Removed | Changesets |
---|
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.
Name | Lines Added | Lines Removed | Changesets |
---|
4.11 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, xtf
[ 2 ] Includes the following repos: xen
[ 3 ] Includes the following repos: mini-os, osstest, raisin, livepatch-build-tools, xtf
[ 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
[ 7 ] Only known employers are used. Unknown employers were discounted.