Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <dc9fb975-6258-0473-3ed9-58d3a74e501a@plvision.eu>
Date:   Wed, 7 Dec 2022 14:36:02 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.13.1
Subject: Re: [PATCH v2] MAINTAINERS: Update maintainer for Marvell Prestera
 Ethernet Switch driver
Content-Language: en-US
To:     Jakub Kicinski <kuba@kernel.org>
Cc:     Vadym Kochan <vadym.kochan@plvision.eu>,
        "David S. Miller" <davem@davemloft.net>,
        Paolo Abeni <pabeni@redhat.com>, netdev@vger.kernel.org,
        Elad Nachman <enachman@marvell.com>,
        Mickey Rachamim <mickeyr@marvell.com>,
        linux-kernel@vger.kernel.org, Taras Chornyi <tchornyi@marvell.com>
References: <20221128093934.1631570-1-vadym.kochan@plvision.eu>
 <20221129211405.7d6de0d5@kernel.org>
 <96e3d5fc-ab8c-2344-3266-3b73664499f1@plvision.eu>
 <20221201131744.6e94c5f7@kernel.org>
From:   Taras Chornyi <taras.chornyi@plvision.eu>
In-Reply-To: <20221201131744.6e94c5f7@kernel.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Uk9tMTkwS211eG41NTB5NmRkK2JDRE9OOFYrREw0ZmlBNGxwNzdSajlhRU5z?=
 =?utf-8?B?V0NZcE5Nam4rRWFoMkxhMFpObTlrVy90ZitDbEZ0WTc2cHhWaXdQaHVLY3F1?=
 =?utf-8?B?TjRFVDAvM3FzWmprbTNpUEJPL3dnUGxjUkNRREkyNFowbncvTGlnbnMzQWxC?=
 =?utf-8?B?VGhMN3JlNGM5K1p6KzMwRFRIc0ZSZlJNT3BEdTdRL0hHZ0VTMG1YcTBwanY5?=
 =?utf-8?B?Z2d2WlZaUmwzQWF4VWUwRk5wbHNYSys3aFB6K3NIeEFsVVpQNUZRc2ZMa2Vi?=
 =?utf-8?B?SStnNU1MWkRhUmFqdGFySVlheHVFdDZETTdIUzFsZXJtcFJhZjg2MTRVek5o?=
 =?utf-8?B?NUNWeGNDMFVJVW5VVldwRFdwbXhwc1pBY1ZsdEl0aGovTkxCa0t1VlI4Ti9h?=
 =?utf-8?B?c0MrZ2RSTXpQQ1UyVEJmZDNBRlBaalJ6SEZlU1I1RDRNQmdBaHI2QVhPQ2N6?=
 =?utf-8?B?YURrY05qS3c2WTZZK3ZlZ1FQNXdHRXlSRGtCVTBZeUtBVVhKSkNtSnowSHlt?=
 =?utf-8?B?aGdtM2JsczhBaTVnZFdCeWJuVHErQmdVYm01Mmp2Y2dibE5la281cjYyVVI0?=
 =?utf-8?B?SEZWUGdsWllOU2F4OG5nMjVDdWI2WUxGeVVhQklEVmZoc2JWYThkZFk0QmdC?=
 =?utf-8?B?N3VOSHRUWWJsanBwcU12aU9HZmhJRHowYmRIbUZpVTVEZ0dmc1pScGFRdHJB?=
 =?utf-8?B?a0NIQTI0QUcwRW5kZ25BSVV3aDVYdXlMek5seWdlWGdVTVhKSTQyY3VXdU5q?=
 =?utf-8?B?U0wzb3dEUG9wUXJ6Smt6NjNOc1FiajFsemFHR3dPVE0zNWMvVXZtWUUvcUtX?=
 =?utf-8?B?Zyt6akMzVHlSbDJUQzIwVlVLZ3ordlQ1Yk4vdWxSSnhvWHFhYThwVEZ6VmhM?=
 =?utf-8?B?aUtWTDdLTjFtcmRIV1FQRVJibDBieEhnZlBIQU9rRk9xL2tJVTVGRXFUWUt3?=
 =?utf-8?B?MHdobHhwTXo0dUxsK3FXMTRuNTd1T0VBdytqYUhzY0x3REJqdEdpZmZXNFp2?=
 =?utf-8?B?VEhhazZ0SDlXdGVyb1dNSG1jN0NEbU1meHlPTitCUENoTnppNDc1UitVd3hO?=
 =?utf-8?B?U1Zscy84b3E1b0R1ZS8zL0prOWZHUlpSdHZ5ZEZSSnFxQzRFVTJtM2ZTUzVL?=
 =?utf-8?B?clo5NnA1am0zalZRREMrM2MvNmR3WmRxZ2dmMHUzOEM0RkM4b09FN2hIOFBm?=
 =?utf-8?B?dXZ3RVQ1MzVzODkyUHIweGk0NE5iMkRYNUlLZEFBSXZTanQvak5ENFVPaW8x?=
 =?utf-8?B?UlVnOGZrMXNZUTBjL2YzRjBqakNrTE5HREFrOWZMNWdRUlNXUHNWaFUxZlpl?=
 =?utf-8?B?am8yRGg2T3NhOWUzL2wwYWl1WjBkYWdLZTJndkswSVhQTVUvUkF0bURnZDlu?=
 =?utf-8?B?QTQveDczb3AxWXdmdlBSZTdyNS9ETkRUV3ZYaUMrVStQY0VZVzNnV2RBK2ZI?=
 =?utf-8?B?SDhxcmIzN2FFdldNNTNMNkt3SVBKdGcydStINHZROU5Uc0prcVFwZVJZSUVN?=
 =?utf-8?B?dG1sOEVPdjRNVnUzRFc1U2hIQ2hiWnB5M0h5S0lvdnVhM05kODQ2MHVjTWFU?=
 =?utf-8?B?Zms3UzlJSnJzcUIvSXE1Zml4MXBhbjBhZXB0UTNVTkg0TzAvdzBkb0V0MFJM?=
 =?utf-8?B?QytERkhoVGM0MmVXaXdQUUFSRGc2TWxzT2xwbXRTR3gzYlk2RHY0WGN5bm41?=
 =?utf-8?B?dTg3d0tWdjU1UDhuQ2F2OTdKV2ljZ3ZvSUV5dktIOTh6WjMyWDJsNWc3eFZB?=
 =?utf-8?B?dTZNaFpwOHFPb2liQUNIRnM4UGZhemNaNFhMTUpkd3RuK3VaQWV4YnNJNzJm?=
 =?utf-8?B?SDJZVHgzSjdsUk5OaHZkTzlsZUtFNzByaEVpSkp1dHhSMWpqaEVmV0NJMm9Q?=
 =?utf-8?B?ejRZWnR5TGFoY3BmZFZsMVBuMTZnL051dEFvN1d4TS9FR0tMTTBqN0tJT3Mz?=
 =?utf-8?B?emU1NDZrakczd3MzNlRxMHZjQVdnemJ3SmU5NFN0Z3ZldDFPZGI5QzRhZ09K?=
 =?utf-8?B?M0tZeTFEWGZ4TmhicDJ5OCtaQ1M0NzJhNjFBRTVIaU9FSldlejBlUjRVRjR6?=
 =?utf-8?B?b0s5bm1HK1lJZkNoK3BsMENxT0RWVEVpK2JmN3NYbEs5WTJlU1RYVjI5QXpC?=
 =?utf-8?B?MWtVNFJiNUhRU2ZrRjZrVmVUUTh5SmxZbnB6eDZvT0JIU0ZKQThHZmdTbCsy?=
 =?utf-8?B?WUE9PQ==?=
X-OriginatorOrg: plvision.eu
X-MS-Exchange-CrossTenant-Network-Message-Id: 558d1b16-8f1e-4e9a-aa11-08dad84f9b28
X-MS-Exchange-CrossTenant-AuthSource: GVXP190MB1990.EURP190.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Dec 2022 12:36:04.5698
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 03707b74-30f3-46b6-a0e0-ff0a7438c9c4
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: FtaxmTNelsVOj/qOnSqI5ucx5l4AxC43CZOLpB1kkupbtQYIG3Zu+xcmejcge8KO0e2NKlFuUKzl6UuE75Usqn0sDDG08LOX/aoJziSvges=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXP190MB1840
X-Spam-Status: No, score=-2.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 01.12.22 23:17, Jakub Kicinski wrote:
> 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
> abstractions.
> 
> 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.

So we will drop this patch and create a new one with changing my email 
to PLVision one.