net: Fix stacked vlan offload features computation
authorToshiaki Makita <makita.toshiaki@lab.ntt.co.jp>
Mon, 22 Dec 2014 10:04:14 +0000 (19:04 +0900)
committerDavid S. Miller <davem@davemloft.net>
Wed, 24 Dec 2014 05:08:33 +0000 (00:08 -0500)
commit796f2da81bead71ffc91ef70912cd8d1827bf756
treeb002f4fc70040e8763d53857582ae6cce5c145da
parente51a08b8df461919451b1a2bf4a730da0b994425
net: Fix stacked vlan offload features computation

When vlan tags are stacked, it is very likely that the outer tag is stored
in skb->vlan_tci and skb->protocol shows the inner tag's vlan_proto.
Currently netif_skb_features() first looks at skb->protocol even if there
is the outer tag in vlan_tci, thus it incorrectly retrieves the protocol
encapsulated by the inner vlan instead of the inner vlan protocol.
This allows GSO packets to be passed to HW and they end up being
corrupted.

Fixes: 58e998c6d239 ("offloading: Force software GSO for multiple vlan tags.")
Signed-off-by: Toshiaki Makita <makita.toshiaki@lab.ntt.co.jp>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c