mbox series

[ovs-dev,V1,0/2] TC offload dump fix and add frag

Message ID 1520859527-9134-1-git-send-email-roid@mellanox.com
Headers show
Series TC offload dump fix and add frag | expand

Message

Roi Dayan March 12, 2018, 12:58 p.m. UTC 
Hi,

The first patch fixing error handling when parsing tc rules for dump flows.
The second patch adds support for IP fragmentation to TC parsing.

V1:
- also support frag first/later

Thanks,
Roi

Roi Dayan (2):
  lib/tc: Handle error parsing action in nl_parse_single_action
  netdev-tc-offloads: Add support for IP fragmentation

 acinclude.m4             |  6 +++---
 include/linux/pkt_cls.h  |  5 +++--
 lib/netdev-tc-offloads.c | 38 ++++++++++++++++++++++++++++++++------
 lib/tc.c                 | 31 +++++++++++++++++++++++++------
 lib/tc.h                 |  1 +
 5 files changed, 64 insertions(+), 17 deletions(-)

Comments

Roi Dayan March 20, 2018, 4:21 p.m. UTC | #1 
On 12/03/2018 14:58, Roi Dayan wrote:
> Hi,
> 
> The first patch fixing error handling when parsing tc rules for dump flows.
> The second patch adds support for IP fragmentation to TC parsing.
> 
> V1:
> - also support frag first/later
> 
> Thanks,
> Roi
> 
> Roi Dayan (2):
>    lib/tc: Handle error parsing action in nl_parse_single_action
>    netdev-tc-offloads: Add support for IP fragmentation
> 
>   acinclude.m4             |  6 +++---
>   include/linux/pkt_cls.h  |  5 +++--
>   lib/netdev-tc-offloads.c | 38 ++++++++++++++++++++++++++++++++------
>   lib/tc.c                 | 31 +++++++++++++++++++++++++------
>   lib/tc.h                 |  1 +
>   5 files changed, 64 insertions(+), 17 deletions(-)
> 


Hi Ben,

any comments about the changes?

Thanks,
Roi
Simon Horman March 20, 2018, 5:16 p.m. UTC | #2 
2018/03/20 17:21 "Roi Dayan" <roid@mellanox.com>:



On 12/03/2018 14:58, Roi Dayan wrote:

> Hi,
>
> The first patch fixing error handling when parsing tc rules for dump flows.
> The second patch adds support for IP fragmentation to TC parsing.
>
> V1:
> - also support frag first/later
>
> Thanks,
> Roi
>
> Roi Dayan (2):
>    lib/tc: Handle error parsing action in nl_parse_single_action
>    netdev-tc-offloads: Add support for IP fragmentation
>
>   acinclude.m4             |  6 +++---
>   include/linux/pkt_cls.h  |  5 +++--
>   lib/netdev-tc-offloads.c | 38 ++++++++++++++++++++++++++++++++------
>   lib/tc.c                 | 31 +++++++++++++++++++++++++------
>   lib/tc.h                 |  1 +
>   5 files changed, 64 insertions(+), 17 deletions(-)
>
>

Hi Ben,

any comments about the changes?


Hi Roi,

Sorry for the delay. I will review these.


Thanks,
Roi