Xen Project 4.11 Acknowledgements: Difference between revisions
Lars.kurth (talk | contribs) |
Lars.kurth (talk | contribs) (→Breakdown by Individual: Change columns) |
||
(5 intermediate revisions by the same user not shown) | |||
Line 1: | Line 1: | ||
{{TODOLeft|The data is based on staging, snapshot June 18, 2018.}} |
|||
== Overview == |
== Overview == |
||
Line 7: | Line 6: | ||
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. |
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. |
||
<b>Note:</b> The data below shows a reduction in public code review activity, as well as development velocity. 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. In addition, we ported many of these changes - including code that enabled these series - to older releases of Xen to uphold our maintenance and security guarantees. |
|||
{|class="zebra" |
{|class="zebra" |
||
Line 19: | Line 18: | ||
|'''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;"|'''222 days <br>139 dev window/days <sup>[ 6 ]</sup><br>7.5 months''' |
||
| |
| |
||
| |
| |
||
Line 28: | Line 27: | ||
|'''Changesets''' |
|'''Changesets''' |
||
|1250 |
|1250 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''1206''' |
||
|987 |
|||
|TODO |
|||
|287 |
|||
|TODO |
|||
!1493 |
|||
!TODO |
|||
|- |
|||
|'''Changesets (including backports)''' <sup>[ 8 ]</sup> |
|||
|1751 |
|||
|style="color: blue;"|'''1971''' |
|||
| |
|||
| |
|||
! |
|||
|- |
|- |
||
|'''Series (=Features)''' <sup>[ 5 ]</sup> |
|'''Series (=Features)''' <sup>[ 5 ]</sup> |
||
|525 |
|525 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''406''' |
||
| |
| |
||
| |
| |
||
Line 42: | Line 48: | ||
|'''Average Size of Feature''' |
|'''Average Size of Feature''' |
||
|2.3 |
|2.3 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''3.0''' |
||
| |
| |
||
| |
| |
||
Line 49: | Line 55: | ||
|'''Changes / Day''' |
|'''Changes / Day''' |
||
|7.2 |
|7.2 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''5.4''' |
||
| |
| |
||
| |
| |
||
Line 56: | Line 62: | ||
|'''Changes / Dev Day''' <sup>[ 6 ]</sup> |
|'''Changes / Dev Day''' <sup>[ 6 ]</sup> |
||
|11.2 |
|11.2 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''8.7''' |
||
| |
| |
||
| |
| |
||
Line 72: | Line 78: | ||
|'''Review Comments''' <sup>[ 5 ]</sup> |
|'''Review Comments''' <sup>[ 5 ]</sup> |
||
|6527 |
|6527 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''7058''' |
||
| |
| |
||
| |
| |
||
Line 79: | Line 85: | ||
|'''Review Comments / Day''' <sup>[ 5 ]</sup> |
|'''Review Comments / Day''' <sup>[ 5 ]</sup> |
||
|37.8 |
|37.8 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''31.8''' |
||
| |
| |
||
| |
| |
||
Line 86: | Line 92: | ||
|'''Review Comments / Dev Day''' <sup>[ 6 ]</sup> |
|'''Review Comments / Dev Day''' <sup>[ 6 ]</sup> |
||
|59.2 |
|59.2 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''50.8''' |
||
| |
| |
||
| |
| |
||
Line 93: | Line 99: | ||
|'''Comments / Series''' <sup>[ 5 ]</sup> |
|'''Comments / Series''' <sup>[ 5 ]</sup> |
||
|12.5 |
|12.5 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''17.4''' |
||
| |
| |
||
| |
| |
||
Line 102: | Line 108: | ||
|'''Lines Added''' |
|'''Lines Added''' |
||
|68199 |
|68199 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''42862''' |
||
|42862 |
|||
|TODO |
|||
|8577 |
|||
|TODO |
|||
! |
!51439 |
||
|- |
|- |
||
|'''Lines Removed''' |
|'''Lines Removed''' |
||
|25777 |
|25777 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''16030''' |
||
|16030 |
|||
|TODO |
|||
|6602 |
|||
|TODO |
|||
!22632 |
|||
!TODO |
|||
|- |
|- |
||
!colspan="6"|Engagement |
!colspan="6"|Engagement |
||
Line 118: | Line 124: | ||
|'''Individuals''' |
|'''Individuals''' |
||
|80 |
|80 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''62''' |
||
| |
|62 |
||
| |
|78 |
||
! |
! |
||
|- |
|- |
||
|'''Employers''' <sup>[ 7 ]</sup> |
|'''Employers''' <sup>[ 7 ]</sup> |
||
|31 |
|31 |
||
|style="color: blue;"|''' |
|style="color: blue;"|'''27''' |
||
| |
|27 |
||
| |
|37 |
||
! |
! |
||
|} |
|} |
||
Line 139: | Line 145: | ||
{|class="zebra sortable" |
{|class="zebra sortable" |
||
!style="min-width: 200px;"|Name {{Sortable}} |
!style="min-width: 200px;"|Name {{Sortable}} |
||
!style="min-width: 110px;"|Changesets {{Sortable}} |
|||
!style="min-width: 110px;"|Lines Added {{Sortable}} |
!style="min-width: 110px;"|Lines Added {{Sortable}} |
||
!style="min-width: 110px;"|Lines Removed {{Sortable}} |
!style="min-width: 110px;"|Lines Removed {{Sortable}} |
||
!style="min-width: 110px;"|Changesets {{Sortable}} |
|||
|- |
|- |
||
| Alan Robinson || 1 || 6 || 4 |
| Alan Robinson || 1 || 6 || 4 |
||
Line 276: | Line 282: | ||
{|class="zebra sortable" |
{|class="zebra sortable" |
||
!style="min-width: 200px;"|Name {{Sortable}} |
!style="min-width: 200px;"|Name {{Sortable}} |
||
!style="min-width: 110px;"|Changesets {{Sortable}} |
|||
!style="min-width: 110px;"|Lines Added {{Sortable}} |
!style="min-width: 110px;"|Lines Added {{Sortable}} |
||
!style="min-width: 110px;"|Lines Removed {{Sortable}} |
!style="min-width: 110px;"|Lines Removed {{Sortable}} |
||
!style="min-width: 110px;"|Changesets {{Sortable}} |
|||
|- |
|- |
||
| Al Viro || 1 || 1 || 1 |
|||
|- |
|||
| Alistair Francis || 2 || 18 || 16 |
|||
|- |
|||
| Andrew Turner || 1 || 1 || 1 |
|||
|- |
|||
| Andy Shevchenko || 1 || 1 || 3 |
|||
|- |
|||
| Arnd Bergmann || 2 || 7 || 9 |
|||
|- |
|||
| Arvind Yadav || 1 || 4 || 1 |
|||
|- |
|||
| Bart Van Assche || 1 || 5 || 5 |
|||
|- |
|||
| Ben Chalmers || 1 || 2 || 2 |
|||
|- |
|||
| Bhavesh Davda || 1 || 8 || 9 |
|||
|- |
|||
| Bjorn Helgaas || 5 || 3 || 2 |
|||
|- |
|||
| Boris Ostrovsky || 3 || 84 || 11 |
|||
|- |
|||
| Manuel Bouyer || 2 || 32 || 11 |
|||
|- |
|||
| Cherry G. Mathew || 1 || 6 || 6 |
|||
|- |
|||
| Chris Patterson || 1 || 6 || 1 |
|||
|- |
|||
| Christoph Hellwig || 2 || 2 || 16 |
|||
|- |
|||
| Christos Zoulas || 6 || 51 || 196 |
|||
|- |
|||
| Colin Ian King || 2 || 2 || 2 |
|||
|- |
|||
| Dan Carpenter || 6 || 12 || 10 |
|||
|- |
|||
| Daniel Vetter || 1 || 1 || 7 |
|||
|- |
|||
| Dave Airlie || 1 || 0 || 0 |
|||
|- |
|||
| David Gibson || 2 || 15 || 15 |
|||
|- |
|||
| David S. Miller || 7 || 0 || 0 |
|||
|- |
|||
| Eduardo Habkost || 2 || 7 || 7 |
|||
|- |
|||
| Eduardo Otubo || 1 || 1 || 0 |
|||
|- |
|||
| Ed Maste || 1 || 4 || 4 |
|||
|- |
|||
| Frederick Lawler || 1 || 9 || 10 |
|||
|- |
|||
| Greg Kurz || 2 || 2 || 3 |
|||
|- |
|||
| Huang Ying || 1 || 4 || 0 |
|||
|- |
|||
| Ian Jackson || 6 || 58 || 74 |
|||
|- |
|||
| Igor Druzhinin || 3 || 34 || 23 |
|||
|- |
|||
| Jan Beulich || 1 || 1 || 1 |
|||
|- |
|||
| Jason Andryuk || 1 || 6 || 1 |
|||
|- |
|||
| Jaromír Doleček || 9 || 457 || 179 |
|||
|- |
|||
| Jeff Roberson || 3 || 6 || 6 |
|||
|- |
|||
| Jens Axboe || 1 || 0 || 0 |
|||
|- |
|||
| Jia-Ju Bai || 5 || 5 || 5 |
|||
|- |
|||
| Joao Martins || 2 || 29 || 0 |
|||
|- |
|||
| Joe Jin || 1 || 1 || 1 |
|||
|- |
|||
| Joe Perches || 6 || 22 || 25 |
|||
|- |
|||
| Juergen Gross || 2 || 225 || 3 |
|||
|- |
|||
| Kees Cook || 5 || 29 || 22 |
|||
|- |
|||
| Konstantin Belousov || 1 || 53 || 2 |
|||
|- |
|||
| Robert Elz || 1 || 5 || 4 |
|||
|- |
|||
| Linus Torvalds || 38 || 12 || 12 |
|||
|- |
|||
| Luc Van Oostenryck || 1 || 1 || 1 |
|||
|- |
|||
| Maarten Lankhorst || 1 || 0 || 0 |
|||
|- |
|||
| Marc-Andre Lureau || 1 || 2 || 2 |
|||
|} |
|} |
||
<br> |
<br> |
||
Line 423: | Line 523: | ||
<span id="Foot6"><sup>[ 6 ]</sup> These are the days/months during which we were not in code freeze</span><br> |
<span id="Foot6"><sup>[ 6 ]</sup> These are the days/months during which we were not in code freeze</span><br> |
||
<span id="Foot7"><sup>[ 7 ]</sup> Only known employers are used. Unknown employers were discounted. </span><br> |
<span id="Foot7"><sup>[ 7 ]</sup> Only known employers are used. Unknown employers were discounted. </span><br> |
||
<span id="Foot8"><sup>[ 8 ]</sup> Includes backports to stable branches </span><br> |
|||
[[Category:Xen]] |
[[Category:Xen]] |
Latest revision as of 12:19, 9 July 2018
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 reduction in public code review activity, as well as development velocity. 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. In addition, we ported many of these changes - including code that enabled these series - to older releases of Xen to uphold our maintenance and security guarantees.
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 |
222 days 139 dev window/days [ 6 ] 7.5 months |
|||
Development Velocity | |||||
Changesets | 1250 | 1206 | 987 | 287 | 1493 |
Changesets (including backports) [ 8 ] | 1751 | 1971 | |||
Series (=Features) [ 5 ] | 525 | 406 | |||
Average Size of Feature | 2.3 | 3.0 | |||
Changes / Day | 7.2 | 5.4 | |||
Changes / Dev Day [ 6 ] | 11.2 | 8.7 | |||
Features / Day [ 5 ] | 3.0 | 3.4 | |||
Code Review Statistics | |||||
Review Comments [ 5 ] | 6527 | 7058 | |||
Review Comments / Day [ 5 ] | 37.8 | 31.8 | |||
Review Comments / Dev Day [ 6 ] | 59.2 | 50.8 | |||
Comments / Series [ 5 ] | 12.5 | 17.4 | |||
Code Churn | |||||
Lines Added | 68199 | 42862 | 42862 | 8577 | 51439 |
Lines Removed | 25777 | 16030 | 16030 | 6602 | 22632 |
Engagement | |||||
Individuals | 80 | 62 | 62 | 78 | |
Employers [ 7 ] | 31 | 27 | 27 | 37 |
4.11 Hypervisor Only Code Contributions [ 1 ]
Breakdown by Individual
This table is sortable : please click on the icon besides the column header, to sort by a specific column.
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.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.
[ 8 ] Includes backports to stable branches