Re: [dpdk-dev] [PATCH v2 1/9] net/mlx5: fix translate vport function name

[Yongseok Koh <yskoh@mellanox.com>]

On Thu, Apr 18, 2019 at 11:28:42AM +0000, Ori Kam wrote:
> Modify the translate vport function to match other translate items
> naming convestions.
> 
> Fixes: 0fe3f18f78d8 ("net/mlx5: add source vport match to the ingress rules")
> Cc: viacheslavo@mellanox.com
> 
> Signed-off-by: Ori Kam <orika@mellanox.com>

You were supposed to change the name to flow_dv_translate_item_src_vport, but
that's fine.

Acked-by: Yongseok Koh <yskoh@mellanox.com>

> ---
> v2:
> * Address ML comments.
> ---
>  drivers/net/mlx5/mlx5_flow_dv.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/drivers/net/mlx5/mlx5_flow_dv.c b/drivers/net/mlx5/mlx5_flow_dv.c
> index 04ab3d6..1e25e0b 100644
> --- a/drivers/net/mlx5/mlx5_flow_dv.c
> +++ b/drivers/net/mlx5/mlx5_flow_dv.c
> @@ -3087,8 +3087,8 @@ struct field_modify_info modify_tcp[] = {
>   *   Mask
>   */
>  static void
> -flow_dv_translate_source_vport(void *matcher, void *key,
> -			      int16_t port, uint16_t mask)
> +flow_dv_translate_item_source_vport(void *matcher, void *key,
> +				    int16_t port, uint16_t mask)
>  {
>  	void *misc_m = MLX5_ADDR_OF(fte_match_param, matcher, misc_parameters);
>  	void *misc_v = MLX5_ADDR_OF(fte_match_param, key, misc_parameters);
> @@ -3492,10 +3492,10 @@ struct field_modify_info modify_tcp[] = {
>  		 * Add matching on source vport index only
>  		 * for ingress rules in E-Switch configurations.
>  		 */
> -		flow_dv_translate_source_vport(matcher.mask.buf,
> -					       dev_flow->dv.value.buf,
> -					       priv->vport_id,
> -					       0xffff);
> +		flow_dv_translate_item_source_vport(matcher.mask.buf,
> +						    dev_flow->dv.value.buf,
> +						    priv->vport_id,
> +						    0xffff);
>  	}
>  	for (; items->type != RTE_FLOW_ITEM_TYPE_END; items++) {
>  		int tunnel = !!(item_flags & MLX5_FLOW_LAYER_TUNNEL);
> -- 
> 1.8.3.1
>