From: Ophir Munk <ophi...@nvidia.com>

This commit implements mlx5_flow_os_create_flow_action_dest_devx_tir()
API as the Linux rdma-core equivalent. Missing rdma-core parameters are
added to file mlx5_win_defs.h. The action TIR id and type
(MLX5_FLOW_CONTEXT_DEST_TYPE_TIR) are saved in the action struct.  The
action struct will be added to array of actions and will be used later
by the flow creation API.

Signed-off-by: Ophir Munk <ophi...@nvidia.com>
Acked-by: Matan Azrad <ma...@nvidia.com>
---
 drivers/common/mlx5/windows/mlx5_win_defs.h | 17 +++++++++++++++++
 drivers/net/mlx5/windows/mlx5_flow_os.c     | 22 +++++++++++++++-------
 2 files changed, 32 insertions(+), 7 deletions(-)

diff --git a/drivers/common/mlx5/windows/mlx5_win_defs.h 
b/drivers/common/mlx5/windows/mlx5_win_defs.h
index 231c3220a3..8a1c2c5af9 100644
--- a/drivers/common/mlx5/windows/mlx5_win_defs.h
+++ b/drivers/common/mlx5/windows/mlx5_win_defs.h
@@ -189,6 +189,17 @@ struct mlx5_matcher {
        uint64_t match_buf[];
 };
 
+/*
+ * Windows mlx5_action. This struct is the
+ * equivalent of rdma-core struct mlx5dv_dr_action.
+ */
+struct mlx5_action {
+       int type;
+       struct {
+               uint32_t id;
+       } dest_tir;
+};
+
 struct mlx5_err_cqe {
        uint8_t         rsvd0[32];
        uint32_t        srqn;
@@ -233,4 +244,10 @@ enum {
        MLX5_FLOW_CONTEXT_DEST_TYPE_TIR                      = 0x2,
        MLX5_FLOW_CONTEXT_DEST_TYPE_QP                       = 0x3,
 };
+
+enum {
+       MLX5_MATCH_OUTER_HEADERS        = 1 << 0,
+       MLX5_MATCH_MISC_PARAMETERS      = 1 << 1,
+       MLX5_MATCH_INNER_HEADERS        = 1 << 2,
+};
 #endif /* __MLX5_WIN_DEFS_H__ */
diff --git a/drivers/net/mlx5/windows/mlx5_flow_os.c 
b/drivers/net/mlx5/windows/mlx5_flow_os.c
index 0ade518910..0c0fba3720 100644
--- a/drivers/net/mlx5/windows/mlx5_flow_os.c
+++ b/drivers/net/mlx5/windows/mlx5_flow_os.c
@@ -134,10 +134,19 @@ int
 mlx5_flow_os_create_flow_action_dest_devx_tir(struct mlx5_devx_obj *tir,
                                              void **action)
 {
-       RTE_SET_USED(tir);
-       *action = NULL;
-       rte_errno = ENOTSUP;
-       return -rte_errno;
+       struct mlx5_action *mlx5_action =
+               mlx5_malloc(MLX5_MEM_ZERO,
+                      sizeof(struct mlx5_action),
+                      0, SOCKET_ID_ANY);
+
+       if (!mlx5_action) {
+               rte_errno = ENOMEM;
+               return -rte_errno;
+       }
+       mlx5_action->type = MLX5_FLOW_CONTEXT_DEST_TYPE_TIR;
+       mlx5_action->dest_tir.id = tir->id;
+       *action = mlx5_action;
+       return 0;
 }
 
 /**
@@ -152,9 +161,8 @@ mlx5_flow_os_create_flow_action_dest_devx_tir(struct 
mlx5_devx_obj *tir,
 int
 mlx5_flow_os_destroy_flow_action(void *action)
 {
-       RTE_SET_USED(action);
-       rte_errno = ENOTSUP;
-       return -rte_errno;
+       mlx5_free(action);
+       return 0;
 }
 
 /**
-- 
2.16.1.windows.4

Reply via email to