powerpc/64s/idle: Run latch switch is done with MSR[EE]=0
authorNicholas Piggin <npiggin@gmail.com>
Tue, 13 Jun 2017 13:05:57 +0000 (23:05 +1000)
committerMichael Ellerman <mpe@ellerman.id.au>
Mon, 19 Jun 2017 09:46:30 +0000 (19:46 +1000)
In the idle sleep/wake code we know that MSR[EE] is clear, so we can
avoid 2 x mfmsr and 2 x mtmsr by calling the double-underscore
versions of the run latch routines which assume interrupts are already
disabled.

Acked-by: Vaidyanathan Srinivasan <svaidy@linux.vnet.ibm.com>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
arch/powerpc/platforms/powernv/idle.c

index 1028df82cd2f906b0e923c6ea613dc51ba1a4b92..2abee070373fb3a8b757b8d3cb269e5d0b89dff6 100644 (file)
@@ -291,9 +291,9 @@ static unsigned long __power7_idle_type(unsigned long type)
        if (!prep_irq_for_idle_irqsoff())
                return 0;
 
-       ppc64_runlatch_off();
+       __ppc64_runlatch_off();
        srr1 = power7_idle_insn(type);
-       ppc64_runlatch_on();
+       __ppc64_runlatch_on();
 
        fini_irq_for_idle_irqsoff();
 
@@ -328,9 +328,9 @@ static unsigned long __power9_idle_type(unsigned long stop_psscr_val,
        psscr = mfspr(SPRN_PSSCR);
        psscr = (psscr & ~stop_psscr_mask) | stop_psscr_val;
 
-       ppc64_runlatch_off();
+       __ppc64_runlatch_off();
        srr1 = power9_idle_stop(psscr);
-       ppc64_runlatch_on();
+       __ppc64_runlatch_on();
 
        fini_irq_for_idle_irqsoff();
 
@@ -365,7 +365,7 @@ unsigned long pnv_cpu_offline(unsigned int cpu)
        unsigned long srr1;
        u32 idle_states = pnv_get_supported_cpuidle_states();
 
-       ppc64_runlatch_off();
+       __ppc64_runlatch_off();
 
        if (cpu_has_feature(CPU_FTR_ARCH_300) && deepest_stop_found) {
                unsigned long psscr;
@@ -392,7 +392,7 @@ unsigned long pnv_cpu_offline(unsigned int cpu)
                HMT_medium();
        }
 
-       ppc64_runlatch_on();
+       __ppc64_runlatch_on();
 
        return srr1;
 }