svn commit: r329712 - in head/sys: conf powerpc/aim powerpc/include powerpc/powernv powerpc/powerpc

Wojciech Macek wma at FreeBSD.org
Wed Feb 21 14:28:42 UTC 2018


Author: wma
Date: Wed Feb 21 14:28:40 2018
New Revision: 329712
URL: https://svnweb.freebsd.org/changeset/base/329712

Log:
  PowerNV: Put processor to power-save state in idle thread
  
  When processor enters power-save state it releases resources shared with other
  cpu threads which makes other cores working much faster.
  
  This patch also implements saving and restoring registers that might get
  corrupted in power-save state.
  
  Submitted by:          Patryk Duda <pdk at semihalf.com>
  Obtained from:         Semihalf
  Reviewed by:           jhibbits, nwhitehorn, wma
  Sponsored by:          IBM, QCM Technologies
  Differential revision: https://reviews.freebsd.org/D14330

Added:
  head/sys/powerpc/powerpc/cpu_subr64.S   (contents, props changed)
Modified:
  head/sys/conf/files.powerpc
  head/sys/powerpc/aim/locore64.S
  head/sys/powerpc/aim/mp_cpudep.c
  head/sys/powerpc/aim/trap_subr64.S
  head/sys/powerpc/include/cpu.h
  head/sys/powerpc/include/spr.h
  head/sys/powerpc/powernv/platform_powernv.c
  head/sys/powerpc/powerpc/cpu.c

Modified: head/sys/conf/files.powerpc
==============================================================================
--- head/sys/conf/files.powerpc	Wed Feb 21 14:17:07 2018	(r329711)
+++ head/sys/conf/files.powerpc	Wed Feb 21 14:28:40 2018	(r329712)
@@ -198,6 +198,7 @@ powerpc/powerpc/clock.c		standard
 powerpc/powerpc/copyinout.c	standard
 powerpc/powerpc/copystr.c	standard
 powerpc/powerpc/cpu.c		standard
+powerpc/powerpc/cpu_subr64.S	optional	powerpc64
 powerpc/powerpc/db_disasm.c	optional	ddb
 powerpc/powerpc/db_hwwatch.c	optional	ddb
 powerpc/powerpc/db_interface.c	optional	ddb

Modified: head/sys/powerpc/aim/locore64.S
==============================================================================
--- head/sys/powerpc/aim/locore64.S	Wed Feb 21 14:17:07 2018	(r329711)
+++ head/sys/powerpc/aim/locore64.S	Wed Feb 21 14:28:40 2018	(r329712)
@@ -53,6 +53,8 @@ GLOBAL(__startkernel)
 	.llong	begin
 GLOBAL(__endkernel)
 	.llong	end
+GLOBAL(can_wakeup)
+	.llong	0x0
 
 	.align	4
 #define	TMPSTKSZ	16384		/* 16K temporary stack */
@@ -60,6 +62,7 @@ GLOBAL(tmpstk)
 	.space	TMPSTKSZ
 
 TOC_ENTRY(tmpstk)
+TOC_ENTRY(can_wakeup)
 
 /*
  * Entry point for bootloaders that do not fully implement ELF and start
@@ -96,6 +99,10 @@ ap_kexec_start:		/* At 0x60 past start, copied to 0x60
 	
 	/* Released */
 	or	2,2,2			/* unyield */
+
+	/* Make sure that it will be software reset. Clear SRR1 */
+	li	%r1,0
+	mtsrr1	%r1
 	ba	EXC_RST
 
 

Modified: head/sys/powerpc/aim/mp_cpudep.c
==============================================================================
--- head/sys/powerpc/aim/mp_cpudep.c	Wed Feb 21 14:17:07 2018	(r329711)
+++ head/sys/powerpc/aim/mp_cpudep.c	Wed Feb 21 14:28:40 2018	(r329712)
@@ -398,8 +398,11 @@ cpudep_ap_setup()
 	case IBMPOWER8:
 	case IBMPOWER8E:
 #ifdef __powerpc64__
-		if (mfmsr() & PSL_HV)
-			mtspr(SPR_LPCR, mfspr(SPR_LPCR) | LPCR_LPES);
+		if (mfmsr() & PSL_HV) {
+			mtspr(SPR_LPCR, mfspr(SPR_LPCR) | LPCR_LPES |
+			    LPCR_PECE_WAKESET);
+			isync();
+		}
 #endif
 		break;
 	default:

Modified: head/sys/powerpc/aim/trap_subr64.S
==============================================================================
--- head/sys/powerpc/aim/trap_subr64.S	Wed Feb 21 14:17:07 2018	(r329711)
+++ head/sys/powerpc/aim/trap_subr64.S	Wed Feb 21 14:28:40 2018	(r329712)
@@ -307,10 +307,24 @@ dtrace_invop_calltrap_addr:
  * once the MMU is turned on.
  */
 	.globl	CNAME(rstcode), CNAME(rstcodeend), CNAME(cpu_reset_handler)
+	.globl	CNAME(cpu_wakeup_handler)
 	.p2align 3
 CNAME(rstcode):
+	/*
+	 * Check if this is software reset or
+	 * processor is waking up from power saving mode
+	 * It is software reset when 46:47 = 0b00
+	 */
+	mfsrr1	%r9			/* Load SRR1 into r9 */
+	andis.	%r9,%r9,0x3		/* Logic AND with 46:47 bits */
+	beq	2f			/* Branch if software reset */
+	bl	1f
+	.llong	cpu_wakeup_handler
+
+	/* It is software reset */
+
 	/* Explicitly set MSR[SF] */
-	mfmsr	%r9
+2:	mfmsr	%r9
 	li	%r8,1
 	insrdi	%r9,%r8,1,0
 	mtmsrd	%r9
@@ -318,6 +332,7 @@ CNAME(rstcode):
 
 	bl	1f
 	.llong	cpu_reset_handler /* Make sure to maintain 8-byte alignment */
+
 1:	mflr	%r9
 	ld	%r9,0(%r9)
 	mtlr	%r9
@@ -358,6 +373,59 @@ cpu_reset_handler:
 	/* Should not be reached */
 9:
 	b	9b
+
+cpu_wakeup_handler:
+	GET_TOCBASE(%r2)
+
+	/* Check for false wake up due to badly SRR1 set (eg. by OPAL) */
+	ld	%r3,TOC_REF(can_wakeup)(%r2)
+	ld	%r3,0(%r3)
+	cmpdi	%r3,0
+	beq	cpu_reset_handler
+
+	/* Turn on MMU after return from interrupt */
+	mfsrr1	%r3
+	ori	%r3,%r3,(PSL_IR | PSL_DR)
+	mtsrr1	%r3
+
+	/* Turn on MMU (needed to access PCB) */
+	mfmsr	%r3
+	ori	%r3,%r3,(PSL_IR | PSL_DR)
+	mtmsr	%r3
+	isync
+
+	mfsprg0	%r3
+
+	ld	%r3,PC_CURTHREAD(%r3)	/* Get current thread */
+	ld	%r3,TD_PCB(%r3)		/* Get PCB of current thread */
+	ld	%r12,PCB_CONTEXT(%r3)	/* Load the non-volatile GP regs. */
+	ld	%r13,PCB_CONTEXT+1*8(%r3)
+	ld	%r14,PCB_CONTEXT+2*8(%r3)
+	ld	%r15,PCB_CONTEXT+3*8(%r3)
+	ld	%r16,PCB_CONTEXT+4*8(%r3)
+	ld	%r17,PCB_CONTEXT+5*8(%r3)
+	ld	%r18,PCB_CONTEXT+6*8(%r3)
+	ld	%r19,PCB_CONTEXT+7*8(%r3)
+	ld	%r20,PCB_CONTEXT+8*8(%r3)
+	ld	%r21,PCB_CONTEXT+9*8(%r3)
+	ld	%r22,PCB_CONTEXT+10*8(%r3)
+	ld	%r23,PCB_CONTEXT+11*8(%r3)
+	ld	%r24,PCB_CONTEXT+12*8(%r3)
+	ld	%r25,PCB_CONTEXT+13*8(%r3)
+	ld	%r26,PCB_CONTEXT+14*8(%r3)
+	ld	%r27,PCB_CONTEXT+15*8(%r3)
+	ld	%r28,PCB_CONTEXT+16*8(%r3)
+	ld	%r29,PCB_CONTEXT+17*8(%r3)
+	ld	%r30,PCB_CONTEXT+18*8(%r3)
+	ld	%r31,PCB_CONTEXT+19*8(%r3)
+	ld	%r5,PCB_CR(%r3)		/* Load the condition register */
+	mtcr	%r5
+	ld	%r5,PCB_LR(%r3)		/* Load the link register */
+	mtsrr0	%r5
+	ld	%r1,PCB_SP(%r3)		/* Load the stack pointer */
+	ld	%r2,PCB_TOC(%r3)	/* Load the TOC pointer */
+
+	rfid
 
 /*
  * This code gets copied to all the trap vectors

Modified: head/sys/powerpc/include/cpu.h
==============================================================================
--- head/sys/powerpc/include/cpu.h	Wed Feb 21 14:17:07 2018	(r329711)
+++ head/sys/powerpc/include/cpu.h	Wed Feb 21 14:28:40 2018	(r329712)
@@ -112,6 +112,11 @@ get_cyclecount(void)
 extern char btext[];
 extern char etext[];
 
+#ifdef __powerpc64__
+extern void enter_idle_powerx(void);
+extern uint64_t can_wakeup;
+#endif
+
 void	cpu_halt(void);
 void	cpu_reset(void);
 void	cpu_sleep(void);

Modified: head/sys/powerpc/include/spr.h
==============================================================================
--- head/sys/powerpc/include/spr.h	Wed Feb 21 14:17:07 2018	(r329711)
+++ head/sys/powerpc/include/spr.h	Wed Feb 21 14:28:40 2018	(r329712)
@@ -201,7 +201,14 @@
 
 #define	SPR_LPCR		0x13e	/* Logical Partitioning Control */
 #define	  LPCR_LPES		0x008	/* Bit 60 */
-
+#define   LPCR_PECE_DRBL        (1ULL << 16)    /* Directed Privileged Doorbell */
+#define   LPCR_PECE_HDRBL       (1ULL << 15)    /* Directed Hypervisor Doorbell */
+#define   LPCR_PECE_EXT         (1ULL << 14)    /* External exceptions */
+#define   LPCR_PECE_DECR        (1ULL << 13)    /* Decrementer exceptions */
+#define   LPCR_PECE_ME          (1ULL << 12)    /* Machine Check and Hypervisor */
+                                                /* Maintenance exceptions */
+#define   LPCR_PECE_WAKESET     (LPCR_PECE_EXT | LPCR_PECE_DECR | LPCR_PECE_ME)
+ 
 #define	SPR_EPCR		0x133
 #define	  EPCR_EXTGS		  0x80000000
 #define	  EPCR_DTLBGS		  0x40000000

Modified: head/sys/powerpc/powernv/platform_powernv.c
==============================================================================
--- head/sys/powerpc/powernv/platform_powernv.c	Wed Feb 21 14:17:07 2018	(r329711)
+++ head/sys/powerpc/powernv/platform_powernv.c	Wed Feb 21 14:28:40 2018	(r329712)
@@ -139,7 +139,9 @@ powernv_attach(platform_t plat)
 	opal_call(OPAL_REINIT_CPUS, 1 /* Big endian */);
 #endif
 
-	cpu_idle_hook = powernv_cpu_idle;
+       if (cpu_idle_hook == NULL)
+                cpu_idle_hook = powernv_cpu_idle;
+
 	powernv_boot_pir = mfspr(SPR_PIR);
 
 	/* LPID must not be altered when PSL_DR or PSL_IR is set */
@@ -150,10 +152,10 @@ powernv_attach(platform_t plat)
 	mtspr(SPR_LPID, 0);
 	isync();
 
-	mtmsr(msr);
-
 	mtspr(SPR_LPCR, LPCR_LPES);
 	isync();
+
+	mtmsr(msr);
 
 	/* Init CPU bits */
 	powernv_smp_ap_init(plat);

Modified: head/sys/powerpc/powerpc/cpu.c
==============================================================================
--- head/sys/powerpc/powerpc/cpu.c	Wed Feb 21 14:17:07 2018	(r329711)
+++ head/sys/powerpc/powerpc/cpu.c	Wed Feb 21 14:28:40 2018	(r329712)
@@ -68,6 +68,8 @@
 #include <sys/kernel.h>
 #include <sys/proc.h>
 #include <sys/sysctl.h>
+#include <sys/sched.h>
+#include <sys/smp.h>
 
 #include <machine/bus.h>
 #include <machine/cpu.h>
@@ -81,11 +83,15 @@
 static void	cpu_6xx_setup(int cpuid, uint16_t vers);
 static void	cpu_970_setup(int cpuid, uint16_t vers);
 static void	cpu_booke_setup(int cpuid, uint16_t vers);
+static void	cpu_powerx_setup(int cpuid, uint16_t vers);
 
 int powerpc_pow_enabled;
 void (*cpu_idle_hook)(sbintime_t) = NULL;
 static void	cpu_idle_60x(sbintime_t);
 static void	cpu_idle_booke(sbintime_t);
+#ifdef __powerpc64__
+static void	cpu_idle_powerx(sbintime_t);
+#endif
 
 struct cputab {
 	const char	*name;
@@ -156,13 +162,13 @@ static const struct cputab models[] = {
 	   PPC_FEATURE_SMT | PPC_FEATURE_ARCH_2_05 | PPC_FEATURE_ARCH_2_06 |
 	   PPC_FEATURE_HAS_VSX,
 	   PPC_FEATURE2_ARCH_2_07 | PPC_FEATURE2_HAS_HTM | PPC_FEATURE2_ISEL |
-	   PPC_FEATURE2_HAS_VCRYPTO, NULL },
+	   PPC_FEATURE2_HAS_VCRYPTO, cpu_powerx_setup },
         { "IBM POWER8",		IBMPOWER8,	REVFMT_MAJMIN,
 	   PPC_FEATURE_64 | PPC_FEATURE_HAS_ALTIVEC | PPC_FEATURE_HAS_FPU |
 	   PPC_FEATURE_SMT | PPC_FEATURE_ARCH_2_05 | PPC_FEATURE_ARCH_2_06 |
 	   PPC_FEATURE_HAS_VSX,
 	   PPC_FEATURE2_ARCH_2_07 | PPC_FEATURE2_HAS_HTM | PPC_FEATURE2_ISEL |
-	   PPC_FEATURE2_HAS_VCRYPTO, NULL },
+	   PPC_FEATURE2_HAS_VCRYPTO, cpu_powerx_setup },
         { "IBM POWER9",		IBMPOWER9,	REVFMT_MAJMIN,
 	   PPC_FEATURE_64 | PPC_FEATURE_HAS_ALTIVEC | PPC_FEATURE_HAS_FPU |
 	   PPC_FEATURE_SMT | PPC_FEATURE_ARCH_2_05 | PPC_FEATURE_ARCH_2_06 |
@@ -610,6 +616,29 @@ cpu_970_setup(int cpuid, uint16_t vers)
 	cpu_idle_hook = cpu_idle_60x;
 }
 
+static void
+cpu_powerx_setup(int cpuid, uint16_t vers)
+{
+
+#ifdef __powerpc64__
+	if ((mfmsr() & PSL_HV) == 0)
+		return;
+
+	/* Configure power-saving */
+	switch (vers) {
+	case IBMPOWER8:
+	case IBMPOWER8E:
+		mtspr(SPR_LPCR, mfspr(SPR_LPCR) | LPCR_PECE_WAKESET);
+		isync();
+		break;
+	default:
+		return;
+	}
+
+	cpu_idle_hook = cpu_idle_powerx;
+#endif
+}
+
 static int
 cpu_feature_bit(SYSCTL_HANDLER_ARGS)
 {
@@ -696,3 +725,26 @@ cpu_idle_booke(sbintime_t sbt)
 #endif
 }
 
+#ifdef __powerpc64__
+static void
+cpu_idle_powerx(sbintime_t sbt)
+{
+
+	/* Sleeping when running on one cpu gives no advantages - avoid it */
+	if (smp_started == 0)
+		return;
+
+	spinlock_enter();
+	if (sched_runnable()) {
+		spinlock_exit();
+		return;
+	}
+
+	if (can_wakeup == 0)
+		can_wakeup = 1;
+	mb();
+
+	enter_idle_powerx();
+	spinlock_exit();
+}
+#endif

Added: head/sys/powerpc/powerpc/cpu_subr64.S
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/sys/powerpc/powerpc/cpu_subr64.S	Wed Feb 21 14:28:40 2018	(r329712)
@@ -0,0 +1,97 @@
+/*-
+ * Copyright (c) 2017-2018 QCM Technologies.
+ * Copyright (c) 2017-2018 Semihalf.
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions and the following disclaimer.
+ * 2. Redistributions in binary form must reproduce the above copyright
+ *    notice, this list of conditions and the following disclaimer in the
+ *    documentation and/or other materials provided with the distribution.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE AUTHOR AND CONTRIBUTORS ``AS IS'' AND
+ * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
+ * ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
+ * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
+ * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
+ * SUCH DAMAGE.
+ *
+ * $FreeBSD$
+ */
+
+#include "assym.s"
+
+#include <machine/asm.h>
+
+	.globl	CNAME(power_save_sequence)
+	.p2align 3
+ENTRY(enter_idle_powerx)
+	mfsprg0	%r3			/* Get the pcpu pointer */
+	ld	%r3,PC_CURTHREAD(%r3)	/* Get current thread */
+	ld	%r3,TD_PCB(%r3)		/* Get PCB of current thread */
+	std	%r12,PCB_CONTEXT(%r3)	/* Save the non-volatile GP regs. */
+	std	%r13,PCB_CONTEXT+1*8(%r3)
+	std	%r14,PCB_CONTEXT+2*8(%r3)
+	std	%r15,PCB_CONTEXT+3*8(%r3)
+	std	%r16,PCB_CONTEXT+4*8(%r3)
+	std	%r17,PCB_CONTEXT+5*8(%r3)
+	std	%r18,PCB_CONTEXT+6*8(%r3)
+	std	%r19,PCB_CONTEXT+7*8(%r3)
+	std	%r20,PCB_CONTEXT+8*8(%r3)
+	std	%r21,PCB_CONTEXT+9*8(%r3)
+	std	%r22,PCB_CONTEXT+10*8(%r3)
+	std	%r23,PCB_CONTEXT+11*8(%r3)
+	std	%r24,PCB_CONTEXT+12*8(%r3)
+	std	%r25,PCB_CONTEXT+13*8(%r3)
+	std	%r26,PCB_CONTEXT+14*8(%r3)
+	std	%r27,PCB_CONTEXT+15*8(%r3)
+	std	%r28,PCB_CONTEXT+16*8(%r3)
+	std	%r29,PCB_CONTEXT+17*8(%r3)
+	std	%r30,PCB_CONTEXT+18*8(%r3)
+	std	%r31,PCB_CONTEXT+19*8(%r3)
+
+	mfcr	%r16			/* Save the condition register */
+	std	%r16,PCB_CR(%r3)
+	mflr	%r16			/* Save the link register */
+	std	%r16,PCB_LR(%r3)
+	std	%r1,PCB_SP(%r3)		/* Save the stack pointer */
+	std	%r2,PCB_TOC(%r3)	/* Save the TOC pointer */
+
+	/* Set where we want to jump */
+	bl	1f
+	.llong power_save_sequence	/* Remember about 8 byte alignment */
+1:	mflr	%r3
+	ld	%r3,0(%r3)
+	mtsrr0	%r3
+
+	/* Set MSR */
+	li	%r3,0
+	ori	%r3,%r3,(PSL_ME | PSL_RI)
+	li	%r8,0x9			/* PSL_SF and PSL_HV */
+	insrdi	%r3,%r8,4,0
+	mtsrr1	%r3
+
+	rfid
+
+	.p2align 2
+CNAME(power_save_sequence):
+	bl	1f
+	.llong	0x0			/* Playground for power-save sequence */
+1:	mflr	%r3
+
+	/* Start power-save sequence */
+	std	%r2,0(%r3)
+	ptesync
+	ld	%r2,0(%r3)
+2:	cmpd	%r2,%r2
+	bne	2b
+	nap
+	b	.


More information about the svn-src-head mailing list