Source-Changes-HG archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
[src/netbsd-7]: src/sys/arch/powerpc Pull up following revision(s) (requested...
details: https://anonhg.NetBSD.org/src/rev/73ee02a37c01
branches: netbsd-7
changeset: 799478:73ee02a37c01
user: snj <snj%NetBSD.org@localhost>
date: Fri Jul 17 03:34:01 2015 +0000
description:
Pull up following revision(s) (requested by matt in ticket #868):
sys/arch/powerpc/include/psl.h: revision 1.20
sys/arch/powerpc/include/userret.h: revisions 1.29, 1.30
sys/arch/powerpc/oea/altivec.c: revision 1.30
sys/arch/powerpc/oea/oea_machdep.c: revision 1.72
sys/arch/powerpc/powerpc/fpu.c: revision 1.36
Add a check to make sure that if PSL_FP is set, we own the FPU.
--
Don't clear PSL_FP/PSL_VEC
--
Don't reload if just re-enabling
--
Don't reload the FPU register if this is just a re-enable.
--
Remove PSL_SPV from BOOKE PSL_USERMOD
--
Don't preserve PSL_FP/PSL_VEC in cpu_setmcontext.
diffstat:
sys/arch/powerpc/include/psl.h | 4 +-
sys/arch/powerpc/include/userret.h | 9 ++++++-
sys/arch/powerpc/oea/altivec.c | 40 +++++++++++++++++++------------------
sys/arch/powerpc/oea/oea_machdep.c | 11 +++------
sys/arch/powerpc/powerpc/fpu.c | 20 ++++++++++--------
5 files changed, 45 insertions(+), 39 deletions(-)
diffs (194 lines):
diff -r 99156a400a0c -r 73ee02a37c01 sys/arch/powerpc/include/psl.h
--- a/sys/arch/powerpc/include/psl.h Thu Jul 16 21:57:42 2015 +0000
+++ b/sys/arch/powerpc/include/psl.h Fri Jul 17 03:34:01 2015 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: psl.h,v 1.19 2014/02/28 05:27:38 matt Exp $ */
+/* $NetBSD: psl.h,v 1.19.4.1 2015/07/17 03:34:01 snj Exp $ */
/*
* Copyright (C) 1995, 1996 Wolfgang Solfrank.
@@ -112,7 +112,7 @@
#elif defined(PPC_BOOKE)
#define PSL_USERSET (PSL_EE | PSL_PR | PSL_IS | PSL_DS | PSL_ME | PSL_CE | PSL_DE)
#define PSL_USERMASK (PSL_SPV | PSL_CE | 0xFFFF)
-#define PSL_USERMOD (PSL_SPV)
+#define PSL_USERMOD (0)
#else /* PPC_IBM4XX */
#ifdef PPC_IBM403
#define PSL_USERSET (PSL_EE | PSL_PR | PSL_IR | PSL_DR | PSL_ME)
diff -r 99156a400a0c -r 73ee02a37c01 sys/arch/powerpc/include/userret.h
--- a/sys/arch/powerpc/include/userret.h Thu Jul 16 21:57:42 2015 +0000
+++ b/sys/arch/powerpc/include/userret.h Fri Jul 17 03:34:01 2015 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: userret.h,v 1.28 2014/03/10 00:04:04 matt Exp $ */
+/* $NetBSD: userret.h,v 1.28.4.1 2015/07/17 03:34:01 snj Exp $ */
/*
* Copyright (C) 1995, 1996 Wolfgang Solfrank.
@@ -61,8 +61,13 @@
KASSERTMSG((tf->tf_srr1 & PSL_PR) != 0,
"tf=%p: srr1 (%#lx): PSL_PR isn't set!",
tf, tf->tf_srr1);
+ KASSERTMSG((tf->tf_srr1 & PSL_FP) == 0
+ || l->l_cpu->ci_data.cpu_pcu_curlwp[PCU_FPU] == l,
+ "tf=%p: srr1 (%#lx): PSL_FP set but FPU curlwp %p is not curlwp %p!",
+ tf, tf->tf_srr1, l->l_cpu->ci_data.cpu_pcu_curlwp[PCU_FPU], l);
- tf->tf_srr1 &= PSL_USERSRR1; /* clear SRR1 status bits */
+ /* clear SRR1 status bits */
+ tf->tf_srr1 &= (PSL_USERSRR1|PSL_FP|PSL_VEC);
#ifdef ALTIVEC
/*
diff -r 99156a400a0c -r 73ee02a37c01 sys/arch/powerpc/oea/altivec.c
--- a/sys/arch/powerpc/oea/altivec.c Thu Jul 16 21:57:42 2015 +0000
+++ b/sys/arch/powerpc/oea/altivec.c Fri Jul 17 03:34:01 2015 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: altivec.c,v 1.29 2014/05/16 00:48:41 rmind Exp $ */
+/* $NetBSD: altivec.c,v 1.29.2.1 2015/07/17 03:34:01 snj Exp $ */
/*
* Copyright (C) 1996 Wolfgang Solfrank.
@@ -32,7 +32,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: altivec.c,v 1.29 2014/05/16 00:48:41 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: altivec.c,v 1.29.2.1 2015/07/17 03:34:01 snj Exp $");
#include "opt_multiprocessor.h"
@@ -82,18 +82,26 @@
vec_mark_used(l);
}
- /*
- * Enable AltiVec temporarily (and disable interrupts).
- */
- const register_t msr = mfmsr();
- mtmsr((msr & ~PSL_EE) | PSL_VEC);
- __asm volatile ("isync");
+ if ((flags & PCU_REENABLE) == 0) {
+ /*
+ * Enable AltiVec temporarily (and disable interrupts).
+ */
+ const register_t msr = mfmsr();
+ mtmsr((msr & ~PSL_EE) | PSL_VEC);
+ __asm volatile ("isync");
- /*
- * Load the vector unit from vreg which is best done in
- * assembly.
- */
- vec_load_from_vreg(&pcb->pcb_vr);
+ /*
+ * Load the vector unit from vreg which is best done in
+ * assembly.
+ */
+ vec_load_from_vreg(&pcb->pcb_vr);
+
+ /*
+ * Restore MSR (turn off AltiVec)
+ */
+ mtmsr(msr);
+ __asm volatile ("isync");
+ }
/*
* VRSAVE will be restored when trap frame returns
@@ -101,12 +109,6 @@
l->l_md.md_utf->tf_vrsave = pcb->pcb_vr.vrsave;
/*
- * Restore MSR (turn off AltiVec)
- */
- mtmsr(msr);
- __asm volatile ("isync");
-
- /*
* Mark vector registers as modified.
*/
l->l_md.md_flags |= PSL_VEC;
diff -r 99156a400a0c -r 73ee02a37c01 sys/arch/powerpc/oea/oea_machdep.c
--- a/sys/arch/powerpc/oea/oea_machdep.c Thu Jul 16 21:57:42 2015 +0000
+++ b/sys/arch/powerpc/oea/oea_machdep.c Fri Jul 17 03:34:01 2015 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: oea_machdep.c,v 1.71 2014/03/24 19:29:59 christos Exp $ */
+/* $NetBSD: oea_machdep.c,v 1.71.4.1 2015/07/17 03:34:01 snj Exp $ */
/*
* Copyright (C) 2002 Matt Thomas
@@ -33,7 +33,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: oea_machdep.c,v 1.71 2014/03/24 19:29:59 christos Exp $");
+__KERNEL_RCSID(0, "$NetBSD: oea_machdep.c,v 1.71.4.1 2015/07/17 03:34:01 snj Exp $");
#include "opt_ppcarch.h"
#include "opt_compat_netbsd.h"
@@ -440,19 +440,16 @@
/*
* Configure a PSL user mask matching this processor.
+ * Don't allow to set PSL_FP/PSL_VEC, since that will affect PCU.
*/
cpu_psluserset = PSL_EE | PSL_PR | PSL_ME | PSL_IR | PSL_DR | PSL_RI;
- cpu_pslusermod = PSL_FP | PSL_FE0 | PSL_FE1 | PSL_LE | PSL_SE | PSL_BE;
+ cpu_pslusermod = PSL_FE0 | PSL_FE1 | PSL_LE | PSL_SE | PSL_BE;
#ifdef PPC_OEA601
if (cpuvers == MPC601) {
cpu_psluserset &= PSL_601_MASK;
cpu_pslusermod &= PSL_601_MASK;
}
#endif
-#ifdef ALTIVEC
- if (cpu_altivec)
- cpu_pslusermod |= PSL_VEC;
-#endif
#ifdef PPC_HIGH_VEC
cpu_psluserset |= PSL_IP; /* XXX ok? */
#endif
diff -r 99156a400a0c -r 73ee02a37c01 sys/arch/powerpc/powerpc/fpu.c
--- a/sys/arch/powerpc/powerpc/fpu.c Thu Jul 16 21:57:42 2015 +0000
+++ b/sys/arch/powerpc/powerpc/fpu.c Fri Jul 17 03:34:01 2015 +0000
@@ -1,4 +1,4 @@
-/* $NetBSD: fpu.c,v 1.35 2014/05/16 00:48:41 rmind Exp $ */
+/* $NetBSD: fpu.c,v 1.35.2.1 2015/07/17 03:34:01 snj Exp $ */
/*
* Copyright (C) 1996 Wolfgang Solfrank.
@@ -32,7 +32,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: fpu.c,v 1.35 2014/05/16 00:48:41 rmind Exp $");
+__KERNEL_RCSID(0, "$NetBSD: fpu.c,v 1.35.2.1 2015/07/17 03:34:01 snj Exp $");
#include "opt_multiprocessor.h"
@@ -84,15 +84,17 @@
memset(&pcb->pcb_fpu, 0, sizeof(pcb->pcb_fpu));
}
- const register_t msr = mfmsr();
- mtmsr((msr & ~PSL_EE) | PSL_FP);
- __asm volatile ("isync");
+ if ((flags & PCU_REENABLE) == 0) {
+ const register_t msr = mfmsr();
+ mtmsr((msr & ~PSL_EE) | PSL_FP);
+ __asm volatile ("isync");
- fpu_load_from_fpreg(&pcb->pcb_fpu);
- __asm volatile ("sync");
+ fpu_load_from_fpreg(&pcb->pcb_fpu);
+ __asm volatile ("sync");
- mtmsr(msr);
- __asm volatile ("isync");
+ mtmsr(msr);
+ __asm volatile ("isync");
+ }
curcpu()->ci_ev_fpusw.ev_count++;
l->l_md.md_utf->tf_srr1 |= PSL_FP|(pcb->pcb_flags & (PCB_FE0|PCB_FE1));
Home |
Main Index |
Thread Index |
Old Index