Re: [PATCH] cpuidle/powernv : init all present cpus for deep states

2018-05-27 Thread Stewart Smith
Michael Ellerman writes: > Akshay Adiga writes: > >> Init all present cpus for deep states instead of "all possible" cpus. >> Init fails if the possible cpu is gaurded. Resulting in making only >> non-deep states available for cpuidle/hotplug. > > This is basically the opposite of what we just di

Re: [PATCH] cpuidle/powernv : init all present cpus for deep states

2018-05-25 Thread Michael Ellerman
Akshay Adiga writes: > Yes this needs to be sent to stable. > > Fixes: d405a98c ("powerpc/powernv: Move cpuidle related code from setup.c > to new file") Is that really the commit that introduced the bug? :) Seems like it's more likely this one: Fixes: 77b54e9f213f ("powernv/powerpc: Add w

Re: [PATCH] cpuidle/powernv : init all present cpus for deep states

2018-05-25 Thread Michael Ellerman
Akshay Adiga writes: > Init all present cpus for deep states instead of "all possible" cpus. > Init fails if the possible cpu is gaurded. Resulting in making only > non-deep states available for cpuidle/hotplug. This is basically the opposite of what we just did for IMC. There we switched from

Re: [PATCH] cpuidle/powernv : init all present cpus for deep states

2018-05-23 Thread Akshay Adiga
On Wed, May 16, 2018 at 05:32:14PM +0530, Akshay Adiga wrote: > Init all present cpus for deep states instead of "all possible" cpus. > Init fails if the possible cpu is gaurded. Resulting in making only > non-deep states available for cpuidle/hotplug. > > Signed-off-by: Akshay Adiga > --- > arc

Re: [PATCH] cpuidle/powernv : init all present cpus for deep states

2018-05-16 Thread Akshay Adiga
Yes this needs to be sent to stable. Fixes: d405a98c ("powerpc/powernv: Move cpuidle related code from setup.c to new file")

Re: [PATCH] cpuidle/powernv : init all present cpus for deep states

2018-05-16 Thread Stewart Smith
Akshay Adiga writes: > Init all present cpus for deep states instead of "all possible" cpus. > Init fails if the possible cpu is gaurded. Resulting in making only > non-deep states available for cpuidle/hotplug. Should this also head to stable? It means that for single threaded workloads, if you

[PATCH] cpuidle/powernv : init all present cpus for deep states

2018-05-16 Thread Akshay Adiga
Init all present cpus for deep states instead of "all possible" cpus. Init fails if the possible cpu is gaurded. Resulting in making only non-deep states available for cpuidle/hotplug. Signed-off-by: Akshay Adiga --- arch/powerpc/platforms/powernv/idle.c | 4 ++-- 1 file changed, 2 insertions(+)