On Thu, Sep 27, 2018 at 05:50:42PM +0300, Dekel Peled wrote:
> This patch adds glue functions for operations:
> - Create packet reformat (encap/decap) flow action.
> - Destroy flow action.
>
> Signed-off-by: Dekel Peled <dek...@mellanox.com>
> ---
> drivers/net/mlx5/mlx5_glue.c | 33 +++++++++++++++++++++++++++++++++
> drivers/net/mlx5/mlx5_glue.h | 7 +++++++
> 2 files changed, 40 insertions(+)
>
> diff --git a/drivers/net/mlx5/mlx5_glue.c b/drivers/net/mlx5/mlx5_glue.c
> index 48590df..caa4c34 100644
> --- a/drivers/net/mlx5/mlx5_glue.c
> +++ b/drivers/net/mlx5/mlx5_glue.c
> @@ -174,6 +174,12 @@
> return ibv_destroy_flow(flow_id);
> }
>
> +static int
> +mlx5_glue_destroy_flow_action(struct ibv_flow_action *action)
> +{
> + return ibv_destroy_flow_action(action);
> +}
> +
> static struct ibv_qp *
> mlx5_glue_create_qp(struct ibv_pd *pd, struct ibv_qp_init_attr *qp_init_attr)
> {
> @@ -388,6 +394,30 @@
> #endif
> }
>
> +static struct ibv_flow_action *
> +mlx5_glue_dv_create_flow_action_packet_reformat
> + (struct ibv_context *ctx,
> + size_t data_sz,
> + void *data,
> + enum mlx5dv_flow_action_packet_reformat_type reformat_type,
> + enum mlx5dv_flow_table_type ft_type)
Aren't these two structs also new? If so, you might have to declare it in
mlx5_glue.h to avoid compilation error. Please refer to mlx5dv_flow_matcher as
an example.
> +{
> +#ifdef HAVE_IBV_FLOW_DV_SUPPORT
> + return mlx5dv_create_flow_action_packet_reformat(ctx,
> + data_sz,
> + data,
> + reformat_type,
> + ft_type);
> +#else
> + (void)ctx;
> + (void)data_sz;
> + (void)data;
> + (void)reformat_type;
> + (void)ft_type;
> + return NULL;
> +#endif
> +}
> +
> alignas(RTE_CACHE_LINE_SIZE)
> const struct mlx5_glue *mlx5_glue = &(const struct mlx5_glue){
> .version = MLX5_GLUE_VERSION,
> @@ -414,6 +444,7 @@
> .modify_wq = mlx5_glue_modify_wq,
> .create_flow = mlx5_glue_create_flow,
> .destroy_flow = mlx5_glue_destroy_flow,
> + .destroy_flow_action = mlx5_glue_destroy_flow_action,
> .create_qp = mlx5_glue_create_qp,
> .create_qp_ex = mlx5_glue_create_qp_ex,
> .destroy_qp = mlx5_glue_destroy_qp,
> @@ -437,4 +468,6 @@
> .dv_create_flow_matcher = mlx5_glue_dv_create_flow_matcher,
> .dv_destroy_flow_matcher = mlx5_glue_dv_destroy_flow_matcher,
> .dv_create_flow = mlx5_glue_dv_create_flow,
> + .dv_create_flow_action_packet_reformat =
> + mlx5_glue_dv_create_flow_action_packet_reformat,
> };
> diff --git a/drivers/net/mlx5/mlx5_glue.h b/drivers/net/mlx5/mlx5_glue.h
> index f6e4e38..7431f58 100644
> --- a/drivers/net/mlx5/mlx5_glue.h
> +++ b/drivers/net/mlx5/mlx5_glue.h
> @@ -85,6 +85,7 @@ struct mlx5_glue {
> struct ibv_flow *(*create_flow)(struct ibv_qp *qp,
> struct ibv_flow_attr *flow);
> int (*destroy_flow)(struct ibv_flow *flow_id);
> + int (*destroy_flow_action)(struct ibv_flow_action *action);
> struct ibv_qp *(*create_qp)(struct ibv_pd *pd,
> struct ibv_qp_init_attr *qp_init_attr);
> struct ibv_qp *(*create_qp_ex)
> @@ -137,6 +138,12 @@ struct mlx5_glue {
> struct mlx5dv_flow_match_parameters *match_value,
> size_t num_actions,
> struct mlx5dv_flow_action_attr *actions_attr);
> + struct ibv_flow_action *(*dv_create_flow_action_packet_reformat)
> + (struct ibv_context *ctx,
> + size_t data_sz,
> + void *data,
> + enum mlx5dv_flow_action_packet_reformat_type reformat_type,
> + enum mlx5dv_flow_table_type ft_type);
> };
>
> const struct mlx5_glue *mlx5_glue;
> --
> 1.8.3.1
>
