lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAL+tcoBzHNt86-4OC7Jck1WBG+YuadWDhQrkyEPBaxQxcET6YQ@mail.gmail.com>
Date: Wed, 13 Mar 2024 10:13:43 +0800
From: Jason Xing <kerneljasonxing@...il.com>
To: Jakub Kicinski <kuba@...nel.org>
Cc: netdev@...r.kernel.org, netdev-driver-reviewers@...r.kernel.org
Subject: Re: [ANN] netdev development stats for 6.9

On Wed, Mar 13, 2024 at 3:44 AM Jakub Kicinski <kuba@...nel.org> wrote:
>
> Intro
> -----
>
> We have posted our pull requests with networking changes for the 6.9
> kernel release last night. As is tradition here are the development
> statistics based on mailing list traffic on netdev@...r.
>
> These stats are somewhat like LWN stats: https://lwn.net/Articles/956765/
> but more focused on review participation.
>
> Previous stats (for 6.8): https://lore.kernel.org/all/20240109134053.33d317dd@kernel.org/
>
> Testing
> -------
>
> The selftest runner went live early during this release.
> I don't have any great ideas on how to quantify the progress
> but, first, I'd like to thank everyone who contributed to improving
> and fixing the tests, and those who contributed to NIPA directly.
> We are reporting results from 243 tests to patchwork (not counting
> kunit tests), each test on two kernel flavors and with many sub-cases.
>
> There's still work to be done fixing some of the tests but we made
> more progress than I anticipated! There are 66 test + kernel config
> combinations which we currently ignore, either because they permanently
> skip / fail or are too flaky. I should mention that those tests do matter,
> recently the ignored pmtu test would have caught a last minute xfrm
> regression :(
>
> Speaking of catching regressions, I do not have an objective count but
> subjectively our tests do in fact catch bugs. In fact the signal to noise
> ratio is higher than I initially expected. I wish it was easier to write
> driver tests and use YNL in the tests, but we'll get there..
>
> General stats
> -------------
>
> Cycle started on Tue, 09 Jan, ended Mon, 11 Mar. That's 63 days,
> one week shorter than the previous cycle.
>
> We have seen 264 msg/day on the list, and 24 commits/day.
> The number of messages is back to the level we have seen in 6.6,
> and has recovered after couple of slower releases (last one being
> particularly slow due to the winter break). The number of commits
> applied directly by netdev maintainers is 9% higher than in 6.6
> and highest on the record.
>
> The total number of review tags in git history have dipped again,
> 61% of commits contain a review tag (down from 69%). The number
> of commits with a review tag from an email domain different than
> the author, however, dropped only by 1% to 53%. Similarly our statistic
> of series which received at least one ack / review tag on the list did
> not change much, increasing by 1% to 67%.
>
> Rankings
> --------
>
> As promised last time, the left side of the stats is now in "change
> sets" (cs) rather than threads, IOW trying to track multiple revisions
> of the same series as one.
>
> Top reviewers (cs):                  Top reviewers (msg):
>    1 ( +1) [30] Jakub Kicinski          1 (   ) [66] Jakub Kicinski
>    2 ( -1) [26] Simon Horman            2 (   ) [45] Simon Horman
>    3 (   ) [14] Andrew Lunn             3 (   ) [44] Andrew Lunn
>    4 ( +1) [12] Paolo Abeni             4 ( +3) [25] Jiri Pirko
>    5 ( +4) [11] Jiri Pirko              5 (   ) [23] Eric Dumazet
>    6 ( -2) [10] Eric Dumazet            6 ( +3) [20] Paolo Abeni
>    7 (   ) [ 5] David Ahern             7 ( +1) [15] Krzysztof Kozlowski
>    8 ( +6) [ 4] Stephen Hemminger       8 ( +2) [11] David Ahern
>    9 ( +2) [ 4] Willem de Bruijn        9 ( +4) [10] Florian Fainelli
>   10 (   ) [ 4] Krzysztof Kozlowski    10 ( -6) [10] Vladimir Oltean
>   11 ( +1) [ 4] Florian Fainelli       11 ( -5) [ 9] Russell King
>   12 ( -6) [ 4] Russell King           12 (   ) [ 9] Willem de Bruijn
>   13 ( +3) [ 2] Jacob Keller           13 ( -2) [ 8] Sergey Shtylyov
>   14 ( +1) [ 2] Rob Herring            14 ( +6) [ 7] Jacob Keller
>   15 ( +2) [ 2] Jamal Hadi Salim       15 (+16) [ 7] Jason Wang
>
> Jiri jumps into top 5, reviewing various driver and netlink changes.
> Stephen largely works on iproute2 reviews. Russell and Vladimir have
> indicated that they are occupied outside of netdev, and slip by a few
> positions.
>
> Thank you all for your work!
>
> Top authors (cs):                    Top authors (msg):
>    1 ( +1) [7] Eric Dumazet             1 (+35) [40] Eric Dumazet
>    2 ( -1) [7] Jakub Kicinski           2 ( +5) [26] Jakub Kicinski
>    3 (+46) [4] Breno Leitao             3 ( +1) [20] Saeed Mahameed
>    4 (   ) [3] Heiner Kallweit          4 (+18) [17] Xuan Zhuo
>    5 ( +8) [2] Stephen Hemminger        5 (***) [15] Jason Xing
>    6 (+24) [2] Paolo Abeni              6 (***) [15] Matthieu Baerts
>    7 (+11) [2] Kuniyuki Iwashima        7 (***) [14] Breno Leitao
>    8 (***) [2] Maks Mishin              8 ( -3) [14] Tony Nguyen
>    9 ( +6) [2] Kunwu Chan               9 ( -3) [13] Kuniyuki Iwashima
>   10 (***) [2] Matthieu Baerts         10 ( -8) [11] Christian Marangi
>
> Thanks to switching from threads to changes sets we see Eric rightfully
> claim the #1 spot, previously occupied by yours truly. Breno added
> missing MODULE_DESCRIPTION()s and refactored drivers using per-cpu
> stats. Paolo contributed a number of MPTCP changes and many selftest
> improvements. Maks Mishin sent a lot of individual iproute2 patches.
> Xuan Zhuo is working on vhost / virtio changes for AF_XDP.
>
> Top scores (positive):               Top scores (negative):
>    1 ( +1) [398] Jakub Kicinski         1 ( +2) [80] Saeed Mahameed
>    2 ( -1) [371] Simon Horman           2 (+17) [60] Xuan Zhuo
>    3 (   ) [243] Andrew Lunn            3 (***) [46] Jason Xing
>    4 ( +8) [160] Jiri Pirko             4 (***) [43] Yang Xiwen via B4 Relay
>    5 ( +2) [152] Paolo Abeni            5 (***) [42] Matthieu Baerts
>    6 ( +2) [ 78] Krzysztof Kozlowski    6 ( -2) [40] Tony Nguyen
>    7 ( +2) [ 76] David Ahern            7 ( -6) [38] David Howells
>    8 ( +2) [ 68] Willem de Bruijn       8 ( -6) [37] Christian Marangi
>    9 ( +2) [ 60] Florian Fainelli       9 (+30) [37] Kory Maincent
>   10 ( -5) [ 54] Russell King          10 (***) [36] Breno Leitao
>
> Corporate stats
> ---------------
>
> Top reviewers (cs):                  Top reviewers (msg):
>    1 (   ) [42] RedHat                  1 (   ) [98] RedHat
>    2 (   ) [32] Meta                    2 (   ) [74] Meta
>    3 ( +2) [16] Google                  3 (   ) [44] Andrew Lunn
>    4 (   ) [14] Andrew Lunn             4 ( +1) [43] Google
>    5 ( -2) [13] Intel                   5 ( -1) [36] Intel
>    6 ( +1) [13] nVidia                  6 ( +2) [31] Linaro
>    7 ( +1) [ 8] Linaro                  7 (   ) [31] nVidia
>
> Top authors (cs):                    Top authors (msg):
>    1 ( +1) [12] RedHat                  1 (   ) [69] Intel
>    2 ( +2) [12] Meta                    2 ( +2) [60] Meta
>    3 ( -2) [10] Intel                   3 ( +2) [55] Google
>    4 ( -1) [10] Google                  4 ( -1) [50] nVidia
>    5 (   ) [ 7] nVidia                  5 ( -3) [47] RedHat
>    6 ( +2) [ 3] Huawei                  6 ( +4) [32] Bootlin
>    7 ( +7) [ 3] Wirecard                7 ( -1) [23] Alibaba
>
> Top scores (positive):               Top scores (negative):
>    1 (   ) [496] RedHat                 1 ( +7) [89] Bootlin
>    2 (   ) [303] Meta                   2 (   ) [79] Alibaba
>    3 (   ) [243] Andrew Lunn            3 (***) [46] Tencent
>    4 ( +2) [138] Linaro                 4 (***) [43] Yang Xiwen
>    5 (   ) [ 79] Google                 5 (***) [42] Tessares
>    6 ( +2) [ 76] Enfabrica              6 ( +6) [38] AMD
>    7 ( -3) [ 60] Oracle                 7 ( -6) [37] Christian Marangi
>
> RedHat remains unbeatable with the combined powers of Simon and Paolo,
> as well as high participation of the less active authors.
> Alibaba maintains its strongly net-negative review score.
> Tencent (Jason Xing) joins them (Tencent doesn't use their email
> domain so it's likely under-counted). Yang Xiwen makes the negative

Interesting numbers.

Well, actually, I always send patches by using the company address and
review patches by using my personal address, which probably makes my
scores negative :S

Next time I'll try to use the same address. Hope I will not appear on
the top of the negative scores list.

> list as well, cost of reposting large series too often...
> --
> Code: https://github.com/kuba-moo/ml-stat
> Raw output: https://netdev.bots.linux.dev/static/nipa/stats-6.9/stdout
>

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ