From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jason Gunthorpe Subject: Re: [PATCH net-next] qed*: Utilize FW 8.33.11.0 Date: Tue, 27 Mar 2018 16:27:16 -0600 Message-ID: <20180327222716.GA3918@ziepe.ca> References: <1522083732-26560-1-git-send-email-Michal.Kalderon@cavium.com> <20180326211827.GG15554@ziepe.ca> <20180327175024.GD1877@mtr-leonro.local> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "Kalderon, Michal" , "davem@davemloft.net" , "netdev@vger.kernel.org" , "linux-rdma@vger.kernel.org" , "linux-scsi@vger.kernel.org" , "Tayar, Tomer" , "Rangankar, Manish" , "Elior, Ariel" To: Leon Romanovsky Return-path: Received: from mail-wr0-f194.google.com ([209.85.128.194]:37904 "EHLO mail-wr0-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752531AbeC0W1W (ORCPT ); Tue, 27 Mar 2018 18:27:22 -0400 Received: by mail-wr0-f194.google.com with SMTP id m13so409045wrj.5 for ; Tue, 27 Mar 2018 15:27:21 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20180327175024.GD1877@mtr-leonro.local> Sender: netdev-owner@vger.kernel.org List-ID: On Tue, Mar 27, 2018 at 08:50:24PM +0300, Leon Romanovsky wrote: > On Tue, Mar 27, 2018 at 05:41:51PM +0000, Kalderon, Michal wrote: > > > From: Jason Gunthorpe [mailto:jgg@ziepe.ca] > > > Sent: Tuesday, March 27, 2018 12:18 AM > > > > diff --git a/drivers/infiniband/hw/qedr/main.c > > > > b/drivers/infiniband/hw/qedr/main.c > > > > index db4bf97..7dbbe6d 100644 > > > > +++ b/drivers/infiniband/hw/qedr/main.c > > > > @@ -51,6 +51,7 @@ > > > > MODULE_DESCRIPTION("QLogic 40G/100G ROCE Driver"); > > > > MODULE_AUTHOR("QLogic Corporation"); MODULE_LICENSE("Dual > > > BSD/GPL"); > > > > +MODULE_VERSION(QEDR_MODULE_VERSION); > > > > > > > > #define QEDR_WQ_MULTIPLIER_DFT (3) > > > > > > > > diff --git a/drivers/infiniband/hw/qedr/qedr.h > > > > b/drivers/infiniband/hw/qedr/qedr.h > > > > index 86d4511..ab0d411 100644 > > > > +++ b/drivers/infiniband/hw/qedr/qedr.h > > > > @@ -43,6 +43,8 @@ > > > > #include "qedr_hsi_rdma.h" > > > > > > > > #define QEDR_NODE_DESC "QLogic 579xx RoCE HCA" > > > > +#define QEDR_MODULE_VERSION "8.33.11.20" > > > > + > > > > > > I thought we had a general prohibition against versions like > > > this in mainline drivers? And what does this hunk have to do > > > with supporting new firmware? > > > > > I'm assuming you refer only to rdma in regards to version > > prohibition right ? as looking at all other vendors (including > > Mellanox) all have module versions under net/ why is rdma > > different in this way ? I now searched back mails on the topic > > and found an email from Leon where he stated: " I am strongly > > against module versions. You should rely on official kernel > > version." But it's not always the inbox driver that is installed > > or probed, the kernel version is not enough. Given different > > distros, vanilla kernels, out of box drivers, etc... it is > > essential for us that based on logs And modinfo we can determine > > the qed* drivers that are running. > > We actually stopped to maintain driver versions, just ensure that inbox, > upstream and MLNX_OFED have different names. > > The discussion thread is here > https://lists.linuxfoundation.org/pipermail/ksummit-discuss/2017-June/004426.html > https://lists.linuxfoundation.org/pipermail/ksummit-discuss/2017-June/004441.html Hmm, Linus pretty clearly said No to MODULE_VERSION and related. So I can't take this hunk, and you shouldn't do in ethernet either, I guess. Honestly the idea that this version will somehow have meaning in the distro kernels is pretty far fetched. You think distros will backport patches changing version # in any way that will make some kind of sense? Jason