On 4/27/22 08:59, Thilak Raj Surendra Babu wrote:
> When a packet is received over an access port that needs to be sent
> over a vxlan tunnel,the access port VLAN id is used in the lookup
> leading to a wrong packet being crafted and sent over the tunnel.
> Clear out the flow 's VLAN field as it should not be used while
> performing mac lookup for the outer tunnel and also at this point
> the VLAN action related to inner flow is already committed.
> 
> Fixes: 7c12dfc527a5 ("tunneling: Avoid datapath-recirc by combining recirc 
> actions at xlate.")
> Signed-off-by: Thilak Raj Surendra Babu <[email protected]>
> Signed-off-by: Rosemarie O'Riorden <[email protected]>
> Co-authored-by: Rosemarie O'Riorden <[email protected]>
> 
> ---
> Version 4:
>   - Address minor review comments.
>   - Replace with a simpler system test.
> 
> Version 3:
>   - Address review comments.
>   - Add unit-test patch from Rosemarie O'Riorden.
> 
> Version 2:
>   - Fixed line length warning from checkpatch.
>   - Dropped unrelated changes.
> 
>  ofproto/ofproto-dpif-xlate.c |  3 +++
>  tests/system-traffic.at      | 47 ++++++++++++++++++++++++++++++++
>  tests/tunnel-push-pop.at     | 52 ++++++++++++++++++++++++++++++++++++
>  3 files changed, 102 insertions(+)

Thanks!  Applied and backported down to 2.13.

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to