[PATCH V5 8/8] cpuidle/powernv: Parse device tree to setup idle states

2014-01-15 Thread Preeti U Murthy
Add deep idle states such as nap and fast sleep to the cpuidle state table only if they are discovered from the device tree during cpuidle initialization. Signed-off-by: Preeti U Murthy --- drivers/cpuidle/cpuidle-powernv.c | 81 + 1 file changed, 64

[PATCH V5 6/8] time/cpuidle: Support in tick broadcast framework in the absence of external clock device

2014-01-15 Thread Preeti U Murthy
broadcast mask. This newly nominated bc_cpu is woken up by an IPI so as to queue the above mentioned hrtimer on it. Signed-off-by: Preeti U Murthy --- include/linux/clockchips.h |4 - kernel/time/clockevents.c|9 +- kernel/time/tick-broadcast.c | 192

[PATCH V5 7/8] cpuidle/powernv: Add "Fast-Sleep" CPU idle state

2014-01-15 Thread Preeti U Murthy
support for fast sleep, enable it in the cpuidle framework on PowerNV. Signed-off-by: Preeti U Murthy --- arch/powerpc/Kconfig |2 ++ arch/powerpc/kernel/time.c|2 +- drivers/cpuidle/cpuidle-powernv.c | 39 + 3 files changed, 42

[PATCH V5 5/8] powermgt: Add OPAL call to resync timebase on wakeup

2014-01-15 Thread Preeti U Murthy
ned-off-by: Vaidyanathan Srinivasan Signed-off-by: Preeti U. Murthy --- arch/powerpc/include/asm/opal.h|2 ++ arch/powerpc/kernel/exceptions-64s.S |2 +- arch/powerpc/kernel/idle_power7.S | 27 arch/powerpc/platforms/po

[PATCH V5 4/8] powernv/cpuidle: Add context management for Fast Sleep

2014-01-15 Thread Preeti U Murthy
-by: Vaidyanathan Srinivasan [Changelog modified by Preeti U. Murthy ] Signed-off-by: Preeti U. Murthy --- arch/powerpc/include/asm/processor.h |1 + arch/powerpc/kernel/exceptions-64s.S | 10 - arch/powerpc/kernel/idle_power7.S| 63 -- 3 files

[PATCH V5 3/8] cpuidle/ppc: Split timer_interrupt() into timer handling and interrupt handling routines

2014-01-15 Thread Preeti U Murthy
timers to directly call into __timer_interupt(). One of the use cases of this is the tick broadcast IPI handling in which the sleeping CPUs need to handle the local timers that have expired. Signed-off-by: Preeti U Murthy --- arch/powerpc/kernel/time.c | 73

[PATCH V5 2/8] powerpc: Implement tick broadcast IPI as a fixed IPI message

2014-01-15 Thread Preeti U Murthy
sa S. Bhat [Functions renamed to tick_broadcast* and Changelog modified by Preeti U. Murthy] Signed-off-by: Preeti U. Murthy Acked-by: Geoff Levand [For the PS3 part] --- arch/powerpc/include/asm/smp.h |2 +- arch/powerpc/include/asm/time.h |1 + arch/powerpc/ke

[PATCH V5 1/8] powerpc: Free up the slot of PPC_MSG_CALL_FUNC_SINGLE IPI message

2014-01-15 Thread Preeti U Murthy
slots are available). So, implement the functionality of PPC_MSG_CALL_FUNC_SINGLE using PPC_MSG_CALL_FUNC itself and release its IPI message slot, so that it can be used for something else in the future, if desired. Signed-off-by: Srivatsa S. Bhat Signed-off-by: Preeti U. Murthy Acked-by: Geoff

[PATCH V5 0/8] cpuidle/ppc: Enable deep idle states on PowerNV

2014-01-15 Thread Preeti U Murthy
ick a broadcast CPU, instead of having a dedicated one. 2. Remove the constraint of having to disable tickless idle on the broadcast CPU by queueing a hrtimer dedicated to do broadcast. V1 posting: https://lkml.org/lkml/2013/7/25/740. 1. Added the infrastructure to wakeup CPUs in deep idle st

Re: Disable sleep states on P7+

2014-01-14 Thread Preeti U Murthy
nel/powersave_nap, in which case the cpu enters low thread priority. Thanks Regards Preeti U Murthy > > ___ > Linuxppc-dev mailing list > Linuxppc-dev@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/linuxppc-dev > __

Re: [PATCH] cpuidle/menu: Fail cpuidle_idle_call() if no idle state is acceptable

2014-01-14 Thread Preeti U Murthy
On 01/14/2014 01:07 PM, Srivatsa S. Bhat wrote: > On 01/14/2014 12:30 PM, Srivatsa S. Bhat wrote: >> On 01/14/2014 11:35 AM, Preeti U Murthy wrote: >>> On PowerPC, in a particular test scenario, all the cpu idle states were >>> disabled. >>> Inspite of this

Re: [PATCH] cpuidle/menu: Fail cpuidle_idle_call() if no idle state is acceptable

2014-01-14 Thread Preeti U Murthy
Hi Srivatsa, On 01/14/2014 12:30 PM, Srivatsa S. Bhat wrote: > On 01/14/2014 11:35 AM, Preeti U Murthy wrote: >> On PowerPC, in a particular test scenario, all the cpu idle states were >> disabled. >> Inspite of this it was observed that the idle state count of the sha

[PATCH] cpuidle/menu: Fail cpuidle_idle_call() if no idle state is acceptable

2014-01-13 Thread Preeti U Murthy
menu governor criteria to be chosen as the next idle state. This patch adds the code to indicate that a valid cpu idle state could not be chosen by the menu governor and reports back to arch so that it can take some default action. Signed-off-by: Preeti U Murthy --- drivers/cpuidle/cpuidle.c

Re: [PATCH] pseries/cpuidle: Remove redundant call to ppc64_runlatch_off() in cpu idle routines

2014-01-12 Thread Preeti U Murthy
Hi Mikey I have the patch with the changelog according to your suggestion below. Thanks On 01/13/2014 11:08 AM, Michael Ellerman wrote: > On Thu, 2014-01-09 at 10:35 +0530, Preeti U Murthy wrote: >> Commit fbd7740fdfdf9475f switched pseries cpu idle handling from complete >> i

Re: [PATCH] pseries/cpuidle: Remove redundant call to ppc64_runlatch_off() in cpu idle routines

2014-01-12 Thread Preeti U Murthy
Hi Deepthi, On 01/13/2014 09:27 AM, Deepthi Dharwar wrote: > On 01/09/2014 10:35 AM, Preeti U Murthy wrote: >> Commit fbd7740fdfdf9475f switched pseries cpu idle handling from complete >> idle >> loops to ppc_md.powersave functions. Earlier to this switch, >> ppc

[PATCH] pseries/cpuidle: Remove redundant call to ppc64_runlatch_off() in cpu idle routines

2014-01-08 Thread Preeti U Murthy
were carried over subsequent enhancements to the pseries cpuidle routines. This patch takes care of eliminating this redundancy. Signed-off-by: Preeti U Murthy --- arch/powerpc/platforms/pseries/processor_idle.c |3 --- 1 file changed, 3 deletions(-) diff --git a/arch/powerpc/platforms

[PATCH V2] time/cpuidle: Support in tick broadcast framework for archs without external clock device

2013-12-31 Thread Preeti U Murthy
mode to periodic. Signed-off-by: Preeti U Murthy --- include/linux/clockchips.h |4 - kernel/time/clockevents.c|8 +- kernel/time/tick-broadcast.c | 180 ++ kernel/time/tick-internal.h |8 +- 4 files changed, 173 insertions(+), 27

Re: [RFC PATCH] time: Support in tick broadcast framework for archs without an external wakeup source

2013-12-15 Thread Preeti U Murthy
Hi, The patch had some compile time fixes to be done. It was accidentally mailed out before doing so. Below is the right patch. Apologies for the same. Thanks Regards Preeti U Murthy - time: Support in tick broadcast

Re: [RFC PATCH] time: Support in tick broadcast framework for archs without an external wakeup source

2013-12-12 Thread Preeti U Murthy
Hi Ben, On 12/13/2013 10:47 AM, Benjamin Herrenschmidt wrote: > On Fri, 2013-12-13 at 09:49 +0530, Preeti U Murthy wrote: >> On some architectures, in certain CPU deep idle states the local timers stop. >> An external clock device is used to wakeup these CPUs. The kernel supp

[RFC PATCH] time: Support in tick broadcast framework for archs without an external wakeup source

2013-12-12 Thread Preeti U Murthy
bc_cpu is woken up by an IPI so as to queue the above mentioned hrtimer on itself. This patch is compile tested only. Signed-off-by: Preeti U Murthy --- include/linux/clockchips.h |4 + kernel/time/clockevents.c|8 +- kernel/time/tick-broadcast.c | 157

Re: [PATCH V4 6/9] cpuidle/ppc: Add basic infrastructure to enable the broadcast framework on ppc

2013-12-02 Thread Preeti U Murthy
Hi Thomas, On 11/29/2013 05:28 PM, Thomas Gleixner wrote: > On Fri, 29 Nov 2013, Preeti U Murthy wrote: >> diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kconfig >> index b44b52c..cafa788 100644 >> --- a/arch/powerpc/Kconfig >> +++ b/arch/powerpc/Kconfig >&g

Re: [PATCH V4 7/9] cpuidle/powernv: Add "Fast-Sleep" CPU idle state

2013-12-02 Thread Preeti U Murthy
Hi Thomas, On 11/29/2013 08:09 PM, Thomas Gleixner wrote: > On Fri, 29 Nov 2013, Preeti U Murthy wrote: >> +static enum hrtimer_restart handle_broadcast(struct hrtimer *hrtimer) >> +{ >> +struct clock_event_device *bc_evt = &bc_timer; >> +ktim

Re: 3.13 Oops on ppc64_cpu --smt=off

2013-12-02 Thread Preeti U Murthy
Hi, On 12/02/2013 03:27 PM, Alexander Graf wrote: > > On 02.12.2013, at 05:01, Preeti U Murthy wrote: > >> Hi, >> >> On 11/30/2013 11:15 PM, Alexander Graf wrote: >>> Hi Ben, >>> >>> With current linus master (3.13-rc2+) I'm facing

Re: 3.13 Oops on ppc64_cpu --smt=off

2013-12-01 Thread Preeti U Murthy
is also pointing to the regression caused by the same commit id. Thanks Regards Preeti U Murthy --- commit 42eb088ed246a5a817bb45a8b32fe234cf1c0f8b Author: Peter Zijlstra Date: Tue Nov 19 16:41:49 2013 +0100 sched: Avoid NULL dereference on sd_busy Commit 37dc6b50cee9 ("

[PATCH V4 9/9] cpuidle/powernv: Parse device tree to setup idle states

2013-11-29 Thread Preeti U Murthy
Add deep idle states such as nap and fast sleep to the cpuidle state table only if they are discovered from the device tree during cpuidle initialization. Signed-off-by: Preeti U. Murthy --- drivers/cpuidle/cpuidle-powerpc-book3s.c | 81 -- 1 file changed, 64

[PATCH V4 8/9] cpuidle/ppc: Nominate new broadcast cpu on hotplug of the old

2013-11-29 Thread Preeti U Murthy
so as to not miss wakeups under such scenarios. Signed-off-by: Preeti U Murthy --- arch/powerpc/include/asm/time.h |1 + arch/powerpc/kernel/time.c |1 + drivers/cpuidle/cpuidle-powerpc-book3s.c | 22 ++ 3 files changed, 24 insertions

[PATCH V4 6/9] cpuidle/ppc: Add basic infrastructure to enable the broadcast framework on ppc

2013-11-29 Thread Preeti U Murthy
ned-off-by: Preeti U Murthy --- arch/powerpc/Kconfig|2 + arch/powerpc/include/asm/time.h |1 + arch/powerpc/kernel/time.c | 58 ++- 3 files changed, 60 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/Kconfig b/arch/powerpc/Kco

[PATCH V4 7/9] cpuidle/powernv: Add "Fast-Sleep" CPU idle state

2013-11-29 Thread Preeti U Murthy
repeats. Protect the region of nomination,de-nomination and check for existence of broadcast CPU with a lock to ensure synchronization between them. Signed-off-by: Preeti U Murthy --- arch/powerpc/include/asm/time.h |1 arch/powerpc/kernel/time.c |2 drivers

[PATCH V4 5/9] powermgt: Add OPAL call to resync timebase on wakeup

2013-11-29 Thread Preeti U Murthy
ned-off-by: Vaidyanathan Srinivasan Signed-off-by: Preeti U. Murthy --- arch/powerpc/include/asm/opal.h|2 ++ arch/powerpc/kernel/exceptions-64s.S |2 +- arch/powerpc/kernel/idle_power7.S | 27 arch/powerpc/platforms/po

[PATCH V4 4/9] powernv/cpuidle: Add context management for Fast Sleep

2013-11-29 Thread Preeti U Murthy
-by: Vaidyanathan Srinivasan [Changelog modified by Preeti U. Murthy ] Signed-off-by: Preeti U. Murthy --- arch/powerpc/include/asm/processor.h |1 + arch/powerpc/kernel/exceptions-64s.S | 10 - arch/powerpc/kernel/idle_power7.S| 63 -- 3 files

[PATCH V4 3/9] cpuidle/ppc: Split timer_interrupt() into timer handling and interrupt handling routines

2013-11-29 Thread Preeti U Murthy
timers to directly call into __timer_interupt(). One of the use cases of this is the tick broadcast IPI handling in which the sleeping CPUs need to handle the local timers that have expired. Signed-off-by: Preeti U Murthy --- arch/powerpc/kernel/time.c | 73

[PATCH V4 2/9] powerpc: Implement tick broadcast IPI as a fixed IPI message

2013-11-29 Thread Preeti U Murthy
sa S. Bhat [Functions renamed to tick_broadcast* and Changelog modified by Preeti U. Murthy] Signed-off-by: Preeti U. Murthy Acked-by: Geoff Levand [For the PS3 part] --- arch/powerpc/include/asm/smp.h |2 +- arch/powerpc/include/asm/time.h |1 + arch/powerpc/ke

[PATCH V4 1/9] powerpc: Free up the slot of PPC_MSG_CALL_FUNC_SINGLE IPI message

2013-11-29 Thread Preeti U Murthy
slots are available). So, implement the functionality of PPC_MSG_CALL_FUNC_SINGLE using PPC_MSG_CALL_FUNC itself and release its IPI message slot, so that it can be used for something else in the future, if desired. Signed-off-by: Srivatsa S. Bhat Signed-off-by: Preeti U. Murthy Acked-by: Geoff

[PATCH V4 0/9] cpuidle/ppc: Enable deep idle states on PowerNV

2013-11-29 Thread Preeti U Murthy
dcast CPU, instead of having a dedicated one. 2. Remove the constraint of having to disable tickless idle on the broadcast CPU by queueing a hrtimer dedicated to do broadcast. V1 posting: https://lkml.org/lkml/2013/7/25/740. 1. Added the infrastructure to wakeup CPUs in deep idle states in which the loc

Re: [PATCH V2 2/2] sched: Remove un-necessary iteration over sched domains to update nr_busy_cpus

2013-10-30 Thread Preeti U Murthy
ed directly when required. >> >> While we are at it, we might as well change the nohz_idle parameter to be >> updated at the sd_busy domain level alone and not the base domain level of a >> CPU. >> This will unify the concept of busy cpus at just one level of sched do

Re: [PATCH V2 2/2] sched: Remove un-necessary iteration over sched domains to update nr_busy_cpus

2013-10-29 Thread Preeti U Murthy
The changelog has missed mentioning the introduction of sd_asym per_cpu sched domain. Apologies for this. The patch with the changelog including mention of sd_asym is pasted below. Regards Preeti U Murthy --- sched: Remove un-necessary iteration over sched domains to update

[PATCH V2 2/2] sched: Remove un-necessary iteration over sched domains to update nr_busy_cpus

2013-10-29 Thread Preeti U Murthy
the sd_busy domain level alone and not the base domain level of a CPU. This will unify the concept of busy cpus at just one level of sched domain where it is currently used. Signed-off-by: Preeti U Murthy --- kernel/sched/core.c |6 ++ kernel/sched/fair.c | 38

[PATCH V2 1/2] sched: Fix asymmetric scheduling for POWER7

2013-10-29 Thread Preeti U Murthy
and() will not yield any set bits if this domain has no idle cpu. Hence, nr_busy check against group weight can be removed. Reported-by: Michael Neuling Signed-off-by: Vaidyanathan Srinivasan Signed-off-by: Preeti U Murthy Tested-by: Michael Neuling --- kernel/sched/fair.c |2 +- 1 file chan

[PATCH V2 0/2] sched: Cleanups,fixes in nohz_kick_needed()

2013-10-29 Thread Preeti U Murthy
domain-sd_busy where it is relevant. 3. Introduce sd_asym to represent the sched domain where asymmetric load balancing has to be done. --- Preeti U Murthy (1): sched: Remove un-necessary iteration over sched domains to update nr_busy_cpus Vaidyanathan Srinivasan (1): sched: Fix

Re: [PATCH 3/3] sched: Aggressive balance in domains whose groups share package resources

2013-10-28 Thread Preeti U Murthy
Hi Peter, On 10/28/2013 09:23 PM, Peter Zijlstra wrote: > On Mon, Oct 21, 2013 at 05:15:02PM +0530, Vaidyanathan Srinivasan wrote: >> From: Preeti U Murthy >> >> The current logic in load balance is such that after picking the >> busiest group, the load is attempted t

Re: [PATCH 1/3] sched: Fix nohz_kick_needed to consider the nr_busy of the parent domain's group

2013-10-28 Thread Preeti U Murthy
Hi Peter, On 10/28/2013 07:20 PM, Peter Zijlstra wrote: > On Thu, Oct 24, 2013 at 01:37:38PM +0530, Preeti U Murthy wrote: >> kernel/sched/core.c |5 + >> kernel/sched/fair.c | 38 -- >> kernel/sched/sched.h |1 + &g

Re: [PATCH 3/3] sched: Aggressive balance in domains whose groups share package resources

2013-10-25 Thread Preeti U Murthy
to_cpumask(group->cpumask))) >> +goto redo_grp; >> + >> goto out_one_pinned; >> } > > Man this retry logic is getting annoying.. isn't there anything saner we > can do?

Re: [PATCH 1/3] sched: Fix nohz_kick_needed to consider the nr_busy of the parent domain's group

2013-10-24 Thread Preeti U Murthy
Hi Vincent, I have addressed your comments and below is the fresh patch. This patch applies on PATCH 2/3 posted in this thread. Regards Preeti U Murthy sched:Remove un-necessary iterations over sched domains to update/query nr_busy_cpus From: Preeti U Murthy nr_busy_cpus parameter is used

Re: [PATCH 3/3] sched: Aggressive balance in domains whose groups share package resources

2013-10-23 Thread Preeti U Murthy
t->lock, >> flags); >> env.flags |= LBF_ALL_PINNED; >> +if (share_pkg_res && >> + cpumask_intersects(cpus, >> +

Re: [PATCH 1/3] sched: Fix nohz_kick_needed to consider the nr_busy of the parent domain's group

2013-10-23 Thread Preeti U Murthy
y does. sd_busy therefore is irrelevant for asymmetric load balancing. Regards Preeti U Murthy START_PATCH--- sched: Fix nohz_kick_needed() --- kernel/sched/core.c |4 kernel/sched/fair.c | 40 ++--

Re: [PATCH 1/3] sched: Fix nohz_kick_needed to consider the nr_busy of the parent domain's group

2013-10-22 Thread Preeti U Murthy
On 10/23/2013 09:30 AM, Preeti U Murthy wrote: > Hi Peter, > > On 10/23/2013 03:41 AM, Peter Zijlstra wrote: >> On Mon, Oct 21, 2013 at 05:14:42PM +0530, Vaidyanathan Srinivasan wrote: >>> kernel/sched/fair.c | 19 +-- >>> 1 file chang

Re: [PATCH 1/3] sched: Fix nohz_kick_needed to consider the nr_busy of the parent domain's group

2013-10-22 Thread Preeti U Murthy
d, that is at nohz_kick_needed(), is done to know the total number of busy cpus at a sched domain level which has the SD_SHARE_PKG_RESOURCES set and not at a sched group level. So why not move nr_busy to struct sched_domain and having the below patch which just updates this parameter for the sched do

Re: [PATCH 1/3] sched: Fix nohz_kick_needed to consider the nr_busy of the parent domain's group

2013-10-22 Thread Preeti U Murthy
to go idle will trigger load balancing without any pre-conditions. In a single socket machine, there will be a CPU domain encompassing the socket and the MC domain will encompass a core. nohz_idle load balancer will kick in if both the threads in the core have tasks running on them. This is fair enough because the threads share the resources of the core. Regards Preeti U Murthy > > Thanks, > Kamalesh. > ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

[PATCH V3 5/6] cpuidle/ppc: Introduce the deep idle state in which the local timers stop

2013-09-10 Thread Preeti U Murthy
above cycle repeats. Protect the region of nomination,de-nomination and check for existence of broadcast cpu with a lock to ensure synchronization between them. [1] tick_handle_oneshot_broadcast() or tick_handle_periodic_broadcast(). Signed-off-by: Preeti U Murthy --- arch/powerpc/include/asm/t

[PATCH V3 6/6] cpuidle/ppc: Nominate new broadcast cpu on hotplug of the old

2013-09-10 Thread Preeti U Murthy
was about to fire on it. Therefore the newly nominated broadcast cpu should set the broadcast hrtimer on itself to expire immediately so as to not miss wakeups under such scenarios. Signed-off-by: Preeti U Murthy --- arch/powerpc/include/asm/time.h |1 + arch/powerpc/kernel/time.c

[PATCH V3 4/6] cpuidle/ppc: Add basic infrastructure to support the broadcast framework on ppc

2013-09-10 Thread Preeti U Murthy
being woken up from the broadcast ipi, set the decrementers_next_tb to now before calling __timer_interrupt(). Signed-off-by: Preeti U Murthy --- arch/powerpc/Kconfig|1 + arch/powerpc/include/asm/time.h |1 + arch/powerpc/kernel/time.c | 69 +++

[PATCH V3 2/6] powerpc: Implement broadcast timer interrupt as an IPI message

2013-09-10 Thread Preeti U Murthy
sa S. Bhat [Changelog modified by pre...@linux.vnet.ibm.com] Signed-off-by: Preeti U Murthy --- arch/powerpc/include/asm/smp.h |3 ++- arch/powerpc/include/asm/time.h |1 + arch/powerpc/kernel/smp.c | 19 +++ arch/powerpc/kernel/time.c

[PATCH V3 3/6] cpuidle/ppc: Split timer_interrupt() into timer handling and interrupt handling routines

2013-09-10 Thread Preeti U Murthy
routines performed during regular interrupt handling and __timer_interrupt(), which takes care of running local timers and collecting time related stats. Now on a broadcast ipi, call __timer_interrupt(). Signed-off-by: Preeti U Murthy --- arch/powerpc/kernel/time.c | 69

[PATCH V3 0/6] cpuidle/ppc: Enable broadcast support for deep idle states

2013-09-10 Thread Preeti U Murthy
vatsa S. Bhat and Vaidyanathan Srinivasan for all their comments and suggestions so far. --- Preeti U Murthy (4): cpuidle/ppc: Split timer_interrupt() into timer handling and interrupt handling routines cpuidle/ppc: Add basic infrastructure to support the broadcast framework on ppc c

[PATCH V3 1/6] powerpc: Free up the IPI message slot of ipi call function (PPC_MSG_CALL_FUNC)

2013-09-10 Thread Preeti U Murthy
available). So, implement the functionality of PPC_MSG_CALL_FUNC using PPC_MSG_CALL_FUNC_SINGLE itself and release its IPI message slot, so that it can be used for something else in the future, if desired. Signed-off-by: Srivatsa S. Bhat Signed-off-by: Preeti U Murthy --- arch/powerpc/include

Re: [RFC V2 PATCH 4/6] cpuidle/ppc: Add longnap state to the idle states on powernv

2013-08-22 Thread Preeti U Murthy
Hi Ben, On 08/22/2013 08:58 AM, Benjamin Herrenschmidt wrote: > On Wed, 2013-08-14 at 17:26 +0530, Preeti U Murthy wrote: >> This patch hooks into the existing broadcast framework along with the support >> that this patchset introduces for ppc, and the cpuidle driver backend >&g

Re: [RFC V2 PATCH 3/6] cpuidle/ppc: Add timer offload framework to support deep idle states

2013-08-22 Thread Preeti U Murthy
Hi Ben, On 08/22/2013 08:57 AM, Benjamin Herrenschmidt wrote: > On Wed, 2013-08-14 at 17:26 +0530, Preeti U Murthy wrote: > >> static irqreturn_t timer_action(int irq, void *data) >> { >> -timer_interrupt(); >> +decrementer_timer_interrupt(); >>

Re: [RFC V2 PATCH 2/6] powerpc: Implement broadcast timer interrupt as an IPI message

2013-08-21 Thread Preeti U Murthy
Hi Ben On 08/22/2013 08:40 AM, Benjamin Herrenschmidt wrote: > On Wed, 2013-08-14 at 17:26 +0530, Preeti U Murthy wrote: >> -static irqreturn_t unused_action(int irq, void *data) >> +static irqreturn_t timer_action(int irq, void *data) >> { >> - /* This slot is

[RFC V2 PATCH 6/6] cpuidle/ppc : Queue a hrtimer on bc_cpu, explicitly to do broadcast handling

2013-08-14 Thread Preeti U Murthy
just wakeup, the new broadcast CPU has to restart the hrtimer on itself so as to continue broadcast handling. Signed-off-by: Preeti U Murthy --- arch/powerpc/include/asm/time.h |5 ++ arch/powerpc/kernel/time.c | 47 --- arch/powerpc

[RFC V2 PATCH 5/6] cpuidle/ppc: Enable dynamic movement of the broadcast functionality across CPUs

2013-08-14 Thread Preeti U Murthy
d-off-by: Preeti U Murthy --- arch/powerpc/include/asm/time.h |1 arch/powerpc/kernel/time.c | 10 ++-- arch/powerpc/platforms/powernv/processor_idle.c | 56 +++ 3 files changed, 53 insertions(+), 14 deletions(-) diff --git a

[RFC V2 PATCH 4/6] cpuidle/ppc: Add longnap state to the idle states on powernv

2013-08-14 Thread Preeti U Murthy
sleep on ppc. Signed-off-by: Preeti U Murthy --- arch/powerpc/platforms/powernv/processor_idle.c | 48 +++ 1 file changed, 47 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/platforms/powernv/processor_idle.c b/arch/powerpc/platforms/powernv/processor_idle.c index

[RFC V2 PATCH 3/6] cpuidle/ppc: Add timer offload framework to support deep idle states

2013-08-14 Thread Preeti U Murthy
or a timer interrupt is called on the cpu in deep idle state to handle the local events. The current design and implementation of the timer offload framework supports the ONESHOT tick mode but not the PERIODIC mode. Signed-off-by: Preeti U. Murthy --- arch/powerpc/include/asm/time.h|3 + ar

[RFC V2 PATCH 2/6] powerpc: Implement broadcast timer interrupt as an IPI message

2013-08-14 Thread Preeti U Murthy
Srivatsa S. Bhat Signed-off-by: Preeti U Murthy --- arch/powerpc/include/asm/smp.h |3 ++- arch/powerpc/kernel/smp.c | 19 +++ arch/powerpc/platforms/cell/interrupt.c |2 +- arch/powerpc/platforms/ps3/smp.c|2 +- 4 files changed, 19

[RFC V2 PATCH 1/6] powerpc: Free up the IPI message slot of ipi call function (PPC_MSG_CALL_FUNC)

2013-08-14 Thread Preeti U Murthy
available). So, implement the functionality of PPC_MSG_CALL_FUNC using PPC_MSG_CALL_FUNC_SINGLE itself and release its IPI message slot, so that it can be used for something else in the future, if desired. Signed-off-by: Srivatsa S. Bhat Signed-off-by: Preeti U Murthy --- arch/powerpc/include

[RFC V2 PATCH 0/6] cpuidle/ppc: Timer offload framework to support deep idle states

2013-08-14 Thread Preeti U Murthy
powernv Patch[5/6]: Dynamically pick a broadcast CPU Patch[6/6]: Remove the constraint of having to disable tickless idle on the broadcast cpu, by queueing a hrtimer exclusively to do broadcast handling. --- Preeti U Murthy (4): cpuidle/ppc: Add timer offload framework to support deep idle

Re: [linux-pm] [PATCH 1/3] cpuidle/powernv: cpuidle backend driver for powernv

2013-08-02 Thread Preeti U Murthy
cpuidle_pause_and_lock(); >> +cpuidle_enable_device(dev); >> +cpuidle_resume_and_unlock(); >> +break; >> + >> + case CPU_DEAD: >> +case CPU_DEAD_FROZEN: >> +

Re: [PATCH V2 4/6] cpuidle/pseries: Move the pseries_idle backend driver to sysdev.

2013-07-31 Thread Preeti U Murthy
Hi Dongsheng, On 07/31/2013 11:16 AM, Wang Dongsheng-B40534 wrote: > Hi Preeti, > >> -Original Message----- >> From: Preeti U Murthy [mailto:pre...@linux.vnet.ibm.com] >> Sent: Wednesday, July 31, 2013 12:00 PM >> To: Wang Dongsheng-B40534 >> Cc: Deept

Re: [PATCH V2 4/6] cpuidle/pseries: Move the pseries_idle backend driver to sysdev.

2013-07-30 Thread Preeti U Murthy
essors idle support IMO should hook onto the backend cpuidle driver that this patchset provides. Regards Preeti U Murthy ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [RFC PATCH 4/5] cpuidle/ppc: CPU goes tickless if there are no arch-specific constraints

2013-07-29 Thread Preeti U Murthy
Hi, On 07/29/2013 10:58 AM, Vaidyanathan Srinivasan wrote: > * Preeti U Murthy [2013-07-27 13:20:37]: > >> Hi Ben, >> >> On 07/27/2013 12:00 PM, Benjamin Herrenschmidt wrote: >>> On Fri, 2013-07-26 at 08:09 +0530, Preeti U Murthy wrote: >>>> *The la

Re: [RFC PATCH 4/5] cpuidle/ppc: CPU goes tickless if there are no arch-specific constraints

2013-07-27 Thread Preeti U Murthy
Hi Ben, On 07/27/2013 12:00 PM, Benjamin Herrenschmidt wrote: > On Fri, 2013-07-26 at 08:09 +0530, Preeti U Murthy wrote: >> *The lapic of a broadcast CPU is active always*. Say CPUX, wants the >> broadcast CPU to wake it up at timeX. Since we cannot program the lapic >>

Re: [RFC PATCH 0/5] cpuidle/ppc: Timer offload framework to support deep idle states

2013-07-26 Thread Preeti U Murthy
Hi Li, On 07/26/2013 03:35 PM, Li Yang-R58472 wrote: > > >> -Original Message- >> From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- >> ow...@vger.kernel.org] On Behalf Of Preeti U Murthy >> Sent: Thursday, July 25, 2013 5:03 PM >> To: b

[Resend RFC PATCH 4/5] cpuidle/ppc: CPU goes tickless if there are no arch-specific constraints

2013-07-25 Thread Preeti U Murthy
disables tickless idle, is a system wide setting. Hence resort to an arch specific call to check if a cpu can go into tickless idle. Signed-off-by: Preeti U Murthy --- arch/powerpc/kernel/time.c |5 + kernel/time/tick-sched.c |7 +++ 2 files changed, 12 insertions(+) diff --git a

[Resend RFC PATCH 1/5] powerpc: Free up the IPI message slot of ipi call function (PPC_MSG_CALL_FUNC)

2013-07-25 Thread Preeti U Murthy
available). So, implement the functionality of PPC_MSG_CALL_FUNC using PPC_MSG_CALL_FUNC_SINGLE itself and release its IPI message slot, so that it can be used for something else in the future, if desired. Signed-off-by: Srivatsa S. Bhat Signed-off-by: Preeti U Murthy --- arch/powerpc/include

[Resend RFC PATCH 5/5] cpuidle/ppc: Add longnap state to the idle states on powernv

2013-07-25 Thread Preeti U Murthy
: Preeti U Murthy --- arch/powerpc/platforms/powernv/processor_idle.c | 48 +++ 1 file changed, 47 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/platforms/powernv/processor_idle.c b/arch/powerpc/platforms/powernv/processor_idle.c index f43ad91a..9aca502 100644 --- a

[Resend RFC PATCH 3/5] cpuidle/ppc: Add timer offload framework to support deep idle states

2013-07-25 Thread Preeti U Murthy
terrupt is called on the cpu in deep idle state to handle the local events. The current design and implementation of the timer offload framework supports the ONESHOT tick mode but not the PERIODIC mode. Signed-off-by: Preeti U. Murthy --- arch/powerpc/include/asm/time.h|3 + arch/powe

[Resend RFC PATCH 2/5] powerpc: Implement broadcast timer interrupt as an IPI message

2013-07-25 Thread Preeti U Murthy
Srivatsa S. Bhat Signed-off-by: Preeti U Murthy --- arch/powerpc/include/asm/smp.h |3 ++- arch/powerpc/kernel/smp.c | 19 +++ arch/powerpc/platforms/cell/interrupt.c |2 +- arch/powerpc/platforms/ps3/smp.c|2 +- 4 files changed, 19

[Resend RFC PATCH 0/5] cpuidle/ppc: Timer offload framework to support deep idle states

2013-07-25 Thread Preeti U Murthy
makes use of the timer offload framework that the patches Patch[1/5] to Patch[4/5] build. This patch series is being resent to clarify certain ambiguity in the patch descriptions from the previous post. Discussion around this: https://lkml.org/lkml/2013/7/25/754 --- Preeti U Murthy (3): cpuidle

Re: [RFC PATCH 4/5] cpuidle/ppc: CPU goes tickless if there are no arch-specific constraints

2013-07-25 Thread Preeti U Murthy
Hi Frederic, I apologise for the confusion. As Paul pointed out maybe the usage of the term lapic is causing a large amount of confusion. So please see the clarification below. Maybe it will help answer your question. On 07/26/2013 08:09 AM, Preeti U Murthy wrote: > Hi Frederic, > >

Re: [RFC PATCH 4/5] cpuidle/ppc: CPU goes tickless if there are no arch-specific constraints

2013-07-25 Thread Preeti U Murthy
Hi Paul, On 07/26/2013 08:49 AM, Paul Mackerras wrote: > On Fri, Jul 26, 2013 at 08:09:23AM +0530, Preeti U Murthy wrote: >> Hi Frederic, >> >> On 07/25/2013 07:00 PM, Frederic Weisbecker wrote: >>> Hi Preeti, >>> >>> I'm not exactly sure w

Re: [RFC PATCH 4/5] cpuidle/ppc: CPU goes tickless if there are no arch-specific constraints

2013-07-25 Thread Preeti U Murthy
Hi Frederic, On 07/25/2013 07:00 PM, Frederic Weisbecker wrote: > On Thu, Jul 25, 2013 at 02:33:02PM +0530, Preeti U Murthy wrote: >> In the current design of timer offload framework, the broadcast cpu should >> *not* go into tickless idle so as to avoid missed wakeups on CPUs i

Re: [RFC PATCH 4/5] cpuidle/ppc: CPU goes tickless if there are no arch-specific constraints

2013-07-25 Thread Preeti U Murthy
Hi Frederic, On 07/25/2013 07:00 PM, Frederic Weisbecker wrote: > On Thu, Jul 25, 2013 at 02:33:02PM +0530, Preeti U Murthy wrote: >> In the current design of timer offload framework, the broadcast cpu should >> *not* go into tickless idle so as to avoid missed wakeups on CPUs i

[RFC PATCH 5/5] cpuidle/ppc: Add longnap state to the idle states on powernv

2013-07-25 Thread Preeti U Murthy
: Preeti U Murthy --- arch/powerpc/platforms/powernv/processor_idle.c | 48 +++ 1 file changed, 47 insertions(+), 1 deletion(-) diff --git a/arch/powerpc/platforms/powernv/processor_idle.c b/arch/powerpc/platforms/powernv/processor_idle.c index f43ad91a..9aca502 100644 --- a

[RFC PATCH 4/5] cpuidle/ppc: CPU goes tickless if there are no arch-specific constraints

2013-07-25 Thread Preeti U Murthy
disables tickless idle, is a system wide setting. Hence resort to an arch specific call to check if a cpu can go into tickless idle. Signed-off-by: Preeti U Murthy --- arch/powerpc/kernel/time.c |5 + kernel/time/tick-sched.c |7 +++ 2 files changed, 12 insertions(+) diff --git a

[RFC PATCH 3/5] cpuidle/ppc: Add timer offload framework to support deep idle states

2013-07-25 Thread Preeti U Murthy
On a broadcast ipi the event handler for a timer interrupt is called on the cpu in deep idle state to handle the local events. The current design and implementation of the timer offload framework supports the ONESHOT tick mode but not the PERIODIC mode. Signed-off-by: Preeti U. Murthy --- arch/po

[RFC PATCH 2/5] powerpc: Implement broadcast timer interrupt as an IPI message

2013-07-25 Thread Preeti U Murthy
Srivatsa S. Bhat Signed-off-by: Preeti U Murthy --- arch/powerpc/include/asm/smp.h |3 ++- arch/powerpc/kernel/smp.c | 19 +++ arch/powerpc/platforms/cell/interrupt.c |2 +- arch/powerpc/platforms/ps3/smp.c|2 +- 4 files changed, 19

[RFC PATCH 0/5] cpuidle/ppc: Timer offload framework to support deep idle states

2013-07-25 Thread Preeti U Murthy
makes use of the timer offload framework that the patches Patch[1/5] to Patch[4/5] build. --- Preeti U Murthy (3): cpuidle/ppc: Add timer offload framework to support deep idle states cpuidle/ppc: CPU goes tickless if there are no arch-specific constraints cpuidle/ppc: Add longnap

[RFC PATCH 1/5] powerpc: Free up the IPI message slot of ipi call function (PPC_MSG_CALL_FUNC)

2013-07-25 Thread Preeti U Murthy
available). So, implement the functionality of PPC_MSG_CALL_FUNC using PPC_MSG_CALL_FUNC_SINGLE itself and release its IPI message slot, so that it can be used for something else in the future, if desired. Signed-off-by: Srivatsa S. Bhat Signed-off-by: Preeti U Murthy --- arch/powerpc/include

<    1   2   3