diff options
author | William Tu <u9012063@gmail.com> | 2018-05-29 05:55:04 -0700 |
---|---|---|
committer | Ben Pfaff <blp@ovn.org> | 2018-05-29 16:24:13 -0700 |
commit | 2274109d5674f1f2b02de98a45acd8137f3df691 (patch) | |
tree | 0325b95a203303cf86353e81acc8d3c9c952c225 /.travis.yml | |
parent | 2f625b34c29e750fef9c0f9f193d49fd007fefea (diff) | |
download | openvswitch-2274109d5674f1f2b02de98a45acd8137f3df691.tar.gz |
datapath: ip6_gre: Fix ip6erspan hlen calculation
commit 2d665034f239412927b1e71329f20f001c92da09
Author: Petr Machata <petrm@mellanox.com>
Date: Thu May 17 16:36:51 2018 +0200
net: ip6_gre: Fix ip6erspan hlen calculation
Even though ip6erspan_tap_init() sets up hlen and tun_hlen according to
what ERSPAN needs, it goes ahead to call ip6gre_tnl_link_config() which
overwrites these settings with GRE-specific ones.
Similarly for changelink callbacks, which are handled by
ip6gre_changelink() calls ip6gre_tnl_change() calls
ip6gre_tnl_link_config() as well.
The difference ends up being 12 vs. 20 bytes, and this is generally not
a problem, because a 12-byte request likely ends up allocating more and
the extra 8 bytes are thus available. However correct it is not.
So replace the newlink and changelink callbacks with an ERSPAN-specific
ones, reusing the newly-introduced _common() functions.
Fixes: 5a963eb61b7c ("ip6_gre: Add ERSPAN native tunnel support")
Signed-off-by: Petr Machata <petrm@mellanox.com>
Acked-by: William Tu <u9012063@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Cc: Greg Rose <gvrose8192@gmail.com>
Signed-off-by: William Tu <u9012063@gmail.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
Reviewed-by: Greg Rose <gvrose8192@gmail.com>
Tested-by: Greg Rose <gvrose8192@gmail.com>
Diffstat (limited to '.travis.yml')
0 files changed, 0 insertions, 0 deletions