From: Martin Spinler <[email protected]> NFB devices do not use separate PCI device for each port; one PCI device contains handles for all ports instead. For some application this approach can be limiting.
The "port" argument can be used to select only desired ports. It can be used multiple times. When is not used at all, the driver selects all ports. Signed-off-by: Martin Spinler <[email protected]> --- drivers/net/nfb/nfb.h | 6 ++++ drivers/net/nfb/nfb_ethdev.c | 55 +++++++++++++++++++++++++++++++++++- drivers/net/nfb/nfb_vdev.c | 3 +- 3 files changed, 62 insertions(+), 2 deletions(-) diff --git a/drivers/net/nfb/nfb.h b/drivers/net/nfb/nfb.h index bd381d4320..04d06a75cb 100644 --- a/drivers/net/nfb/nfb.h +++ b/drivers/net/nfb/nfb.h @@ -42,6 +42,12 @@ extern int nfb_logtype; #define RTE_NFB_DRIVER_NAME net_nfb +/* Device arguments */ +#define NFB_ARG_PORT "port" + +#define NFB_COMMON_ARGS \ + NFB_ARG_PORT "=<number>" \ + "" struct pmd_internals { uint16_t max_rxmac; diff --git a/drivers/net/nfb/nfb_ethdev.c b/drivers/net/nfb/nfb_ethdev.c index 2afd2a42db..0a948e4a2a 100644 --- a/drivers/net/nfb/nfb_ethdev.c +++ b/drivers/net/nfb/nfb_ethdev.c @@ -22,6 +22,11 @@ #include "nfb_rxmode.h" #include "nfb.h" +static const char * const VALID_KEYS[] = { + NFB_ARG_PORT, + NULL +}; + /* The TAILQ entries are used for cleanup of allocated resources * in situations, where dev_close is not called. */ @@ -657,6 +662,24 @@ nfb_eth_dev_uninit(struct rte_eth_dev *dev) return 0; } +static int fill_port_mask(const char *key __rte_unused, const char *value, void *opaque) +{ + int ret = 0; + char *end = NULL; + uint64_t *port_mask = opaque; + int port; + + port = strtol(value, &end, 0); + if ((value[0] == '\0') || end == NULL || (*end != '\0')) + ret = -1; + + if (ret != 0 || port >= 64 || port < 0) + return -1; + + *port_mask |= (1ull << port); + return 0; +} + RTE_EXPORT_INTERNAL_SYMBOL(nfb_eth_common_probe) int nfb_eth_common_probe(struct rte_device *device, @@ -664,7 +687,7 @@ nfb_eth_common_probe(struct rte_device *device, struct nfb_init_params *params, int ep_index) { int i; - int ret; + int ret = 0; int basename_len; struct nc_ifc_info *ifc; @@ -672,6 +695,9 @@ nfb_eth_common_probe(struct rte_device *device, struct rte_eth_dev *eth_dev; struct pmd_internals *p; + struct rte_kvargs *kvlist; + uint64_t port_mask = 0xFFFFFFFFFFFFFFFFull; + basename_len = strlen(params->name); nfb_dev = nfb_open(params->path); @@ -682,6 +708,27 @@ nfb_eth_common_probe(struct rte_device *device, nc_ifc_map_info_create_ordinary(nfb_dev, ¶ms->map_info); + if (params->args != NULL && strlen(params->args) > 0) { + kvlist = rte_kvargs_parse(params->args, VALID_KEYS); + if (kvlist == NULL) { + NFB_LOG(ERR, "Failed to parse device arguments %s", params->args); + return -EINVAL; + } + if (rte_kvargs_count(kvlist, NFB_ARG_PORT)) { + port_mask = 0; + if (rte_kvargs_process(kvlist, NFB_ARG_PORT, fill_port_mask, + (void *)&port_mask)) + ret = -1; + if (ret || port_mask >= (1ull << params->map_info.ifc_cnt)) + port_mask = 0; + } + rte_kvargs_free(kvlist); + if (port_mask == 0) { + NFB_LOG(ERR, "Failed to parse device port argument"); + return -EINVAL; + } + } + for (i = 0; i < params->map_info.ifc_cnt; i++) { ifc = ¶ms->map_info.ifc[i]; params->ifc_info = ifc; @@ -691,6 +738,9 @@ nfb_eth_common_probe(struct rte_device *device, (ifc->flags & NC_IFC_INFO_FLAG_ACTIVE) == 0) continue; + if ((port_mask & (1ull << i)) == 0) + continue; + snprintf(params->name + basename_len, sizeof(params->name) - basename_len, "_eth%d", params->ifc_info->id); @@ -809,3 +859,6 @@ RTE_PMD_REGISTER_PCI(RTE_NFB_DRIVER_NAME, nfb_eth_driver); RTE_PMD_REGISTER_PCI_TABLE(RTE_NFB_DRIVER_NAME, nfb_pci_id_table); RTE_PMD_REGISTER_KMOD_DEP(RTE_NFB_DRIVER_NAME, "* nfb"); RTE_LOG_REGISTER_DEFAULT(nfb_logtype, NOTICE); +RTE_PMD_REGISTER_PARAM_STRING(RTE_NFB_DRIVER_NAME, + NFB_COMMON_ARGS + ); diff --git a/drivers/net/nfb/nfb_vdev.c b/drivers/net/nfb/nfb_vdev.c index 3a8d8b0531..c815ca08f6 100644 --- a/drivers/net/nfb/nfb_vdev.c +++ b/drivers/net/nfb/nfb_vdev.c @@ -91,5 +91,6 @@ static struct rte_vdev_driver vdev_nfb_vdev = { RTE_PMD_REGISTER_VDEV(VDEV_NFB_DRIVER, vdev_nfb_vdev); RTE_PMD_REGISTER_ALIAS(VDEV_NFB_DRIVER, eth_vdev_nfb); RTE_PMD_REGISTER_PARAM_STRING(net_vdev_nfb, - VDEV_NFB_ARG_DEV "=<string>" + VDEV_NFB_ARG_DEV "=<string> " + NFB_COMMON_ARGS ); -- 2.52.0
