Re: [ANN] bpf development stats for 6.6

[Alexei Starovoitov]

On Tue, Aug 29, 2023 at 5:35 PM Stanislav Fomichev <sdf@xxxxxxxxxx> wrote:
>
> See the netdev posting for more info and context:
> https://lore.kernel.org/netdev/20230829150539.6f998d1f@xxxxxxxxxx/T/#u
>
> As last time, I'm presenting raw stats without any evaluation. I'm
> posting a link to the previous cycle below so people can do the
> comparison if needed. I'm also trying to present the same data as we
> have in netdev email so you can follow Jakub's comments about script
> changes/etc.
>
> Previous cycle:
> 27 Apr to 28 Jun: 4234 mailing list messages, 62 days, 68 messages per day
> 664 repo commits (11 commits/day)
>
> Current cycle:
> 28 Jun to 29 Aug: 5240 mailing list messages, 62 days, 85 messages per day
> 384 repo commits (6 commits/day)

That is a big jump in activity.
Every bpf developer needs to internalize what "85 messages per day"
really means.
Put yourself in maintainer's shoes.
Patches will be applied only after they're reviewed.
When developers only send patches they slow down themselves and
all other developers. Every active developer needs to code review as well.

We all want patches to move swiftly, so please help maintainers and
yourself: review as many patches as you can.
This will speed up the patch queue and every developer will benefit.
win/win for both developers and maintainers.

>
> 6.5 stats: https://lore.kernel.org/netdev/20230829150539.6f998d1f@xxxxxxxxxx/T/#u
>
> Rankings
> --------
>
> Top reviewers (thr):                 Top reviewers (msg):
>    1 ( +1) [11] Alexei Starovoitov      1 ( +1) [23] Alexei Starovoitov
>    2 ( -1) [ 7] Yonghong Song           2 ( +1) [15] Yonghong Song
>    3 ( +5) [ 4] Martin KaFai Lau        3 ( +4) [ 9] Stanislav Fomichev
>    4 ( +3) [ 4] Stanislav Fomichev      4 ( -3) [ 7] Andrii Nakryiko
>    5 ( -1) [ 4] Daniel Borkmann         5 ( +5) [ 7] Martin KaFai Lau
>    6 ( +3) [ 3] Jakub Kicinski          6 (+15) [ 5] Hou Tao
>    7 ( -2) [ 3] Jiri Olsa               7 ( +7) [ 5] Jakub Kicinski
>    8 ( -5) [ 2] Andrii Nakryiko         8 ( -2) [ 4] Daniel Borkmann
>    9 (***) [ 2] Hou Tao                 9 (+13) [ 4] Jesper Dangaard Brouer
>   10 (+16) [ 2] Jesper Dangaard Brouer   10 (+17) [ 4] Eduard Zingerman
>   11 ( +7) [ 2] John Fastabend         11 ( -2) [ 4] Steven Rostedt
>   12 ( +3) [ 2] Eduard Zingerman       12 (***) [ 4] Alan Maguire

BIG thank you to all reviewers.
You're doing an excellent job.

>
> Top authors (thr):                   Top authors (msg):
>    1 ( +9) [3] Yonghong Song            1 ( +5) [20] Jiri Olsa
>    2 (   ) [2] Yafang Shao              2 (+30) [17] Yonghong Song
>    3 (***) [2] Geliang Tang             3 ( +8) [10] Larysa Zaremba
>    4 ( +5) [1] Daniel Borkmann          4 (   ) [10] Masami Hiramatsu (Google)
>    5 ( +2) [1] Masami Hiramatsu (Google)    5 (***) [ 8] Geliang Tang
>    6 ( -2) [1] Jiri Olsa                6 ( -3) [ 8] Maciej Fijalkowski
>    7 (+19) [1] Kui-Feng Lee             7 ( -5) [ 6] Yafang Shao
>    8 (+40) [1] Dave Marchevsky          8 (+12) [ 6] Dave Marchevsky
>    9 (***) [1] Rong Tao                 9 (+15) [ 5] Daniel Borkmann
>   10 (***) [1] Daniel Xu               10 (***) [ 5] Hao Xu
>
> Company rankings
> ----------------
>
> Top reviewers (thr):                 Top reviewers (msg):
>    1 (   ) [20] Meta                    1 (   ) [50] Meta
>    2 (   ) [ 8] Isovalent               2 (   ) [16] RedHat
>    3 (   ) [ 7] Google                  3 ( +1) [15] Google
>    4 (   ) [ 6] RedHat                  4 ( -1) [12] Isovalent
>    5 (   ) [ 3] Intel                   5 (   ) [ 7] Intel
>    6 ( +3) [ 3] Huawei                  6 ( +2) [ 7] Huawei
>    7 (+24) [ 2] Oracle                  7 (+41) [ 4] Oracle
>    8 ( -2) [ 2] Corigine                8 (+49) [ 3] Rivos
>    9 ( -1) [ 1] nVidia                  9 ( -3) [ 3] Corigine
>   10 (+34) [ 1] Rivos                  10 ( -3) [ 2] nVidia
>
> Top authors (thr):                   Top authors (msg):
>    1 (   ) [9] Meta                     1 (   ) [40] Meta
>    2 ( +3) [6] Huawei                   2 ( +1) [31] Isovalent
>    3 ( -1) [4] Isovalent                3 ( +1) [24] Intel
>    4 ( -1) [3] Google                   4 ( +2) [19] Huawei
>    5 ( +1) [3] Intel                    5 ( -3) [17] Google
>    6 ( -2) [2] RedHat                   6 ( +2) [ 9] RedHat
>    7 (***) [2] Juniper Networks         7 (***) [ 8] SUSE
>    8 (***) [2] SUSE                     8 ( +5) [ 7] Bytedance
>    9 ( +8) [1] Oracle                   9 (***) [ 6] Juniper Networks
>   10 (+42) [1] Linaro                  10 (***) [ 5] Hao Xu
>
> Development vs reviewing scores
> -------------------------------
>
> Top scores (positive):               Top scores (negative):
>    1 ( +1) [144] Alexei Starovoitov     1 (***) [46] Jiri Olsa
>    2 ( +4) [ 50] Martin KaFai Lau       2 ( +4) [39] Larysa Zaremba
>    3 (+15) [ 40] Stanislav Fomichev     3 (   ) [37] Masami Hiramatsu (Google)
>    4 ( -3) [ 38] Yonghong Song          4 (***) [34] Geliang Tang
>    5 ( +4) [ 37] Jakub Kicinski         5 ( +7) [22] Dave Marchevsky
>    6 ( -1) [ 34] Andrii Nakryiko        6 ( -4) [22] Maciej Fijalkowski
>    7 (***) [ 30] Jesper Dangaard Brouer    7 (***) [19] Hao Xu
>    8 (***) [ 25] John Fastabend         8 (+26) [18] Roberto Sassu
>    9 (***) [ 24] Hou Tao                9 (***) [16] Valentin Schneider
>   10 ( -7) [ 23] Daniel Borkmann       10 (***) [16] Tejun Heo

Jiri,
you've managed to be in the top reviewers and top negative :)
I guess that means that we need to tweak the positive/negative formula
to more accurately represent the value each person brings
to the community.

>
> Top scores (positive):               Top scores (negative):
>    1 (   ) [142] Meta                   1 ( +1) [49] Intel
>    2 (   ) [ 52] RedHat                 2 (+11) [38] Huawei
>    3 (***) [ 28] Google                 3 ( +5) [28] Bytedance
>    4 ( -1) [ 22] Corigine               4 (***) [22] SUSE
>    5 (***) [ 18] Oracle                 5 (***) [19] Isovalent
>    6 (+37) [  8] Rivos                  6 (***) [19] Hao Xu
>    7 ( -2) [  8] Microsoft              7 ( -3) [13] Alibaba
>    8 (***) [  7] Markus Elfring         8 (***) [12] Aviatrix
>    9 (   ) [  6] Amazon                 9 (***) [12] Juniper Networks
>   10 (***) [  5] Huacai Chen           10 (***) [11] Leon Hwang
>   11 (+28) [  5] Dave Thaler           11 (***) [10] IBM
>
> More raw stats
> --------------
>
> Prev: start: Thu, 27 Apr 2023 08:55:55 +0200
>         end: Wed, 28 Jun 2023 17:27:38 +0200
> Prev: messages: 4234 days: 62 (68 msg/day)
> Prev: direct commits: 664 (11 commits/day)
> Prev: people/aliases: 241  {'author': 89, 'commenter': 103, 'both': 49}
> Prev: review pct: 12.95%  x-corp pct: 12.05%
>
> Curr: start: Wed, 28 Jun 2023 22:18:25 -0700
>         end: Tue, 29 Aug 2023 15:24:18 -0700
> Curr: messages: 5240 days: 62 (85 msg/day)
> Curr: direct commits: 384 (6 commits/day)
> Curr: people/aliases: 279  {'author': 112, 'commenter': 120, 'both': 47}
> Curr: review pct: 24.22%  x-corp pct: 23.44%
>
> Diff: +23.8% msg/day
> Diff: -42.2% commits/day
> Diff: +15.8% people/day
> Diff: review pct: +11.3%
>       x-corp pct: +11.4%
>