On Fri, Nov 20, 2020 at 05:12:46AM -0800, Toms Atteka wrote:
> This optimization caused FLOW_TNL_F_UDPIF flag not to be used in
> hash calculation for geneve tunnel when revalidating flows which
> resulted in different cache hash values and incorrect behaviour.
>
> Reported-at: https://github.com/vmware-tanzu/antrea/issues/897
> Signed-off-by: Toms Atteka <[email protected]>
Why was the check there? It seems strange to have a very specific "if"
test and then just remove it without some idea of why it was there to
begin with.
...
> - } else if (flow->metadata.present.len || is_mask) {
> + } else {
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
