net/mlx5: add support for configuration through kvargs
The intent is to replace the remaining compile-time options and environment variables with a common mean of runtime configuration. This commit only adds the kvargs handling code, subsequent commits will update the rest. Signed-off-by: Nelio Laranjeiro <nelio.laranjeiro@6wind.com> Signed-off-by: Adrien Mazarguil <adrien.mazarguil@6wind.com>
This commit is contained in:
parent
51a50a3d9b
commit
e72dd09b61
@ -54,6 +54,7 @@ DEPDIRS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) += lib/librte_ether
|
|||||||
DEPDIRS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) += lib/librte_mbuf
|
DEPDIRS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) += lib/librte_mbuf
|
||||||
DEPDIRS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) += lib/librte_eal
|
DEPDIRS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) += lib/librte_eal
|
||||||
DEPDIRS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) += lib/librte_mempool
|
DEPDIRS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) += lib/librte_mempool
|
||||||
|
DEPDIRS-$(CONFIG_RTE_LIBRTE_MLX5_PMD) += lib/librte_kvargs
|
||||||
|
|
||||||
# Basic CFLAGS.
|
# Basic CFLAGS.
|
||||||
CFLAGS += -O3
|
CFLAGS += -O3
|
||||||
|
@ -37,6 +37,7 @@
|
|||||||
#include <assert.h>
|
#include <assert.h>
|
||||||
#include <stdint.h>
|
#include <stdint.h>
|
||||||
#include <stdlib.h>
|
#include <stdlib.h>
|
||||||
|
#include <errno.h>
|
||||||
#include <net/if.h>
|
#include <net/if.h>
|
||||||
|
|
||||||
/* Verbs header. */
|
/* Verbs header. */
|
||||||
@ -57,6 +58,7 @@
|
|||||||
#include <rte_ethdev.h>
|
#include <rte_ethdev.h>
|
||||||
#include <rte_pci.h>
|
#include <rte_pci.h>
|
||||||
#include <rte_common.h>
|
#include <rte_common.h>
|
||||||
|
#include <rte_kvargs.h>
|
||||||
#ifdef PEDANTIC
|
#ifdef PEDANTIC
|
||||||
#pragma GCC diagnostic error "-pedantic"
|
#pragma GCC diagnostic error "-pedantic"
|
||||||
#endif
|
#endif
|
||||||
@ -237,6 +239,71 @@ mlx5_dev_idx(struct rte_pci_addr *pci_addr)
|
|||||||
return ret;
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Verify and store value for device argument.
|
||||||
|
*
|
||||||
|
* @param[in] key
|
||||||
|
* Key argument to verify.
|
||||||
|
* @param[in] val
|
||||||
|
* Value associated with key.
|
||||||
|
* @param opaque
|
||||||
|
* User data.
|
||||||
|
*
|
||||||
|
* @return
|
||||||
|
* 0 on success, negative errno value on failure.
|
||||||
|
*/
|
||||||
|
static int
|
||||||
|
mlx5_args_check(const char *key, const char *val, void *opaque)
|
||||||
|
{
|
||||||
|
struct priv *priv = opaque;
|
||||||
|
|
||||||
|
/* No parameters are expected at the moment. */
|
||||||
|
(void)priv;
|
||||||
|
(void)val;
|
||||||
|
WARN("%s: unknown parameter", key);
|
||||||
|
return -EINVAL;
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Parse device parameters.
|
||||||
|
*
|
||||||
|
* @param priv
|
||||||
|
* Pointer to private structure.
|
||||||
|
* @param devargs
|
||||||
|
* Device arguments structure.
|
||||||
|
*
|
||||||
|
* @return
|
||||||
|
* 0 on success, errno value on failure.
|
||||||
|
*/
|
||||||
|
static int
|
||||||
|
mlx5_args(struct priv *priv, struct rte_devargs *devargs)
|
||||||
|
{
|
||||||
|
const char **params = (const char *[]){
|
||||||
|
NULL,
|
||||||
|
};
|
||||||
|
struct rte_kvargs *kvlist;
|
||||||
|
int ret = 0;
|
||||||
|
int i;
|
||||||
|
|
||||||
|
if (devargs == NULL)
|
||||||
|
return 0;
|
||||||
|
/* Following UGLY cast is done to pass checkpatch. */
|
||||||
|
kvlist = rte_kvargs_parse(devargs->args, params);
|
||||||
|
if (kvlist == NULL)
|
||||||
|
return 0;
|
||||||
|
/* Process parameters. */
|
||||||
|
for (i = 0; (params[i] != NULL); ++i) {
|
||||||
|
if (rte_kvargs_count(kvlist, params[i])) {
|
||||||
|
ret = rte_kvargs_process(kvlist, params[i],
|
||||||
|
mlx5_args_check, priv);
|
||||||
|
if (ret != 0)
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
rte_kvargs_free(kvlist);
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
static struct eth_driver mlx5_driver;
|
static struct eth_driver mlx5_driver;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@ -408,6 +475,12 @@ mlx5_pci_devinit(struct rte_pci_driver *pci_drv, struct rte_pci_device *pci_dev)
|
|||||||
priv->port = port;
|
priv->port = port;
|
||||||
priv->pd = pd;
|
priv->pd = pd;
|
||||||
priv->mtu = ETHER_MTU;
|
priv->mtu = ETHER_MTU;
|
||||||
|
err = mlx5_args(priv, pci_dev->devargs);
|
||||||
|
if (err) {
|
||||||
|
ERROR("failed to process device arguments: %s",
|
||||||
|
strerror(err));
|
||||||
|
goto port_error;
|
||||||
|
}
|
||||||
if (ibv_exp_query_device(ctx, &exp_device_attr)) {
|
if (ibv_exp_query_device(ctx, &exp_device_attr)) {
|
||||||
ERROR("ibv_exp_query_device() failed");
|
ERROR("ibv_exp_query_device() failed");
|
||||||
goto port_error;
|
goto port_error;
|
||||||
|
Loading…
Reference in New Issue
Block a user