Re: [RESEND] [v3 PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-26 Thread Nicholas Piggin
On Fri, 21 Jul 2017 16:31:34 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > Currently we use the stop-api provided by the firmware to program the > SLW engine to restore the values of hypervisor resources that get lost > on

Re: [RESEND] [v3 PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-26 Thread Nicholas Piggin
On Fri, 21 Jul 2017 16:31:34 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > Currently we use the stop-api provided by the firmware to program the > SLW engine to restore the values of hypervisor resources that get lost > on deeper idle states (such as winkle). Since the deep

Re: [PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-25 Thread Michael Ellerman
Nicholas Piggin writes: ... > I wonder if we should think about a more structured kernel API for > modifying these kind of system registers so we always have the > up-to-date values stored in memory. Many of them do need to be > restored after sleep, but they don't need to be

Re: [PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-25 Thread Michael Ellerman
Nicholas Piggin writes: ... > I wonder if we should think about a more structured kernel API for > modifying these kind of system registers so we always have the > up-to-date values stored in memory. Many of them do need to be > restored after sleep, but they don't need to be saved per-thread >

[RESEND] [v3 PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-21 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" Currently we use the stop-api provided by the firmware to program the SLW engine to restore the values of hypervisor resources that get lost on deeper idle states (such as winkle). Since the deep states were only used for CPU-Hotplug on POWER8

[RESEND] [v3 PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-21 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" Currently we use the stop-api provided by the firmware to program the SLW engine to restore the values of hypervisor resources that get lost on deeper idle states (such as winkle). Since the deep states were only used for CPU-Hotplug on POWER8 systems, we would program

Re: [v2 PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-19 Thread Nicholas Piggin
On Wed, 19 Jul 2017 13:48:50 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > Currently we use the stop-api provided by the firmware to program the > SLW engine to restore the values of hypervisor resources that get lost > on

Re: [v2 PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-19 Thread Nicholas Piggin
On Wed, 19 Jul 2017 13:48:50 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > Currently we use the stop-api provided by the firmware to program the > SLW engine to restore the values of hypervisor resources that get lost > on deeper idle states (such as winkle). Since the deep

[v2 PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-19 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" Currently we use the stop-api provided by the firmware to program the SLW engine to restore the values of hypervisor resources that get lost on deeper idle states (such as winkle). Since the deep states were only used for CPU-Hotplug on POWER8

[v2 PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-19 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" Currently we use the stop-api provided by the firmware to program the SLW engine to restore the values of hypervisor resources that get lost on deeper idle states (such as winkle). Since the deep states were only used for CPU-Hotplug on POWER8 systems, we would program

Re: [PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-18 Thread Nicholas Piggin
On Wed, 19 Jul 2017 10:34:59 +0530 Gautham R Shenoy wrote: > Hello Nicholas, > > On Wed, Jul 19, 2017 at 12:14:12PM +1000, Nicholas Piggin wrote: > > Thanks for working on these patches. We really need to get this stuff > > merged and tested asap :) > > > > > On Tue,

Re: [PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-18 Thread Nicholas Piggin
On Wed, 19 Jul 2017 10:34:59 +0530 Gautham R Shenoy wrote: > Hello Nicholas, > > On Wed, Jul 19, 2017 at 12:14:12PM +1000, Nicholas Piggin wrote: > > Thanks for working on these patches. We really need to get this stuff > > merged and tested asap :) > > > > > On Tue, 18 Jul 2017 19:58:49

Re: [PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-18 Thread Gautham R Shenoy
Hello Nicholas, On Wed, Jul 19, 2017 at 12:14:12PM +1000, Nicholas Piggin wrote: > Thanks for working on these patches. We really need to get this stuff > merged and tested asap :) > > On Tue, 18 Jul 2017 19:58:49 +0530 [..snip..] > > diff --git a/arch/powerpc/platforms/powernv/smp.c > >

Re: [PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-18 Thread Gautham R Shenoy
Hello Nicholas, On Wed, Jul 19, 2017 at 12:14:12PM +1000, Nicholas Piggin wrote: > Thanks for working on these patches. We really need to get this stuff > merged and tested asap :) > > On Tue, 18 Jul 2017 19:58:49 +0530 [..snip..] > > diff --git a/arch/powerpc/platforms/powernv/smp.c > >

Re: [PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-18 Thread Nicholas Piggin
Thanks for working on these patches. We really need to get this stuff merged and tested asap :) On Tue, 18 Jul 2017 19:58:49 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > > Currently we use the stop-api provided by the

Re: [PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-18 Thread Nicholas Piggin
Thanks for working on these patches. We really need to get this stuff merged and tested asap :) On Tue, 18 Jul 2017 19:58:49 +0530 "Gautham R. Shenoy" wrote: > From: "Gautham R. Shenoy" > > > Currently we use the stop-api provided by the firmware to program the > SLW engine to restore the

[PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-18 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" Currently we use the stop-api provided by the firmware to program the SLW engine to restore the values of hypervisor resources that get lost on deeper idle states (such as winkle). Since the deep states were only used for CPU-Hotplug on POWER8

[PATCH 2/2] powernv/powerpc: Clear PECE1 in LPCR via stop-api only on Hotplug

2017-07-18 Thread Gautham R. Shenoy
From: "Gautham R. Shenoy" Currently we use the stop-api provided by the firmware to program the SLW engine to restore the values of hypervisor resources that get lost on deeper idle states (such as winkle). Since the deep states were only used for CPU-Hotplug on POWER8 systems, we would program