> From: Jason Gunthorpe [mailto:jgg@xxxxxxxx] > 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? > > Jason 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 have received complaints that our qedr module doesn't have a version whereas all of our other components do (qed, qede, qedi, qedf). We decided to add the qedr version with the next version update to align with the rest of the components. We can move the driver version bump into a different commit for all components, just made sense to Add it to this one as it is the root of the version update. Let me know if you think it is essential and I'll make the change for v2 Thanks, Michal -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
