Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756240AbdDFSCI (ORCPT ); Thu, 6 Apr 2017 14:02:08 -0400 Received: from mail.skyhub.de ([5.9.137.197]:50366 "EHLO mail.skyhub.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756190AbdDFSBv (ORCPT ); Thu, 6 Apr 2017 14:01:51 -0400 Date: Thu, 6 Apr 2017 20:01:13 +0200 From: Borislav Petkov To: "Kirill A. Shutemov" Cc: Wei Yang , tglx@linutronix.de, mingo@redhat.com, hpa@zytor.com, tj@kernel.org, linux-kernel@vger.kernel.org Subject: Re: [Patch V2 2/2] x86/mm/numa: remove the numa_nodemask_from_meminfo() Message-ID: <20170406180113.hvcydzrjldodosfo@pd.tnic> References: <20170314030801.13656-1-richard.weiyang@gmail.com> <20170314030801.13656-2-richard.weiyang@gmail.com> <20170406124459.dwn5zhpr2xqg3lqm@node.shutemov.name> <20170406145937.docce7sa5tuqyon4@pd.tnic> <20170406154216.a4um6ftjyia5wxya@node.shutemov.name> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20170406154216.a4um6ftjyia5wxya@node.shutemov.name> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1656 Lines: 44 On Thu, Apr 06, 2017 at 06:42:16PM +0300, Kirill A. Shutemov wrote: > Config is attached. Thanks! > Looks like fake numa is the key. ... > NUMA: Warning: node ids are out of bound, from=0 to=1 distance=20 [ 0.000000] numa_register_memblks: numa_nodes_parsed: 0 > numa_register_memblks: nid: 0 > numa_register_memblks: nid: 1 > numa_register_memblks: nid: 2 > numa_register_memblks: nid: 3 Yeah, the fake numa thing calls emu_setup_memblk() and that doesn't set numa_nodes_parsed to the number of fake numa nodes. And since with that "cleanup" which opened more work than it saved (btw, this is the last time I'm looking at crap like that) we got rid of the "enlarging" of the node mask to the actual nodes count and *that* blows up with numa_nodes_parsed having only node 0 in there. Long story short, something as trivial as this helps here: --- diff --git a/arch/x86/mm/numa_emulation.c b/arch/x86/mm/numa_emulation.c index a8f90ce3dedf..60d82b8baaa3 100644 --- a/arch/x86/mm/numa_emulation.c +++ b/arch/x86/mm/numa_emulation.c @@ -70,6 +70,9 @@ static int __init emu_setup_memblk(struct numa_meminfo *ei, printk(KERN_INFO "Faking node %d at [mem %#018Lx-%#018Lx] (%LuMB)\n", nid, eb->start, eb->end - 1, (eb->end - eb->start) >> 20); + + node_set(nid, numa_nodes_parsed); + return 0; } -- Regards/Gruss, Boris. Good mailing practices for 400: avoid top-posting and trim the reply.