From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tang Chen Subject: Re: [PATCH] Update sched_domains_numa_masks when new cpus are onlined. Date: Tue, 25 Sep 2012 20:02:35 +0800 Message-ID: <50619D5B.1040101@cn.fujitsu.com> References: <1347963128-25942-1-git-send-email-tangchen@cn.fujitsu.com> <1348479536.11847.25.camel@twins> <50611946.9080601@cn.fujitsu.com> <1348569232.3881.1.camel@twins> <5061996E.8070301@cn.fujitsu.com> <1348573806.3881.35.camel@twins> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1348573806.3881.35.camel@twins> Sender: linux-numa-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii"; format="flowed" To: Peter Zijlstra Cc: "Srivatsa S. Bhat" , linux-kernel@vger.kernel.org, x86@kernel.org, linux-numa@vger.kernel.org, wency@cn.fujitsu.com, mingo@kernel.org, Thomas Gleixner Hi Peter~ I will make a v2 patch-set following your comments and resent it soon. :) Thanks. :) On 09/25/2012 07:50 PM, Peter Zijlstra wrote: > On Tue, 2012-09-25 at 19:45 +0800, Tang Chen wrote: > > >> Let's have an example here. >> >> sched_init_numa() >> { >> ... >> // A loop set sched_domains_numa_levels to level.-------------1 >> >> // I set sched_domains_numa_levels to 0. >> sched_domains_numa_levels = 0;--------------------------------2 > > Ah, ok, so this 'idiot in a hurry' (aka. me) placed the hunk wrong and > thought it was at the start of the function. > >> // A loop allocating memory for sched_domains_numa_masks[][] >> for (i = 0; i< level; i++) { >> ...... >> // Allocate memory for sched_domains_numa_masks[i]----3 >> ...... >> } >> ...... >> >> // I reset sched_domains_numa_levels to level. >> sched_domains_numa_levels = level;----------------------------4 >> } > > Yes this makes sense, it might be best to have this as a separate patch. > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ >