diff options
author | Ilya Maximets <i.maximets@ovn.org> | 2022-08-19 21:51:27 +0200 |
---|---|---|
committer | Ilya Maximets <i.maximets@ovn.org> | 2022-08-30 22:37:35 +0200 |
commit | 1336eeb5705f7b4856731cb39d92faf1fb68bbbb (patch) | |
tree | 7318fd17f32f8606772ad5a7947429a5fdb915a6 /ovsdb/raft.c | |
parent | e5f79eaea5fc668bffc4f2bbc060428fbce3cb92 (diff) | |
download | openvswitch-1336eeb5705f7b4856731cb39d92faf1fb68bbbb.tar.gz |
netdev-offload-tc: Parse tunnel options only for geneve ports.
Cited commit correctly fixed the issue of incorrect reporting
of zero-length geneve option matches as wildcarded. But as a
side effect, exact match on the metadata length was added to
every tunnel flow match, even if the tunnel is not geneve.
That doesn't generate any functional issues, but it maybe
confusing to see 'tunnel(...,geneve(),...)' while looking at
datapath flow dumps for, e.g., vxlan tunnel flows.
Fix that by checking the port type before parsing geneve options.
tunnel() attribute itself doesn't have enough information to
figure out the tunnel type.
Fixes: 7a6c8074c5d2 ("netdev-offload-tc: Fix the mask for tunnel metadata length.")
Acked-by: Eelco Chaudron <echaudro@redhat.com>
Reviewed-by: Roi Dayan <roid@nvidia.com>
Signed-off-by: Ilya Maximets <i.maximets@ovn.org>
Diffstat (limited to 'ovsdb/raft.c')
0 files changed, 0 insertions, 0 deletions