Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Herbert Xu
Date: Wednesday, April 22, 2009 - 3:48 am

On Tue, Apr 21, 2009 at 03:19:14PM -0400, Andrew Gallatin wrote:

I was sure I had tested my case with the IRQs bound (using a cxgb3),
but when I tried it again today GRO was indeed slower (8G vs. 9.4G)!
I fiddled with it all day and couldn't figure out why this was
so.  We weren't spending any more time in the GRO code than LRO,
and in fact we were aggregating more packets with GRO (700k segments
instead of 900k segments).  GRO was also sending a lot less ACKs
than LRO.

It finally dawned on me that my sender had been upgraded from 2.6.18
to 2.6.30-rc1.  Indeed, rebooting into 2.6.18 seems to restore
the balance between GRO and LRO.  I wonder if the ACK reduction
has anything to do with this.

Hopefully tomorrow I'll get my hands onto a myricom and try to
replicate your problem.

In the mean time, can you see if there is any disparity in the
number of aggregated segments and ACKs between GRO and LRO?
netstat -s should be sufficient to measure this (TCP segments
received and sent).

Thanks,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH] myr10ge: again fix lro_gen_skb() alignment, Stanislaw Gruszka, (Wed Apr 15, 1:09 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, David Miller, (Wed Apr 15, 2:28 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Brice Goglin, (Wed Apr 15, 2:48 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, David Miller, (Wed Apr 15, 3:02 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Wed Apr 15, 6:01 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Wed Apr 15, 2:04 pm)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, David Miller, (Wed Apr 15, 4:42 pm)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, David Miller, (Thu Apr 16, 2:02 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Tue Apr 21, 12:19 pm)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Herbert Xu, (Wed Apr 22, 3:48 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Wed Apr 22, 8:37 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Herbert Xu, (Thu Apr 23, 10:45 pm)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Fri Apr 24, 5:45 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Fri Apr 24, 9:16 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Rick Jones, (Fri Apr 24, 10:13 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, David Miller, (Mon Apr 27, 2:32 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, David Miller, (Mon Apr 27, 5:45 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, David Miller, (Mon Apr 27, 5:45 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Herbert Xu, (Mon Apr 27, 11:12 pm)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Tue Apr 28, 8:00 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, David Miller, (Tue Apr 28, 8:02 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Tue Apr 28, 8:44 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Tue Apr 28, 2:12 pm)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Wed Apr 29, 6:42 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Eric Dumazet, (Wed Apr 29, 6:53 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Wed Apr 29, 7:18 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Eric Dumazet, (Wed Apr 29, 8:26 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Wed Apr 29, 10:28 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Eric Dumazet, (Thu Apr 30, 1:17 am)
Re: [PATCH] myr10ge: again fix lro_gen_skb() alignment, Andrew Gallatin, (Thu Apr 30, 12:14 pm)