From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ralf Baechle Subject: Re: [PATCH] CPUFREQ: Loongson2: drop set_cpus_allowed_ptr() Date: Mon, 10 Apr 2017 13:05:11 +0200 Message-ID: <20170410110511.GE24214@linux-mips.org> References: <20170404154957.19678-1-bigeasy@linutronix.de> <20170410103153.GK24555@vireshk-i7> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from eddie.linux-mips.org ([148.251.95.138]:36576 "EHLO cvs.linux-mips.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753470AbdDJLFX (ORCPT ); Mon, 10 Apr 2017 07:05:23 -0400 Received: from localhost.localdomain ([127.0.0.1]:58248 "EHLO linux-mips.org" rhost-flags-OK-OK-OK-FAIL) by eddie.linux-mips.org with ESMTP id S23993922AbdDJLFO58XwI (ORCPT ); Mon, 10 Apr 2017 13:05:14 +0200 Content-Disposition: inline In-Reply-To: <20170410103153.GK24555@vireshk-i7> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Viresh Kumar Cc: Sebastian Andrzej Siewior , wuzhangjin@gmail.com, linux-mips@linux-mips.org, tglx@linutronix.de, "Rafael J. Wysocki" , linux-pm@vger.kernel.org On Mon, Apr 10, 2017 at 04:01:53PM +0530, Viresh Kumar wrote: > On 04-04-17, 17:49, Sebastian Andrzej Siewior wrote: > > It is pure mystery to me why we need to be on a specific CPU while > > looking up a value in an array. > > My best shot at this is that before commit d4019f0a92ab ("cpufreq: move > > freq change notifications to cpufreq core") it was required to invoke > > cpufreq_notify_transition() on a special CPU. > > > > Since it looks like a waste, remove it. > > > > Cc: "Rafael J. Wysocki" > > Cc: Viresh Kumar > > Cc: linux-pm@vger.kernel.org > > Signed-off-by: Sebastian Andrzej Siewior > > --- > > drivers/cpufreq/loongson2_cpufreq.c | 7 ------- > > 1 file changed, 7 deletions(-) > > > > diff --git a/drivers/cpufreq/loongson2_cpufreq.c b/drivers/cpufreq/loongson2_cpufreq.c > > index 6bbdac1065ff..9ac27b22476c 100644 > > --- a/drivers/cpufreq/loongson2_cpufreq.c > > +++ b/drivers/cpufreq/loongson2_cpufreq.c > > @@ -51,19 +51,12 @@ static int loongson2_cpu_freq_notifier(struct notifier_block *nb, > > static int loongson2_cpufreq_target(struct cpufreq_policy *policy, > > unsigned int index) > > { > > - unsigned int cpu = policy->cpu; > > - cpumask_t cpus_allowed; > > unsigned int freq; > > > > - cpus_allowed = current->cpus_allowed; > > - set_cpus_allowed_ptr(current, cpumask_of(cpu)); > > - > > freq = > > ((cpu_clock_freq / 1000) * > > loongson2_clockmod_table[index].driver_data) / 8; > > > > - set_cpus_allowed_ptr(current, &cpus_allowed); > > - > > /* setting the cpu frequency */ > > clk_set_rate(policy->clk, freq * 1000); > > It was Zhangjin who wrote the first version and he may be able to > answer the questions we have. Anyway, it is safe to apply this patch > right now. > > Acked-by: Viresh Kumar Thanks for the ack, patch applied. I noticed some of the other cpufreq drivers seem to be using similar constructs. Ralf