Messages in this thread | | | Subject | Re: [PATCH] tcp: fix setting csum_start in tcp_gso_segment | From | Eric Dumazet <> | Date | Wed, 25 Jun 2014 00:19:04 -0700 |
| |
On Tue, 2014-06-24 at 21:03 -0700, Tom Herbert wrote:
> > 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 <davej@redhat.com> > Signed-off-by: Tom Herbert <therbert@google.com> > --- > 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()"
| |