[v2,1/9] net/mlx5: fix translate vport function name
Checks
Commit Message
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>
---
v2:
* Address ML comments.
---
drivers/net/mlx5/mlx5_flow_dv.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
Comments
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
>
@@ -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);