linux-kernel - Re: [PATCH v2] MAINTAINERS: Update maintainer for Marvell Prestera Ethernet Switch driver

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: <Y4kR+qxXCnM/5mde@lunn.ch>
Date:   Thu, 1 Dec 2022 21:43:38 +0100
From:   Andrew Lunn <andrew@...n.ch>
To:     Taras Chornyi <taras.chornyi@...ision.eu>
Cc:     Jakub Kicinski <kuba@...nel.org>,
        Vadym Kochan <vadym.kochan@...ision.eu>,
        "David S. Miller" <davem@...emloft.net>,
        Paolo Abeni <pabeni@...hat.com>, netdev@...r.kernel.org,
        Elad Nachman <enachman@...vell.com>,
        Mickey Rachamim <mickeyr@...vell.com>,
        linux-kernel@...r.kernel.org, Taras Chornyi <tchornyi@...vell.com>
Subject: Re: [PATCH v2] MAINTAINERS: Update maintainer for Marvell Prestera
 Ethernet Switch driver

On Thu, Dec 01, 2022 at 10:39:07AM +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.
> 
> 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.

Hi Taras

The problem we have is that all this is totally opaque to us, in
netdev. The name Elad Nachman does not appear on a single patch in
mainline git for the Prestera. All i can find anywhere in mainline is
one patch in 2018 for stmmac.

The community bases a lot of its judgements on trust.  We look at
contributions, be it patches, review comments, or helping others
finding bugs etc, and slowly build up a level of trust. And at the
moment there is nothing to base trust on.

So lets wait until there are a number of merged patchsets, trust has
been established, before adding this entry. It should not take too
long, given his level of experience. As i've said before, you become a
Maintainer by being a Maintainer.

      Andrew