[ovs-dev] [PATCH] netdev-tc-offloads: Fix parsing SCTP in dump flows

[Simon Horman]

On Sun, Aug 06, 2017 at 10:54:59AM +0300, Roi Dayan wrote:
> After splitting the unions of tcp/udp the sctp was forgotten
> when parsing flower back to match.
> 
> Fixes: 2b1d9fa90909 ("tc: Split IPs and transport layer ports unions in flower struct")
> Signed-off-by: Roi Dayan <roid at mellanox.com>
> Reviewed-by: Paul Blakey <paulb at mellanox.com>

Acked-by: Simon Horman <simon.horman at netronome.com>

> ---
>  lib/netdev-tc-offloads.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/lib/netdev-tc-offloads.c b/lib/netdev-tc-offloads.c
> index e51e638..2af1546 100644
> --- a/lib/netdev-tc-offloads.c
> +++ b/lib/netdev-tc-offloads.c
> @@ -326,6 +326,9 @@ parse_tc_flower_to_match(struct tc_flower *flower,
>          } else if (key->ip_proto == IPPROTO_UDP) {
>              match_set_tp_dst_masked(match, key->udp_dst, mask->udp_dst);
>              match_set_tp_src_masked(match, key->udp_src, mask->udp_src);
> +        } else if (key->ip_proto == IPPROTO_SCTP) {
> +            match_set_tp_dst_masked(match, key->sctp_dst, mask->sctp_dst);
> +            match_set_tp_src_masked(match, key->sctp_src, mask->sctp_src);
>          }
>      }
>  
> -- 
> 1.7.1
>