Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
Date:   Wed, 18 Oct 2023 15:56:29 -0300
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc:     Saeed Mahameed <saeed@kernel.org>, Arnd Bergmann <arnd@arndb.de>,
        linux-kernel@vger.kernel.org, Leon Romanovsky <leonro@nvidia.com>,
        Jiri Pirko <jiri@nvidia.com>,
        Saeed Mahameed <saeedm@nvidia.com>
Subject: Re: [PATCH 2/5] misc: mlx5ctl: Add mlx5ctl misc driver
Message-ID: <20231018185629.GD3952@nvidia.com>
References: <20231018081941.475277-1-saeed@kernel.org>
 <20231018081941.475277-3-saeed@kernel.org>
 <2023101835-trapdoor-unicycle-788a@gregkh>
 <20231018180128.GA719006@nvidia.com>
 <2023101808-quicksand-roman-0da7@gregkh>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <2023101808-quicksand-roman-0da7@gregkh>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?GWwr1X/f7tH4aq0z2DESOsR4n7n27/GgLbBqMOjbcOJu8+H3aMsaj0g8a6w/?=
 =?us-ascii?Q?EYUgUaec6ap8SyAe/t9hIWoZmOTojEJ9J18F19H9r2/7w8m7SXu01TwznYqv?=
 =?us-ascii?Q?aXE6P0511sdHf2SP0v2TaQbmLnA8WdlyMn9rR1zTZHW5LkyK7Sr6tda08+aU?=
 =?us-ascii?Q?BeZkXfjwn/p3Mx0/XK8zkAuSQWHWwAmLKnpCHDiE7xaY2p68Avy3iHGtKNau?=
 =?us-ascii?Q?drWA0Pm/E/xtRyy/EnaoYnVcRDfMIrCvDg+KOSRCF2hk8ZdGPUjSWUXmloXX?=
 =?us-ascii?Q?MPbCY2Vqiq8tA3KXiEwsHmMVnJ5BsetHFADLy89cuPjVh5ntPsACLixw1t8T?=
 =?us-ascii?Q?3S+nzM2sI5r2BW9QvsTvBn6HWqHNpek2XI0ts6UWghuRCrOLdC/wKwAkXOej?=
 =?us-ascii?Q?85LC+IdB4g7QDrL63/Ctzm49Q6diDbbFh6E7oSJ4EWI0PDNcDEOEND9pTJcl?=
 =?us-ascii?Q?XPFoeNe5wibyriIowJdsN85HdOy8EcLNKdxEkOFXgo1PDezao6yY8Nks34u8?=
 =?us-ascii?Q?f8ttoGgaR457srHA3HCCFi/oTq9T3QbZUFSqHn87sbNMiV6LyVOVbWBxa8rJ?=
 =?us-ascii?Q?JVDT6HhIWQeid7lxOlMVxQE/SQcoemq6gFcSb5XQtBNPYKd13/86WcKG+IQ9?=
 =?us-ascii?Q?kEW1pFc/ydQ+9jcGe5OTcSkwue2IrzRSjheGoefHlL0ojIkQF7x9T3eci+w/?=
 =?us-ascii?Q?cX99zXKTvRseUFHiUwANTBm9XkTVxM5lZvW3rIu5WnEJ+/5ENnGhi5e437nn?=
 =?us-ascii?Q?DEs2nOeSnkv78FsVvtfZ0pYCMIx4kiyoI/bNUZ08BSu8xhBEOBKh55Oxm7ZP?=
 =?us-ascii?Q?qWNro5ScmC+MSgNA1qzwh27Prsso/0KZS5HR2/00CMKePEZ282wP8LfKZTQP?=
 =?us-ascii?Q?jF3fx3JldwpQqPH8NtnvJB62FwbuzIXgO40oSyYC8sJZzUNq5qOOQeSuy5k1?=
 =?us-ascii?Q?pFReXmaJZwv2jtpWEdpkJRQI9FoF4+m0KyXeQAr5HcH2MCaugHBM2aW4HkAk?=
 =?us-ascii?Q?FtQ39P6uoem0xQf5P0DWXDAHmCQzrI/gGwy6ycfGwpvFpF2ABMRnTRiV/7Is?=
 =?us-ascii?Q?Si4UbGdqw9+9C0LQr6W/j8qsCmtBnCquKMnzaGv+aB70UTkA6GB2tkb0yuu8?=
 =?us-ascii?Q?ymp/NVJys+lrKpbK1nBBn8Yt/r5QgHv/nhNJ3L9TU+M+C8AJMjGEqoXxlgFO?=
 =?us-ascii?Q?7iv96UHYe6XewTejYVs4g/s4JlPZMuNr6995J7MCPowUK2z9jTHjS+TCEYCd?=
 =?us-ascii?Q?E1yIMQK9f5E4ZPuxA3WLOez85DSSsJ0AMmWXXsufYKvDhFwMI9jpvOam0EbK?=
 =?us-ascii?Q?LEup4eh2mqq1IjQ3BuL9UVdppbg3QSEbntyh8Ar5rhx+1qaCV49bW9gOf1bq?=
 =?us-ascii?Q?n3B+TJZ8JJqmNxr++aoWw+gDeDjOLXSXIWE8BTkCWx265plsr2nZFHK9kV5y?=
 =?us-ascii?Q?OvidtTKcYDM2x7ZgRnZnVv/lssTZNApUqe/UENu4qyVqdZwuq1DPfRKPYvEt?=
 =?us-ascii?Q?mCZNCm9nwcmpAsAdhnchrAOUl9r3V6hTQTShDPvvhcQRbECJ865p0gS492TX?=
 =?us-ascii?Q?4e9Vd79XhVymyhxAdmYJ3GxkpO1b+JdEekQPBrx/?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 248a6a41-4642-4f34-e6a6-08dbd00bf155
X-MS-Exchange-CrossTenant-AuthSource: LV2PR12MB5869.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Oct 2023 18:56:31.6524
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Rf41EOQDb30wcHUbbeDtc7f7TBekbszMvVm83g7GSiQAMGyO12UjawHdGfdJFaXe
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR12MB4851
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable
	autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Wed, 18 Oct 2023 11:56:53 -0700 (PDT)

On Wed, Oct 18, 2023 at 08:22:39PM +0200, Greg Kroah-Hartman wrote:
> On Wed, Oct 18, 2023 at 03:01:28PM -0300, Jason Gunthorpe wrote:
> > On Wed, Oct 18, 2023 at 10:30:00AM +0200, Greg Kroah-Hartman wrote:
> > > On Wed, Oct 18, 2023 at 01:19:38AM -0700, Saeed Mahameed wrote:
> > > > +// SPDX-License-Identifier: GPL-2.0 OR Linux-OpenIB
> > > 
> > > For dual-licensed code, I need a LOT of documentation as to why this
> > > must be dual-licensed, AND a signed-off-by from your corporate lawyer
> > > agreeing to it so they convey an understanding of just how complex and
> > > messy this will get over time and what you are agreeing to do here.
> > 
> > Can you provide a brief or whitepaper discussing this complexity
> > please? This pushback is news to me, Mellanox and the RDMA ecosystem
> > has been doing this for over 15 years now. I would need something
> > substantive to have a conversation with our legal.
> 
> Have your legal talk to the LF legal working group, they are the ones
> that told me never to mess with this license again.  I'm sure that
> nvidia's lawyers are part of this group, so let's let them hash it
> out.

Are you talking about OpenIB specifically or the concept of dual
license (eg GPL/MIT) in general?

> > However, I believe we can get an exception approval for single license
> > MIT or BSD-3-Clause for this code.
> 
> GPLv2 please, otherwise again, I'm going to demand a really really good
> reason why Linux kernel code needs a non-GPL license and again, a sign
> off from your lawyers explaining why it must be so.

All of the Mellanox driver stack (over 400 files now!) is dual
licensed because we have a large team of people working the Mellanox
driver for many operating systems with many different licenses. We
want the certainty of a permissive license for the driver code we
supply to Linux as the team routinely references and/or re-uses
Mellanox authored Linux driver code into other scenarios under the
permissive side of the dual license.

For instance I could easily see the work Saeed has done here finding
its way into FreeBSD. We significantly support FreeBSD employing
maintainers and develop a sophisticated Mellanox driver over
there. This would not be possible without the Linux driver being dual
licensed.

This has been the direction from our legal for a long time.

AFAIK this has also been a long time accepted Linux practice, there
are many examples in the driver tree. What has changed now that Saeed
tries to add 3 more files the giant driver? I need a bigger
explanation if we are going to revisit this practice with our legal.

To be clear, we can surely get the approvals to remove the offensive
OpenIB from these files. eg mlxsw is already approved using
"BSD-3-Clause OR GPL-2.0".

Further, as a maintainer myself, is this now the community consensus
expected review standard?  When was it discussed? I do not see
evidence of a ban on dual licensing in
https://docs.kernel.org/process/license-rules.html ?

Thanks,
Jason