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: Wed, 21 Sep 2016 12:42:04 -0400 From: Jon Masters <jcm@...hat.com> To: Hanjun Guo <hanjun.guo@...aro.org>, Robert Richter <rric@...nel.org> Cc: David Daney <ddaney.cavm@...il.com>, linux-kernel@...r.kernel.org, Marc Zyngier <marc.zyngier@....com>, "Rafael J. Wysocki" <rafael.j.wysocki@...el.com>, Will Deacon <will.deacon@....com>, Ganapatrao Kulkarni <gkulkarni@...iumnetworks.com>, linux-arm-kernel@...ts.infradead.org, David Daney <david.daney@...ium.com>, xieyisheng1@...wei.com Subject: Re: [PATCH] arm64, numa: Add cpu_to_node() implementation. On 09/20/2016 10:12 AM, Hanjun Guo wrote: > On 09/20/2016 09:38 PM, Robert Richter wrote: >> On 20.09.16 19:32:34, Hanjun Guo wrote: >>> On 09/20/2016 06:43 PM, Robert Richter wrote: >> >>>> Instead we need to make sure the set_*numa_node() functions are called >>>> earlier before secondary cpus are booted. My suggested change for that >>>> is this: >>>> >>>> >>>> diff --git a/arch/arm64/kernel/smp.c b/arch/arm64/kernel/smp.c >>>> index d93d43352504..952365c2f100 100644 >>>> --- a/arch/arm64/kernel/smp.c >>>> +++ b/arch/arm64/kernel/smp.c >>>> @@ -204,7 +204,6 @@ int __cpu_up(unsigned int cpu, struct >>>> task_struct *idle) >>>> static void smp_store_cpu_info(unsigned int cpuid) >>>> { >>>> store_cpu_topology(cpuid); >>>> - numa_store_cpu_info(cpuid); >>>> } >>>> >>>> /* >>>> @@ -719,6 +718,7 @@ void __init smp_prepare_cpus(unsigned int max_cpus) >>>> continue; >>>> >>>> set_cpu_present(cpu, true); >>>> + numa_store_cpu_info(cpu); >>>> } >>>> } >>> >>> We tried a similar approach which add numa_store_cpu_info() in >>> early_map_cpu_to_node(), and remove it from smp_store_cpu_info, >>> but didn't work for us, we will try your approach to see if works. > > And it works :) Great. I'm curious for further (immediate) feedback on David's updated patch in the other thread due to some time sensitive needs on our end. Jon. -- Computer Architect | Sent from my Fedora powered laptop
Powered by blists - more mailing lists