Re: [PATCH 13/14] powerpc/64: runlatch CTRL[RUN] set optimisation

2017-06-13 Thread Michael Ellerman
Benjamin Herrenschmidt writes: > On Tue, 2017-06-13 at 20:04 +1000, Michael Ellerman wrote: >> > Good idea.  Writing to CTRL register can change only the RUN field. >> > Was this any different in older generations? >> >> No AFAICS back to 2.02. >> >> > Anton and Ben kept the mfspr/mtspr part in

Re: [PATCH 13/14] powerpc/64: runlatch CTRL[RUN] set optimisation

2017-06-13 Thread Benjamin Herrenschmidt
On Tue, 2017-06-13 at 20:04 +1000, Michael Ellerman wrote: > > Good idea.  Writing to CTRL register can change only the RUN field. > > Was this any different in older generations? > > No AFAICS back to 2.02. > > > Anton and Ben kept the mfspr/mtspr part in earlier updates to this > > routine. >

Re: [PATCH 13/14] powerpc/64: runlatch CTRL[RUN] set optimisation

2017-06-13 Thread Nicholas Piggin
On Tue, 13 Jun 2017 20:04:27 +1000 Michael Ellerman wrote: > Vaidyanathan Srinivasan writes: > > * Nicholas Piggin [2017-06-12 09:58:34]: > > > >> The CTRL register is read-only except bit 63 which is the run latch > >> control. This means it can be updated with a mtspr rather than > >> mfspr

Re: [PATCH 13/14] powerpc/64: runlatch CTRL[RUN] set optimisation

2017-06-13 Thread Michael Ellerman
Vaidyanathan Srinivasan writes: > * Nicholas Piggin [2017-06-12 09:58:34]: > >> The CTRL register is read-only except bit 63 which is the run latch >> control. This means it can be updated with a mtspr rather than >> mfspr/mtspr. >> >> Signed-off-by: Nicholas Piggin > >> diff --git a/arch/power

Re: [PATCH 13/14] powerpc/64: runlatch CTRL[RUN] set optimisation

2017-06-12 Thread Vaidyanathan Srinivasan
* Nicholas Piggin [2017-06-12 09:58:34]: > The CTRL register is read-only except bit 63 which is the run latch > control. This means it can be updated with a mtspr rather than > mfspr/mtspr. > > Signed-off-by: Nicholas Piggin Reviewed-by: Vaidyanathan Srinivasan > --- > arch/powerpc/kernel/

[PATCH 13/14] powerpc/64: runlatch CTRL[RUN] set optimisation

2017-06-11 Thread Nicholas Piggin
The CTRL register is read-only except bit 63 which is the run latch control. This means it can be updated with a mtspr rather than mfspr/mtspr. Signed-off-by: Nicholas Piggin --- arch/powerpc/kernel/process.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/arch/

[PATCH 13/14] powerpc/64: runlatch CTRL[RUN] set optimisation

2017-06-08 Thread Nicholas Piggin
The CTRL register is read-only except bit 63 which is the run latch control. This means it can be updated with a mtspr rather than mfspr/mtspr. Signed-off-by: Nicholas Piggin --- arch/powerpc/kernel/process.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/arch/