[dpdk-dev] [PATCH v3 3/9] ethdev: support sub function representor

[Xueming Li]

SubFunction is a portion of the PCI device, created on demand, a SF
netdev has its own dedicated queues(txq, rxq). A SF netdev supports
eswitch representation offload similar to existing PF and VF
representors.

To support SF representor, this patch introduces new devargs syntax:

sf#: new SF port representor/s, example: sf[0-3], sf2

Signed-off-by: Xueming Li <xuemi...@nvidia.com>
---
 doc/guides/prog_guide/poll_mode_drv.rst |  1 +
 lib/librte_ethdev/ethdev_private.c      | 14 +++++++++++---
 lib/librte_ethdev/rte_ethdev_driver.h   |  1 +
 3 files changed, 13 insertions(+), 3 deletions(-)

diff --git a/doc/guides/prog_guide/poll_mode_drv.rst 
b/doc/guides/prog_guide/poll_mode_drv.rst
index 979a2bab9d..831d620c8d 100644
--- a/doc/guides/prog_guide/poll_mode_drv.rst
+++ b/doc/guides/prog_guide/poll_mode_drv.rst
@@ -377,6 +377,7 @@ parameters to those ports.
    -a DBDF,representor=vf0
    -a DBDF,representor=vf[0,4,6,9]
    -a DBDF,representor=vf[0-31]
+   -a DBDF,representor=sf[0-1023]
 
 Note: PMDs are not required to support the standard device arguments and users
 should consult the relevant PMD documentation to see support devargs.
diff --git a/lib/librte_ethdev/ethdev_private.c 
b/lib/librte_ethdev/ethdev_private.c
index 54f565ca37..551a43738a 100644
--- a/lib/librte_ethdev/ethdev_private.c
+++ b/lib/librte_ethdev/ethdev_private.c
@@ -96,6 +96,7 @@ rte_eth_devargs_process_list(char *str, uint16_t *list, 
uint16_t *len_list,
  * representor format:
  *   #: range or single number of VF representor - legacy
  *   vf#: VF port representor/s
+ *   sf#: SF port representor/s
  */
 int
 rte_eth_devargs_parse_representor_ports(char *str, void *data)
@@ -103,10 +104,17 @@ rte_eth_devargs_parse_representor_ports(char *str, void 
*data)
        struct rte_eth_devargs *eth_da = data;
        int ret;
 
-       /* Parse vf# or number # alone implies VF */
-       eth_da->type = RTE_ETH_REPRESENTOR_VF;
-       if (str[0] == 'v'  && str[1] == 'f')
+       eth_da->type = RTE_ETH_REPRESENTOR_NONE;
+       /* Parse vf# and sf#, number # alone implies VF */
+       if (str[0] == 'v'  && str[1] == 'f') {
+               eth_da->type = RTE_ETH_REPRESENTOR_VF;
                str += 2;
+       } else if (str[0] == 's'  && str[1] == 'f') {
+               eth_da->type = RTE_ETH_REPRESENTOR_SF;
+               str += 2;
+       } else {
+               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)
diff --git a/lib/librte_ethdev/rte_ethdev_driver.h 
b/lib/librte_ethdev/rte_ethdev_driver.h
index b66a955b18..086d64223a 100644
--- a/lib/librte_ethdev/rte_ethdev_driver.h
+++ b/lib/librte_ethdev/rte_ethdev_driver.h
@@ -1197,6 +1197,7 @@ rte_eth_switch_domain_free(uint16_t domain_id);
 enum rte_eth_representor_type {
        RTE_ETH_REPRESENTOR_NONE, /* not a representor */
        RTE_ETH_REPRESENTOR_VF,   /* representor of VF */
+       RTE_ETH_REPRESENTOR_SF,   /* representor of SF */
 };
 
 /** Generic Ethernet device arguments  */
-- 
2.25.1