gre: Improve headroom calculation to reducing copying.
authorJesse Gross <jesse@nicira.com>
Fri, 11 Jun 2010 22:10:35 +0000 (15:10 -0700)
committerJesse Gross <jesse@nicira.com>
Fri, 11 Jun 2010 22:54:44 +0000 (15:54 -0700)
commit7e7d587d68a22aa47745d5acadd9790d1a34934f
tree2172ae7c87adb56afc5c2694ba11d413a7fc9cac
parent26bb189de27c51a210d64ddace8669cef50c0f9c
gre: Improve headroom calculation to reducing copying.

The first change is use skb_header_cloned() instead of (skb_cloned()
 && skb_clone_writable()).  While these are effectively the same
checks, skb_clone_writable() does not exist until 2.6.23 and relies
on information not available in earlier kernels and therefore the
compat version always returns false.  Since GRE only adds data to
the front of the packet and this is on the transmit path (meaning
there is no important data in the SKB head) it is safe to use
skb_header_cloned().  This avoids a copy for locally generated
TCP packets on the affected kernels.

The second change is to always allocate 64 bytes of headroom when
a copy needs to be made.  There is no cost to allocating extra
space once we have decided to make a copy and can avoid another
copy later if we need to add a VLAN tag.
datapath/vport-gre.c