diff mbox

[rdma-next] RDMA: Remove useless MODULE_VERSION

Message ID 20170626055822.18946-1-leon@kernel.org (mailing list archive)
State Accepted
Headers show

Commit Message

Leon Romanovsky June 26, 2017, 5:58 a.m. UTC
All modules in drivers/infiniband defined and used MODULE_VERSION, which
was pointless because the kernel version describes their state more accurate
then those arbitrary numbers.

Signed-off-by: Leon Romanovsky <leon@kernel.org>
---
Followup of the following discussion:
https://lists.linuxfoundation.org/pipermail/ksummit-discuss/2017-June/004421.html
---
 drivers/infiniband/hw/bnxt_re/main.c            | 1 -
 drivers/infiniband/hw/cxgb3/iwch.c              | 1 -
 drivers/infiniband/hw/cxgb4/device.c            | 1 -
 drivers/infiniband/hw/hfi1/driver.c             | 1 -
 drivers/infiniband/hw/i40iw/i40iw_main.c        | 1 -
 drivers/infiniband/hw/mlx4/main.c               | 1 -
 drivers/infiniband/hw/mlx5/main.c               | 1 -
 drivers/infiniband/hw/mthca/mthca_main.c        | 1 -
 drivers/infiniband/hw/nes/nes.c                 | 1 -
 drivers/infiniband/hw/ocrdma/ocrdma_main.c      | 1 -
 drivers/infiniband/hw/qedr/main.c               | 1 -
 drivers/infiniband/hw/qedr/qedr.h               | 1 -
 drivers/infiniband/hw/qib/qib_driver.c          | 1 -
 drivers/infiniband/hw/usnic/usnic_ib_main.c     | 1 -
 drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c  | 1 -
 drivers/infiniband/sw/rxe/rxe.c                 | 1 -
 drivers/infiniband/ulp/ipoib/ipoib_main.c       | 1 -
 drivers/infiniband/ulp/iser/iscsi_iser.c        | 1 -
 drivers/infiniband/ulp/isert/ib_isert.c         | 1 -
 drivers/infiniband/ulp/opa_vnic/opa_vnic_vema.c | 1 -
 drivers/infiniband/ulp/srp/ib_srp.c             | 1 -
 21 files changed, 21 deletions(-)

--
2.13.1

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Comments

Sagi Grimberg June 27, 2017, 9:59 a.m. UTC | #1
for iser/isert

Acked-by: Sagi Grimbrg <sagi@grimberg.me>

For the rest,

Reviewed-by: Sagi Grimberg <sagi@grimbeg.me>
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Dennis Dalessandro June 27, 2017, 6:28 p.m. UTC | #2
On 6/26/2017 1:58 AM, Leon Romanovsky wrote:
> All modules in drivers/infiniband defined and used MODULE_VERSION, which
> was pointless because the kernel version describes their state more accurate
> then those arbitrary numbers.

For hfi1:

Acked-by: Dennis Dalessandro <dennis.dalessandro@intel.com>

For qib and vnic:

Reviewed-by: Dennis Dalessandro <dennis.dalessandro@intel.com>

-Denny
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Selvin Xavier June 28, 2017, 10:09 a.m. UTC | #3
For bnxt_re and ocrdma changes:

Acked-by: Selvin Xavier <selvin.xavier@broadcom.com>

On Mon, Jun 26, 2017 at 11:28 AM, Leon Romanovsky <leon@kernel.org> wrote:
> All modules in drivers/infiniband defined and used MODULE_VERSION, which
> was pointless because the kernel version describes their state more accurate
> then those arbitrary numbers.
>
> Signed-off-by: Leon Romanovsky <leon@kernel.org>
> ---
> Followup of the following discussion:
> https://lists.linuxfoundation.org/pipermail/ksummit-discuss/2017-June/004421.html
> ---
>  drivers/infiniband/hw/bnxt_re/main.c            | 1 -
>  drivers/infiniband/hw/cxgb3/iwch.c              | 1 -
>  drivers/infiniband/hw/cxgb4/device.c            | 1 -
>  drivers/infiniband/hw/hfi1/driver.c             | 1 -
>  drivers/infiniband/hw/i40iw/i40iw_main.c        | 1 -
>  drivers/infiniband/hw/mlx4/main.c               | 1 -
>  drivers/infiniband/hw/mlx5/main.c               | 1 -
>  drivers/infiniband/hw/mthca/mthca_main.c        | 1 -
>  drivers/infiniband/hw/nes/nes.c                 | 1 -
>  drivers/infiniband/hw/ocrdma/ocrdma_main.c      | 1 -
>  drivers/infiniband/hw/qedr/main.c               | 1 -
>  drivers/infiniband/hw/qedr/qedr.h               | 1 -
>  drivers/infiniband/hw/qib/qib_driver.c          | 1 -
>  drivers/infiniband/hw/usnic/usnic_ib_main.c     | 1 -
>  drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c  | 1 -
>  drivers/infiniband/sw/rxe/rxe.c                 | 1 -
>  drivers/infiniband/ulp/ipoib/ipoib_main.c       | 1 -
>  drivers/infiniband/ulp/iser/iscsi_iser.c        | 1 -
>  drivers/infiniband/ulp/isert/ib_isert.c         | 1 -
>  drivers/infiniband/ulp/opa_vnic/opa_vnic_vema.c | 1 -
>  drivers/infiniband/ulp/srp/ib_srp.c             | 1 -
>  21 files changed, 21 deletions(-)
>
> diff --git a/drivers/infiniband/hw/bnxt_re/main.c b/drivers/infiniband/hw/bnxt_re/main.c
> index 5d355401179b..239b51685884 100644
> --- a/drivers/infiniband/hw/bnxt_re/main.c
> +++ b/drivers/infiniband/hw/bnxt_re/main.c
> @@ -70,7 +70,6 @@ static char version[] =
>  MODULE_AUTHOR("Eddie Wai <eddie.wai@broadcom.com>");
>  MODULE_DESCRIPTION(BNXT_RE_DESC " Driver");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(ROCE_DRV_MODULE_VERSION);
>
>  /* globals */
>  static struct list_head bnxt_re_dev_list = LIST_HEAD_INIT(bnxt_re_dev_list);
> diff --git a/drivers/infiniband/hw/cxgb3/iwch.c b/drivers/infiniband/hw/cxgb3/iwch.c
> index 47b2ce2ef203..591de319c178 100644
> --- a/drivers/infiniband/hw/cxgb3/iwch.c
> +++ b/drivers/infiniband/hw/cxgb3/iwch.c
> @@ -45,7 +45,6 @@
>  MODULE_AUTHOR("Boyd Faulkner, Steve Wise");
>  MODULE_DESCRIPTION("Chelsio T3 RDMA Driver");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(DRV_VERSION);
>
>  static void open_rnic_dev(struct t3cdev *);
>  static void close_rnic_dev(struct t3cdev *);
> diff --git a/drivers/infiniband/hw/cxgb4/device.c b/drivers/infiniband/hw/cxgb4/device.c
> index f96a96dbcf1f..cef48d1d83b9 100644
> --- a/drivers/infiniband/hw/cxgb4/device.c
> +++ b/drivers/infiniband/hw/cxgb4/device.c
> @@ -44,7 +44,6 @@
>  MODULE_AUTHOR("Steve Wise");
>  MODULE_DESCRIPTION("Chelsio T4/T5 RDMA Driver");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(DRV_VERSION);
>
>  static int allow_db_fc_on_t5;
>  module_param(allow_db_fc_on_t5, int, 0644);
> diff --git a/drivers/infiniband/hw/hfi1/driver.c b/drivers/infiniband/hw/hfi1/driver.c
> index a50870e455a3..79a8b05855ac 100644
> --- a/drivers/infiniband/hw/hfi1/driver.c
> +++ b/drivers/infiniband/hw/hfi1/driver.c
> @@ -96,7 +96,6 @@ MODULE_PARM_DESC(cap_mask, "Bit mask of enabled/disabled HW features");
>
>  MODULE_LICENSE("Dual BSD/GPL");
>  MODULE_DESCRIPTION("Intel Omni-Path Architecture driver");
> -MODULE_VERSION(HFI1_DRIVER_VERSION);
>
>  /*
>   * MAX_PKT_RCV is the max # if packets processed per receive interrupt.
> diff --git a/drivers/infiniband/hw/i40iw/i40iw_main.c b/drivers/infiniband/hw/i40iw/i40iw_main.c
> index a3f18a22f5ed..3b22d925baa5 100644
> --- a/drivers/infiniband/hw/i40iw/i40iw_main.c
> +++ b/drivers/infiniband/hw/i40iw/i40iw_main.c
> @@ -77,7 +77,6 @@ MODULE_PARM_DESC(mpa_version, "MPA version to be used in MPA Req/Resp 1 or 2");
>  MODULE_AUTHOR("Intel Corporation, <e1000-rdma@lists.sourceforge.net>");
>  MODULE_DESCRIPTION("Intel(R) Ethernet Connection X722 iWARP RDMA Driver");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(DRV_VERSION);
>
>  static struct i40e_client i40iw_client;
>  static char i40iw_client_name[I40E_CLIENT_STR_LENGTH] = "i40iw";
> diff --git a/drivers/infiniband/hw/mlx4/main.c b/drivers/infiniband/hw/mlx4/main.c
> index 521d0def2d9e..93601cdb6532 100644
> --- a/drivers/infiniband/hw/mlx4/main.c
> +++ b/drivers/infiniband/hw/mlx4/main.c
> @@ -71,7 +71,6 @@
>  MODULE_AUTHOR("Roland Dreier");
>  MODULE_DESCRIPTION("Mellanox ConnectX HCA InfiniBand driver");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(DRV_VERSION);
>
>  int mlx4_ib_sm_guid_assign = 0;
>  module_param_named(sm_guid_assign, mlx4_ib_sm_guid_assign, int, 0444);
> diff --git a/drivers/infiniband/hw/mlx5/main.c b/drivers/infiniband/hw/mlx5/main.c
> index 0c79983c8b1a..a7a0756a6996 100644
> --- a/drivers/infiniband/hw/mlx5/main.c
> +++ b/drivers/infiniband/hw/mlx5/main.c
> @@ -66,7 +66,6 @@
>  MODULE_AUTHOR("Eli Cohen <eli@mellanox.com>");
>  MODULE_DESCRIPTION("Mellanox Connect-IB HCA IB driver");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(DRIVER_VERSION);
>
>  static char mlx5_version[] =
>         DRIVER_NAME ": Mellanox Connect-IB Infiniband driver v"
> diff --git a/drivers/infiniband/hw/mthca/mthca_main.c b/drivers/infiniband/hw/mthca/mthca_main.c
> index c309e5c96383..1b10d21c8026 100644
> --- a/drivers/infiniband/hw/mthca/mthca_main.c
> +++ b/drivers/infiniband/hw/mthca/mthca_main.c
> @@ -49,7 +49,6 @@
>  MODULE_AUTHOR("Roland Dreier");
>  MODULE_DESCRIPTION("Mellanox InfiniBand HCA low-level driver");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(DRV_VERSION);
>
>  #ifdef CONFIG_INFINIBAND_MTHCA_DEBUG
>
> diff --git a/drivers/infiniband/hw/nes/nes.c b/drivers/infiniband/hw/nes/nes.c
> index 5b9601014f0c..ab6232ab136b 100644
> --- a/drivers/infiniband/hw/nes/nes.c
> +++ b/drivers/infiniband/hw/nes/nes.c
> @@ -63,7 +63,6 @@
>  MODULE_AUTHOR("NetEffect");
>  MODULE_DESCRIPTION("NetEffect RNIC Low-level iWARP Driver");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(DRV_VERSION);
>
>  int interrupt_mod_interval = 0;
>
> diff --git a/drivers/infiniband/hw/ocrdma/ocrdma_main.c b/drivers/infiniband/hw/ocrdma/ocrdma_main.c
> index 57c9a2ad0260..757c65816295 100644
> --- a/drivers/infiniband/hw/ocrdma/ocrdma_main.c
> +++ b/drivers/infiniband/hw/ocrdma/ocrdma_main.c
> @@ -58,7 +58,6 @@
>  #include "ocrdma_stats.h"
>  #include <rdma/ocrdma-abi.h>
>
> -MODULE_VERSION(OCRDMA_ROCE_DRV_VERSION);
>  MODULE_DESCRIPTION(OCRDMA_ROCE_DRV_DESC " " OCRDMA_ROCE_DRV_VERSION);
>  MODULE_AUTHOR("Emulex Corporation");
>  MODULE_LICENSE("Dual BSD/GPL");
> diff --git a/drivers/infiniband/hw/qedr/main.c b/drivers/infiniband/hw/qedr/main.c
> index 6a72095d6c7a..6c5639caab20 100644
> --- a/drivers/infiniband/hw/qedr/main.c
> +++ b/drivers/infiniband/hw/qedr/main.c
> @@ -47,7 +47,6 @@
>  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 aa08c76a4245..0e95ffd6c14a 100644
> --- a/drivers/infiniband/hw/qedr/qedr.h
> +++ b/drivers/infiniband/hw/qedr/qedr.h
> @@ -41,7 +41,6 @@
>  #include <linux/qed/roce_common.h>
>  #include "qedr_hsi_rdma.h"
>
> -#define QEDR_MODULE_VERSION    "8.10.10.0"
>  #define QEDR_NODE_DESC "QLogic 579xx RoCE HCA"
>  #define DP_NAME(dev) ((dev)->ibdev.name)
>
> diff --git a/drivers/infiniband/hw/qib/qib_driver.c b/drivers/infiniband/hw/qib/qib_driver.c
> index 2b5982f743ef..719906a9fd51 100644
> --- a/drivers/infiniband/hw/qib/qib_driver.c
> +++ b/drivers/infiniband/hw/qib/qib_driver.c
> @@ -66,7 +66,6 @@ MODULE_PARM_DESC(compat_ddr_negotiate,
>  MODULE_LICENSE("Dual BSD/GPL");
>  MODULE_AUTHOR("Intel <ibsupport@intel.com>");
>  MODULE_DESCRIPTION("Intel IB driver");
> -MODULE_VERSION(QIB_DRIVER_VERSION);
>
>  /*
>   * QIB_PIO_MAXIBHDR is the max IB header size allowed for in our
> diff --git a/drivers/infiniband/hw/usnic/usnic_ib_main.c b/drivers/infiniband/hw/usnic/usnic_ib_main.c
> index c0c1e8b027b1..08cbbe224e11 100644
> --- a/drivers/infiniband/hw/usnic/usnic_ib_main.c
> +++ b/drivers/infiniband/hw/usnic/usnic_ib_main.c
> @@ -720,7 +720,6 @@ static void __exit usnic_ib_destroy(void)
>  MODULE_DESCRIPTION("Cisco VIC (usNIC) Verbs Driver");
>  MODULE_AUTHOR("Upinder Malhi <umalhi@cisco.com>");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(DRV_VERSION);
>  module_param(usnic_log_lvl, uint, S_IRUGO | S_IWUSR);
>  module_param(usnic_ib_share_vf, uint, S_IRUGO | S_IWUSR);
>  MODULE_PARM_DESC(usnic_log_lvl, " Off=0, Err=1, Info=2, Debug=3");
> diff --git a/drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c b/drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c
> index 34ebc7615411..e76565280afa 100644
> --- a/drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c
> +++ b/drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c
> @@ -1119,5 +1119,4 @@ module_exit(pvrdma_cleanup);
>
>  MODULE_AUTHOR("VMware, Inc");
>  MODULE_DESCRIPTION("VMware Paravirtual RDMA driver");
> -MODULE_VERSION(DRV_VERSION);
>  MODULE_LICENSE("Dual BSD/GPL");
> diff --git a/drivers/infiniband/sw/rxe/rxe.c b/drivers/infiniband/sw/rxe/rxe.c
> index c21c913f911a..8c3d30b3092d 100644
> --- a/drivers/infiniband/sw/rxe/rxe.c
> +++ b/drivers/infiniband/sw/rxe/rxe.c
> @@ -38,7 +38,6 @@
>  MODULE_AUTHOR("Bob Pearson, Frank Zago, John Groves, Kamal Heib");
>  MODULE_DESCRIPTION("Soft RDMA transport");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION("0.2");
>
>  /* free resources for all ports on a device */
>  static void rxe_cleanup_ports(struct rxe_dev *rxe)
> diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c
> index a115c0b7a310..b11c46db7cc9 100644
> --- a/drivers/infiniband/ulp/ipoib/ipoib_main.c
> +++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c
> @@ -60,7 +60,6 @@ const char ipoib_driver_version[] = DRV_VERSION;
>  MODULE_AUTHOR("Roland Dreier");
>  MODULE_DESCRIPTION("IP-over-InfiniBand net driver");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(DRV_VERSION);
>
>  int ipoib_sendq_size __read_mostly = IPOIB_TX_RING_SIZE;
>  int ipoib_recvq_size __read_mostly = IPOIB_RX_RING_SIZE;
> diff --git a/drivers/infiniband/ulp/iser/iscsi_iser.c b/drivers/infiniband/ulp/iser/iscsi_iser.c
> index 5a887efb4bdf..8ef3c1bd78bf 100644
> --- a/drivers/infiniband/ulp/iser/iscsi_iser.c
> +++ b/drivers/infiniband/ulp/iser/iscsi_iser.c
> @@ -77,7 +77,6 @@
>  MODULE_DESCRIPTION("iSER (iSCSI Extensions for RDMA) Datamover");
>  MODULE_LICENSE("Dual BSD/GPL");
>  MODULE_AUTHOR("Alex Nezhinsky, Dan Bar Dov, Or Gerlitz");
> -MODULE_VERSION(DRV_VER);
>
>  static struct scsi_host_template iscsi_iser_sht;
>  static struct iscsi_transport iscsi_iser_transport;
> diff --git a/drivers/infiniband/ulp/isert/ib_isert.c b/drivers/infiniband/ulp/isert/ib_isert.c
> index fcbed35e95a8..d7cc7c6190ba 100644
> --- a/drivers/infiniband/ulp/isert/ib_isert.c
> +++ b/drivers/infiniband/ulp/isert/ib_isert.c
> @@ -2710,7 +2710,6 @@ static void __exit isert_exit(void)
>  }
>
>  MODULE_DESCRIPTION("iSER-Target for mainline target infrastructure");
> -MODULE_VERSION("1.0");
>  MODULE_AUTHOR("nab@Linux-iSCSI.org");
>  MODULE_LICENSE("GPL");
>
> diff --git a/drivers/infiniband/ulp/opa_vnic/opa_vnic_vema.c b/drivers/infiniband/ulp/opa_vnic/opa_vnic_vema.c
> index 875694f9a7f9..f9910a6c3291 100644
> --- a/drivers/infiniband/ulp/opa_vnic/opa_vnic_vema.c
> +++ b/drivers/infiniband/ulp/opa_vnic/opa_vnic_vema.c
> @@ -1053,4 +1053,3 @@ module_exit(opa_vnic_deinit);
>  MODULE_LICENSE("Dual BSD/GPL");
>  MODULE_AUTHOR("Intel Corporation");
>  MODULE_DESCRIPTION("Intel OPA Virtual Network driver");
> -MODULE_VERSION(DRV_VERSION);
> diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers/infiniband/ulp/srp/ib_srp.c
> index 2354c742caa1..fa5ccdb3bb2a 100644
> --- a/drivers/infiniband/ulp/srp/ib_srp.c
> +++ b/drivers/infiniband/ulp/srp/ib_srp.c
> @@ -62,7 +62,6 @@
>  MODULE_AUTHOR("Roland Dreier");
>  MODULE_DESCRIPTION("InfiniBand SCSI RDMA Protocol initiator");
>  MODULE_LICENSE("Dual BSD/GPL");
> -MODULE_VERSION(DRV_VERSION);
>  MODULE_INFO(release_date, DRV_RELDATE);
>
>  #if !defined(CONFIG_DYNAMIC_DEBUG)
> --
> 2.13.1
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Amrani, Ram June 28, 2017, 12:11 p.m. UTC | #4
For qedr changes:

Acked-by: Ram Amrani <Ram.Amrani@cavium.com>

 
> diff --git a/drivers/infiniband/hw/qedr/main.c b/drivers/infiniband/hw/qedr/main.c
> index 6a72095d6c7a..6c5639caab20 100644
> --- a/drivers/infiniband/hw/qedr/main.c
> +++ b/drivers/infiniband/hw/qedr/main.c
> @@ -47,7 +47,6 @@
>  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 aa08c76a4245..0e95ffd6c14a 100644
> --- a/drivers/infiniband/hw/qedr/qedr.h
> +++ b/drivers/infiniband/hw/qedr/qedr.h
> @@ -41,7 +41,6 @@
>  #include <linux/qed/roce_common.h>
>  #include "qedr_hsi_rdma.h"
> 
> -#define QEDR_MODULE_VERSION	"8.10.10.0"
>  #define QEDR_NODE_DESC "QLogic 579xx RoCE HCA"
>  #define DP_NAME(dev) ((dev)->ibdev.name)
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Johannes Thumshirn June 28, 2017, 12:37 p.m. UTC | #5
On Mon, Jun 26, 2017 at 08:58:22AM +0300, Leon Romanovsky wrote:
> All modules in drivers/infiniband defined and used MODULE_VERSION, which
> was pointless because the kernel version describes their state more accurate
> then those arbitrary numbers.
> 
> Signed-off-by: Leon Romanovsky <leon@kernel.org>
> ---

Highly appreciated :-)
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
Adit Ranadive July 3, 2017, 7:27 p.m. UTC | #6
> All modules in drivers/infiniband defined and used MODULE_VERSION, 
> which was pointless because the kernel version describes their state more 
> accurate then those arbitrary numbers.

> Signed-off-by: Leon Romanovsky <leon@kernel.org>

For vmw_pvrdma -
Acked-by: Adit Ranadive <aditr@vmware.com>
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Doug Ledford July 28, 2017, 5:56 p.m. UTC | #7
On Mon, 2017-06-26 at 08:58 +0300, Leon Romanovsky wrote:
> All modules in drivers/infiniband defined and used MODULE_VERSION,
> which
> was pointless because the kernel version describes their state more
> accurate
> then those arbitrary numbers.
> 
> Signed-off-by: Leon Romanovsky <leon@kernel.org>

Thanks, applied.
diff mbox

Patch

diff --git a/drivers/infiniband/hw/bnxt_re/main.c b/drivers/infiniband/hw/bnxt_re/main.c
index 5d355401179b..239b51685884 100644
--- a/drivers/infiniband/hw/bnxt_re/main.c
+++ b/drivers/infiniband/hw/bnxt_re/main.c
@@ -70,7 +70,6 @@  static char version[] =
 MODULE_AUTHOR("Eddie Wai <eddie.wai@broadcom.com>");
 MODULE_DESCRIPTION(BNXT_RE_DESC " Driver");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(ROCE_DRV_MODULE_VERSION);

 /* globals */
 static struct list_head bnxt_re_dev_list = LIST_HEAD_INIT(bnxt_re_dev_list);
diff --git a/drivers/infiniband/hw/cxgb3/iwch.c b/drivers/infiniband/hw/cxgb3/iwch.c
index 47b2ce2ef203..591de319c178 100644
--- a/drivers/infiniband/hw/cxgb3/iwch.c
+++ b/drivers/infiniband/hw/cxgb3/iwch.c
@@ -45,7 +45,6 @@ 
 MODULE_AUTHOR("Boyd Faulkner, Steve Wise");
 MODULE_DESCRIPTION("Chelsio T3 RDMA Driver");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(DRV_VERSION);

 static void open_rnic_dev(struct t3cdev *);
 static void close_rnic_dev(struct t3cdev *);
diff --git a/drivers/infiniband/hw/cxgb4/device.c b/drivers/infiniband/hw/cxgb4/device.c
index f96a96dbcf1f..cef48d1d83b9 100644
--- a/drivers/infiniband/hw/cxgb4/device.c
+++ b/drivers/infiniband/hw/cxgb4/device.c
@@ -44,7 +44,6 @@ 
 MODULE_AUTHOR("Steve Wise");
 MODULE_DESCRIPTION("Chelsio T4/T5 RDMA Driver");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(DRV_VERSION);

 static int allow_db_fc_on_t5;
 module_param(allow_db_fc_on_t5, int, 0644);
diff --git a/drivers/infiniband/hw/hfi1/driver.c b/drivers/infiniband/hw/hfi1/driver.c
index a50870e455a3..79a8b05855ac 100644
--- a/drivers/infiniband/hw/hfi1/driver.c
+++ b/drivers/infiniband/hw/hfi1/driver.c
@@ -96,7 +96,6 @@  MODULE_PARM_DESC(cap_mask, "Bit mask of enabled/disabled HW features");

 MODULE_LICENSE("Dual BSD/GPL");
 MODULE_DESCRIPTION("Intel Omni-Path Architecture driver");
-MODULE_VERSION(HFI1_DRIVER_VERSION);

 /*
  * MAX_PKT_RCV is the max # if packets processed per receive interrupt.
diff --git a/drivers/infiniband/hw/i40iw/i40iw_main.c b/drivers/infiniband/hw/i40iw/i40iw_main.c
index a3f18a22f5ed..3b22d925baa5 100644
--- a/drivers/infiniband/hw/i40iw/i40iw_main.c
+++ b/drivers/infiniband/hw/i40iw/i40iw_main.c
@@ -77,7 +77,6 @@  MODULE_PARM_DESC(mpa_version, "MPA version to be used in MPA Req/Resp 1 or 2");
 MODULE_AUTHOR("Intel Corporation, <e1000-rdma@lists.sourceforge.net>");
 MODULE_DESCRIPTION("Intel(R) Ethernet Connection X722 iWARP RDMA Driver");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(DRV_VERSION);

 static struct i40e_client i40iw_client;
 static char i40iw_client_name[I40E_CLIENT_STR_LENGTH] = "i40iw";
diff --git a/drivers/infiniband/hw/mlx4/main.c b/drivers/infiniband/hw/mlx4/main.c
index 521d0def2d9e..93601cdb6532 100644
--- a/drivers/infiniband/hw/mlx4/main.c
+++ b/drivers/infiniband/hw/mlx4/main.c
@@ -71,7 +71,6 @@ 
 MODULE_AUTHOR("Roland Dreier");
 MODULE_DESCRIPTION("Mellanox ConnectX HCA InfiniBand driver");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(DRV_VERSION);

 int mlx4_ib_sm_guid_assign = 0;
 module_param_named(sm_guid_assign, mlx4_ib_sm_guid_assign, int, 0444);
diff --git a/drivers/infiniband/hw/mlx5/main.c b/drivers/infiniband/hw/mlx5/main.c
index 0c79983c8b1a..a7a0756a6996 100644
--- a/drivers/infiniband/hw/mlx5/main.c
+++ b/drivers/infiniband/hw/mlx5/main.c
@@ -66,7 +66,6 @@ 
 MODULE_AUTHOR("Eli Cohen <eli@mellanox.com>");
 MODULE_DESCRIPTION("Mellanox Connect-IB HCA IB driver");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(DRIVER_VERSION);

 static char mlx5_version[] =
 	DRIVER_NAME ": Mellanox Connect-IB Infiniband driver v"
diff --git a/drivers/infiniband/hw/mthca/mthca_main.c b/drivers/infiniband/hw/mthca/mthca_main.c
index c309e5c96383..1b10d21c8026 100644
--- a/drivers/infiniband/hw/mthca/mthca_main.c
+++ b/drivers/infiniband/hw/mthca/mthca_main.c
@@ -49,7 +49,6 @@ 
 MODULE_AUTHOR("Roland Dreier");
 MODULE_DESCRIPTION("Mellanox InfiniBand HCA low-level driver");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(DRV_VERSION);

 #ifdef CONFIG_INFINIBAND_MTHCA_DEBUG

diff --git a/drivers/infiniband/hw/nes/nes.c b/drivers/infiniband/hw/nes/nes.c
index 5b9601014f0c..ab6232ab136b 100644
--- a/drivers/infiniband/hw/nes/nes.c
+++ b/drivers/infiniband/hw/nes/nes.c
@@ -63,7 +63,6 @@ 
 MODULE_AUTHOR("NetEffect");
 MODULE_DESCRIPTION("NetEffect RNIC Low-level iWARP Driver");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(DRV_VERSION);

 int interrupt_mod_interval = 0;

diff --git a/drivers/infiniband/hw/ocrdma/ocrdma_main.c b/drivers/infiniband/hw/ocrdma/ocrdma_main.c
index 57c9a2ad0260..757c65816295 100644
--- a/drivers/infiniband/hw/ocrdma/ocrdma_main.c
+++ b/drivers/infiniband/hw/ocrdma/ocrdma_main.c
@@ -58,7 +58,6 @@ 
 #include "ocrdma_stats.h"
 #include <rdma/ocrdma-abi.h>

-MODULE_VERSION(OCRDMA_ROCE_DRV_VERSION);
 MODULE_DESCRIPTION(OCRDMA_ROCE_DRV_DESC " " OCRDMA_ROCE_DRV_VERSION);
 MODULE_AUTHOR("Emulex Corporation");
 MODULE_LICENSE("Dual BSD/GPL");
diff --git a/drivers/infiniband/hw/qedr/main.c b/drivers/infiniband/hw/qedr/main.c
index 6a72095d6c7a..6c5639caab20 100644
--- a/drivers/infiniband/hw/qedr/main.c
+++ b/drivers/infiniband/hw/qedr/main.c
@@ -47,7 +47,6 @@ 
 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 aa08c76a4245..0e95ffd6c14a 100644
--- a/drivers/infiniband/hw/qedr/qedr.h
+++ b/drivers/infiniband/hw/qedr/qedr.h
@@ -41,7 +41,6 @@ 
 #include <linux/qed/roce_common.h>
 #include "qedr_hsi_rdma.h"

-#define QEDR_MODULE_VERSION	"8.10.10.0"
 #define QEDR_NODE_DESC "QLogic 579xx RoCE HCA"
 #define DP_NAME(dev) ((dev)->ibdev.name)

diff --git a/drivers/infiniband/hw/qib/qib_driver.c b/drivers/infiniband/hw/qib/qib_driver.c
index 2b5982f743ef..719906a9fd51 100644
--- a/drivers/infiniband/hw/qib/qib_driver.c
+++ b/drivers/infiniband/hw/qib/qib_driver.c
@@ -66,7 +66,6 @@  MODULE_PARM_DESC(compat_ddr_negotiate,
 MODULE_LICENSE("Dual BSD/GPL");
 MODULE_AUTHOR("Intel <ibsupport@intel.com>");
 MODULE_DESCRIPTION("Intel IB driver");
-MODULE_VERSION(QIB_DRIVER_VERSION);

 /*
  * QIB_PIO_MAXIBHDR is the max IB header size allowed for in our
diff --git a/drivers/infiniband/hw/usnic/usnic_ib_main.c b/drivers/infiniband/hw/usnic/usnic_ib_main.c
index c0c1e8b027b1..08cbbe224e11 100644
--- a/drivers/infiniband/hw/usnic/usnic_ib_main.c
+++ b/drivers/infiniband/hw/usnic/usnic_ib_main.c
@@ -720,7 +720,6 @@  static void __exit usnic_ib_destroy(void)
 MODULE_DESCRIPTION("Cisco VIC (usNIC) Verbs Driver");
 MODULE_AUTHOR("Upinder Malhi <umalhi@cisco.com>");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(DRV_VERSION);
 module_param(usnic_log_lvl, uint, S_IRUGO | S_IWUSR);
 module_param(usnic_ib_share_vf, uint, S_IRUGO | S_IWUSR);
 MODULE_PARM_DESC(usnic_log_lvl, " Off=0, Err=1, Info=2, Debug=3");
diff --git a/drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c b/drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c
index 34ebc7615411..e76565280afa 100644
--- a/drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c
+++ b/drivers/infiniband/hw/vmw_pvrdma/pvrdma_main.c
@@ -1119,5 +1119,4 @@  module_exit(pvrdma_cleanup);

 MODULE_AUTHOR("VMware, Inc");
 MODULE_DESCRIPTION("VMware Paravirtual RDMA driver");
-MODULE_VERSION(DRV_VERSION);
 MODULE_LICENSE("Dual BSD/GPL");
diff --git a/drivers/infiniband/sw/rxe/rxe.c b/drivers/infiniband/sw/rxe/rxe.c
index c21c913f911a..8c3d30b3092d 100644
--- a/drivers/infiniband/sw/rxe/rxe.c
+++ b/drivers/infiniband/sw/rxe/rxe.c
@@ -38,7 +38,6 @@ 
 MODULE_AUTHOR("Bob Pearson, Frank Zago, John Groves, Kamal Heib");
 MODULE_DESCRIPTION("Soft RDMA transport");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION("0.2");

 /* free resources for all ports on a device */
 static void rxe_cleanup_ports(struct rxe_dev *rxe)
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_main.c b/drivers/infiniband/ulp/ipoib/ipoib_main.c
index a115c0b7a310..b11c46db7cc9 100644
--- a/drivers/infiniband/ulp/ipoib/ipoib_main.c
+++ b/drivers/infiniband/ulp/ipoib/ipoib_main.c
@@ -60,7 +60,6 @@  const char ipoib_driver_version[] = DRV_VERSION;
 MODULE_AUTHOR("Roland Dreier");
 MODULE_DESCRIPTION("IP-over-InfiniBand net driver");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(DRV_VERSION);

 int ipoib_sendq_size __read_mostly = IPOIB_TX_RING_SIZE;
 int ipoib_recvq_size __read_mostly = IPOIB_RX_RING_SIZE;
diff --git a/drivers/infiniband/ulp/iser/iscsi_iser.c b/drivers/infiniband/ulp/iser/iscsi_iser.c
index 5a887efb4bdf..8ef3c1bd78bf 100644
--- a/drivers/infiniband/ulp/iser/iscsi_iser.c
+++ b/drivers/infiniband/ulp/iser/iscsi_iser.c
@@ -77,7 +77,6 @@ 
 MODULE_DESCRIPTION("iSER (iSCSI Extensions for RDMA) Datamover");
 MODULE_LICENSE("Dual BSD/GPL");
 MODULE_AUTHOR("Alex Nezhinsky, Dan Bar Dov, Or Gerlitz");
-MODULE_VERSION(DRV_VER);

 static struct scsi_host_template iscsi_iser_sht;
 static struct iscsi_transport iscsi_iser_transport;
diff --git a/drivers/infiniband/ulp/isert/ib_isert.c b/drivers/infiniband/ulp/isert/ib_isert.c
index fcbed35e95a8..d7cc7c6190ba 100644
--- a/drivers/infiniband/ulp/isert/ib_isert.c
+++ b/drivers/infiniband/ulp/isert/ib_isert.c
@@ -2710,7 +2710,6 @@  static void __exit isert_exit(void)
 }

 MODULE_DESCRIPTION("iSER-Target for mainline target infrastructure");
-MODULE_VERSION("1.0");
 MODULE_AUTHOR("nab@Linux-iSCSI.org");
 MODULE_LICENSE("GPL");

diff --git a/drivers/infiniband/ulp/opa_vnic/opa_vnic_vema.c b/drivers/infiniband/ulp/opa_vnic/opa_vnic_vema.c
index 875694f9a7f9..f9910a6c3291 100644
--- a/drivers/infiniband/ulp/opa_vnic/opa_vnic_vema.c
+++ b/drivers/infiniband/ulp/opa_vnic/opa_vnic_vema.c
@@ -1053,4 +1053,3 @@  module_exit(opa_vnic_deinit);
 MODULE_LICENSE("Dual BSD/GPL");
 MODULE_AUTHOR("Intel Corporation");
 MODULE_DESCRIPTION("Intel OPA Virtual Network driver");
-MODULE_VERSION(DRV_VERSION);
diff --git a/drivers/infiniband/ulp/srp/ib_srp.c b/drivers/infiniband/ulp/srp/ib_srp.c
index 2354c742caa1..fa5ccdb3bb2a 100644
--- a/drivers/infiniband/ulp/srp/ib_srp.c
+++ b/drivers/infiniband/ulp/srp/ib_srp.c
@@ -62,7 +62,6 @@ 
 MODULE_AUTHOR("Roland Dreier");
 MODULE_DESCRIPTION("InfiniBand SCSI RDMA Protocol initiator");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(DRV_VERSION);
 MODULE_INFO(release_date, DRV_RELDATE);

 #if !defined(CONFIG_DYNAMIC_DEBUG)