From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e28smtp06.in.ibm.com (e28smtp06.in.ibm.com [122.248.162.6]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by ozlabs.org (Postfix) with ESMTPS id 89F732C00A8 for ; Wed, 19 Mar 2014 17:35:37 +1100 (EST) Received: from /spool/local by e28smtp06.in.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Wed, 19 Mar 2014 12:05:33 +0530 Received: from d28relay02.in.ibm.com (d28relay02.in.ibm.com [9.184.220.59]) by d28dlp01.in.ibm.com (Postfix) with ESMTP id 3D1AEE0053 for ; Wed, 19 Mar 2014 12:09:23 +0530 (IST) Received: from d28av03.in.ibm.com (d28av03.in.ibm.com [9.184.220.65]) by d28relay02.in.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id s2J6ZPmJ13041838 for ; Wed, 19 Mar 2014 12:05:25 +0530 Received: from d28av03.in.ibm.com (localhost [127.0.0.1]) by d28av03.in.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id s2J6ZTQY024440 for ; Wed, 19 Mar 2014 12:05:29 +0530 Message-ID: <53293AA8.2040609@linux.vnet.ibm.com> Date: Wed, 19 Mar 2014 12:05:20 +0530 From: "Srivatsa S. Bhat" MIME-Version: 1.0 To: Benjamin Herrenschmidt Subject: Re: [PATCH v2 2/6] powernv:cpufreq: Create a powernv_cpu_to_core_mask() helper. References: <1394449861-8688-1-git-send-email-ego@linux.vnet.ibm.com> <1394449861-8688-3-git-send-email-ego@linux.vnet.ibm.com> <1395185847.30666.22.camel@pasglop> In-Reply-To: <1395185847.30666.22.camel@pasglop> Content-Type: text/plain; charset=UTF-8 Cc: "Gautham R. Shenoy" , linuxppc-dev@ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 03/19/2014 05:07 AM, Benjamin Herrenschmidt wrote: > On Mon, 2014-03-10 at 16:40 +0530, Gautham R. Shenoy wrote: >> From: "Srivatsa S. Bhat" >> >> Create a helper method that computes the cpumask corresponding to the >> thread-siblings of a cpu. Use this for initializing the policy->cpus >> mask for a given cpu. >> >> (Original code written by Srivatsa S. Bhat. Gautham moved this to a >> helper function!) > > How does that differ from the existing entry in the sibling map which > you can obtain with the helper cpu_sibling_mask() ? (You probably want > to *copy* the mask of course but I don't see the need of re-creating > one from scratch). > The intent here was to have a sibling mask that is invariant of CPU hotplug. cpu_sibling_mask is updated on every hotplug operation, and this would break cpufreq, since policy->cpus has to be hotplug invariant. This should have been noted in the changelog of patch 1 as well as this patch. (The earlier (internal) versions of this patchset had the description in the changelogs, but somehow it got dropped accidentally). I'll work with Gautham and ensure that we have this important info included in the changelog. Thanks for pointing it out! Regards, Srivatsa S. Bhat > Also, this should have been CCed to the cpufreq mailing list and maybe > the relevant maintainer too. > > Cheers, > Ben. > >> Signed-off-by: Srivatsa S. Bhat >> Signed-off-by: Gautham R. Shenoy >> --- >> drivers/cpufreq/powernv-cpufreq.c | 24 ++++++++++++++++++------ >> 1 file changed, 18 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/cpufreq/powernv-cpufreq.c b/drivers/cpufreq/powernv-cpufreq.c >> index ab1551f..4cad727 100644 >> --- a/drivers/cpufreq/powernv-cpufreq.c >> +++ b/drivers/cpufreq/powernv-cpufreq.c >> @@ -115,6 +115,23 @@ static struct freq_attr *powernv_cpu_freq_attr[] = { >> >> /* Helper routines */ >> >> +/** >> + * Sets the bits corresponding to the thread-siblings of cpu in its core >> + * in 'cpus'. >> + */ >> +static void powernv_cpu_to_core_mask(unsigned int cpu, cpumask_var_t cpus) >> +{ >> + int base, i; >> + >> + base = cpu_first_thread_sibling(cpu); >> + >> + for (i = 0; i < threads_per_core; i++) { >> + cpumask_set_cpu(base + i, cpus); >> + } >> + >> + return; >> +} >> + >> /* Access helpers to power mgt SPR */ >> >> static inline unsigned long get_pmspr(unsigned long sprn) >> @@ -180,13 +197,8 @@ static int powernv_set_freq(cpumask_var_t cpus, unsigned int new_index) >> >> static int powernv_cpufreq_cpu_init(struct cpufreq_policy *policy) >> { >> - int base, i; >> - >> #ifdef CONFIG_SMP >> - base = cpu_first_thread_sibling(policy->cpu); >> - >> - for (i = 0; i < threads_per_core; i++) >> - cpumask_set_cpu(base + i, policy->cpus); >> + powernv_cpu_to_core_mask(policy->cpu, policy->cpus); >> #endif >> policy->cpuinfo.transition_latency = 25000; >> > >