On Wed, Sep 13, 2017 at 05:04:58PM +0300, Roi Dayan wrote:
> From: Paul Blakey <pa...@mellanox.com>
>
> This commit fixes offloading when tunnel id is 0 by checking
> for valid tunnel dst ipv4 or ipv6, like done in the normal flow.
>
> Also need to reset the tunnel mask outside the condition
> because in the special case, when adding a port with key=flow,
> the match for tunnel_set gets a tunnel key id 0 with a mask.
> If we don't reset the mask we'll fail later when we check
> for unknown attributes that the mask was not reset.
>
> Fixes: 8f283af89298 ("netdev-tc-offloads: Implement netdev flow put using tc
> interface")
> Signed-off-by: Paul Blakey <pa...@mellanox.com>
> Reviewed-by: Roi Dayan <r...@mellanox.com>
> ---
> Hi,
>
> This is V2 for the prev patch with
> title "netdev-tc-offloads: Fix offloading when tunnel id is 0"
>
> V1->V2:
> - Use flow_tnl_dst_is_set() instead of checking mask
> - Move memset of maks tunnel to outside the condition
> - Update commit message
Thanks, applied to master and branch-2.8.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
