Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S936745Ab3DJOLi (ORCPT ); Wed, 10 Apr 2013 10:11:38 -0400 Received: from a9-74.smtp-out.amazonses.com ([54.240.9.74]:34786 "EHLO a9-74.smtp-out.amazonses.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S935800Ab3DJOLh (ORCPT ); Wed, 10 Apr 2013 10:11:37 -0400 X-Greylist: delayed 664 seconds by postgrey-1.27 at vger.kernel.org; Wed, 10 Apr 2013 10:11:36 EDT Date: Wed, 10 Apr 2013 14:00:31 +0000 From: Christoph Lameter X-X-Sender: cl@gentwo.org To: Joonsoo Kim cc: Pekka Enberg , Steven Rostedt , Paul Gortmaker , LKML , RT , Thomas Gleixner , Clark Williams Subject: Re: [RT LATENCY] 249 microsecond latency caused by slub's unfreeze_partials() code. In-Reply-To: <20130410073143.GF5872@lge.com> Message-ID: <0000013df43f6fe0-8b71a9f4-bad1-417e-ab44-797c5ba2aecd-000000@email.amazonses.com> References: <1364355032.6345.200.camel@gandalf.local.home> <20130327061351.GB17125@lge.com> <0000013db20ca149-0064fbb8-2f81-4323-9095-a38f6abb79c5-000000@email.amazonses.com> <0000013dc63b3a87-6ce88b75-d011-407e-8dde-da73c3a7f5fd-000000@email.amazonses.com> <20130402004217.GA16699@lge.com> <1365424372.25498.6.camel@gandalf.local.home> <20130410073143.GF5872@lge.com> User-Agent: Alpine 2.02 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII X-SES-Outgoing: 2013.04.10-54.240.9.74 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 597 Lines: 20 On Wed, 10 Apr 2013, Joonsoo Kim wrote: > > Christoph, Joonsoo, comments? > > Steven's comment is reasonable to me. > > If there is no objection from Christoph, > let's drop a patch in which I implement Christoph's idea. Fine with me. I do not like passing a reference just because we have to return an additional counter. Adds a lot of overhead. -- 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/