mbox series

[ovs-dev,v4,0/2] add TC offload support for geneve with options

Message ID 1537259780-9769-1-git-send-email-pieter.jansenvanvuuren@netronome.com
Headers show
Series add TC offload support for geneve with options | expand

Message

Pieter Jansen van Vuuren Sept. 18, 2018, 8:36 a.m. UTC 
This series adds match and action support for TC offload of
geneve tunnel with options.

Pieter Jansen van Vuuren (2):
  lib/tc: add geneve with option encap action offload
  lib/tc: add geneve with option match offload

 include/linux/pkt_cls.h              |  25 ++
 include/linux/tc_act/tc_tunnel_key.h |  23 ++
 lib/netdev-tc-offloads.c             |  94 +++++++
 lib/tc.c                             | 355 ++++++++++++++++++++++++++-
 lib/tc.h                             |   3 +
 5 files changed, 492 insertions(+), 8 deletions(-)

Comments

Simon Horman Sept. 18, 2018, 10:03 a.m. UTC | #1 
On Tue, Sep 18, 2018 at 09:36:18AM +0100, Pieter Jansen van Vuuren wrote:
> This series adds match and action support for TC offload of
> geneve tunnel with options.
> 
> Pieter Jansen van Vuuren (2):
>   lib/tc: add geneve with option encap action offload
>   lib/tc: add geneve with option match offload

Thanks Pieter,

these look good to me. I will, however, wait to allow review from others
before applying them.

> 
>  include/linux/pkt_cls.h              |  25 ++
>  include/linux/tc_act/tc_tunnel_key.h |  23 ++
>  lib/netdev-tc-offloads.c             |  94 +++++++
>  lib/tc.c                             | 355 ++++++++++++++++++++++++++-
>  lib/tc.h                             |   3 +
>  5 files changed, 492 insertions(+), 8 deletions(-)
> 
> -- 
> 2.17.0
>
Simon Horman Sept. 20, 2018, 2 p.m. UTC | #2 
On Tue, Sep 18, 2018 at 12:03:11PM +0200, Simon Horman wrote:
> On Tue, Sep 18, 2018 at 09:36:18AM +0100, Pieter Jansen van Vuuren wrote:
> > This series adds match and action support for TC offload of
> > geneve tunnel with options.
> > 
> > Pieter Jansen van Vuuren (2):
> >   lib/tc: add geneve with option encap action offload
> >   lib/tc: add geneve with option match offload
> 
> Thanks Pieter,
> 
> these look good to me. I will, however, wait to allow review from others
> before applying them.

Thanks again Pieter, applied to master.