[PATCH] net/mlx5/hws: add support for multi pattern

[Raslan Darawsheh]

Hi,

> -----Original Message-----
> From: Alex Vesker <valex at nvidia.com>
> Sent: Sunday, July 9, 2023 5:20 PM
> To: Alex Vesker <valex at nvidia.com>; Slava Ovsiienko
> <viacheslavo at nvidia.com>; NBU-Contact-Thomas Monjalon (EXTERNAL)
> <thomas at monjalon.net>; Suanming Mou <suanmingm at nvidia.com>; Matan
> Azrad <matan at nvidia.com>; Ori Kam <orika at nvidia.com>
> Cc: dev at dpdk.org
> Subject: [PATCH] net/mlx5/hws: add support for multi pattern
> 
> When creating an action it can contain a bulk of objects, but the objects are
> fixed to a specific pattern and cannot be reused. This can lead to inefficient
> usage of the HW resources, This support allows creating multiple patterns over
> a single mlx5dr action.
> 
> Signed-off-by: Alex Vesker <valex at nvidia.com>
> Reviewed-by: Erez Shitrit <erezsh at nvidia.com>
> Acked-by: Matan Azrad <matan at nvidia.com>
> ---
....
> 
> @@ -503,10 +518,10 @@ mlx5dr_action_create_reformat(struct
> mlx5dr_context *ctx,
>   *
>   * @param[in] ctx
>   *	The context in which the new action will be created.
> - * @param[in] pattern_sz
> - *	Byte size of the pattern array.
> - * @param[in] pattern
> - *	PRM format modify pattern action array.
> + * @param[in] num_of_patterns
> + * 	Number of provided patterns in "patterns" array.
Fixed an issue with the wrong indentation usage of space 

> + * @param[in] patterns
> + *	Patterns array containing pattern information.
>   * @param[in] log_bulk_size
>   *	Number of unique values used with this pattern.
>   * @param[in] flags
> @@ -515,8 +530,8 @@ mlx5dr_action_create_reformat(struct
....
> +static int
> +mlx5dr_action_create_modify_header_hws(struct mlx5dr_action *action,
> +				       uint8_t num_of_patterns,
> +				       struct mlx5dr_action_mh_pattern
> *pattern,
> +				       uint32_t log_bulk_size)
> +{
> +	struct mlx5dr_devx_obj *pat_obj, *arg_obj = NULL;
> +	struct mlx5dr_context *ctx = action->ctx;
> +	uint16_t max_mh_actions = 0;
> +	int i, ret;
> +
> +	/* Caclulate maximum number of mh actions for shared arg allocation
Fixed a typo Caclulate =-> Calculate. 
....

Patch applied to next-net-mlx,

Kindest regards,
Raslan Darawsheh