[PATCH v2 1/2] cpu: Offline state Framework.

that the following operation puts the CPU in the same state as it did in the absence of the driver. echo 0 /sys/devices/system/cpu/cpunumber/online This framework also serializes the writes to the current_state with respect to with the writes to the online sysfs tunable. Signed-off-by: Gautham R

[PATCH v2 0/2] cpu: pseries: Offline state framework.

in turn can put the vCPU time to the best use. NOTE: This option DOES NOT result in a configuration change and the vCPU would be still entitled to the LPAR to which it earlier belong to. Awaiting your feedback. --- Gautham R Shenoy (2): cpu: Implement cpu-offline

[PATCH v2 2/2] cpu: Implement cpu-offline-state driver for pSeries.

it earlier belong to. Signed-off-by: Gautham R Shenoy e...@in.ibm.com --- arch/powerpc/platforms/pseries/Makefile |2 arch/powerpc/platforms/pseries/hotplug-cpu.c| 76 ++- arch/powerpc/platforms/pseries/offline_driver.c | 161 +++ arch/powerpc

[PATCH v3 0/3] cpu: pseries: Cpu offline states framework

function. Gautham R Shenoy (2): cpu: Implement cpu-offline-state callbacks for pSeries. cpu: Offline state Framework. Documentation/cpu-hotplug.txt | 22 +++ arch/powerpc/include/asm/lppaca.h |9 + arch/powerpc/platforms/pseries/Makefile

[PATCH v3 1/3] pSeries: cede latency specifier helper function.

unused during the subsequent H_CEDE hypercall. Signed-off-by: Arun R Bharadwaj a...@linux.vnet.ibm.com Signed-off-by: Gautham R Shenoy e...@in.ibm.com --- arch/powerpc/include/asm/lppaca.h |9 - arch/powerpc/platforms/pseries/plpar_wrappers.h | 17 + arch

[PATCH v3 2/3] cpu: Offline state Framework.

-by: Gautham R Shenoy e...@in.ibm.com --- Documentation/cpu-hotplug.txt | 22 + drivers/base/cpu.c| 181 +++-- include/linux/cpu.h | 10 ++ 3 files changed, 204 insertions(+), 9 deletions(-) diff --git a/Documentation/cpu-hotplug.txt b

[PATCH v3 3/3] cpu: Implement cpu-offline-state callbacks for pSeries.

to the hypervisor with a latency hint specifier value. Hypervisor may use this hint to provide better energy savings. In this state, the control of the vCPU will continue to be with the LPAR. Signed-off-by: Gautham R Shenoy e...@in.ibm.com --- arch/powerpc/platforms/pseries/Makefile |2 arch

[PATCH v4 0/4] pseries: Add cede support for cpu-offline

platforms and it works as per the expectations. Comments are very much appreciated. --- Gautham R Shenoy (4): pseries: Serialize cpu hotplug operations during deactivate Vs deallocate pseries: Add code to online/offline CPUs of a DLPAR node. pSeries: Add hooks to put the CPU

[PATCH v4 2/4] pSeries: Add hooks to put the CPU into an appropriate offline state

(). Signed-off-by: Gautham R Shenoy e...@in.ibm.com Signed-off-by: Nathan Fontenot nf...@austin.ibm.com --- arch/powerpc/platforms/pseries/hotplug-cpu.c| 157 ++- arch/powerpc/platforms/pseries/offline_states.h | 18 +++ arch/powerpc/platforms/pseries/smp.c| 19

[PATCH v4 3/4] pseries: Add code to online/offline CPUs of a DLPAR node.

probe and release. Signed-off-by: Gautham R Shenoy e...@in.ibm.com Signed-off-by: Nathan Fontenot nf...@austin.ibm.com --- arch/powerpc/platforms/pseries/dlpar.c | 103 1 files changed, 102 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/platforms/pseries

[PATCH v4 4/4] pseries: Serialize cpu hotplug operations during deactivate Vs deallocate

this serialization, which is a no-op on all architectures except PPC_PSERIES Signed-off-by: Gautham R Shenoy e...@in.ibm.com --- arch/powerpc/platforms/pseries/dlpar.c | 26 ++ drivers/base/cpu.c |2 ++ include/linux/cpu.h| 13

[PATCH v5 1/4] pSeries: extended_cede_processor() helper function.

This patch provides an extended_cede_processor() helper function which takes the cede latency hint as an argument. This hint is to be passed on to the hypervisor to cede to the corresponding state on platforms which support it. Signed-off-by: Gautham R Shenoy e...@in.ibm.com Signed-off-by: Arun R

[PATCH v5 2/4] pSeries: Add hooks to put the CPU into an appropriate offline state

(). The patch also provides a boot-time command line argument to disable/enable CPU_STATE_INACTIVE. Signed-off-by: Gautham R Shenoy e...@in.ibm.com Signed-off-by: Nathan Fontenot nf...@austin.ibm.com --- Documentation/cpu-hotplug.txt |6 + arch/powerpc/platforms/pseries/hotplug-cpu.c

[PATCH v5 4/4] pseries: Serialize cpu hotplug operations during deactivate Vs deallocate

this serialization, which is a no-op on all architectures except PPC_PSERIES Signed-off-by: Gautham R Shenoy e...@in.ibm.com --- arch/powerpc/platforms/pseries/dlpar.c | 28 +++- drivers/base/cpu.c |2 ++ include/linux/cpu.h| 13

Re: [PATCH v5 0/4] pseries: Add cede support for cpu-offline

On Fri, Oct 30, 2009 at 10:52:43AM +0530, Gautham R Shenoy wrote: Hi, Hi Peter, Did you get a chance to look at this new design ? This is version 5 of patch series that provides a framework to choose the state a pseries CPU must be put to when it is offlined. Previous versions can

[PATCH 1/2] pseries: Don't panic when H_PROD fails during cpu-online.

CPU_UP_CANCELED. Signed-off-by: Gautham R Shenoy e...@in.ibm.com --- arch/powerpc/platforms/pseries/smp.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/platforms/pseries/smp.c b/arch/powerpc/platforms/pseries/smp.c index 8868c01..b488663 100644 --- a/arch/powerpc

[PATCH 2/2] pseries: Make declarations of cpu_hotplug_driver_lock() ANSI compatible.

And add the __acquires() and __releases() annotations, while at it. Signed-off-by: Gautham R Shenoy e...@in.ibm.com --- arch/powerpc/platforms/pseries/dlpar.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/platforms/pseries/dlpar.c b/arch/powerpc

Re: [PATCH] powerpc/cpufreq: Add pr_warn() on OPAL firmware failures

the default loglevel corresponds to KERN_WARNING so this warning should get printed anyway. However, don't you think it would be better if we make it a pr_err( ) since it's a platform error that's causing the pstate_id to go out of bounds ? Otherwise it looks ok. Acked-by: Gautham R. Shenoy e

[PATCH 1/4] powernv:cpufreq: Create pstate_id_to_freq() helper

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Create a helper routine that can return the cpu-frequency for the corresponding pstate_id. Also, cache the values of the pstate_max, pstate_min and pstate_nominal and nr_pstates in a static structure so that they can be reused in the future

[PATCH 2/4] powernv:cpufreq: Export nominal frequency via sysfs.

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Create a driver attribute named cpuinfo_nominal_frequency which creates a sysfs read-only file named cpuinfo_nominal_frequency. Export the frequency corresponding to the nominal_pstate through this interface. Signed-off-by: Gautham R. Shenoy e

[PATCH 0/4] powernv:cpufreq: Export nominal and current frequency via sysfs

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Hi, On IBM POWERNV platforms[1], presently we do not report the current operating frequency of the processor through the sysfs interface cpuinfo_cur_freq since the cpu frequency driver[1] has not implemented the -get(unsigned int cpu) method. Fix

[PATCH 4/4] powernv:cpufreq: Implement the driver-get() method

From: Gautham R. Shenoy e...@linux.vnet.ibm.com The current frequency of a cpu is reported through the sysfs file cpuinfo_cur_freq. This requires the driver to implement a -get(unsigned int cpu) method which will return the current operating frequency. Implement a function named

Re: [PATCH v2 14/52] powerpc, sysfs: Fix CPU hotplug callback registration

Hello Ben, On Fri, Mar 07, 2014 at 01:57:31PM +1100, Benjamin Herrenschmidt wrote: On Fri, 2014-02-14 at 13:22 +0530, Srivatsa S. Bhat wrote: Subsystems that want to register CPU hotplug callbacks, as well as perform initialization for the CPUs that are already online, often do it as shown

[PATCH v2 0/6] powernv:cpufreq: Dynamic cpu-frequency scaling

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Hi, This is the v2 of the consolidated patchset consisting patches for enabling cpufreq on IBM POWERNV platforms along with some enhancements. The v1 of these patches have been previously submitted on linuxppc-dev [1][2]. - This patchset

[PATCH v2 5/6] powernv:cpufreq: Export nominal frequency via sysfs.

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Create a driver attribute named cpuinfo_nominal_freq which creates a sysfs read-only file named cpuinfo_nominal_freq. Export the frequency corresponding to the nominal_pstate through this interface. Nominal frequency is the highest non-turbo

[PATCH v2 1/6] powernv: cpufreq driver for powernv platform

-by: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com Signed-off-by: Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com Signed-off-by: Anton Blanchard an...@samba.org Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- arch/powerpc/include/asm/reg.h | 4 + arch/powerpc/platforms

[PATCH v2 3/6] powernv, cpufreq:Add per-core locking to serialize frequency transitions

-by: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- drivers/cpufreq/powernv-cpufreq.c | 21 - 1 file changed, 16 insertions(+), 5 deletions(-) diff --git a/drivers/cpufreq/powernv-cpufreq.c b/drivers/cpufreq/powernv-cpufreq.c

[PATCH v2 2/6] powernv:cpufreq: Create a powernv_cpu_to_core_mask() helper.

function!) Signed-off-by: Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- drivers/cpufreq/powernv-cpufreq.c | 24 ++-- 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/drivers/cpufreq/powernv-cpufreq.c b

[PATCH v2 6/6] powernv:cpufreq: Implement the driver-get() method

From: Gautham R. Shenoy e...@linux.vnet.ibm.com The current frequency of a cpu is reported through the sysfs file cpuinfo_cur_freq. This requires the driver to implement a -get(unsigned int cpu) method which will return the current operating frequency. Implement a function named

[PATCH v2 4/6] powernv:cpufreq: Create pstate_id_to_freq() helper

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Create a helper routine that can return the cpu-frequency for the corresponding pstate_id. Also, cache the values of the pstate_max, pstate_min and pstate_nominal and nr_pstates in a static structure so that they can be reused in the future

[RFC PATCH 1/2] powerpc: powernv: Framework to show the correct clock in /proc/cpuinfo

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Currently, the code in setup-common.c for powerpc assumes that all clock rates are same in a smp system. This value is cached in the variable named ppc_proc_freq and is the value that is reported in /proc/cpuinfo. However on the PowerNV platform

[RFC PATCH 2/2] powerpc: powernv: Implement ppc_md.get_proc_freq()

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Implement a method named pnv_get_proc_freq(unsigned int cpu) which returns the current clock rate on the 'cpu' in Hz to be reported in /proc/cpuinfo. This method uses the value reported by cpufreq when such a value is sane. Otherwise it falls back

[RFC PATCH 0/2] powernv: Show the correct clock value in /proc/cpuinfo

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Hi, Currently, the code in setup-common.c for powerpc assumes that all clock rates are same in a smp system. This value is cached in the variable named ppc_proc_freq and is the value that is reported in /proc/cpuinfo. However on the PowerNV

Re: [PATCH v2 2/6] powernv:cpufreq: Create a powernv_cpu_to_core_mask() helper.

On Wed, Mar 19, 2014 at 12:05:20PM +0530, Srivatsa S. Bhat wrote: 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 srivatsa.b...@linux.vnet.ibm.com Create a helper method that computes the cpumask

[PATCH v3 0/5] powernv: Enable Dynamic Frequency

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Hi, This is the v3 of the consolidated patchset consisting patches for enabling cpufreq on IBM POWERNV platforms along with some enhancements. I have incorporated the review for v2 (which can be found at [1]). - This patchset contains code

[PATCH v3 1/5] powernv: cpufreq driver for powernv platform

] Signed-off-by: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com Signed-off-by: Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com Signed-off-by: Anton Blanchard an...@samba.org Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- arch/powerpc/include/asm/reg.h | 4 + arch

[PATCH v3 2/5] powernv, cpufreq:Add per-core locking to serialize frequency transitions

-by: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- drivers/cpufreq/powernv-cpufreq.c | 21 - 1 file changed, 16 insertions(+), 5 deletions(-) diff --git a/drivers/cpufreq/powernv-cpufreq.c b/drivers/cpufreq/powernv-cpufreq.c

[PATCH v3 3/5] powernv:cpufreq: Create pstate_id_to_freq() helper

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Create a helper routine that can return the cpu-frequency for the corresponding pstate_id. Also, cache the values of the pstate_max, pstate_min and pstate_nominal and nr_pstates in a static structure so that they can be reused in the future

[PATCH v3 4/5] powernv:cpufreq: Export nominal frequency via sysfs.

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Create a driver attribute named cpuinfo_nominal_freq which creates a sysfs read-only file named cpuinfo_nominal_freq. Export the frequency corresponding to the nominal_pstate through this interface. Nominal frequency is the highest non-turbo

[PATCH v3 5/5] powernv:cpufreq: Implement the driver-get() method

From: Gautham R. Shenoy e...@linux.vnet.ibm.com The current frequency of a cpu is reported through the sysfs file cpuinfo_cur_freq. This requires the driver to implement a -get(unsigned int cpu) method which will return the current operating frequency. Implement a function named

Re: [PATCH v3 2/5] powernv, cpufreq:Add per-core locking to serialize frequency transitions

On Thu, Mar 20, 2014 at 05:40:57PM +0530, Gautham R. Shenoy wrote: From: Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com On POWER systems, the CPU frequency is controlled at a core-level and hence we need to serialize so that only one of the threads in the core switches the core's

Re: [PATCH v3 3/5] powernv:cpufreq: Create pstate_id_to_freq() helper

On Thu, Mar 20, 2014 at 05:40:58PM +0530, Gautham R. Shenoy wrote: From: Gautham R. Shenoy e...@linux.vnet.ibm.com Create a helper routine that can return the cpu-frequency for the corresponding pstate_id. Also, cache the values of the pstate_max, pstate_min and pstate_nominal

Re: [PATCH v3 5/5] powernv:cpufreq: Implement the driver-get() method

On Thu, Mar 20, 2014 at 05:41:00PM +0530, Gautham R. Shenoy wrote: From: Gautham R. Shenoy e...@linux.vnet.ibm.com The current frequency of a cpu is reported through the sysfs file cpuinfo_cur_freq. This requires the driver to implement a -get(unsigned int cpu) method which will return

Re: [PATCH v3 0/5] powernv: Enable Dynamic Frequency

Hi Viresh, On Fri, Mar 21, 2014 at 01:16:03PM +0530, Viresh Kumar wrote: On Thu, Mar 20, 2014 at 5:40 PM, Gautham R. Shenoy e...@linux.vnet.ibm.com wrote: This is the v3 of the consolidated patchset consisting patches for enabling cpufreq on IBM POWERNV platforms along with some

Re: [PATCH v3 4/5] powernv:cpufreq: Export nominal frequency via sysfs.

On Fri, Mar 21, 2014 at 02:17:28PM +0530, Viresh Kumar wrote: +static ssize_t show_cpuinfo_nominal_freq(struct cpufreq_policy *policy, + char *buf) +{ + int nominal_freq; + nominal_freq =

Re: [PATCH v3 1/5] powernv: cpufreq driver for powernv platform

Hi Viresh, On Fri, Mar 21, 2014 at 02:11:32PM +0530, Viresh Kumar wrote: On Thu, Mar 20, 2014 at 5:40 PM, Gautham R. Shenoy e...@linux.vnet.ibm.com wrote: From: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com Hi Vaidy, diff --git a/drivers/cpufreq/Kconfig b/drivers/cpufreq/Kconfig

Re: [PATCH v3 5/5] powernv:cpufreq: Implement the driver-get() method

On Fri, Mar 21, 2014 at 03:01:29PM +0530, Viresh Kumar wrote: On Thu, Mar 20, 2014 at 5:41 PM, Gautham R. Shenoy e...@linux.vnet.ibm.com wrote: From: Gautham R. Shenoy e...@linux.vnet.ibm.com The current frequency of a cpu is reported through the sysfs file cpuinfo_cur_freq

Re: [PATCH v3 5/5] powernv:cpufreq: Implement the driver-get() method

On Fri, Mar 21, 2014 at 12:01:07PM +, David Laight wrote: From: Viresh Kumar On 21 March 2014 16:34, Gautham R Shenoy e...@linux.vnet.ibm.com wrote: Heh! Well, that wasn't the reason why this was sent out as a separate patch, but never mind. Though I don't understand why it would

Re: [PATCH v3 5/5] powernv:cpufreq: Implement the driver-get() method

On Fri, Mar 21, 2014 at 05:15:34PM +0530, Viresh Kumar wrote: On 21 March 2014 16:34, Gautham R Shenoy e...@linux.vnet.ibm.com wrote: Heh! Well, that wasn't the reason why this was sent out as a separate patch, but never mind. Though I don't understand why it would be difficult to review

Re: [PATCH v3 1/5] powernv: cpufreq driver for powernv platform

Hi Viresh, On Fri, Mar 21, 2014 at 04:24:27PM +0530, Viresh Kumar wrote: On 21 March 2014 16:13, Gautham R Shenoy e...@linux.vnet.ibm.com wrote: On Fri, Mar 21, 2014 at 02:11:32PM +0530, Viresh Kumar wrote: + pr_debug(PState id %d freq %d MHz\n, id, freq

Re: [PATCH v3 5/5] powernv:cpufreq: Implement the driver-get() method

On Fri, Mar 21, 2014 at 06:42:50PM +0530, Viresh Kumar wrote: On 21 March 2014 18:34, Gautham R Shenoy e...@linux.vnet.ibm.com wrote: Consider the case when pmspr = 0x00fe; We are interested in extracting the value 'fe'. And ensure that when we store this value into an int, we

Re: [PATCH v3 1/5] powernv: cpufreq driver for powernv platform

On Fri, Mar 21, 2014 at 04:24:27PM +0530, Viresh Kumar wrote: +static int powernv_cpufreq_cpu_init(struct cpufreq_policy *policy) +{ + int base, i; + + base = cpu_first_thread_sibling(policy-cpu); + + for (i = 0; i threads_per_core; i++) +

Re: [PATCH v3 5/5] powernv:cpufreq: Implement the driver-get() method

Hi Ben, On Sat, Mar 22, 2014 at 09:56:30AM +1100, Benjamin Herrenschmidt wrote: On Fri, 2014-03-21 at 16:34 +0530, Gautham R Shenoy wrote: +/* + * Computes the current frequency on this cpu + * and stores the result in *ret_freq. + */ +static void powernv_read_cpu_freq

[PATCH v4] powernv: Dynamic Frequency Scaling Enablement

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Hi, This is the v4 of the patchset to enable Dynamic Frequency Scaling on IBM PowerNV Platforms. I have incorporated the review comments from the previous version (can be found at [1]). In this version, * Multiple patches from the previous

[PATCH v4] powernv, cpufreq: cpufreq driver for powernv platform

pre...@linux.vnet.ibm.com Signed-off-by: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com Signed-off-by: Srivatsa S. Bhat srivatsa.b...@linux.vnet.ibm.com Signed-off-by: Anton Blanchard an...@samba.org Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- arch/powerpc/configs

Re: [PATCH v4] powernv, cpufreq: cpufreq driver for powernv platform

On Thu, Mar 27, 2014 at 12:09:53PM +0530, Viresh Kumar wrote: Cc'ing Rafael. Thanks. It was a lapse on my part by not Cc'ing Rafael in the first place. On 26 March 2014 22:25, Gautham R. Shenoy e...@linux.vnet.ibm.com wrote: From: Vaidyanathan Srinivasan sva...@linux.vnet.ibm.com

Re: [PATCH v4] powernv, cpufreq: cpufreq driver for powernv platform

On Thu, Mar 27, 2014 at 03:29:36PM +0530, Viresh Kumar wrote: On 27 March 2014 15:00, Gautham R Shenoy e...@linux.vnet.ibm.com wrote: As of now, I prefer this patch be based on code that is in the -next tree. I'll get rid of the per-core locking once the serialization patch of the core

[PATCH v5 0/3] powernv,cpufreq: Dynamic Frequency Scaling support

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Hi, This is v5 of the patchset to enable dynamic frequency scaling on IBM PowerNV platforms. This patchset does address all the review comments obtained for v4 (which can be found at [1]). Changes from v4: * Created a separate patch to select

[PATCH v5 2/3] powernv, cpufreq: cpufreq driver for powernv platform

srivatsa.b...@linux.vnet.ibm.com Signed-off-by: Anton Blanchard an...@samba.org Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- arch/powerpc/include/asm/reg.h| 4 + drivers/cpufreq/Kconfig.powerpc | 8 + drivers/cpufreq/Makefile | 1 + drivers/cpufreq/powernv

[PATCH v5 1/3] powernv, cpufreq: Select CPUFreq related Kconfig options for powernv

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Enable CPUFreq for PowerNV. Select performance, powersave, userspace and ondemand governors. Choose ondemand to be the default governor. Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com Signed-off-by: Srivatsa S. Bhat srivatsa.b

[PATCH v5 3/3] powernv, cpufreq: Use cpufreq_frequency_table.driver_data to store pstate ids

From: Gautham R. Shenoy e...@linux.vnet.ibm.com The .driver_data field in the cpufreq_frequency_table was supposed to be private to the drivers. However at some later point, it was being used to indicate if the particular frequency in the table is the BOOST_FREQUENCY. After patches [1] and [2

[PATCH REPOST v5 2/3] powernv, cpufreq: cpufreq driver for powernv platform

srivatsa.b...@linux.vnet.ibm.com Signed-off-by: Anton Blanchard an...@samba.org Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- arch/powerpc/include/asm/reg.h| 4 + drivers/cpufreq/Kconfig.powerpc | 8 + drivers/cpufreq/Makefile | 1 + drivers/cpufreq/powernv

[PATCH REPOST v5 3/3] powernv, cpufreq: Use cpufreq_frequency_table.driver_data to store pstate ids

From: Gautham R. Shenoy e...@linux.vnet.ibm.com The .driver_data field in the cpufreq_frequency_table was supposed to be private to the drivers. However at some later point, it was being used to indicate if the particular frequency in the table is the BOOST_FREQUENCY. After patches [1] and [2

[PATCH REPOST v5 1/3] powernv, cpufreq: Select CPUFreq related Kconfig options for powernv

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Enable CPUFreq for PowerNV. Select performance, powersave, userspace and ondemand governors. Choose ondemand to be the default governor. Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com Signed-off-by: Srivatsa S. Bhat srivatsa.b

[PATCH 3/4] powernv:cpufreq: Create a powernv_cpu_to_core_mask() helper.

From: Gautham R. Shenoy e...@linux.vnet.ibm.com Move the code that computes the cpumask corresponding to the thread-siblings of a cpu to a new method named powernv_cpu_to_core_mask() so that it can be used by other places in the code. Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com

[PATCH] pseries: cpu: Reduce the polling interval in __cpu_up()

R Shenoy e...@in.ibm.com --- arch/powerpc/kernel/smp.c |5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/arch/powerpc/kernel/smp.c b/arch/powerpc/kernel/smp.c index 65484b2..00c13a1 100644 --- a/arch/powerpc/kernel/smp.c +++ b/arch/powerpc/kernel/smp.c @@ -411,9 +411,8

[PATCH 0/3] cpu: idle state framework for offline CPUs.

, Shaohua: new ACPI processor driver to force CPUs idle (URL: http://www.spinics.net/lists/linux-acpi/msg22863.html) Changelog: --- Gautham R Shenoy (3): pSeries: cpu: Cede CPU during a deactivate-offline cpu: Implement cpu-offline-state callbacks for pSeries. cpu: Offline state

[PATCH 2/3] cpu: Implement cpu-offline-state callbacks for pSeries.

rtas_stop_self() or H_CEDE, depending on the preferred_offline_state is implemented in the next patch. Signed-off-by: Gautham R Shenoy e...@in.ibm.com --- arch/powerpc/platforms/pseries/hotplug-cpu.c| 90 +++ arch/powerpc/platforms/pseries/offline_driver.h | 16 2

[PATCH 1/3] cpu: Offline state Framework.

the callbacks when the above mentioned sysfs interfaces are read or written into. Thus the values provided by reading available_offline_states are expected to vary with the architecture. Signed-off-by: Gautham R Shenoy e...@in.ibm.com --- drivers/base/cpu.c | 111

[PATCH 3/3] pSeries: cpu: Cede CPU during a deactivate-offline

Implements the pSeries specific code bits to put the CPU into rtas_stop_self() state or H_CEDE state depending on the preferred_offline_state value for that CPU. Signed-off-by: Gautham R Shenoy e...@in.ibm.com --- arch/powerpc/platforms/pseries/hotplug-cpu.c| 70

Re: [PATCH 0/3] cpu: idle state framework for offline CPUs.

Hi Shaohua, On Thu, Aug 06, 2009 at 09:58:55AM +0800, Shaohua Li wrote: Hi, On Wed, Aug 05, 2009 at 10:25:53PM +0800, Gautham R Shenoy wrote: In this patch-series, we propose to extend the CPU-Hotplug infrastructure and allow the system administrator to choose the desired state the CPU

[PATCH] book3s_hv_rmhandlers:Pass the correct trap argument to kvmhv_commence_exit

as r12 contains the trap number Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- arch/powerpc/kvm/book3s_hv_rmhandlers.S | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/kvm/book3s_hv_rmhandlers.S b/arch/powerpc/kvm/book3s_hv_rmhandlers.S index 4d70df2..f0d7c54 100644

Re: [PATCH] book3s_hv_rmhandlers:Pass the correct trap argument to kvmhv_commence_exit

Hi Sam, On Fri, Aug 14, 2015 at 03:07:28PM +1000, Sam Bobroff wrote: On Thu, May 21, 2015 at 01:57:04PM +0530, Gautham R. Shenoy wrote: In guest_exit_cont we call kvmhv_commence_exit which expects the trap number as the argument. However r3 doesn't contain the trap number at this point

Re: powerpc: Add an inline function to update HID0

Hi Michael, On Tue, Aug 04, 2015 at 08:08:58PM +1000, Michael Ellerman wrote: On Tue, 2015-04-08 at 08:30:58 UTC, Gautham R. Shenoy wrote: Section 3.7 of Version 1.2 of the Power8 Processor User's Manual prescribes that updates to HID0 be preceded by a SYNC instruction and followed

[PATCH v2] powerpc: Add an inline function to update HID0

. Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- [v1-- v2: Moved defn of update_hid0 to reg.h from kvm_ppc.h] arch/powerpc/include/asm/reg.h | 13 + arch/powerpc/platforms/powernv/subcore.c | 4 ++-- 2 files changed, 15 insertions(+), 2 deletions(-) diff --git

[PATCH] powerpc: Add an inline function to update HID0

-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- arch/powerpc/include/asm/kvm_ppc.h | 11 +++ arch/powerpc/platforms/powernv/subcore.c | 4 ++-- 2 files changed, 13 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/include/asm/kvm_ppc.h b/arch/powerpc/include/asm

Re: powerpc: Add an inline function to update HID0

Hi Segher, Thanks for the suggestions. I will rename the function to update_power8_hid0() and use asm volatile. On Tue, Aug 04, 2015 at 09:30:57PM -0500, Segher Boessenkool wrote: On Tue, Aug 04, 2015 at 08:08:58PM +1000, Michael Ellerman wrote: +static inline void update_hid0(unsigned long

[PATCH v3] powerpc: Add an inline function to update POWER8 HID0

path. Signed-off-by: Gautham R. Shenoy e...@linux.vnet.ibm.com --- [v1 -- v2: Moved defn of update_hid0 to reg.h from kvm_ppc.h] [v2 -- v3: Renamed to update_power8_hid0 and used asm volatile] arch/powerpc/include/asm/reg.h | 9 + arch/powerpc/platforms/powernv/subcore.c | 4

[PATCH] book3s_hv: Handle H_DOORBELL on the guest exit path

can unconditionally clear the doorbell message in kvmppc_check_wake_reason. Signed-off-by: Gautham R. Shenoy <e...@linux.vnet.ibm.com> --- arch/powerpc/kvm/book3s_hv_rmhandlers.S | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/arch/power

Re: [PATCH RESEND v4 3/4] cpufreq: powernv: Add a trace print for the throttle event

Hi Shilpa, Just saw this resend! On Tue, Jan 12, 2016 at 04:24:26AM -0600, Shilpasri G Bhat wrote: > Record the throttle event with a trace print replacing the printk, > except for events like throttling below nominal and occ reset > event which print a warning message. > > Signed-off-by:

Re: [PATCH RESEND v4 4/4] cpufreq: powernv: Add sysfs attributes to show throttle stats

Hi Shilpa, On Tue, Jan 12, 2016 at 04:24:27AM -0600, Shilpasri G Bhat wrote: > +static inline int get_chip_index(struct kobject *kobj) Probably have "get_chip_index(int id)". See the reason below. > +{ > + int i, id; > + > + i = kstrtoint(kobj->name + 4, 0, ); > + if (i) > +

[PATCH 2/2] workqueue:Fix affinity of an unbound worker of a node with 1 online CPU

Cc: Tejun Heo <hte...@gmail.com> Cc: Michael Ellerman <m...@ellerman.id.au> Signed-off-by: Gautham R. Shenoy <e...@linux.vnet.ibm.com> --- kernel/workqueue.c | 19 +++ 1 file changed, 15 insertions(+), 4 deletions(-) diff --git a/kernel/workqueue.c b/kernel/workque

[PATCH 1/2] workqueue: Move wq_update_unbound_numa() to the beginning of CPU_ONLINE

Thomas Gleixner <t...@linutronix.de> Cc: Tejun Heo <hte...@gmail.com> Cc: Michael Ellerman <m...@ellerman.id.au> Signed-off-by: Gautham R. Shenoy <e...@linux.vnet.ibm.com> --- kernel/workqueue.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git

[PATCH 0/2] Fix CPU Online handling for unbounded worker threads

a00800 388a0998 7f63db78 484abfa1 6000 2fa3 409eff1c 813f0378 2f890001 419eff10 <0fe0> 4b08 6000 6000 ---[ end trace cbc1c5cfbc9591d0 ]--- The patches are based on 4.7-rc2. I have tested the patches on a multi-node x86_64 and a ppc64 Gautham R. Shenoy (2): wor

Re: [PATCH 0/2] Fix CPU Online handling for unbounded worker threads

Hi Peter, Thomas, On Tue, Jun 07, 2016 at 08:44:01PM +0530, Gautham R. Shenoy wrote: > Hi, > > This patchset fixes a couple of issues in the CPU_ONLINE notification > handling for the workqueues with respect to unbounded worker threads. Any thoughts on these patches ? They fix

Re: [PATCH v4 08/10] powerpc/powernv: Add platform support for stop instruction

> Bits 60:63 - Requested Level > Used to specify which power-saving level must be entered on executing > stop instruction > > This patch adds support for stop instruction and PSSCR handling. This version looks good to me.

Re: [PATCH 2/2] workqueue:Fix affinity of an unbound worker of a node with 1 online CPU

Hi Peter, On Tue, Jun 14, 2016 at 01:22:34PM +0200, Peter Zijlstra wrote: > On Tue, Jun 07, 2016 at 08:44:03PM +0530, Gautham R. Shenoy wrote: > > I'm still puzzled why we don't see this on x86. Afaict there's nothing > PPC specific about this. You are right. On PPC, at boot

Re: [PATCH 2/2] workqueue:Fix affinity of an unbound worker of a node with 1 online CPU

On Wed, Jun 15, 2016 at 01:32:49PM +0200, Peter Zijlstra wrote: > On Wed, Jun 15, 2016 at 03:49:36PM +0530, Gautham R Shenoy wrote: > > > Also, with the first patch in the series (which ensures that > > restore_unbound_workers are called *after* the new workers for the >

Re: [PATCH 1/2] workqueue: Move wq_update_unbound_numa() to the beginning of CPU_ONLINE

On Tue, Jun 21, 2016 at 09:47:19PM +0200, Peter Zijlstra wrote: > On Tue, Jun 21, 2016 at 03:43:56PM -0400, Tejun Heo wrote: > > On Tue, Jun 21, 2016 at 09:37:09PM +0200, Peter Zijlstra wrote: > > > Hurm.. So I've applied it, just to get this issue sorted, but I'm not > > > entirely sure I like

Re: [PATCH 1/2] workqueue: Move wq_update_unbound_numa() to the beginning of CPU_ONLINE

Hello Tejun, On Wed, Jun 15, 2016 at 11:53:50AM -0400, Tejun Heo wrote: > Hello, > > On Tue, Jun 07, 2016 at 08:44:02PM +0530, Gautham R. Shenoy wrote: > > Currently in the CPU_ONLINE workqueue handler, the > > restore_unbound_workers_cpumask() will never call >

Re: [PATCH 1/2] workqueue: Move wq_update_unbound_numa() to the beginning of CPU_ONLINE

d mask and ends up overriding it to cpu_possible_mask. > > CPU_ONLINE callbacks should be able to put kthreads on the CPU which > is coming online. Update select_fallback_rq() so that it follows > cpu_online() rather than cpu_active() for kthreads. > > Signed-off-by: Tejun Heo &

Re: [PATCH v7 6/6] cpufreq: powernv: Add sysfs attributes to show throttle stats

Hi Shilpa, A minor nit. On Thu, Jan 28, 2016 at 12:55:41PM +0530, Shilpasri G Bhat wrote: [..snip..] > + > +What: > /sys/devices/system/cpu/cpufreq/chip*/throttle_reasons/ > +Date:Jan 2016 > +Contact: Linux kernel mailing list >

Re: [PATCH v7 1/6] cpufreq: powernv: Free 'chips' on module exit

On Thu, Jan 28, 2016 at 12:55:36PM +0530, Shilpasri G Bhat wrote: > This will free the dynamically allocated memory of'chips' on > module exit. > > Signed-off-by: Shilpasri G Bhat <shilpa.b...@linux.vnet.ibm.com> Reviewed-by: Gautham R. Shenoy <e...@linux.vnet.ibm.com>

Re: [PATCH v7 3/6] cpufreq: powernv: Remove cpu_to_chip_id() from hot-path

the chip ids for all cores in the array 'core_to_chip_map' and use it > in the hotpath. > > Reported-by: Anton Blanchard <an...@samba.org> > Signed-off-by: Shilpasri G Bhat <shilpa.b...@linux.vnet.ibm.com> Reviewed-by: Gautham R. Shenoy <e...@linux.vnet.

Re: [PATCH v7 5/6] cpufreq: powernv: Replace pr_info with trace print for throttle event

shilpa.b...@linux.vnet.ibm.com> Reviewed-by: Gautham R. Shenoy <e...@linux.vnet.ibm.com> -- Thanks and Regards gautham. ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH v8 6/6] cpufreq: powernv: Add sysfs attributes to show throttle stats

Hi Viresh, > > What I can suggest is: > - Move this directory inside cpuX/cpufreq/ directory, in a similar way > as to how we create 'stats' directory today. > - You can then get policy->cpu, to get chip->id out of it. > - The only disadvantage here is that the same chip directory will be >

Re: [PATCH v5 5/5] cpufreq: powernv: Add sysfs attributes to show throttle stats

On Thu, Jan 21, 2016 at 03:08:59PM +0530, Shilpasri G Bhat wrote: > Signed-off-by: Shilpasri G Bhat <shilpa.b...@linux.vnet.ibm.com> Reviewed-by: Gautham R. Shenoy <e...@linux.vnet.ibm.com> -- Thanks and Regards gautham. ___ Linuxppc-d

Re: [PATCH v6 2/5] cpufreq: powernv: Remove cpu_to_chip_id() from hot-path

...@samba.org> > > Signed-off-by: Shilpasri G Bhat <shilpa.b...@linux.vnet.ibm.com> > > Reviewed-by: Gautham R. Shenoy <e...@linux.vnet.ibm.com> > > snip > > Does the core_to_chip_map need to be updated/refreshed on/after/ a > cpu (core) hotplug? I presume

Re: [PATCH v2 2/9] powerpc/kvm: make hypervisor state restore a function

Hi Shreyas, On Tue, May 03, 2016 at 01:54:31PM +0530, Shreyas B. Prabhu wrote: > In the current code, when the thread wakes up in reset vector, some > of the state restore code and check for whether a thread needs to > branch to kvm is duplicated. Reorder the code such that this > duplication is

Re: [PATCH v2 6/9] powerpc/powernv: set power_save func after the idle states are initialized

reyas B. Prabhy <shre...@linux.vnet.ibm.com> Reviewed-by: Gautham R. Shenoy <e...@linux.vnet.ibm.com> > --- > arch/powerpc/platforms/powernv/idle.c | 3 +++ > arch/powerpc/platforms/powernv/setup.c | 2 +- > 2 files changed, 4 insertions(+), 1 deletion(-) > > diff --git

  1   2   3   4   5   6   7   >