From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f41.google.com (mail-pa0-f41.google.com [209.85.220.41]) by kanga.kvack.org (Postfix) with ESMTP id D0B876B0038 for ; Wed, 19 Aug 2015 08:45:20 -0400 (EDT) Received: by pawq9 with SMTP id q9so2630325paw.3 for ; Wed, 19 Aug 2015 05:45:20 -0700 (PDT) Received: from mga09.intel.com (mga09.intel.com. [134.134.136.24]) by mx.google.com with ESMTP id jv9si1006560pbb.201.2015.08.19.05.45.19 for ; Wed, 19 Aug 2015 05:45:19 -0700 (PDT) Subject: Re: [Patch V3 3/9] sgi-xp: Replace cpu_to_node() with cpu_to_mem() to support memoryless node References: <1439781546-7217-1-git-send-email-jiang.liu@linux.intel.com> <1439781546-7217-4-git-send-email-jiang.liu@linux.intel.com> From: Jiang Liu Message-ID: <55D47A57.3030702@linux.intel.com> Date: Wed, 19 Aug 2015 20:45:11 +0800 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org List-ID: To: Robin Holt Cc: Andrew Morton , Mel Gorman , David Rientjes , Mike Galbraith , Peter Zijlstra , "Rafael J . Wysocki" , Tang Chen , Tejun Heo , Cliff Whickman , Tony Luck , linux-mm@kvack.org, linux-hotplug@vger.kernel.org, LKML , x86@kernel.org On 2015/8/19 19:52, Robin Holt wrote: > On Sun, Aug 16, 2015 at 10:19 PM, Jiang Liu wrote: >> Function xpc_create_gru_mq_uv() allocates memory with __GFP_THISNODE >> flag set, which may cause permanent memory allocation failure on >> memoryless node. So replace cpu_to_node() with cpu_to_mem() to better >> support memoryless node. For node with memory, cpu_to_mem() is the same >> as cpu_to_node(). >> >> Signed-off-by: Jiang Liu >> --- >> drivers/misc/sgi-xp/xpc_uv.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/misc/sgi-xp/xpc_uv.c b/drivers/misc/sgi-xp/xpc_uv.c >> index 95c894482fdd..9210981c0d5b 100644 >> --- a/drivers/misc/sgi-xp/xpc_uv.c >> +++ b/drivers/misc/sgi-xp/xpc_uv.c >> @@ -238,7 +238,7 @@ xpc_create_gru_mq_uv(unsigned int mq_size, int cpu, char *irq_name, >> >> mq->mmr_blade = uv_cpu_to_blade_id(cpu); >> >> - nid = cpu_to_node(cpu); >> + nid = cpu_to_mem(cpu); > > I would recommend rejecting this. First, SGI's UV system does not and > can not support memory-less nodes. Additionally the hardware _REALLY_ > wants the memory to be local to the CPU. We will register this memory > region with the node firmware. That will set the hardware up to watch > this memory block and raise an IRQ targeting the registered CPU when > anything is written into the memory block. This is all part of how > cross-partition communications expects to work. > > Additionally, the interrupt handler will read the memory region, so > having node-local memory is extremely helpful. Hi Robin, Thanks for review, I will drop this patch in next version. Actually, if SGI UV systems don't support memoryless node, cpu_to_mem() is the same as cpu_to_node(). Thanks! Gerry > > Thanks, > Robin > -- 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