net/mlx5: remove limitation on number of instances

This artificial limitation was inherited from the mlx4 code base and has no
purpose other than adding unnecessary noise.

This patch is a port of commit f2318196c71a ("net/mlx4: remove limitation
on number of instances").

Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
Acked-by: Yongseok Koh <yskoh@mellanox.com>

diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c
index c933e27..60575db 100644 (file)
--- a/drivers/net/mlx5/mlx5.c
+++ b/drivers/net/mlx5/mlx5.c
@@ -400,39 +400,6 @@ const struct eth_dev_ops mlx5_dev_ops_isolate = {
        .is_removed = mlx5_is_removed,
 };
 
-static struct {
-       struct rte_pci_addr pci_addr; /* associated PCI address */
-       uint32_t ports; /* physical ports bitfield. */
-} mlx5_dev[32];
-
-/**
- * Get device index in mlx5_dev[] from PCI bus address.
- *
- * @param[in] pci_addr
- *   PCI bus address to look for.
- *
- * @return
- *   mlx5_dev[] index on success, -1 on failure.
- */
-static int
-mlx5_dev_idx(struct rte_pci_addr *pci_addr)
-{
-       unsigned int i;
-       int ret = -1;
-
-       assert(pci_addr != NULL);
-       for (i = 0; (i != RTE_DIM(mlx5_dev)); ++i) {
-               if ((mlx5_dev[i].pci_addr.domain == pci_addr->domain) &&
-                   (mlx5_dev[i].pci_addr.bus == pci_addr->bus) &&
-                   (mlx5_dev[i].pci_addr.devid == pci_addr->devid) &&
-                   (mlx5_dev[i].pci_addr.function == pci_addr->function))
-                       return i;
-               if ((mlx5_dev[i].ports == 0) && (ret == -1))
-                       ret = i;
-       }
-       return ret;
-}
-
 /**
  * Verify and store value for device argument.
  *
@@ -702,7 +669,6 @@ mlx5_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
        unsigned int mprq_max_stride_size_n = 0;
        unsigned int mprq_min_stride_num_n = 0;
        unsigned int mprq_max_stride_num_n = 0;
-       int idx;
        int i;
        struct mlx5dv_context attrs_out = {0};
 #ifdef HAVE_IBV_DEVICE_COUNTERS_SET_SUPPORT
@@ -712,16 +678,6 @@ mlx5_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
        /* Prepare shared data between primary and secondary process. */
        mlx5_prepare_shared_data();
        assert(pci_drv == &mlx5_driver);
-       /* Get mlx5_dev[] index. */
-       idx = mlx5_dev_idx(&pci_dev->addr);
-       if (idx == -1) {
-               DRV_LOG(ERR, "this driver cannot support any more adapters");
-               err = ENOMEM;
-               goto error;
-       }
-       DRV_LOG(DEBUG, "using driver device index %d", idx);
-       /* Save PCI address. */
-       mlx5_dev[idx].pci_addr = pci_dev->addr;
        list = mlx5_glue->get_device_list(&i);
        if (list == NULL) {
                assert(errno);
@@ -877,7 +833,6 @@ mlx5_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
                char name[RTE_ETH_NAME_MAX_LEN];
                int len;
                uint32_t port = i + 1; /* ports are indexed from one */
-               uint32_t test = (1 << i);
                struct ibv_context *ctx = NULL;
                struct ibv_port_attr port_attr;
                struct ibv_pd *pd = NULL;
@@ -912,7 +867,6 @@ mlx5_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
                         pci_dev->addr.devid, pci_dev->addr.function);
                if (device_attr.orig_attr.phys_port_cnt > 1)
                        snprintf(name + len, sizeof(name), " port %u", i);
-               mlx5_dev[idx].ports |= test;
                if (rte_eal_process_type() == RTE_PROC_SECONDARY) {
                        eth_dev = rte_eth_dev_attach_secondary(name);
                        if (eth_dev == NULL) {
@@ -952,7 +906,7 @@ mlx5_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
                        rte_eth_dev_probing_finish(eth_dev);
                        continue;
                }
-               DRV_LOG(DEBUG, "using port %u (%08" PRIx32 ")", port, test);
+               DRV_LOG(DEBUG, "using port %u", port);
                ctx = mlx5_glue->open_device(ibv_dev);
                if (ctx == NULL) {
                        err = ENODEV;
@@ -983,7 +937,6 @@ mlx5_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
                        err = ENOMEM;
                        goto port_error;
                }
-               mlx5_dev[idx].ports |= test;
                /* from rte_ethdev.c */
                priv = rte_zmalloc("ethdev private structure",
                                   sizeof(*priv),
@@ -1232,11 +1185,6 @@ port_error:
         * long as the dpdk does not provide a way to deallocate a ethdev and a
         * way to enumerate the registered ethdevs to free the previous ones.
         */
-       /* no port found, complain */
-       if (!mlx5_dev[idx].ports) {
-               rte_errno = ENODEV;
-               err = rte_errno;
-       }
 error:
        if (attr_ctx)
                claim_zero(mlx5_glue->close_device(attr_ctx));