netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* IGB handling of zero length checksumming?
@ 2010-04-11  9:40 David Miller
  2010-04-11 21:42 ` David Miller
  0 siblings, 1 reply; 2+ messages in thread
From: David Miller @ 2010-04-11  9:40 UTC (permalink / raw)
  To: netdev
  Cc: jeffrey.t.kirsher, jesse.brandeburg, bruce.w.allan,
	alexander.h.duyck, peter.p.waskiewicz.jr, john.ronciak


If the IGB is given a "skb->ip_summed === CHECKSUM_PARTIAL" packet and
the data area past the TCP header is of zero length, will it do the
right thing?

I'm asking because I made a change recently in net-2.6 that causes TCP
packets with no data (ACKs, etc.) to start using CHECKSUM_PARTIAL.

And a user with IGB interfaces on a system running net-2.6 is getting
failures connecting to SSH and stuff like that and it is known that
this TCP commit of mine is the culprit.  See:

http://marc.info/?l=linux-netdev&m=127097744616962&w=2

I'm very likely to just revert this change regardless of what the
problem is, and try to sort this out in net-next-2.6, but it'd be nice
to know if IGB indeed has a problem in this area and that's the true
cause of the issue.

Thanks!

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: IGB handling of zero length checksumming?
  2010-04-11  9:40 IGB handling of zero length checksumming? David Miller
@ 2010-04-11 21:42 ` David Miller
  0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2010-04-11 21:42 UTC (permalink / raw)
  To: netdev
  Cc: jeffrey.t.kirsher, jesse.brandeburg, bruce.w.allan,
	alexander.h.duyck, peter.p.waskiewicz.jr, john.ronciak

From: David Miller <davem@davemloft.net>
Date: Sun, 11 Apr 2010 02:40:27 -0700 (PDT)

> 
> If the IGB is given a "skb->ip_summed === CHECKSUM_PARTIAL" packet and
> the data area past the TCP header is of zero length, will it do the
> right thing?

Hey guys you don't have to worry about this.

The problem turned out to be somewhere else. :-)


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-04-11 21:42 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-04-11  9:40 IGB handling of zero length checksumming? David Miller
2010-04-11 21:42 ` David Miller

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).