From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH 2/8] numa: x86_64: use generic percpu var numa_node_id() implementation Date: Fri, 30 Apr 2010 06:58:10 +0200 Message-ID: <4BDA6362.4030505@kernel.org> References: <20100415172950.8801.60358.sendpatchset@localhost.localdomain> <20100415173003.8801.48519.sendpatchset@localhost.localdomain> <4BCA74D8.3030503@kernel.org> <1272560208.4927.39.camel@useless.americas.hpqcorp.net> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1272560208.4927.39.camel@useless.americas.hpqcorp.net> Sender: owner-linux-mm@kvack.org List-Id: Content-Type: text/plain; charset="us-ascii" To: Lee Schermerhorn Cc: Christoph Lameter , linux-mm@kvack.org, linux-numa@vger.kernel.org, Mel Gorman , andi@firstfloor.org, Nick Piggin , David Rientjes , eric.whitney@hp.com, Andrew Morton , KAMEZAWA Hiroyuki Hello, On 04/29/2010 06:56 PM, Lee Schermerhorn wrote: > Tejun: do you mean: > > #ifdef CONFIG_NUMA > if (cpu != 0 && percpu_read(numa_node) == 0 && > ........................^ here? > early_cpu_to_node(cpu) != NUMA_NO_NODE) > set_numa_node(early_cpu_to_node(cpu)); > #endif > > Looks like 'numa_node_id()' would work there. Yeah, it just looked weird to use raw variable when an access wrapper is there. > But, I wonder what the "cpu != 0 && percpu_read(numa_node) == 0" is > trying to do? That I have don't have any clue about. :-) > Just trying to grok the intent. Maybe someone will chime in. Christoph? Mel? Thanks. -- tejun -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org