mbox series

[PATCHv2,net-next,0/4] net: sched: support vxlan and erspan options

Message ID cover.1574330535.git.lucien.xin@gmail.com
Headers show
Series net: sched: support vxlan and erspan options | expand

Message

Xin Long Nov. 21, 2019, 10:03 a.m. UTC 
This patchset is to add vxlan and erspan options support in
cls_flower and act_tunnel_key. The form is pretty much like
geneve_opts in:

  https://patchwork.ozlabs.org/patch/935272/
  https://patchwork.ozlabs.org/patch/954564/

but only one option is allowed for vxlan and erspan.

v1->v2:
  - see each patch changelog.

Xin Long (4):
  net: sched: add vxlan option support to act_tunnel_key
  net: sched: add erspan option support to act_tunnel_key
  net: sched: allow flower to match vxlan options
  net: sched: allow flower to match erspan options

 include/uapi/linux/pkt_cls.h              |  29 ++++
 include/uapi/linux/tc_act/tc_tunnel_key.h |  29 ++++
 net/sched/act_tunnel_key.c                | 203 +++++++++++++++++++++++-
 net/sched/cls_flower.c                    | 254 ++++++++++++++++++++++++++++++
 4 files changed, 514 insertions(+), 1 deletion(-)

Comments

Jakub Kicinski Nov. 21, 2019, 6:54 p.m. UTC | #1 
On Thu, 21 Nov 2019 18:03:25 +0800, Xin Long wrote:
> This patchset is to add vxlan and erspan options support in
> cls_flower and act_tunnel_key. The form is pretty much like
> geneve_opts in:
> 
>   https://patchwork.ozlabs.org/patch/935272/
>   https://patchwork.ozlabs.org/patch/954564/
> 
> but only one option is allowed for vxlan and erspan.

I have only cursory understanding of the flower parts, but looks good
to me:

Acked-by: Jakub Kicinski <jakub.kicinski@netronome.com>
David Miller Nov. 21, 2019, 7:46 p.m. UTC | #2 
From: Xin Long <lucien.xin@gmail.com>
Date: Thu, 21 Nov 2019 18:03:25 +0800

> This patchset is to add vxlan and erspan options support in
> cls_flower and act_tunnel_key. The form is pretty much like
> geneve_opts in:
> 
>   https://patchwork.ozlabs.org/patch/935272/
>   https://patchwork.ozlabs.org/patch/954564/
> 
> but only one option is allowed for vxlan and erspan.
> 
> v1->v2:
>   - see each patch changelog.

Series applied.