[dpdk-dev] [PATCH v1 2/2] net/mlx5: support new device global syntax

[Xueming Li]

This patch support new device global syntax like:
        bus=pci,addr=BB:DD.F/class=eth/driver=mlx5,devargs,..

Ignore "driver" key as part of new global device syntax in devargs.

The representor devarg is supposed to come from either class section or
driver section.

Signed-off-by: Xueming Li <xuemi...@nvidia.com>
---
 drivers/net/mlx5/linux/mlx5_os.c | 18 ++++++++++++++++--
 drivers/net/mlx5/mlx5.c          |  6 +++++-
 2 files changed, 21 insertions(+), 3 deletions(-)

diff --git a/drivers/net/mlx5/linux/mlx5_os.c b/drivers/net/mlx5/linux/mlx5_os.c
index 6812a1f215..f1ed3505b1 100644
--- a/drivers/net/mlx5/linux/mlx5_os.c
+++ b/drivers/net/mlx5/linux/mlx5_os.c
@@ -699,13 +699,27 @@ mlx5_dev_spawn(struct rte_device *dpdk_dev,
        if (switch_info->representor && dpdk_dev->devargs) {
                struct rte_eth_devargs eth_da;
 
-               err = rte_eth_devargs_parse(dpdk_dev->devargs->args, &eth_da);
+               /* Representer should come from class argument or driver */
+               if (dpdk_dev->devargs->cls_str)
+                       err = rte_eth_devargs_parse(dpdk_dev->devargs->cls_str,
+                                                   &eth_da);
                if (err) {
                        rte_errno = -err;
                        DRV_LOG(ERR, "failed to process device arguments: %s",
-                               strerror(rte_errno));
+                               dpdk_dev->devargs->cls_str);
                        return NULL;
                }
+               if (eth_da.type == RTE_ETH_REPRESENTOR_NONE) {
+                       /* Support legacy device argument */
+                       err = rte_eth_devargs_parse(dpdk_dev->devargs->args,
+                                                   &eth_da);
+                       if (err) {
+                               rte_errno = -err;
+                               DRV_LOG(ERR, "failed to process device 
arguments: %s",
+                                       dpdk_dev->devargs->args);
+                               return NULL;
+                       }
+               }
                for (i = 0; i < eth_da.nb_representor_ports; ++i)
                        if (eth_da.representor_ports[i] ==
                            (uint16_t)switch_info->port_name)
diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c
index 023ef50a77..f2b6cf9fd6 100644
--- a/drivers/net/mlx5/mlx5.c
+++ b/drivers/net/mlx5/mlx5.c
@@ -41,6 +41,9 @@
 #include "mlx5_flow_os.h"
 #include "rte_pmd_mlx5.h"
 
+/* Driver type key for new device global syntax. */
+#define MLX5_DRIVER_KEY "driver"
+
 /* Device parameter to enable RX completion queue compression. */
 #define MLX5_RXQ_CQE_COMP_EN "rxq_cqe_comp_en"
 
@@ -1600,7 +1603,7 @@ mlx5_args_check(const char *key, const char *val, void 
*opaque)
        signed long tmp;
 
        /* No-op, port representors are processed in mlx5_dev_spawn(). */
-       if (!strcmp(MLX5_REPRESENTOR, key))
+       if (!strcmp(MLX5_DRIVER_KEY, key) || !strcmp(MLX5_REPRESENTOR, key))
                return 0;
        errno = 0;
        tmp = strtol(val, NULL, 0);
@@ -1754,6 +1757,7 @@ int
 mlx5_args(struct mlx5_dev_config *config, struct rte_devargs *devargs)
 {
        const char **params = (const char *[]){
+               MLX5_DRIVER_KEY,
                MLX5_RXQ_CQE_COMP_EN,
                MLX5_RXQ_CQE_PAD_EN,
                MLX5_RXQ_PKT_PAD_EN,
-- 
2.25.1