[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <48EB7464.9090206@linux-foundation.org>
Date: Tue, 07 Oct 2008 09:38:28 -0500
From: Christoph Lameter <cl@...ux-foundation.org>
To: Evgeniy Polyakov <johnpol@....mipt.ru>
CC: Corey Minyard <minyard@....org>,
David Miller <davem@...emloft.net>, dada1@...mosbay.com,
linux-kernel@...r.kernel.org, netdev@...r.kernel.org,
shemminger@...tta.com, paulmck@...ux.vnet.ibm.com,
Hugh Dickins <hugh@...itas.com>
Subject: Re: [PATCH 3/3] Convert the UDP hash lock to RCU
Evgeniy Polyakov wrote:
> On Tue, Oct 07, 2008 at 09:16:13AM -0500, Christoph Lameter (cl@...ux-foundation.org) wrote:
>>> I tested skb destruction via RCU path, and got 2.5 times worse numbers
>>> with small-packets-bulk-transfer workload.
>> Was this with regular RCU freeing? This will cool down the cacheline before
>> frees. You need SLAB_DESTROY_BY_RCU to keep the objects cache hot.
>
> I believe there were no SLAB_DESTROY_BY_RCU 2 yars ago :)
Its been awhile. Hugh created it
> It was pure call_rcu(&skb->rcu, real_skb_freeing), where
> real_skb_freeing() just did usual kfree().
Right. That results in cacheline cooldown. You'd want to recycle the object as
they are cache hot on a per cpu basis. That is screwed up by the delayed
regular rcu processing. We have seen multiple regressions due to cacheline
cooldown.
The only choice in cacheline hot sensitive areas is to deal with the
complexity that comes with SLAB_DESTROY_BY_RCU or give up on RCU.
--
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