LKML Archive mirror
 help / color / mirror / Atom feed
* Re: [ANN] bpf development stats for 6.6
       [not found] <ZO6O27Z9RvresmiV@google.com>
@ 2023-08-30 21:24 ` Alexei Starovoitov
  2023-08-31  8:24   ` Jiri Olsa
  0 siblings, 1 reply; 2+ messages in thread
From: Alexei Starovoitov @ 2023-08-30 21:24 UTC (permalink / raw
  To: Stanislav Fomichev, Network Development, LKML; +Cc: bpf, Jakub Kicinski

On Tue, Aug 29, 2023 at 5:35 PM Stanislav Fomichev <sdf@google.com> wrote:
>
> See the netdev posting for more info and context:
> https://lore.kernel.org/netdev/20230829150539.6f998d1f@kernel.org/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@kernel.org/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%
>

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [ANN] bpf development stats for 6.6
  2023-08-30 21:24 ` [ANN] bpf development stats for 6.6 Alexei Starovoitov
@ 2023-08-31  8:24   ` Jiri Olsa
  0 siblings, 0 replies; 2+ messages in thread
From: Jiri Olsa @ 2023-08-31  8:24 UTC (permalink / raw
  To: Alexei Starovoitov
  Cc: Stanislav Fomichev, Network Development, LKML, bpf,
	Jakub Kicinski

On Wed, Aug 30, 2023 at 02:24:19PM -0700, Alexei Starovoitov wrote:

SNIP

> > 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.

heh nice, win-win ;-)

jirka

> 
> >
> > 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%
> >
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-08-31  8:24 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <ZO6O27Z9RvresmiV@google.com>
2023-08-30 21:24 ` [ANN] bpf development stats for 6.6 Alexei Starovoitov
2023-08-31  8:24   ` Jiri Olsa

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).