On Fri, 7 Feb 2025 12:25:28 -0800 Saeed Mahameed wrote:
> >nVidia is already refusing to add basic minoring features to their
> >upstream driver, and keeps asking its customers to migrate to libdoca.
>
> nVidia is one of the top contributers to netdev,
That's inaccurate. I can't think of a single meaningful contribution
from nVidia's NIC team outside of your own driver in the last 2 years.
> we submit patches on weekly bases and due to netdev mailing list
> review backlog and policy we barely make quota,
Luckily we have development statistics so we don't have to argue:
Top reviewers (cs): Top reviewers (msg):
1 ( +1) [27] Meta 1 ( +1) [68] Meta
2 ( -1) [25] RedHat 2 ( -1) [57] RedHat
3 ( +1) [19] Intel 3 ( +2) [49] Intel
4 ( -1) [15] Andrew Lunn 4 ( ) [43] Andrew Lunn
5 ( ) [12] Google 5 ( -2) [32] Google
6 ( +2) [ 5] Linaro 6 ( +3) [13] NXP
7 ( +3) [ 4] Oracle 7 ( +5) [13] Oracle
Top authors (cs): Top authors (msg):
1 ( ) [9] RedHat 1 ( ) [48] Intel
2 ( +2) [8] Google 2 ( ) [42] RedHat
3 ( -1) [7] Intel 3 ( +1) [39] Meta
4 ( -1) [7] Meta 4 ( -1) [31] Huawei
5 ( +2) [5] nVidia 5 ( ) [31] nVidia
6 ( +7) [3] Oracle 6 (+11) [28] Oracle
7 ( +9) [2] Linaro 7 (+15) [23] Pengutronix
Top scores (positive): Top scores (negative):
1 ( +1) [329] Meta 1 ( ) [92] Huawei
2 ( +1) [265] Andrew Lunn 2 ( +1) [76] OpenVPN
3 ( -2) [238] RedHat 3 (***) [54] Pengutronix
4 ( +3) [125] Intel 4 ( +2) [53] Marvell
5 ( -1) [116] Google 5 ( +5) [50] Dent
6 ( -1) [ 70] Linaro 6 (***) [45] nVidia
7 ( -1) [ 39] Broadcom 7 (+12) [43] AMD
https://lore.kernel.org/all/20250121200710.19126f7d@xxxxxxxxxx/
nVidia has a negative review vs authorship score. It'd probably
be much worse if it wasn't for the work of the switch team.
> so please elaborate on what features we are refusing to do ??
nVidia likes to send these threads to my management so I need
to be careful. An issue was discovered during new platform evaluation.
That's all I'm gonna say.
> As explained above, netdev doesn't need it, but netdev subsystem also
> hosts the pci base drivers, so you are going to see fwctl patches the
> same as you see rdma and other non netdev patches flowing through
> netdev ML.
Sure, but we're deadlocked here. It may be a slight inconvenience to
redo the interface so that its not a standalone aux bus driver. But if
you agree the netdev doesn't need it seems like a fairly straightforward
way to unblock your progress.
I am glad that you at least agree now that nedev doesn't need it.
