mbox series

[v2,0/3] ethdev: add generic TTL rewrite actions

Message ID 20180925143719.23339-1-jackmin@mellanox.com (mailing list archive)
Headers
Series ethdev: add generic TTL rewrite actions |

Message

Xiaoyu Min Sept. 25, 2018, 2:37 p.m. UTC 
  This patch series is for RFC[1]

Patch 1 adds generic TTL rewrite actions to flow API
Patch 2 adds corresponding testpmd commands
Patch 3 implements the offloading logic of E-Switch rules on Mellanox MLX5

[1]: https://patches.dpdk.org/patch/43617/

V2:
 * fix misspelled issues reported by checkpatch

Xiaoyu Min (3):
  ethdev: add generic TTL rewrite actions
  app/testpmd: add commands of modify TTL
  net/mlx5: eswitch-modify TTL actions

 app/test-pmd/cmdline_flow.c                 | 34 ++++++++++
 app/test-pmd/config.c                       |  3 +
 doc/guides/prog_guide/rte_flow.rst          | 30 +++++++++
 doc/guides/testpmd_app_ug/testpmd_funcs.rst |  5 ++
 drivers/net/mlx5/mlx5_flow.h                |  2 +
 drivers/net/mlx5/mlx5_flow_tcf.c            | 74 ++++++++++++++++++++-
 lib/librte_ethdev/rte_flow.c                |  2 +
 lib/librte_ethdev/rte_flow.h                | 31 +++++++++
 8 files changed, 179 insertions(+), 2 deletions(-)

Comments

Thomas Monjalon Oct. 3, 2018, 8:35 p.m. UTC | #1 
25/09/2018 16:37, Xiaoyu Min:
> Xiaoyu Min (3):
>   ethdev: add generic TTL rewrite actions
>   app/testpmd: add commands of modify TTL
>   net/mlx5: eswitch-modify TTL actions

If no more comment, I think we should accept this series.
Ferruh Yigit Oct. 5, 2018, 12:52 p.m. UTC | #2 
On 9/25/2018 3:37 PM, Xiaoyu Min wrote:
> This patch series is for RFC[1]
> 
> Patch 1 adds generic TTL rewrite actions to flow API
> Patch 2 adds corresponding testpmd commands
> Patch 3 implements the offloading logic of E-Switch rules on Mellanox MLX5
> 
> [1]: https://patches.dpdk.org/patch/43617/
> 
> V2:
>  * fix misspelled issues reported by checkpatch
> 
> Xiaoyu Min (3):
>   ethdev: add generic TTL rewrite actions
>   app/testpmd: add commands of modify TTL
>   net/mlx5: eswitch-modify TTL actions

Can you please rebase on top of latest next-net? I guess Adrien's "flow API
object converter" causing conflict with existing rte_flow patches.
Xiaoyu Min Oct. 8, 2018, 2:30 a.m. UTC | #3 
On 18-10-05 13:52:03, Ferruh Yigit wrote:
> On 9/25/2018 3:37 PM, Xiaoyu Min wrote:
> > This patch series is for RFC[1]
> > 
> > Patch 1 adds generic TTL rewrite actions to flow API
> > Patch 2 adds corresponding testpmd commands
> > Patch 3 implements the offloading logic of E-Switch rules on Mellanox MLX5
> > 
> > [1]: https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatches.dpdk.org%2Fpatch%2F43617%2F&data=02%7C01%7Cjackmin%40mellanox.com%7Cf0b174a17cc549c8c6a108d62ac15e73%7Ca652971c7d2e4d9ba6a4d149256f461b%7C0%7C0%7C636743407334308856&sdata=WQy0UYGPK2uZOJ7hhDsMHjNhj05vx9BVpvzCwKlYjWE%3D&reserved=0
> > 
> > V2:
> >  * fix misspelled issues reported by checkpatch
> > 
> > Xiaoyu Min (3):
> >   ethdev: add generic TTL rewrite actions
> >   app/testpmd: add commands of modify TTL
> >   net/mlx5: eswitch-modify TTL actions
> 
> Can you please rebase on top of latest next-net? I guess Adrien's "flow API
> object converter" causing conflict with existing rte_flow patches.
Sure, I'll rebase on latest next-net and address change requests in next version

-Jack