DPDK patches and discussions
 help / color / mirror / Atom feed
From: Slava Ovsiienko <viacheslavo@mellanox.com>
To: Vu Pham <vuhuong@mellanox.com>, "dev@dpdk.org" <dev@dpdk.org>
Cc: Ori Kam <orika@mellanox.com>, Matan Azrad <matan@mellanox.com>,
	Raslan Darawsheh <rasland@mellanox.com>,
	Vu Pham <vuhuong@mellanox.com>
Subject: Re: [dpdk-dev] [PATCH v4 1/2] common/mlx5: refactor multi-process IPC handling codes to common driver
Date: Tue, 14 Apr 2020 07:26:41 +0000	[thread overview]
Message-ID: <AM4PR05MB3265B6C2E59A76EAFD09CDC5D2DA0@AM4PR05MB3265.eurprd05.prod.outlook.com> (raw)
In-Reply-To: <20200413211748.20500-2-vuhuong@mellanox.com>

> -----Original Message-----
> From: Vu Pham <vuhuong@mellanox.com>
> Sent: Tuesday, April 14, 2020 0:18
> To: dev@dpdk.org
> Cc: Slava Ovsiienko <viacheslavo@mellanox.com>; Ori Kam
> <orika@mellanox.com>; Matan Azrad <matan@mellanox.com>; Raslan
> Darawsheh <rasland@mellanox.com>; Vu Pham <vuhuong@mellanox.com>
> Subject: [PATCH v4 1/2] common/mlx5: refactor multi-process IPC handling
> codes to common driver
> 
> Refactor common multi-process handling codes from net PMD to common
> driver. Using tuple mp_id{name, port_id} as standard input parameter for all
> multi-process IPC APIs instead of using rte_eth_dev.
> 
> Modify net PMD to use multi-process APIs from common driver.
> 
> Signed-off-by: Vu Pham <vuhuong@mellanox.com>
Acked-by: Viacheslav Ovsiienko <viacheslavo@mellanox.com>

> ---
>  drivers/common/mlx5/Makefile                    |   3 +-
>  drivers/common/mlx5/meson.build                 |   1 +
>  drivers/common/mlx5/mlx5_common_mp.c            | 188
> +++++++++++++++++++
>  drivers/common/mlx5/mlx5_common_mp.h            |  98 ++++++++++
>  drivers/common/mlx5/rte_common_mlx5_version.map |  13 ++
>  drivers/net/mlx5/mlx5.c                         |  15 +-
>  drivers/net/mlx5/mlx5.h                         |  43 +----
>  drivers/net/mlx5/mlx5_mp.c                      | 234 ++----------------------
>  drivers/net/mlx5/mlx5_mr.c                      |   2 +-
>  drivers/net/mlx5/mlx5_rxtx.c                    |   3 +-
>  10 files changed, 336 insertions(+), 264 deletions(-)  create mode 100644
> drivers/common/mlx5/mlx5_common_mp.c
>  create mode 100644 drivers/common/mlx5/mlx5_common_mp.h
> 
> diff --git a/drivers/common/mlx5/Makefile b/drivers/common/mlx5/Makefile
> index f32933d592..2a88492731 100644
> --- a/drivers/common/mlx5/Makefile
> +++ b/drivers/common/mlx5/Makefile
> @@ -17,6 +17,7 @@ endif
>  SRCS-y += mlx5_devx_cmds.c
>  SRCS-y += mlx5_common.c
>  SRCS-y += mlx5_nl.c
> +SRCS-y += mlx5_common_mp.c
>  ifeq ($(CONFIG_RTE_IBVERBS_LINK_DLOPEN),y)
>  INSTALL-y-lib += $(LIB_GLUE)
>  endif
> @@ -46,7 +47,7 @@ endif
>  LDLIBS += -lrte_eal -lrte_pci -lrte_kvargs -lrte_net
> 
>  # A few warnings cannot be avoided in external headers.
> -CFLAGS += -Wno-error=cast-qual -UPEDANTIC
> +CFLAGS += -Wno-error=cast-qual  -UPEDANTIC -DALLOW_EXPERIMENTAL_API
> 
>  EXPORT_MAP := rte_common_mlx5_version.map
> 
> diff --git a/drivers/common/mlx5/meson.build
> b/drivers/common/mlx5/meson.build index f671710714..83671861c9 100644
> --- a/drivers/common/mlx5/meson.build
> +++ b/drivers/common/mlx5/meson.build
> @@ -55,6 +55,7 @@ sources = files(
>  	'mlx5_devx_cmds.c',
>  	'mlx5_common.c',
>  	'mlx5_nl.c',
> +	'mlx5_common_mp.c',
>  )
>  if not dlopen_ibverbs
>  	sources += files('mlx5_glue.c')
> diff --git a/drivers/common/mlx5/mlx5_common_mp.c
> b/drivers/common/mlx5/mlx5_common_mp.c
> new file mode 100644
> index 0000000000..da55143bc1
> --- /dev/null
> +++ b/drivers/common/mlx5/mlx5_common_mp.c
> @@ -0,0 +1,188 @@
> +/* SPDX-License-Identifier: BSD-3-Clause
> + * Copyright 2019 6WIND S.A.
> + * Copyright 2019 Mellanox Technologies, Ltd  */
> +
> +#include <stdio.h>
> +#include <time.h>
> +
> +#include <rte_eal.h>
> +#include <rte_errno.h>
> +
> +#include "mlx5_common_mp.h"
> +#include "mlx5_common_utils.h"
> +
> +/**
> + * Request Memory Region creation to the primary process.
> + *
> + * @param[in] mp_id
> + *   ID of the MP process.
> + * @param addr
> + *   Target virtual address to register.
> + *
> + * @return
> + *   0 on success, a negative errno value otherwise and rte_errno is set.
> + */
> +int
> +mlx5_mp_req_mr_create(struct mlx5_mp_id *mp_id, uintptr_t addr) {
> +	struct rte_mp_msg mp_req;
> +	struct rte_mp_msg *mp_res;
> +	struct rte_mp_reply mp_rep;
> +	struct mlx5_mp_param *req = (struct mlx5_mp_param
> *)mp_req.param;
> +	struct mlx5_mp_param *res;
> +	struct timespec ts = {.tv_sec = MLX5_MP_REQ_TIMEOUT_SEC, .tv_nsec
> = 0};
> +	int ret;
> +
> +	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> +	mp_init_msg(mp_id, &mp_req, MLX5_MP_REQ_CREATE_MR);
> +	req->args.addr = addr;
> +	ret = rte_mp_request_sync(&mp_req, &mp_rep, &ts);
> +	if (ret) {
> +		DRV_LOG(ERR, "port %u request to primary process failed",
> +			mp_id->port_id);
> +		return -rte_errno;
> +	}
> +	MLX5_ASSERT(mp_rep.nb_received == 1);
> +	mp_res = &mp_rep.msgs[0];
> +	res = (struct mlx5_mp_param *)mp_res->param;
> +	ret = res->result;
> +	if (ret)
> +		rte_errno = -ret;
> +	free(mp_rep.msgs);
> +	return ret;
> +}
> +
> +/**
> + * Request Verbs queue state modification to the primary process.
> + *
> + * @param[in] mp_id
> + *   ID of the MP process.
> + * @param sm
> + *   State modify parameters.
> + *
> + * @return
> + *   0 on success, a negative errno value otherwise and rte_errno is set.
> + */
> +int
> +mlx5_mp_req_queue_state_modify(struct mlx5_mp_id *mp_id,
> +			       struct mlx5_mp_arg_queue_state_modify *sm) {
> +	struct rte_mp_msg mp_req;
> +	struct rte_mp_msg *mp_res;
> +	struct rte_mp_reply mp_rep;
> +	struct mlx5_mp_param *req = (struct mlx5_mp_param
> *)mp_req.param;
> +	struct mlx5_mp_param *res;
> +	struct timespec ts = {.tv_sec = MLX5_MP_REQ_TIMEOUT_SEC, .tv_nsec
> = 0};
> +	int ret;
> +
> +	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> +	mp_init_msg(mp_id, &mp_req,
> MLX5_MP_REQ_QUEUE_STATE_MODIFY);
> +	req->args.state_modify = *sm;
> +	ret = rte_mp_request_sync(&mp_req, &mp_rep, &ts);
> +	if (ret) {
> +		DRV_LOG(ERR, "port %u request to primary process failed",
> +			mp_id->port_id);
> +		return -rte_errno;
> +	}
> +	MLX5_ASSERT(mp_rep.nb_received == 1);
> +	mp_res = &mp_rep.msgs[0];
> +	res = (struct mlx5_mp_param *)mp_res->param;
> +	ret = res->result;
> +	free(mp_rep.msgs);
> +	return ret;
> +}
> +
> +/**
> + * Request Verbs command file descriptor for mmap to the primary process.
> + *
> + * @param[in] mp_id
> + *   ID of the MP process.
> + *
> + * @return
> + *   fd on success, a negative errno value otherwise and rte_errno is set.
> + */
> +int
> +mlx5_mp_req_verbs_cmd_fd(struct mlx5_mp_id *mp_id) {
> +	struct rte_mp_msg mp_req;
> +	struct rte_mp_msg *mp_res;
> +	struct rte_mp_reply mp_rep;
> +	struct mlx5_mp_param *res;
> +	struct timespec ts = {.tv_sec = MLX5_MP_REQ_TIMEOUT_SEC, .tv_nsec
> = 0};
> +	int ret;
> +
> +	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> +	mp_init_msg(mp_id, &mp_req, MLX5_MP_REQ_VERBS_CMD_FD);
> +	ret = rte_mp_request_sync(&mp_req, &mp_rep, &ts);
> +	if (ret) {
> +		DRV_LOG(ERR, "port %u request to primary process failed",
> +			mp_id->port_id);
> +		return -rte_errno;
> +	}
> +	MLX5_ASSERT(mp_rep.nb_received == 1);
> +	mp_res = &mp_rep.msgs[0];
> +	res = (struct mlx5_mp_param *)mp_res->param;
> +	if (res->result) {
> +		rte_errno = -res->result;
> +		DRV_LOG(ERR,
> +			"port %u failed to get command FD from primary
> process",
> +			mp_id->port_id);
> +		ret = -rte_errno;
> +		goto exit;
> +	}
> +	MLX5_ASSERT(mp_res->num_fds == 1);
> +	ret = mp_res->fds[0];
> +	DRV_LOG(DEBUG, "port %u command FD from primary is %d",
> +		mp_id->port_id, ret);
> +exit:
> +	free(mp_rep.msgs);
> +	return ret;
> +}
> +
> +/**
> + * Initialize by primary process.
> + */
> +int
> +mlx5_mp_init_primary(const char *name, const rte_mp_t primary_action) {
> +	int ret;
> +
> +	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_PRIMARY);
> +
> +	/* primary is allowed to not support IPC */
> +	ret = rte_mp_action_register(name, primary_action);
> +	if (ret && rte_errno != ENOTSUP)
> +		return -1;
> +	return 0;
> +}
> +
> +/**
> + * Un-initialize by primary process.
> + */
> +void
> +mlx5_mp_uninit_primary(const char *name) {
> +	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_PRIMARY);
> +	rte_mp_action_unregister(name);
> +}
> +
> +/**
> + * Initialize by secondary process.
> + */
> +int
> +mlx5_mp_init_secondary(const char *name, const rte_mp_t
> +secondary_action) {
> +	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> +	return rte_mp_action_register(name, secondary_action); }
> +
> +/**
> + * Un-initialize by secondary process.
> + */
> +void
> +mlx5_mp_uninit_secondary(const char *name) {
> +	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> +	rte_mp_action_unregister(name);
> +}
> diff --git a/drivers/common/mlx5/mlx5_common_mp.h
> b/drivers/common/mlx5/mlx5_common_mp.h
> new file mode 100644
> index 0000000000..7aab77acb2
> --- /dev/null
> +++ b/drivers/common/mlx5/mlx5_common_mp.h
> @@ -0,0 +1,98 @@
> +/* SPDX-License-Identifier: BSD-3-Clause
> + * Copyright 2018 6WIND S.A.
> + * Copyright 2018 Mellanox Technologies, Ltd  */
> +
> +#ifndef RTE_PMD_MLX5_COMMON_MP_H_
> +#define RTE_PMD_MLX5_COMMON_MP_H_
> +
> +/* Verbs header. */
> +/* ISO C doesn't support unnamed structs/unions, disabling -pedantic.
> +*/ #ifdef PEDANTIC #pragma GCC diagnostic ignored "-Wpedantic"
> +#endif
> +#include <infiniband/verbs.h>
> +#ifdef PEDANTIC
> +#pragma GCC diagnostic error "-Wpedantic"
> +#endif
> +
> +#include <rte_eal.h>
> +#include <rte_string_fns.h>
> +
> +/* Request types for IPC. */
> +enum mlx5_mp_req_type {
> +	MLX5_MP_REQ_VERBS_CMD_FD = 1,
> +	MLX5_MP_REQ_CREATE_MR,
> +	MLX5_MP_REQ_START_RXTX,
> +	MLX5_MP_REQ_STOP_RXTX,
> +	MLX5_MP_REQ_QUEUE_STATE_MODIFY,
> +};
> +
> +struct mlx5_mp_arg_queue_state_modify {
> +	uint8_t is_wq; /* Set if WQ. */
> +	uint16_t queue_id; /* DPDK queue ID. */
> +	enum ibv_wq_state state; /* WQ requested state. */ };
> +
> +/* Pameters for IPC. */
> +struct mlx5_mp_param {
> +	enum mlx5_mp_req_type type;
> +	int port_id;
> +	int result;
> +	RTE_STD_C11
> +	union {
> +		uintptr_t addr; /* MLX5_MP_REQ_CREATE_MR */
> +		struct mlx5_mp_arg_queue_state_modify state_modify;
> +		/* MLX5_MP_REQ_QUEUE_STATE_MODIFY */
> +	} args;
> +};
> +
> +/*  Identifier of a MP process */
> +struct mlx5_mp_id {
> +	char name[RTE_MP_MAX_NAME_LEN];
> +	uint16_t port_id;
> +};
> +
> +/** Request timeout for IPC. */
> +#define MLX5_MP_REQ_TIMEOUT_SEC 5
> +
> +/**
> + * Initialize IPC message.
> + *
> + * @param[in] port_id
> + *   Port ID of the device.
> + * @param[out] msg
> + *   Pointer to message to fill in.
> + * @param[in] type
> + *   Message type.
> + */
> +static inline void
> +mp_init_msg(struct mlx5_mp_id *mp_id, struct rte_mp_msg *msg,
> +	    enum mlx5_mp_req_type type)
> +{
> +	struct mlx5_mp_param *param = (struct mlx5_mp_param *)msg-
> >param;
> +
> +	memset(msg, 0, sizeof(*msg));
> +	strlcpy(msg->name, mp_id->name, sizeof(msg->name));
> +	msg->len_param = sizeof(*param);
> +	param->type = type;
> +	param->port_id = mp_id->port_id;
> +}
> +
> +__rte_experimental
> +int mlx5_mp_init_primary(const char *name, const rte_mp_t
> +primary_action); __rte_experimental void mlx5_mp_uninit_primary(const
> +char *name); __rte_experimental int mlx5_mp_init_secondary(const char
> +*name, const rte_mp_t secondary_action); __rte_experimental void
> +mlx5_mp_uninit_secondary(const char *name); __rte_experimental int
> +mlx5_mp_req_mr_create(struct mlx5_mp_id *mp_id, uintptr_t addr);
> +__rte_experimental int mlx5_mp_req_queue_state_modify(struct
> mlx5_mp_id
> +*mp_id,
> +				   struct mlx5_mp_arg_queue_state_modify
> *sm); __rte_experimental
> +int mlx5_mp_req_verbs_cmd_fd(struct mlx5_mp_id *mp_id);
> +
> +#endif /* RTE_PMD_MLX5_COMMON_MP_H_ */
> diff --git a/drivers/common/mlx5/rte_common_mlx5_version.map
> b/drivers/common/mlx5/rte_common_mlx5_version.map
> index aede2a0a51..265703d1c9 100644
> --- a/drivers/common/mlx5/rte_common_mlx5_version.map
> +++ b/drivers/common/mlx5/rte_common_mlx5_version.map
> @@ -48,4 +48,17 @@ DPDK_20.0.1 {
>  	mlx5_nl_vlan_vmwa_delete;
> 
>  	mlx5_translate_port_name;
> +
> +};
> +
> +EXPERIMENTAL {
> +        global:
> +
> +	mlx5_mp_init_primary;
> +	mlx5_mp_uninit_primary;
> +	mlx5_mp_init_secondary;
> +	mlx5_mp_uninit_secondary;
> +	mlx5_mp_req_mr_create;
> +	mlx5_mp_req_queue_state_modify;
> +	mlx5_mp_req_verbs_cmd_fd;
>  };
> diff --git a/drivers/net/mlx5/mlx5.c b/drivers/net/mlx5/mlx5.c index
> 293d316413..d87c384422 100644
> --- a/drivers/net/mlx5/mlx5.c
> +++ b/drivers/net/mlx5/mlx5.c
> @@ -38,6 +38,7 @@
>  #include <mlx5_glue.h>
>  #include <mlx5_devx_cmds.h>
>  #include <mlx5_common.h>
> +#include <mlx5_common_mp.h>
> 
>  #include "mlx5_defs.h"
>  #include "mlx5.h"
> @@ -1722,7 +1723,8 @@ mlx5_init_once(void)
>  		rte_rwlock_init(&sd->mem_event_rwlock);
>  		rte_mem_event_callback_register("MLX5_MEM_EVENT_CB",
>  						mlx5_mr_mem_event_cb,
> NULL);
> -		ret = mlx5_mp_init_primary();
> +		ret = mlx5_mp_init_primary(MLX5_MP_NAME,
> +					   mlx5_mp_primary_handle);
>  		if (ret)
>  			goto out;
>  		sd->init_done = true;
> @@ -1730,7 +1732,8 @@ mlx5_init_once(void)
>  	case RTE_PROC_SECONDARY:
>  		if (ld->init_done)
>  			break;
> -		ret = mlx5_mp_init_secondary();
> +		ret = mlx5_mp_init_secondary(MLX5_MP_NAME,
> +					     mlx5_mp_secondary_handle);
>  		if (ret)
>  			goto out;
>  		++sd->secondary_cnt;
> @@ -2205,6 +2208,8 @@ mlx5_dev_spawn(struct rte_device *dpdk_dev,
>  	}
>  	DRV_LOG(DEBUG, "naming Ethernet device \"%s\"", name);
>  	if (rte_eal_process_type() == RTE_PROC_SECONDARY) {
> +		struct mlx5_mp_id mp_id;
> +
>  		eth_dev = rte_eth_dev_attach_secondary(name);
>  		if (eth_dev == NULL) {
>  			DRV_LOG(ERR, "can not attach rte ethdev"); @@ -
> 2216,8 +2221,10 @@ mlx5_dev_spawn(struct rte_device *dpdk_dev,
>  		err = mlx5_proc_priv_init(eth_dev);
>  		if (err)
>  			return NULL;
> +		mp_id.port_id = eth_dev->data->port_id;
> +		strlcpy(mp_id.name, MLX5_MP_NAME,
> RTE_MP_MAX_NAME_LEN);
>  		/* Receive command fd from primary process */
> -		err = mlx5_mp_req_verbs_cmd_fd(eth_dev);
> +		err = mlx5_mp_req_verbs_cmd_fd(&mp_id);
>  		if (err < 0)
>  			return NULL;
>  		/* Remap UAR for Tx queues. */
> @@ -2379,6 +2386,8 @@ mlx5_dev_spawn(struct rte_device *dpdk_dev,
>  	priv->ibv_port = spawn->ibv_port;
>  	priv->pci_dev = spawn->pci_dev;
>  	priv->mtu = RTE_ETHER_MTU;
> +	priv->mp_id.port_id = port_id;
> +	strlcpy(priv->mp_id.name, MLX5_MP_NAME,
> RTE_MP_MAX_NAME_LEN);
>  #ifndef RTE_ARCH_64
>  	/* Initialize UAR access locks for 32bit implementations. */
>  	rte_spinlock_init(&priv->uar_lock_cq);
> diff --git a/drivers/net/mlx5/mlx5.h b/drivers/net/mlx5/mlx5.h index
> fccfe47341..e9d5868883 100644
> --- a/drivers/net/mlx5/mlx5.h
> +++ b/drivers/net/mlx5/mlx5.h
> @@ -36,43 +36,13 @@
>  #include <mlx5_devx_cmds.h>
>  #include <mlx5_prm.h>
>  #include <mlx5_nl.h>
> +#include <mlx5_common_mp.h>
> 
>  #include "mlx5_defs.h"
>  #include "mlx5_utils.h"
>  #include "mlx5_mr.h"
>  #include "mlx5_autoconf.h"
> 
> -/* Request types for IPC. */
> -enum mlx5_mp_req_type {
> -	MLX5_MP_REQ_VERBS_CMD_FD = 1,
> -	MLX5_MP_REQ_CREATE_MR,
> -	MLX5_MP_REQ_START_RXTX,
> -	MLX5_MP_REQ_STOP_RXTX,
> -	MLX5_MP_REQ_QUEUE_STATE_MODIFY,
> -};
> -
> -struct mlx5_mp_arg_queue_state_modify {
> -	uint8_t is_wq; /* Set if WQ. */
> -	uint16_t queue_id; /* DPDK queue ID. */
> -	enum ibv_wq_state state; /* WQ requested state. */
> -};
> -
> -/* Pameters for IPC. */
> -struct mlx5_mp_param {
> -	enum mlx5_mp_req_type type;
> -	int port_id;
> -	int result;
> -	RTE_STD_C11
> -	union {
> -		uintptr_t addr; /* MLX5_MP_REQ_CREATE_MR */
> -		struct mlx5_mp_arg_queue_state_modify state_modify;
> -		/* MLX5_MP_REQ_QUEUE_STATE_MODIFY */
> -	} args;
> -};
> -
> -/** Request timeout for IPC. */
> -#define MLX5_MP_REQ_TIMEOUT_SEC 5
> -
>  /** Key string for IPC. */
>  #define MLX5_MP_NAME "net_mlx5_mp"
> 
> @@ -583,6 +553,7 @@ struct mlx5_priv {
>  #endif
>  	uint8_t skip_default_rss_reta; /* Skip configuration of default reta. */
>  	uint8_t fdb_def_rule; /* Whether fdb jump to table 1 is configured. */
> +	struct mlx5_mp_id mp_id; /* ID of a multi-process process */
>  };
> 
>  #define PORT_ID(priv) ((priv)->dev_data->port_id) @@ -783,16 +754,10 @@
> int mlx5_flow_dev_dump(struct rte_eth_dev *dev, FILE *file,
>  		       struct rte_flow_error *error);
> 
>  /* mlx5_mp.c */
> +int mlx5_mp_primary_handle(const struct rte_mp_msg *mp_msg, const void
> +*peer); int mlx5_mp_secondary_handle(const struct rte_mp_msg *mp_msg,
> +const void *peer);
>  void mlx5_mp_req_start_rxtx(struct rte_eth_dev *dev);  void
> mlx5_mp_req_stop_rxtx(struct rte_eth_dev *dev); -int
> mlx5_mp_req_mr_create(struct rte_eth_dev *dev, uintptr_t addr); -int
> mlx5_mp_req_verbs_cmd_fd(struct rte_eth_dev *dev); -int
> mlx5_mp_req_queue_state_modify(struct rte_eth_dev *dev,
> -				   struct mlx5_mp_arg_queue_state_modify
> *sm);
> -int mlx5_mp_init_primary(void);
> -void mlx5_mp_uninit_primary(void);
> -int mlx5_mp_init_secondary(void);
> -void mlx5_mp_uninit_secondary(void);
> 
>  /* mlx5_socket.c */
> 
> diff --git a/drivers/net/mlx5/mlx5_mp.c b/drivers/net/mlx5/mlx5_mp.c index
> 55d408fe95..43684dbc3a 100644
> --- a/drivers/net/mlx5/mlx5_mp.c
> +++ b/drivers/net/mlx5/mlx5_mp.c
> @@ -10,46 +10,14 @@
>  #include <rte_ethdev_driver.h>
>  #include <rte_string_fns.h>
> 
> +#include <mlx5_common_mp.h>
> +
>  #include "mlx5.h"
>  #include "mlx5_rxtx.h"
>  #include "mlx5_utils.h"
> 
> -/**
> - * Initialize IPC message.
> - *
> - * @param[in] dev
> - *   Pointer to Ethernet structure.
> - * @param[out] msg
> - *   Pointer to message to fill in.
> - * @param[in] type
> - *   Message type.
> - */
> -static inline void
> -mp_init_msg(struct rte_eth_dev *dev, struct rte_mp_msg *msg,
> -	    enum mlx5_mp_req_type type)
> -{
> -	struct mlx5_mp_param *param = (struct mlx5_mp_param *)msg-
> >param;
> -
> -	memset(msg, 0, sizeof(*msg));
> -	strlcpy(msg->name, MLX5_MP_NAME, sizeof(msg->name));
> -	msg->len_param = sizeof(*param);
> -	param->type = type;
> -	param->port_id = dev->data->port_id;
> -}
> -
> -/**
> - * IPC message handler of primary process.
> - *
> - * @param[in] dev
> - *   Pointer to Ethernet structure.
> - * @param[in] peer
> - *   Pointer to the peer socket path.
> - *
> - * @return
> - *   0 on success, a negative errno value otherwise and rte_errno is set.
> - */
> -static int
> -mp_primary_handle(const struct rte_mp_msg *mp_msg, const void *peer)
> +int
> +mlx5_mp_primary_handle(const struct rte_mp_msg *mp_msg, const void
> +*peer)
>  {
>  	struct rte_mp_msg mp_res;
>  	struct mlx5_mp_param *res = (struct mlx5_mp_param
> *)mp_res.param; @@ -71,21 +39,21 @@ mp_primary_handle(const struct
> rte_mp_msg *mp_msg, const void *peer)
>  	priv = dev->data->dev_private;
>  	switch (param->type) {
>  	case MLX5_MP_REQ_CREATE_MR:
> -		mp_init_msg(dev, &mp_res, param->type);
> +		mp_init_msg(&priv->mp_id, &mp_res, param->type);
>  		lkey = mlx5_mr_create_primary(dev, &entry, param-
> >args.addr);
>  		if (lkey == UINT32_MAX)
>  			res->result = -rte_errno;
>  		ret = rte_mp_reply(&mp_res, peer);
>  		break;
>  	case MLX5_MP_REQ_VERBS_CMD_FD:
> -		mp_init_msg(dev, &mp_res, param->type);
> +		mp_init_msg(&priv->mp_id, &mp_res, param->type);
>  		mp_res.num_fds = 1;
>  		mp_res.fds[0] = priv->sh->ctx->cmd_fd;
>  		res->result = 0;
>  		ret = rte_mp_reply(&mp_res, peer);
>  		break;
>  	case MLX5_MP_REQ_QUEUE_STATE_MODIFY:
> -		mp_init_msg(dev, &mp_res, param->type);
> +		mp_init_msg(&priv->mp_id, &mp_res, param->type);
>  		res->result = mlx5_queue_state_modify_primary
>  					(dev, &param->args.state_modify);
>  		ret = rte_mp_reply(&mp_res, peer);
> @@ -110,14 +78,15 @@ mp_primary_handle(const struct rte_mp_msg
> *mp_msg, const void *peer)
>   * @return
>   *   0 on success, a negative errno value otherwise and rte_errno is set.
>   */
> -static int
> -mp_secondary_handle(const struct rte_mp_msg *mp_msg, const void *peer)
> +int
> +mlx5_mp_secondary_handle(const struct rte_mp_msg *mp_msg, const void
> +*peer)
>  {
>  	struct rte_mp_msg mp_res;
>  	struct mlx5_mp_param *res = (struct mlx5_mp_param
> *)mp_res.param;
>  	const struct mlx5_mp_param *param =
>  		(const struct mlx5_mp_param *)mp_msg->param;
>  	struct rte_eth_dev *dev;
> +	struct mlx5_priv *priv;
>  	int ret;
> 
>  	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> @@ -127,13 +96,14 @@ mp_secondary_handle(const struct rte_mp_msg
> *mp_msg, const void *peer)
>  		return -rte_errno;
>  	}
>  	dev = &rte_eth_devices[param->port_id];
> +	priv = dev->data->dev_private;
>  	switch (param->type) {
>  	case MLX5_MP_REQ_START_RXTX:
>  		DRV_LOG(INFO, "port %u starting datapath", dev->data-
> >port_id);
>  		rte_mb();
>  		dev->rx_pkt_burst = mlx5_select_rx_function(dev);
>  		dev->tx_pkt_burst = mlx5_select_tx_function(dev);
> -		mp_init_msg(dev, &mp_res, param->type);
> +		mp_init_msg(&priv->mp_id, &mp_res, param->type);
>  		res->result = 0;
>  		ret = rte_mp_reply(&mp_res, peer);
>  		break;
> @@ -142,7 +112,7 @@ mp_secondary_handle(const struct rte_mp_msg
> *mp_msg, const void *peer)
>  		dev->rx_pkt_burst = removed_rx_burst;
>  		dev->tx_pkt_burst = removed_tx_burst;
>  		rte_mb();
> -		mp_init_msg(dev, &mp_res, param->type);
> +		mp_init_msg(&priv->mp_id, &mp_res, param->type);
>  		res->result = 0;
>  		ret = rte_mp_reply(&mp_res, peer);
>  		break;
> @@ -171,6 +141,7 @@ mp_req_on_rxtx(struct rte_eth_dev *dev, enum
> mlx5_mp_req_type type)
>  	struct rte_mp_reply mp_rep;
>  	struct mlx5_mp_param *res;
>  	struct timespec ts = {.tv_sec = MLX5_MP_REQ_TIMEOUT_SEC, .tv_nsec
> = 0};
> +	struct mlx5_priv *priv = dev->data->dev_private;
>  	int ret;
>  	int i;
> 
> @@ -182,7 +153,7 @@ mp_req_on_rxtx(struct rte_eth_dev *dev, enum
> mlx5_mp_req_type type)
>  			dev->data->port_id, type);
>  		return;
>  	}
> -	mp_init_msg(dev, &mp_req, type);
> +	mp_init_msg(&priv->mp_id, &mp_req, type);
>  	ret = rte_mp_request_sync(&mp_req, &mp_rep, &ts);
>  	if (ret) {
>  		if (rte_errno != ENOTSUP)
> @@ -234,178 +205,3 @@ mlx5_mp_req_stop_rxtx(struct rte_eth_dev *dev)  {
>  	mp_req_on_rxtx(dev, MLX5_MP_REQ_STOP_RXTX);  }
> -
> -/**
> - * Request Memory Region creation to the primary process.
> - *
> - * @param[in] dev
> - *   Pointer to Ethernet structure.
> - * @param addr
> - *   Target virtual address to register.
> - *
> - * @return
> - *   0 on success, a negative errno value otherwise and rte_errno is set.
> - */
> -int
> -mlx5_mp_req_mr_create(struct rte_eth_dev *dev, uintptr_t addr) -{
> -	struct rte_mp_msg mp_req;
> -	struct rte_mp_msg *mp_res;
> -	struct rte_mp_reply mp_rep;
> -	struct mlx5_mp_param *req = (struct mlx5_mp_param
> *)mp_req.param;
> -	struct mlx5_mp_param *res;
> -	struct timespec ts = {.tv_sec = MLX5_MP_REQ_TIMEOUT_SEC, .tv_nsec
> = 0};
> -	int ret;
> -
> -	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> -	mp_init_msg(dev, &mp_req, MLX5_MP_REQ_CREATE_MR);
> -	req->args.addr = addr;
> -	ret = rte_mp_request_sync(&mp_req, &mp_rep, &ts);
> -	if (ret) {
> -		DRV_LOG(ERR, "port %u request to primary process failed",
> -			dev->data->port_id);
> -		return -rte_errno;
> -	}
> -	MLX5_ASSERT(mp_rep.nb_received == 1);
> -	mp_res = &mp_rep.msgs[0];
> -	res = (struct mlx5_mp_param *)mp_res->param;
> -	ret = res->result;
> -	if (ret)
> -		rte_errno = -ret;
> -	free(mp_rep.msgs);
> -	return ret;
> -}
> -
> -/**
> - * Request Verbs queue state modification to the primary process.
> - *
> - * @param[in] dev
> - *   Pointer to Ethernet structure.
> - * @param sm
> - *   State modify parameters.
> - *
> - * @return
> - *   0 on success, a negative errno value otherwise and rte_errno is set.
> - */
> -int
> -mlx5_mp_req_queue_state_modify(struct rte_eth_dev *dev,
> -			       struct mlx5_mp_arg_queue_state_modify *sm)
> -{
> -	struct rte_mp_msg mp_req;
> -	struct rte_mp_msg *mp_res;
> -	struct rte_mp_reply mp_rep;
> -	struct mlx5_mp_param *req = (struct mlx5_mp_param
> *)mp_req.param;
> -	struct mlx5_mp_param *res;
> -	struct timespec ts = {.tv_sec = MLX5_MP_REQ_TIMEOUT_SEC, .tv_nsec
> = 0};
> -	int ret;
> -
> -	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> -	mp_init_msg(dev, &mp_req, MLX5_MP_REQ_QUEUE_STATE_MODIFY);
> -	req->args.state_modify = *sm;
> -	ret = rte_mp_request_sync(&mp_req, &mp_rep, &ts);
> -	if (ret) {
> -		DRV_LOG(ERR, "port %u request to primary process failed",
> -			dev->data->port_id);
> -		return -rte_errno;
> -	}
> -	MLX5_ASSERT(mp_rep.nb_received == 1);
> -	mp_res = &mp_rep.msgs[0];
> -	res = (struct mlx5_mp_param *)mp_res->param;
> -	ret = res->result;
> -	free(mp_rep.msgs);
> -	return ret;
> -}
> -
> -/**
> - * Request Verbs command file descriptor for mmap to the primary process.
> - *
> - * @param[in] dev
> - *   Pointer to Ethernet structure.
> - *
> - * @return
> - *   fd on success, a negative errno value otherwise and rte_errno is set.
> - */
> -int
> -mlx5_mp_req_verbs_cmd_fd(struct rte_eth_dev *dev) -{
> -	struct rte_mp_msg mp_req;
> -	struct rte_mp_msg *mp_res;
> -	struct rte_mp_reply mp_rep;
> -	struct mlx5_mp_param *res;
> -	struct timespec ts = {.tv_sec = MLX5_MP_REQ_TIMEOUT_SEC, .tv_nsec
> = 0};
> -	int ret;
> -
> -	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> -	mp_init_msg(dev, &mp_req, MLX5_MP_REQ_VERBS_CMD_FD);
> -	ret = rte_mp_request_sync(&mp_req, &mp_rep, &ts);
> -	if (ret) {
> -		DRV_LOG(ERR, "port %u request to primary process failed",
> -			dev->data->port_id);
> -		return -rte_errno;
> -	}
> -	MLX5_ASSERT(mp_rep.nb_received == 1);
> -	mp_res = &mp_rep.msgs[0];
> -	res = (struct mlx5_mp_param *)mp_res->param;
> -	if (res->result) {
> -		rte_errno = -res->result;
> -		DRV_LOG(ERR,
> -			"port %u failed to get command FD from primary
> process",
> -			dev->data->port_id);
> -		ret = -rte_errno;
> -		goto exit;
> -	}
> -	MLX5_ASSERT(mp_res->num_fds == 1);
> -	ret = mp_res->fds[0];
> -	DRV_LOG(DEBUG, "port %u command FD from primary is %d",
> -		dev->data->port_id, ret);
> -exit:
> -	free(mp_rep.msgs);
> -	return ret;
> -}
> -
> -/**
> - * Initialize by primary process.
> - */
> -int
> -mlx5_mp_init_primary(void)
> -{
> -	int ret;
> -
> -	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_PRIMARY);
> -
> -	/* primary is allowed to not support IPC */
> -	ret = rte_mp_action_register(MLX5_MP_NAME, mp_primary_handle);
> -	if (ret && rte_errno != ENOTSUP)
> -		return -1;
> -	return 0;
> -}
> -
> -/**
> - * Un-initialize by primary process.
> - */
> -void
> -mlx5_mp_uninit_primary(void)
> -{
> -	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_PRIMARY);
> -	rte_mp_action_unregister(MLX5_MP_NAME);
> -}
> -
> -/**
> - * Initialize by secondary process.
> - */
> -int
> -mlx5_mp_init_secondary(void)
> -{
> -	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> -	return rte_mp_action_register(MLX5_MP_NAME,
> mp_secondary_handle);
> -}
> -
> -/**
> - * Un-initialize by secondary process.
> - */
> -void
> -mlx5_mp_uninit_secondary(void)
> -{
> -	MLX5_ASSERT(rte_eal_process_type() == RTE_PROC_SECONDARY);
> -	rte_mp_action_unregister(MLX5_MP_NAME);
> -}
> diff --git a/drivers/net/mlx5/mlx5_mr.c b/drivers/net/mlx5/mlx5_mr.c index
> a8f185a208..9151992a72 100644
> --- a/drivers/net/mlx5/mlx5_mr.c
> +++ b/drivers/net/mlx5/mlx5_mr.c
> @@ -540,7 +540,7 @@ mlx5_mr_create_secondary(struct rte_eth_dev *dev,
> struct mlx5_mr_cache *entry,
> 
>  	DEBUG("port %u requesting MR creation for address (%p)",
>  	      dev->data->port_id, (void *)addr);
> -	ret = mlx5_mp_req_mr_create(dev, addr);
> +	ret = mlx5_mp_req_mr_create(&priv->mp_id, addr);
>  	if (ret) {
>  		DEBUG("port %u fail to request MR creation for address
> (%p)",
>  		      dev->data->port_id, (void *)addr); diff --git
> a/drivers/net/mlx5/mlx5_rxtx.c b/drivers/net/mlx5/mlx5_rxtx.c index
> 7ce3732fd3..42d7da8a4b 100644
> --- a/drivers/net/mlx5/mlx5_rxtx.c
> +++ b/drivers/net/mlx5/mlx5_rxtx.c
> @@ -1000,6 +1000,7 @@ static int
>  mlx5_queue_state_modify(struct rte_eth_dev *dev,
>  			struct mlx5_mp_arg_queue_state_modify *sm)  {
> +	struct mlx5_priv *priv = dev->data->dev_private;
>  	int ret = 0;
> 
>  	switch (rte_eal_process_type()) {
> @@ -1007,7 +1008,7 @@ mlx5_queue_state_modify(struct rte_eth_dev *dev,
>  		ret = mlx5_queue_state_modify_primary(dev, sm);
>  		break;
>  	case RTE_PROC_SECONDARY:
> -		ret = mlx5_mp_req_queue_state_modify(dev, sm);
> +		ret = mlx5_mp_req_queue_state_modify(&priv->mp_id, sm);
>  		break;
>  	default:
>  		break;
> --
> 2.16.6


  reply	other threads:[~2020-04-14  7:26 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-02 19:21 [dpdk-dev] [PATCH 0/4] refactor multi-process IPC and memory management " Vu Pham
2020-04-02 19:21 ` [dpdk-dev] [PATCH 1/4] common/mlx5: refactor multi-process IPC handling " Vu Pham
2020-04-02 19:21 ` [dpdk-dev] [PATCH 2/4] net/mlx5: modify net PMD to use common multi-process APIs Vu Pham
2020-04-02 19:21 ` [dpdk-dev] [PATCH 3/4] common/mlx5: refactor memory management codes Vu Pham
2020-04-02 19:21 ` [dpdk-dev] [PATCH 4/4] net/mlx5: modify net PMD to use common memory management driver Vu Pham
2020-04-07 16:48 ` [dpdk-dev] [PATCH v2 0/4] refactor multi-process IPC and memory management codes to common driver Vu Pham
2020-04-07 16:48   ` [dpdk-dev] [PATCH v2 1/4] common/mlx5: refactor MP IPC handling " Vu Pham
2020-04-08  9:05     ` Slava Ovsiienko
2020-04-07 16:48   ` [dpdk-dev] [PATCH v2 2/4] net/mlx5: modify net pmd to use common multi-process APIs Vu Pham
2020-04-07 16:48   ` [dpdk-dev] [PATCH v2 3/4] common/mlx5: refactor memory management codes Vu Pham
2020-04-07 16:48   ` [dpdk-dev] [PATCH v2 4/4] net/mlx5: modify net pmd to use common MR driver Vu Pham
2020-04-07 17:00 ` [dpdk-dev] [PATCH v3 0/4] refactor multi-process IPC and memory management codes to common driver Vu Pham
2020-04-07 17:00   ` [dpdk-dev] [PATCH v3 1/4] common/mlx5: refactor multi-process IPC handling " Vu Pham
2020-04-08  9:05     ` Slava Ovsiienko
2020-04-07 17:00   ` [dpdk-dev] [PATCH v3 2/4] net/mlx5: modify net PMD to use common multi-process APIs Vu Pham
2020-04-08  9:05     ` Slava Ovsiienko
2020-04-07 17:00   ` [dpdk-dev] [PATCH v3 3/4] common/mlx5: refactor memory management codes Vu Pham
2020-04-08  9:04     ` Slava Ovsiienko
2020-04-07 17:00   ` [dpdk-dev] [PATCH v3 4/4] net/mlx5: modify net PMD to use common MR driver Vu Pham
2020-04-08  9:06     ` Slava Ovsiienko
2020-04-13 21:17 ` [dpdk-dev] [PATCH v4 0/2] refactor multi-process IPC and memory management codes to common driver Vu Pham
2020-04-13 21:17   ` [dpdk-dev] [PATCH v4 1/2] common/mlx5: refactor multi-process IPC handling " Vu Pham
2020-04-14  7:26     ` Slava Ovsiienko [this message]
2020-04-13 21:17   ` [dpdk-dev] [PATCH v4 2/2] common/mlx5: refactor memory management codes Vu Pham
2020-04-14  7:27     ` Slava Ovsiienko
2020-04-15  9:30   ` [dpdk-dev] [PATCH v4 0/2] refactor multi-process IPC and memory management codes to common driver Raslan Darawsheh

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=AM4PR05MB3265B6C2E59A76EAFD09CDC5D2DA0@AM4PR05MB3265.eurprd05.prod.outlook.com \
    --to=viacheslavo@mellanox.com \
    --cc=dev@dpdk.org \
    --cc=matan@mellanox.com \
    --cc=orika@mellanox.com \
    --cc=rasland@mellanox.com \
    --cc=vuhuong@mellanox.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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).