archive mirror
 help / color / mirror / Atom feed
From: Jakub Kicinski <>
To: Taras Chornyi <>
Cc: Vadym Kochan <>,
	"David S. Miller" <>,
	Paolo Abeni <>,, Elad Nachman <>,
	Mickey Rachamim <>,,
	Taras Chornyi <>
Subject: Re: [PATCH v2] MAINTAINERS: Update maintainer for Marvell Prestera Ethernet Switch driver
Date: Thu, 1 Dec 2022 13:17:44 -0800	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

On Thu, 1 Dec 2022 10:39:07 +0200 Taras Chornyi wrote:
> On 30.11.22 07:14, Jakub Kicinski wrote:
> > On Mon, 28 Nov 2022 11:39:34 +0200 Vadym Kochan wrote:  
> >> Add Elad Nachman as maintainer for Marvell Prestera Ethernet Switch driver.
> >>
> >> Change Taras Chornyi mailbox to plvision.  
> > This is a patch, so the description needs to explain why...
> > and who these people are. It would seem more natural if you,
> > Oleksandr and Yevhen were the maintainers.
> >
> > Seriously, this is a community project please act the part.  
> The Marvell Prestera Switchdev Kernel Driver's focus and maintenance are 
> shifted from PLVision (Marvell Contractors) to the Marvell team in Israel.
> In the last 12 months, the driver's development efforts have been shared 
> between the PLVision team and Elad Nachman from the Marvell Israel group.

Ah, damn, I was worried that's what you'd say :(

> Elad Nachman is a veteran with over ten years of experience in Linux 
> kernel development.
> He has made many Linux kernel contributions to several community 
> projects, including the Linux kernel, DPDK (KNI Linux Kernel driver) and 
> the DENT project.
> Elad has done reviews and technical code contributions on Armada 3700, 
> Helping Pali Rohár, who is the maintainer of the Armada 3700 PCI 
> sub-system, as well as others in the Armada 3700 cpufreq sub-system.
> In the last year and a half, Elad has internally dealt extensively with 
> the Marvell Prestera sub-system and has led various upstreaming 
> sub-projects related to the Prestera sub-system, Including Prestera 
> sub-system efforts related to the Marvell AC5/X SOC drivers upstreaming. 
> This included technical review and guidance on the technical aspects and 
> code content of the patches sent for review.
> In addition, Elad is a member of the internal review group of code 
> before it applies as a PR.

I see 4 mentions of Elad Nachman in the entire git history.

The distinction between the kernel community and the corporate Linux
involvement is something I don't quite know how to verbalize.
And I don't know whether my perspective is shared by others.

Linux has taken over the world (at least the technical world) so having
Linux kernel exposure is common. But building a product based on Linux
which is then packaged and shipped to customers, in the usual corp
product development methodology, translates very poorly to developing
upstream. This is more true in networking that other parts of the
kernel, to my knowledge, because we attempt to build vendor-independent

While I do not mean to question Elad's expertise and capability as an
engineer/lead/manager, and very much appreciate Marvell's investment 
in the upstream drivers for Prestera and in DENT -- I think the
community involvement is lacking. Short to medium term we should try to
find a way of improving this situation, we can clarify what we expect
from you and if you have ideas on how we can make the involvement**
easier - we'd love to hear them.

** community involvement ideas, less interested in how we can make the
   "ship products" part easier, but you can share those too

> Finally, do note the fact that I will continue to maintain/support this 
> driver, but I would like to have someone that I can share the effort with.

Understandable. I hope PLVision does not disappear form the picture.
We are really allergic to the "push the driver upstream and disengage"
or "throw it over the wall open source" model, if you will.

Unfortunately we only have one nuclear button for discouraging such
arrangements (git-rm), which will hopefully never be used.

To conclude, I think we should have a call first, and then decide
who the best choice for a maintainer is. 

  parent reply	other threads:[~2022-12-01 21:17 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-11-28  9:39 [PATCH v2] MAINTAINERS: Update maintainer for Marvell Prestera Ethernet Switch driver Vadym Kochan
2022-11-30  5:14 ` Jakub Kicinski
2022-12-01  8:39   ` Taras Chornyi
2022-12-01 20:43     ` Andrew Lunn
2022-12-01 21:17     ` Jakub Kicinski [this message]
2022-12-07 12:36       ` Taras Chornyi
2022-12-07 16:53         ` Jakub Kicinski

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \ \ \ \ \ \ \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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 NNTP newsgroup(s).