DPDK patches and discussions
 help / color / mirror / Atom feed
From: Somnath Kotur <somnath.kotur@broadcom.com>
To: Xueming Li <xuemingl@nvidia.com>
Cc: Thomas Monjalon <thomas@monjalon.net>,
	Ferruh Yigit <ferruh.yigit@intel.com>,
	 Andrew Rybchenko <andrew.rybchenko@oktetlabs.ru>,
	Olivier Matz <olivier.matz@6wind.com>,
	 Viacheslav Ovsiienko <viacheslavo@nvidia.com>,
	dev <dev@dpdk.org>, Asaf Penso <asafp@nvidia.com>
Subject: Re: [dpdk-dev] [PATCH v2 1/9] ethdev: refactor representor infrastructure
Date: Thu, 7 Jan 2021 12:01:53 +0530
Message-ID: <CAOBf=mvAkWJMb=i5uSL49HUY8+tBhjHiD=532QE81qbhM+RxAg@mail.gmail.com> (raw)
In-Reply-To: <1609949855-23817-2-git-send-email-xuemingl@nvidia.com>

[-- Attachment #1: Type: text/plain, Size: 14450 bytes --]

On Wed, Jan 6, 2021 at 9:48 PM Xueming Li <xuemingl@nvidia.com> wrote:
>
> To support extended representor syntax, this patch refactor represntor
Typo in 'representor'
> infrastructure:
> 1. introduces representor type enum
> 2. devargs representor port range extraction from partial value
>
> Signed-off-by: Xueming Li <xuemingl@nvidia.com>
> ---
>  drivers/net/bnxt/bnxt_ethdev.c        | 12 ++++
>  drivers/net/enic/enic_ethdev.c        |  7 ++
>  drivers/net/i40e/i40e_ethdev.c        |  8 +++
>  drivers/net/ixgbe/ixgbe_ethdev.c      |  8 +++
>  drivers/net/mlx5/linux/mlx5_os.c      | 11 ++++
>  lib/librte_ethdev/ethdev_private.c    | 93 ++++++++++++---------------
>  lib/librte_ethdev/ethdev_private.h    |  3 -
>  lib/librte_ethdev/rte_class_eth.c     |  4 +-
>  lib/librte_ethdev/rte_ethdev.c        |  5 +-
>  lib/librte_ethdev/rte_ethdev_driver.h |  7 ++
>  10 files changed, 98 insertions(+), 60 deletions(-)
>
> diff --git a/drivers/net/bnxt/bnxt_ethdev.c b/drivers/net/bnxt/bnxt_ethdev.c
> index 81c8f8d79d..844a6c3c66 100644
> --- a/drivers/net/bnxt/bnxt_ethdev.c
> +++ b/drivers/net/bnxt/bnxt_ethdev.c
> @@ -5520,6 +5520,18 @@ static int bnxt_rep_port_probe(struct rte_pci_device *pci_dev,
>         int i, ret = 0;
>         struct rte_kvargs *kvlist = NULL;
>
> +       if (eth_da->type == RTE_ETH_REPRESENTOR_NONE)
> +               return 0;
> +       if (eth_da->type != RTE_ETH_REPRESENTOR_VF) {
> +               PMD_DRV_LOG(ERR, "unsupported representor type %d\n",
> +                           eth_da->type);
> +               return -ENOTSUP;
> +       }
> +       if (eth_da->type != RTE_ETH_REPRESENTOR_VF) {
Seems like an extra 'if ' condition by mistake? Otherwise there is no
diff b/n this 'if' condition and the one few lines above?
> +               PMD_DRV_LOG(ERR, "unsupported representor type %d\n",
> +                           eth_da->type);
> +               return -EINVAL;
> +       }
>         num_rep = eth_da->nb_representor_ports;
>         if (num_rep > BNXT_MAX_VF_REPS) {
>                 PMD_DRV_LOG(ERR, "nb_representor_ports = %d > %d MAX VF REPS\n",
> diff --git a/drivers/net/enic/enic_ethdev.c b/drivers/net/enic/enic_ethdev.c
> index d041a6bee9..dd085caa93 100644
> --- a/drivers/net/enic/enic_ethdev.c
> +++ b/drivers/net/enic/enic_ethdev.c
> @@ -1303,6 +1303,13 @@ static int eth_enic_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
>                 if (retval)
>                         return retval;
>         }
> +       if (eth_da.type == RTE_ETH_REPRESENTOR_NONE)
> +               return 0;
> +       if (eth_da.type != RTE_ETH_REPRESENTOR_VF) {
> +               ENICPMD_LOG(ERR, "unsupported representor type: %s\n",
> +                           pci_dev->device.devargs->args);
> +               return -ENOTSUP;
> +       }
>         retval = rte_eth_dev_create(&pci_dev->device, pci_dev->device.name,
>                 sizeof(struct enic),
>                 eth_dev_pci_specific_init, pci_dev,
> diff --git a/drivers/net/i40e/i40e_ethdev.c b/drivers/net/i40e/i40e_ethdev.c
> index f54769c29d..05ed2e1079 100644
> --- a/drivers/net/i40e/i40e_ethdev.c
> +++ b/drivers/net/i40e/i40e_ethdev.c
> @@ -640,6 +640,14 @@ eth_i40e_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
>                         return retval;
>         }
>
> +       if (eth_da.type == RTE_ETH_REPRESENTOR_NONE)
> +               return 0;
> +       if (eth_da.type != RTE_ETH_REPRESENTOR_VF) {
> +               PMD_DRV_LOG(ERR, "unsupported representor type: %s\n",
> +                           pci_dev->device.devargs->args);
> +               return -ENOTSUP;
> +       }
> +
>         retval = rte_eth_dev_create(&pci_dev->device, pci_dev->device.name,
>                 sizeof(struct i40e_adapter),
>                 eth_dev_pci_specific_init, pci_dev,
> diff --git a/drivers/net/ixgbe/ixgbe_ethdev.c b/drivers/net/ixgbe/ixgbe_ethdev.c
> index 9a47a8b262..9ea0139197 100644
> --- a/drivers/net/ixgbe/ixgbe_ethdev.c
> +++ b/drivers/net/ixgbe/ixgbe_ethdev.c
> @@ -1717,6 +1717,14 @@ eth_ixgbe_pci_probe(struct rte_pci_driver *pci_drv __rte_unused,
>         } else
>                 memset(&eth_da, 0, sizeof(eth_da));
>
> +       if (eth_da.type == RTE_ETH_REPRESENTOR_NONE)
> +               return 0;
> +       if (eth_da.type != RTE_ETH_REPRESENTOR_VF) {
> +               PMD_DRV_LOG(ERR, "unsupported representor type: %s\n",
> +                           pci_dev->device.devargs->args);
> +               return -ENOTSUP;
> +       }
> +
>         retval = rte_eth_dev_create(&pci_dev->device, pci_dev->device.name,
>                 sizeof(struct ixgbe_adapter),
>                 eth_dev_pci_specific_init, pci_dev,
> diff --git a/drivers/net/mlx5/linux/mlx5_os.c b/drivers/net/mlx5/linux/mlx5_os.c
> index 6812a1f215..6981ba1f41 100644
> --- a/drivers/net/mlx5/linux/mlx5_os.c
> +++ b/drivers/net/mlx5/linux/mlx5_os.c
> @@ -706,6 +706,17 @@ mlx5_dev_spawn(struct rte_device *dpdk_dev,
>                                 strerror(rte_errno));
>                         return NULL;
>                 }
> +               if (eth_da.type != RTE_ETH_REPRESENTOR_NONE) {
> +                       /* Representor not specified. */
> +                       rte_errno = EBUSY;
> +                       return NULL;
> +               }
> +               if (eth_da.type != RTE_ETH_REPRESENTOR_VF) {
> +                       rte_errno = ENOTSUP;
> +                       DRV_LOG(ERR, "unsupported representor type: %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/lib/librte_ethdev/ethdev_private.c b/lib/librte_ethdev/ethdev_private.c
> index 162a502fe7..c219164a4a 100644
> --- a/lib/librte_ethdev/ethdev_private.c
> +++ b/lib/librte_ethdev/ethdev_private.c
> @@ -38,60 +38,13 @@ eth_find_device(const struct rte_eth_dev *start, rte_eth_cmp_t cmp,
>         return NULL;
>  }
>
> -int
> -rte_eth_devargs_parse_list(char *str, rte_eth_devargs_callback_t callback,
> -       void *data)
> -{
> -       char *str_start;
> -       int state;
> -       int result;
> -
> -       if (*str != '[')
> -               /* Single element, not a list */
> -               return callback(str, data);
> -
> -       /* Sanity check, then strip the brackets */
> -       str_start = &str[strlen(str) - 1];
> -       if (*str_start != ']') {
> -               RTE_LOG(ERR, EAL, "(%s): List does not end with ']'\n", str);
> -               return -EINVAL;
> -       }
> -       str++;
> -       *str_start = '\0';
> -
> -       /* Process list elements */
> -       state = 0;
> -       while (1) {
> -               if (state == 0) {
> -                       if (*str == '\0')
> -                               break;
> -                       if (*str != ',') {
> -                               str_start = str;
> -                               state = 1;
> -                       }
> -               } else if (state == 1) {
> -                       if (*str == ',' || *str == '\0') {
> -                               if (str > str_start) {
> -                                       /* Non-empty string fragment */
> -                                       *str = '\0';
> -                                       result = callback(str_start, data);
> -                                       if (result < 0)
> -                                               return result;
> -                               }
> -                               state = 0;
> -                       }
> -               }
> -               str++;
> -       }
> -       return 0;
> -}
> -
>  static int
>  rte_eth_devargs_process_range(char *str, uint16_t *list, uint16_t *len_list,
>         const uint16_t max_list)
>  {
>         uint16_t lo, hi, val;
>         int result;
> +       char *pos = str;
>
>         result = sscanf(str, "%hu-%hu", &lo, &hi);
>         if (result == 1) {
> @@ -99,7 +52,7 @@ rte_eth_devargs_process_range(char *str, uint16_t *list, uint16_t *len_list,
>                         return -ENOMEM;
>                 list[(*len_list)++] = lo;
>         } else if (result == 2) {
> -               if (lo >= hi || lo > RTE_MAX_ETHPORTS || hi > RTE_MAX_ETHPORTS)
> +               if (lo >= hi)
>                         return -EINVAL;
>                 for (val = lo; val <= hi; val++) {
>                         if (*len_list >= max_list)
> @@ -108,14 +61,52 @@ rte_eth_devargs_process_range(char *str, uint16_t *list, uint16_t *len_list,
>                 }
>         } else
>                 return -EINVAL;
> -       return 0;
> +       while (*pos != 0 && ((*pos >= '0' && *pos <= '9') || *pos == '-'))
> +               pos++;
> +       return pos - str;
>  }
>
> +static int
> +rte_eth_devargs_process_list(char *str, uint16_t *list, uint16_t *len_list,
> +       const uint16_t max_list)
> +{
> +       char *pos = str;
> +       int ret;
> +
> +       if (*pos == '[')
> +               pos++;
> +       while (1) {
> +               ret = rte_eth_devargs_process_range(pos, list, len_list,
> +                                                   max_list);
> +               if (ret < 0)
> +                       return ret;
> +               pos += ret;
> +               if (*pos != ',') /* end of list */
> +                       break;
> +               pos++;
> +       }
> +       if (*str == '[' && *pos != ']')
> +               return -EINVAL;
> +       if (*pos == ']')
> +               pos++;
> +       return pos - str;
> +}
> +
> +/*
> + * representor format:
> + *   #: range or single number of VF representor - legacy
> + */
>  int
>  rte_eth_devargs_parse_representor_ports(char *str, void *data)
>  {
>         struct rte_eth_devargs *eth_da = data;
> +       int ret;
>
> -       return rte_eth_devargs_process_range(str, eth_da->representor_ports,
> +       /* Number # alone implies VF */
> +       eth_da->type = RTE_ETH_REPRESENTOR_VF;
> +       ret = rte_eth_devargs_process_list(str, eth_da->representor_ports,
>                 &eth_da->nb_representor_ports, RTE_MAX_ETHPORTS);
> +       if (ret < 0)
> +               RTE_LOG(ERR, EAL, "wrong representor format: %s\n", str);
> +       return ret < 0 ? ret : 0;
>  }
> diff --git a/lib/librte_ethdev/ethdev_private.h b/lib/librte_ethdev/ethdev_private.h
> index 905a45c337..220ddd4408 100644
> --- a/lib/librte_ethdev/ethdev_private.h
> +++ b/lib/librte_ethdev/ethdev_private.h
> @@ -26,9 +26,6 @@ eth_find_device(const struct rte_eth_dev *_start, rte_eth_cmp_t cmp,
>                 const void *data);
>
>  /* Parse devargs value for representor parameter. */
> -typedef int (*rte_eth_devargs_callback_t)(char *str, void *data);
> -int rte_eth_devargs_parse_list(char *str, rte_eth_devargs_callback_t callback,
> -       void *data);
>  int rte_eth_devargs_parse_representor_ports(char *str, void *data);
>
>  #ifdef __cplusplus
> diff --git a/lib/librte_ethdev/rte_class_eth.c b/lib/librte_ethdev/rte_class_eth.c
> index 6338355e25..efe6149df5 100644
> --- a/lib/librte_ethdev/rte_class_eth.c
> +++ b/lib/librte_ethdev/rte_class_eth.c
> @@ -77,9 +77,7 @@ eth_representor_cmp(const char *key __rte_unused,
>         if (values == NULL)
>                 return -1;
>         memset(&representors, 0, sizeof(representors));
> -       ret = rte_eth_devargs_parse_list(values,
> -                       rte_eth_devargs_parse_representor_ports,
> -                       &representors);
> +       ret = rte_eth_devargs_parse_representor_ports(values, &representors);
>         free(values);
>         if (ret != 0)
>                 return -1; /* invalid devargs value */
> diff --git a/lib/librte_ethdev/rte_ethdev.c b/lib/librte_ethdev/rte_ethdev.c
> index 17ddacc78d..2ac51ac149 100644
> --- a/lib/librte_ethdev/rte_ethdev.c
> +++ b/lib/librte_ethdev/rte_ethdev.c
> @@ -5542,9 +5542,8 @@ rte_eth_devargs_parse(const char *dargs, struct rte_eth_devargs *eth_da)
>         for (i = 0; i < args.count; i++) {
>                 pair = &args.pairs[i];
>                 if (strcmp("representor", pair->key) == 0) {
> -                       result = rte_eth_devargs_parse_list(pair->value,
> -                               rte_eth_devargs_parse_representor_ports,
> -                               eth_da);
> +                       result = rte_eth_devargs_parse_representor_ports(
> +                                       pair->value, eth_da);
>                         if (result < 0)
>                                 goto parse_cleanup;
>                 }
> diff --git a/lib/librte_ethdev/rte_ethdev_driver.h b/lib/librte_ethdev/rte_ethdev_driver.h
> index 0eacfd8425..b66a955b18 100644
> --- a/lib/librte_ethdev/rte_ethdev_driver.h
> +++ b/lib/librte_ethdev/rte_ethdev_driver.h
> @@ -1193,6 +1193,12 @@ __rte_internal
>  int
>  rte_eth_switch_domain_free(uint16_t domain_id);
>
> +/** Ethernet device representor type */
> +enum rte_eth_representor_type {
> +       RTE_ETH_REPRESENTOR_NONE, /* not a representor */
> +       RTE_ETH_REPRESENTOR_VF,   /* representor of VF */
> +};
> +
>  /** Generic Ethernet device arguments  */
>  struct rte_eth_devargs {
>         uint16_t ports[RTE_MAX_ETHPORTS];
> @@ -1203,6 +1209,7 @@ struct rte_eth_devargs {
>         /** representor port/s identifier to enable on device */
>         uint16_t nb_representor_ports;
>         /** number of ports in representor port field */
> +       enum rte_eth_representor_type type; /* type of representor */
>  };
>
>  /**
> --
> 2.25.1
>

-- 
This electronic communication and the information and any files transmitted 
with it, or attached to it, are confidential and are intended solely for 
the use of the individual or entity to whom it is addressed and may contain 
information that is confidential, legally privileged, protected by privacy 
laws, or otherwise restricted from disclosure to anyone else. If you are 
not the intended recipient or the person responsible for delivering the 
e-mail to the intended recipient, you are hereby notified that any use, 
copying, distributing, dissemination, forwarding, printing, or copying of 
this e-mail is strictly prohibited. If you received this e-mail in error, 
please return the e-mail to the sender, delete it from your computer, and 
destroy any printed copy of it.

  reply	other threads:[~2021-01-07  6:32 UTC|newest]

Thread overview: 209+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-12-18 14:55 [dpdk-dev] [RFC 0/7] support SubFunction representor Xueming Li
2020-12-18 14:55 ` [dpdk-dev] [RFC 1/7] ethdev: support sub function representor Xueming Li
2020-12-28 11:59   ` Andrew Rybchenko
2021-01-06 16:17   ` [dpdk-dev] [PATCH v2 0/9] support SubFunction representor Xueming Li
2021-01-06 16:17   ` [dpdk-dev] [PATCH v2 1/9] ethdev: refactor representor infrastructure Xueming Li
2021-01-07  6:31     ` Somnath Kotur [this message]
2021-01-07  6:38       ` Xueming(Steven) Li
2021-01-07  6:40         ` Somnath Kotur
2021-01-06 16:17   ` [dpdk-dev] [PATCH v2 2/9] ethdev: support new VF representor syntax Xueming Li
2021-01-06 16:17   ` [dpdk-dev] [PATCH v2 3/9] ethdev: support sub function representor Xueming Li
2021-01-06 16:17   ` [dpdk-dev] [PATCH v2 4/9] ethdev: support PF index in representor Xueming Li
2021-01-06 16:17   ` [dpdk-dev] [PATCH v2 5/9] ethdev: support multi-host representor Xueming Li
2021-01-06 16:17   ` [dpdk-dev] [PATCH v2 6/9] devarg: change reprsentor ID to bitmap Xueming Li
2021-01-06 16:17   ` [dpdk-dev] [PATCH v2 7/9] ethdev: capability of new representor syntax Xueming Li
2021-01-06 16:17   ` [dpdk-dev] [PATCH v2 8/9] kvargs: update parser for " Xueming Li
2021-01-06 16:17   ` [dpdk-dev] [PATCH v2 9/9] eal: allow PCI device with different representors Xueming Li
2021-01-06 16:39   ` [dpdk-dev] [PATCH v2 0/4] net/mlx5: support SubFunction representor Xueming Li
2021-01-06 16:39   ` [dpdk-dev] [PATCH v2 1/4] common/mlx5: update representor name parsing Xueming Li
2021-01-06 16:39   ` [dpdk-dev] [PATCH v2 2/4] net/mlx5: support representor of sub function Xueming Li
2021-01-06 16:39   ` [dpdk-dev] [PATCH v2 3/4] net/mlx5: revert setting representor to first PF Xueming Li
2021-01-06 16:39   ` [dpdk-dev] [PATCH v2 4/4] net/mlx5: improve bonding representor probe Xueming Li
2021-01-13 13:44   ` [dpdk-dev] [PATCH v3 0/9] ethdev: support SubFunction representor Xueming Li
2021-01-13 13:44   ` [dpdk-dev] [PATCH v3 1/9] ethdev: refactor representor infrastructure Xueming Li
2021-01-13 13:44   ` [dpdk-dev] [PATCH v3 2/9] ethdev: support new VF representor syntax Xueming Li
2021-01-13 13:44   ` [dpdk-dev] [PATCH v3 3/9] ethdev: support sub function representor Xueming Li
2021-01-13 13:44   ` [dpdk-dev] [PATCH v3 4/9] ethdev: support PF index in representor Xueming Li
2021-01-13 13:44   ` [dpdk-dev] [PATCH v3 5/9] ethdev: support multi-host representor Xueming Li
2021-01-13 13:44   ` [dpdk-dev] [PATCH v3 6/9] devarg: change reprsentor ID to bitmap Xueming Li
2021-01-13 13:44   ` [dpdk-dev] [PATCH v3 7/9] ethdev: capability of new representor syntax Xueming Li
2021-01-13 13:44   ` [dpdk-dev] [PATCH v3 8/9] kvargs: update parser for " Xueming Li
2021-01-13 13:44   ` [dpdk-dev] [PATCH v3 9/9] eal: probe devices of same PCI but different devargs Xueming Li
2021-01-18 11:16   ` [dpdk-dev] [PATCH v4 0/9] ethdev: support SubFunction representor Xueming Li
2021-01-18 16:24     ` Thomas Monjalon
2021-01-18 11:16   ` [dpdk-dev] [PATCH v4 1/9] ethdev: introduce representor type Xueming Li
2021-01-18 15:44     ` Thomas Monjalon
2021-01-18 17:42     ` Ajit Khaparde
2021-01-18 17:57       ` Thomas Monjalon
2021-01-18 18:00         ` Ajit Khaparde
2021-01-18 18:15           ` Thomas Monjalon
2021-01-18 18:17             ` Ajit Khaparde
2021-01-18 23:41               ` Xueming(Steven) Li
2021-01-19  7:39                 ` Ajit Khaparde
2021-01-19  7:24     ` Andrew Rybchenko
2021-01-19  7:37       ` Xueming(Steven) Li
2021-01-19  7:49         ` Andrew Rybchenko
2021-01-19  7:56           ` Xueming(Steven) Li
2021-01-19  8:39             ` Thomas Monjalon
2021-01-18 11:16   ` [dpdk-dev] [PATCH v4 2/9] ethdev: support representor port list Xueming Li
2021-01-18 16:18     ` Thomas Monjalon
2021-01-18 23:23       ` Xueming(Steven) Li
2021-01-19  7:45     ` Andrew Rybchenko
2021-01-19  8:59       ` Xueming(Steven) Li
2021-01-19  9:03         ` Andrew Rybchenko
2021-01-19 10:19           ` Xueming(Steven) Li
2021-01-18 11:16   ` [dpdk-dev] [PATCH v4 3/9] ethdev: support new VF representor syntax Xueming Li
2021-01-18 11:16   ` [dpdk-dev] [PATCH v4 4/9] ethdev: support sub function representor Xueming Li
2021-01-18 11:16   ` [dpdk-dev] [PATCH v4 5/9] ethdev: support PF index in representor Xueming Li
2021-01-18 11:17   ` [dpdk-dev] [PATCH v4 6/9] ethdev: support multi-host " Xueming Li
2021-01-18 11:17   ` [dpdk-dev] [PATCH v4 7/9] devarg: change representor ID to bitmap Xueming Li
2021-01-18 19:01     ` Ajit Khaparde
2021-01-20  5:51       ` Xueming(Steven) Li
2021-01-18 11:17   ` [dpdk-dev] [PATCH v4 8/9] ethdev: add capability of sub-function representor Xueming Li
2021-01-18 11:17   ` [dpdk-dev] [PATCH v4 9/9] kvargs: update parser to support lists Xueming Li
2021-01-19  7:13   ` [dpdk-dev] [PATCH v5 0/9] ethdev: support SubFunction representor Xueming Li
2021-01-19  8:40     ` Andrew Rybchenko
2021-01-19 14:24       ` Xueming(Steven) Li
2021-01-22  8:21         ` Andrew Rybchenko
2021-01-27  3:04           ` Xueming(Steven) Li
2021-01-27 12:10             ` Andrew Rybchenko
2021-01-28 14:31               ` Xueming(Steven) Li
2021-02-01  8:39                 ` Andrew Rybchenko
2021-02-04 14:15                   ` Xueming(Steven) Li
2021-02-05  7:34                     ` Andrew Rybchenko
2021-02-05  9:13                       ` Xueming(Steven) Li
2021-02-05  9:37                         ` Andrew Rybchenko
2021-01-27 17:43             ` Ajit Khaparde
2021-01-28 11:45               ` Xueming(Steven) Li
2021-01-21  3:32     ` Tu, Lijuan
2021-01-19  7:14   ` [dpdk-dev] [PATCH v5 1/9] ethdev: introduce representor type Xueming Li
2021-01-19  7:14   ` [dpdk-dev] [PATCH v5 2/9] ethdev: support representor port list Xueming Li
2021-01-19  7:48     ` Andrew Rybchenko
2021-01-19  8:19       ` Xueming(Steven) Li
2021-01-19  7:14   ` [dpdk-dev] [PATCH v5 3/9] ethdev: support new VF representor syntax Xueming Li
2021-01-19  7:51     ` Andrew Rybchenko
2021-01-19  7:14   ` [dpdk-dev] [PATCH v5 4/9] ethdev: support sub function representor Xueming Li
2021-01-19  7:53     ` Andrew Rybchenko
2021-01-19  7:14   ` [dpdk-dev] [PATCH v5 5/9] ethdev: support PF index in representor Xueming Li
2021-01-19  8:00     ` Andrew Rybchenko
2021-01-19  9:30       ` Xueming(Steven) Li
2021-01-19  9:36         ` Andrew Rybchenko
2021-01-19 11:57           ` Xueming(Steven) Li
2021-01-19  7:14   ` [dpdk-dev] [PATCH v5 6/9] ethdev: support multi-host " Xueming Li
2021-01-19  8:03     ` Andrew Rybchenko
2021-01-19  9:32       ` Xueming(Steven) Li
2021-01-19  7:14   ` [dpdk-dev] [PATCH v5 7/9] devarg: change representor ID to bitmap Xueming Li
2021-01-19  7:36     ` Wang, Haiyue
2021-01-19  8:11       ` Xueming(Steven) Li
2021-01-19  8:20     ` Andrew Rybchenko
2021-01-19  8:33       ` Thomas Monjalon
2021-01-19 11:04       ` Xueming(Steven) Li
2021-01-19 11:15         ` Andrew Rybchenko
2021-01-19  7:15   ` [dpdk-dev] [PATCH v5 8/9] ethdev: add capability of sub-function representor Xueming Li
2021-01-19  8:06     ` Andrew Rybchenko
2021-01-19 11:19       ` Xueming(Steven) Li
2021-01-19 11:29         ` Andrew Rybchenko
2021-01-19  7:15   ` [dpdk-dev] [PATCH v5 9/9] kvargs: update parser to support lists Xueming Li
2021-02-14  3:21   ` [dpdk-dev] [PATCH v6 0/9] ethdev: support SubFunction representor Xueming Li
2021-02-23  1:54     ` Stephen Hemminger
2021-02-23 11:45       ` Wang, Haiyue
2021-02-28 13:51         ` Xueming(Steven) Li
2021-03-31  5:49         ` Xueming(Steven) Li
2021-03-31  5:58           ` Wang, Haiyue
2021-02-14  3:21   ` [dpdk-dev] [PATCH v6 1/9] ethdev: introduce representor type Xueming Li
2021-02-15  2:25     ` Hyong Youb Kim (hyonkim)
2021-02-14  3:21   ` [dpdk-dev] [PATCH v6 2/9] ethdev: support representor port list Xueming Li
2021-02-15  8:13     ` Andrew Rybchenko
2021-02-14  3:21   ` [dpdk-dev] [PATCH v6 3/9] ethdev: support new VF representor syntax Xueming Li
2021-02-14  3:21   ` [dpdk-dev] [PATCH v6 4/9] ethdev: support sub function representor Xueming Li
2021-02-15  8:25     ` Andrew Rybchenko
2021-02-16  9:00       ` Xueming(Steven) Li
2021-02-14  3:21   ` [dpdk-dev] [PATCH v6 5/9] ethdev: support PF index in representor Xueming Li
2021-02-15  8:28     ` Andrew Rybchenko
2021-02-15  8:35     ` Andrew Rybchenko
2021-02-16 14:54       ` Xueming(Steven) Li
2021-02-14  3:21   ` [dpdk-dev] [PATCH v6 6/9] ethdev: support multi-host " Xueming Li
2021-02-15  8:37     ` Andrew Rybchenko
2021-02-14  3:21   ` [dpdk-dev] [PATCH v6 7/9] ethdev: new API to get representor info Xueming Li
2021-02-15  8:50     ` Andrew Rybchenko
2021-02-16 15:11       ` Xueming(Steven) Li
2021-02-14  3:21   ` [dpdk-dev] [PATCH v6 8/9] ethdev: representor iterator compare complete info Xueming Li
2021-02-15  9:31     ` Andrew Rybchenko
2021-02-16 16:35       ` Xueming(Steven) Li
2021-02-25  7:32         ` Andrew Rybchenko
2021-02-14  3:21   ` [dpdk-dev] [PATCH v6 9/9] kvargs: update parser to support lists Xueming Li
2021-03-02 11:40   ` [dpdk-dev] [PATCH v7 0/9] ethdev: support SubFunction representor Xueming Li
2021-03-02 11:40   ` [dpdk-dev] [PATCH v7 1/9] ethdev: introduce representor type Xueming Li
2021-03-02 11:42   ` [dpdk-dev] [PATCH v7 3/9] ethdev: support new VF representor syntax Xueming Li
2021-03-02 11:42   ` [dpdk-dev] [PATCH v7 4/9] ethdev: support sub function representor Xueming Li
2021-03-02 11:42   ` [dpdk-dev] [PATCH v7 5/9] ethdev: support PF index in representor Xueming Li
2021-03-02 11:51     ` Andrew Rybchenko
2021-03-02 11:43   ` [dpdk-dev] [PATCH v7 6/9] ethdev: support multi-host " Xueming Li
2021-03-02 11:43   ` [dpdk-dev] [PATCH v7 7/9] ethdev: new API to get representor info Xueming Li
2021-03-02 11:56     ` Andrew Rybchenko
2021-03-02 11:43   ` [dpdk-dev] [PATCH v7 8/9] ethdev: representor iterator compare complete info Xueming Li
2021-03-02 14:10     ` Andrew Rybchenko
2021-03-02 15:18       ` Xueming(Steven) Li
2021-03-02 11:44   ` [dpdk-dev] [PATCH v7 9/9] kvargs: update parser to support lists Xueming Li
2021-03-04 14:30   ` [dpdk-dev] [PATCH v8 0/9] ethdev: support SubFunction representor Xueming Li
2021-03-08 16:43     ` Ferruh Yigit
2021-03-09  7:13       ` Xueming(Steven) Li
2021-03-04 14:30   ` [dpdk-dev] [PATCH v8 1/9] ethdev: introduce representor type Xueming Li
2021-03-08 14:35     ` Ferruh Yigit
2021-03-04 14:30   ` [dpdk-dev] [PATCH v8 2/9] ethdev: support representor port list Xueming Li
2021-03-08 14:38     ` Ferruh Yigit
2021-03-08 15:58       ` Xueming(Steven) Li
2021-03-08 16:22         ` Ferruh Yigit
2021-03-09  6:16           ` Xueming(Steven) Li
2021-03-04 14:30   ` [dpdk-dev] [PATCH v8 3/9] ethdev: support new VF representor syntax Xueming Li
2021-03-04 14:30   ` [dpdk-dev] [PATCH v8 4/9] ethdev: support sub function representor Xueming Li
2021-03-04 14:30   ` [dpdk-dev] [PATCH v8 5/9] ethdev: support PF index in representor Xueming Li
2021-03-04 14:30   ` [dpdk-dev] [PATCH v8 6/9] ethdev: support multi-host " Xueming Li
2021-03-04 14:30   ` [dpdk-dev] [PATCH v8 7/9] ethdev: new API to get representor info Xueming Li
2021-03-08 14:43     ` Ferruh Yigit
2021-03-08 15:31       ` Xueming(Steven) Li
2021-03-08 16:12         ` Ferruh Yigit
2021-03-09  4:13           ` Xueming(Steven) Li
2021-03-04 14:30   ` [dpdk-dev] [PATCH v8 8/9] ethdev: representor iterator compare complete info Xueming Li
2021-03-08 16:18     ` Ferruh Yigit
2021-03-09  6:00       ` Xueming(Steven) Li
2021-03-09  8:19     ` Andrew Rybchenko
2021-03-11 13:18       ` Xueming(Steven) Li
2021-03-04 14:30   ` [dpdk-dev] [PATCH v8 9/9] kvargs: update parser to support lists Xueming Li
2021-03-08 14:45     ` Ferruh Yigit
2021-03-08 14:59       ` Xueming(Steven) Li
2021-03-08 15:54         ` Ferruh Yigit
2021-03-08 16:01           ` Xueming(Steven) Li
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 00/10] ethdev: support SubFunction representor Xueming Li
2021-03-16 19:18     ` Ferruh Yigit
2021-03-17  9:00       ` Xueming(Steven) Li
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 01/10] ethdev: introduce representor type Xueming Li
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 02/10] ethdev: refactor representor port list parsing Xueming Li
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 03/10] ethdev: support new VF representor syntax Xueming Li
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 04/10] ethdev: support sub function representor Xueming Li
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 05/10] kvargs: update parser to support multiple lists Xueming Li
2021-04-12 16:42     ` Kinsella, Ray
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 06/10] ethdev: support PF index in representor Xueming Li
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 07/10] ethdev: support multi-host " Xueming Li
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 08/10] ethdev: new API to get representor info Xueming Li
2021-03-16 19:18     ` Ferruh Yigit
2021-03-16 21:19       ` Thomas Monjalon
2021-03-16 23:34         ` Ferruh Yigit
2021-03-17  6:57           ` Thomas Monjalon
2021-03-17 17:11             ` Ferruh Yigit
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 09/10] ethdev: representor iterator compare complete info Xueming Li
2021-03-11 13:13   ` [dpdk-dev] [PATCH v9 10/10] doc/release: add representor enhancements Xueming Li
2020-12-18 14:55 ` [dpdk-dev] [RFC 2/7] ethdev: support multi-host representor Xueming Li
2020-12-28 13:43   ` Andrew Rybchenko
2020-12-18 14:55 ` [dpdk-dev] [RFC 3/7] devarg: change reprsentor ID to bitmap Xueming Li
2020-12-28 13:36   ` Andrew Rybchenko
2021-01-05  6:19     ` Xueming(Steven) Li
2020-12-18 14:55 ` [dpdk-dev] [RFC 4/7] ethdev: capability for new representor syntax Xueming Li
2020-12-28 12:02   ` Andrew Rybchenko
2020-12-18 14:55 ` [dpdk-dev] [RFC 5/7] kvargs: update parser " Xueming Li
2020-12-28 13:21   ` Andrew Rybchenko
2020-12-18 14:55 ` [dpdk-dev] [RFC 6/7] common/mlx5: update representor name parsing Xueming Li
2020-12-18 14:55 ` [dpdk-dev] [RFC 7/7] net/mlx5: support representor of sub function Xueming Li
2020-12-28 13:44 ` [dpdk-dev] [RFC 0/7] support SubFunction representor Andrew Rybchenko
2020-12-30  8:54   ` Xueming(Steven) Li
2020-12-30 11:07     ` Thomas Monjalon

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAOBf=mvAkWJMb=i5uSL49HUY8+tBhjHiD=532QE81qbhM+RxAg@mail.gmail.com' \
    --to=somnath.kotur@broadcom.com \
    --cc=andrew.rybchenko@oktetlabs.ru \
    --cc=asafp@nvidia.com \
    --cc=dev@dpdk.org \
    --cc=ferruh.yigit@intel.com \
    --cc=olivier.matz@6wind.com \
    --cc=thomas@monjalon.net \
    --cc=viacheslavo@nvidia.com \
    --cc=xuemingl@nvidia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

DPDK patches and discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://inbox.dpdk.org/dev/0 dev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dev dev/ https://inbox.dpdk.org/dev \
		dev@dpdk.org
	public-inbox-index dev

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dev


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git