On Tue, 2014-06-24 at 21:03 -0700, Tom Herbert wrote:
quoted hunk ↗ jump to hunk
It looks like a likely culprit is that SKB_GSO_CB()->csum_start is
not set correctly when doing non-scatter gather. We are using
offset as opposed to doffset.
Reported-by: Dave Jones <redacted>
Signed-off-by: Tom Herbert <redacted>
---
net/core/skbuff.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/core/skbuff.c b/net/core/skbuff.c
index 9cd5344..c1a3303 100644
--- a/net/core/skbuff.c
+++ b/net/core/skbuff.c
@@ -2993,7 +2993,7 @@ struct sk_buff *skb_segment(struct sk_buff *head_skb,
skb_put(nskb, len),
len, 0);
SKB_GSO_CB(nskb)->csum_start =
- skb_headroom(nskb) + offset;
+ skb_headroom(nskb) + doffset;
continue;
}
Yes, seems an obvious typo, but please change patch title.
This is not "tcp: fix setting csum_start in tcp_gso_segment"
Maybe "net: fix setting csum_start in skb_segment()"