lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 25 Feb 2019 07:30:40 -0800 From: Dave Hansen <dave.hansen@...el.com> To: Pingfan Liu <kernelfans@...il.com>, x86@...nel.org, linux-mm@...ck.org Cc: Thomas Gleixner <tglx@...utronix.de>, Ingo Molnar <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>, "H. Peter Anvin" <hpa@...or.com>, Dave Hansen <dave.hansen@...ux.intel.com>, Vlastimil Babka <vbabka@...e.cz>, Mike Rapoport <rppt@...ux.vnet.ibm.com>, Andrew Morton <akpm@...ux-foundation.org>, Mel Gorman <mgorman@...e.de>, Joonsoo Kim <iamjoonsoo.kim@....com>, Andy Lutomirski <luto@...nel.org>, Andi Kleen <ak@...ux.intel.com>, Petr Tesarik <ptesarik@...e.cz>, Michal Hocko <mhocko@...e.com>, Stephen Rothwell <sfr@...b.auug.org.au>, Jonathan Corbet <corbet@....net>, Nicholas Piggin <npiggin@...il.com>, Daniel Vacek <neelx@...hat.com>, linux-kernel@...r.kernel.org Subject: Re: [PATCH 5/6] x86/numa: push forward the setup of node to cpumask map On 2/24/19 4:34 AM, Pingfan Liu wrote: > At present the node to cpumask map is set up until the secondary > cpu boot up. But it is too late for the purpose of building node fall back > list at early boot stage. Considering that init_cpu_to_node() already owns > cpu to node map, it is a good place to set up node to cpumask map too. So > do it by calling numa_add_cpu(cpu) in init_cpu_to_node(). It sounds like you have carefully considered the ordering and dependencies here. However, none of that consideration has made it into the code. Could you please add some comments to the new call-sites to explain why the *must* be where they are?
Powered by blists - more mailing lists