Messages in this thread | | | Date | Wed, 14 Nov 2007 04:10:08 -0800 (PST) | Subject | Re: 2.6.24-rc2: Network commit causes SLUB performance regression with tbench | From | David Miller <> |
| |
From: Nick Piggin <nickpiggin@yahoo.com.au> Date: Wed, 14 Nov 2007 11:02:11 +1100
> On Wednesday 14 November 2007 22:48, Herbert Xu wrote: > > On Wed, Nov 14, 2007 at 03:10:22AM -0800, David Miller wrote: > > > So the thing that's being effected here in TCP is > > > net/ipv4/tcp.c:select_size(), specifically the else branch: > > > > Thanks for the pointer. Indeed there is a bug in that area. > > I'm not sure whether it's causing the problem at hand but it's > > certainly suboptimal. > > > > [TCP]: Fix size calculation in sk_stream_alloc_pskb > > This looks like it fixes the problem!
Great, thanks for testing. I'll apply Herbert's patch tomorrow as I've already sent Linus a bug fix pull request tonight.
> Still interested to know why SLAB didn't see the same thing...
Yes, I wonder why too. I bet objects just got packed differently.
There is this fugly "LOOPBACK_OVERHEAD" macro define in drivers/net/loopback.c that is trying to figure out the various overheads that we should subtract from the loopback MTU we use by default.
It's almost guarenteed to be wrong for the way the allocators work now. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |