On Wed, Jan 10, 2018 at 01:13:11PM -0800, William Tu wrote: > Existing code only set these tunnel flags (df, csum, and key) when the > flag is set in the output tunnel port, but did not clear when the flag > is unset. The patch fixes it by setting and clearing it accordingly. > > Two existing testcases need to fix: > 'tunnel - Geneve option present' has no key set up, so we should match > 'flags(df)' instead of 'flags(df|key)'. The second case > 'tunnel - concomitant IPv6 and IPv4 tunnels' follows the same pattern. > One additional test case 'tunnel - Mix Geneve/GRE options' is added. > > Signed-off-by: William Tu <[email protected]> > VMWare-BZ: #2019012 > --- > v1->v2: > refactor the tunnel flag set/unset code.
Thank you. I applied this to master and branch-2.8. It did not apply to branch-2.7 because of conflicts in the test case. I don't know whether it is important to backport to that and earlier branches. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
