Merge branch 'ip6_gre-Fixes-in-headroom-handling'
authorDavid S. Miller <davem@davemloft.net>
Thu, 17 May 2018 20:50:20 +0000 (16:50 -0400)
committerDavid S. Miller <davem@davemloft.net>
Thu, 17 May 2018 20:50:20 +0000 (16:50 -0400)
commit374edea4aaedb92964d35f68318c094b6d5c0f63
tree69b5c67423f30b1a6dccb2c152a15fec0195d19b
parent02f99df1875c11330cd0be69a40fa8ccd14749b2
parent2d665034f239412927b1e71329f20f001c92da09
Merge branch 'ip6_gre-Fixes-in-headroom-handling'

Petr Machata says:

====================
net: ip6_gre: Fixes in headroom handling

This series mends some problems in headroom management in ip6_gre
module. The current code base has the following three closely-related
problems:

- ip6gretap tunnels neglect to ensure there's enough writable headroom
  before pushing GRE headers.

- ip6erspan does this, but assumes that dev->needed_headroom is primed.
  But that doesn't happen until ip6_tnl_xmit() is called later. Thus for
  the first packet, ip6erspan actually behaves like ip6gretap above.

- ip6erspan shares some of the code with ip6gretap, including
  calculations of needed header length. While there is custom
  ERSPAN-specific code for calculating the headroom, the computed
  values are overwritten by the ip6gretap code.

The first two issues lead to a kernel panic in situations where a packet
is mirrored from a veth device to the device in question. They are
fixed, respectively, in patches #1 and #2, which include the full panic
trace and a reproducer.

The rest of the patchset deals with the last issue. In patches #3 to #6,
several functions are split up into reusable parts. Finally in patch #7
these blocks are used to compose ERSPAN-specific callbacks where
necessary to fix the hlen calculation.
====================

Signed-off-by: David S. Miller <davem@davemloft.net>