2009-04-23 08:01:40

by Chris Wright

[permalink] [raw]
Subject: [patch 095/100] gso: Fix support for linear packets

-stable review patch. If anyone has any objections, please let us know.
---------------------

From: Herbert Xu <[email protected]>

upstream commit: 2f181855a0b3c2b39314944add7b41c15647cf86

When GRO/frag_list support was added to GSO, I made an error
which broke the support for segmenting linear GSO packets (GSO
packets are normally non-linear in the payload).

These days most of these packets are constructed by the tun
driver, which prefers to allocate linear memory if possible.
This is fixed in the latest kernel, but for 2.6.29 and earlier
it is still the norm.

Therefore this bug causes failures with GSO when used with tun
in 2.6.29.

Reported-by: James Huang <[email protected]>
Signed-off-by: Herbert Xu <[email protected]>
Signed-off-by: David S. Miller <[email protected]>
Signed-off-by: Chris Wright <[email protected]>
---
net/core/skbuff.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

--- a/net/core/skbuff.c
+++ b/net/core/skbuff.c
@@ -2496,7 +2496,7 @@ struct sk_buff *skb_segment(struct sk_bu
skb_network_header_len(skb));
skb_copy_from_linear_data(skb, nskb->data, doffset);

- if (pos >= offset + len)
+ if (fskb != skb_shinfo(skb)->frag_list)
continue;

if (!sg) {