diff options
author | Michał Mirosław <mirq-linux@rere.qmqm.pl> | 2010-12-14 10:24:08 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-12-16 17:43:15 -0500 |
commit | 0d0b16727f24f8258eeb33818347ca0f4557f982 (patch) | |
tree | e89fec837700b02aff0d08de1c5cbae3ff5060a8 /drivers/net/sunhme.c | |
parent | 55508d601dab7df5cbcc7a63f4be8620eface204 (diff) |
net: Fix drivers advertising HW_CSUM feature to use csum_start
Some drivers are using skb_transport_offset(skb) instead of skb->csum_start
for NETIF_F_HW_CSUM offload. This does not matter now, but if someone
implements checksumming of encapsulated packets then this will break silently.
TSO output paths are left as they are, since they are for IP+TCP only
(might be worth converting though).
Signed-off-by: Michał Mirosław <mirq-linux@rere.qmqm.pl>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/sunhme.c')
-rw-r--r-- | drivers/net/sunhme.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/sunhme.c b/drivers/net/sunhme.c index 5e28c414421a..55bbb9c15d96 100644 --- a/drivers/net/sunhme.c +++ b/drivers/net/sunhme.c | |||
@@ -2266,7 +2266,7 @@ static netdev_tx_t happy_meal_start_xmit(struct sk_buff *skb, | |||
2266 | 2266 | ||
2267 | tx_flags = TXFLAG_OWN; | 2267 | tx_flags = TXFLAG_OWN; |
2268 | if (skb->ip_summed == CHECKSUM_PARTIAL) { | 2268 | if (skb->ip_summed == CHECKSUM_PARTIAL) { |
2269 | const u32 csum_start_off = skb_transport_offset(skb); | 2269 | const u32 csum_start_off = skb_checksum_start_offset(skb); |
2270 | const u32 csum_stuff_off = csum_start_off + skb->csum_offset; | 2270 | const u32 csum_stuff_off = csum_start_off + skb->csum_offset; |
2271 | 2271 | ||
2272 | tx_flags = (TXFLAG_OWN | TXFLAG_CSENABLE | | 2272 | tx_flags = (TXFLAG_OWN | TXFLAG_CSENABLE | |