[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20080109094725.GA22140@2ka.mipt.ru>
Date: Wed, 9 Jan 2008 12:47:26 +0300
From: Evgeniy Polyakov <johnpol@....mipt.ru>
To: Andi Kleen <andi@...stfloor.org>
Cc: David Miller <davem@...emloft.net>, jheffner@....edu,
ilpo.jarvinen@...sinki.fi, lachlan.andrew@...il.com,
netdev@...r.kernel.org, quetchen@...tech.edu
Subject: Re: SACK scoreboard
Hi.
On Wed, Jan 09, 2008 at 08:03:18AM +0100, Andi Kleen (andi@...stfloor.org) wrote:
> > It adds severe spikes in CPU utilization that are even moderate
> > line rates begins to affect RTTs.
> >
> > Or do you think it's OK to process 500,000 SKBs while locked
> > in a software interrupt.
>
> You can always push it into a work queue. Even put it to
> other cores if you want.
>
> In fact this is already done partly for the ->completion_queue.
> Wouldn't be a big change to queue it another level down.
>
> Also even freeing a lot of objects doesn't have to be
> that expensive. I suspect the most cost is in taking
> the slab locks, but that could be batched. Without
> that the kmem_free fast path isn't particularly
> expensive, as long as the headers are still in cache.
Postponing freeing of the skb has major drawbacks. Some time ago I
made a patch to postpone skb freeing behind rcu and got 2.5 times slower
connection speed on some machines with decreased CPU usage though.
So, queueing solution has to be proven with real data and although it
looks good in one situation, it can be really bad in another.
For interested reader: results of the RCUfication of the kfree_skbmem()
http://tservice.net.ru/~s0mbre/blog/devel/networking/2006/12/05
--
Evgeniy Polyakov
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists