Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src libpthread sets initial value of MSR for lwp's. However, app...



details:   https://anonhg.NetBSD.org/src/rev/3c1a35295da1
branches:  trunk
changeset: 849055:3c1a35295da1
user:      rin <rin%NetBSD.org@localhost>
date:      Thu Feb 20 07:07:02 2020 +0000

description:
libpthread sets initial value of MSR for lwp's. However, appropriate
value differs b/w oea/booke/ibm4xx, and there's no way to obtain it
from userland. Therefore, this initial value should be corrected by
cpu_setmcontext().

- Comment this in libpthread
- Add KASSERT in cpu_mcontext_validate()

diffstat:

 lib/libpthread/arch/powerpc/pthread_md.h |  7 +++++--
 sys/arch/powerpc/powerpc/sig_machdep.c   |  6 ++++--
 2 files changed, 9 insertions(+), 4 deletions(-)

diffs (48 lines):

diff -r a5d49d36e6cd -r 3c1a35295da1 lib/libpthread/arch/powerpc/pthread_md.h
--- a/lib/libpthread/arch/powerpc/pthread_md.h  Thu Feb 20 06:36:04 2020 +0000
+++ b/lib/libpthread/arch/powerpc/pthread_md.h  Thu Feb 20 07:07:02 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: pthread_md.h,v 1.7 2011/01/25 19:12:05 christos Exp $  */
+/*     $NetBSD: pthread_md.h,v 1.8 2020/02/20 07:07:02 rin Exp $       */
 
 /*
  * Copyright (c) 2001 Wasabi Systems, Inc.
@@ -53,7 +53,10 @@
 /*
  * Set initial, sane values for registers whose values aren't just
  * "don't care".
- * 0xd032 is PSL_USERSET from arch/powerpc/include/psl.h
+ *
+ * XXX
+ * "Sane value" for MSR differs between oea/booke/ibm4xx, but no way to
+ * obtain from userland. It should be corrected by cpu_setmcontext().
  */
 #define _INITCONTEXT_U_MD(ucp)                                         \
        (ucp)->uc_mcontext.__gregs[_REG_MSR] = 0xd032;
diff -r a5d49d36e6cd -r 3c1a35295da1 sys/arch/powerpc/powerpc/sig_machdep.c
--- a/sys/arch/powerpc/powerpc/sig_machdep.c    Thu Feb 20 06:36:04 2020 +0000
+++ b/sys/arch/powerpc/powerpc/sig_machdep.c    Thu Feb 20 07:07:02 2020 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: sig_machdep.c,v 1.46 2018/11/27 14:09:54 maxv Exp $    */
+/*     $NetBSD: sig_machdep.c,v 1.47 2020/02/20 07:07:02 rin Exp $     */
 
 /*
  * Copyright (C) 1995, 1996 Wolfgang Solfrank.
@@ -32,7 +32,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: sig_machdep.c,v 1.46 2018/11/27 14:09:54 maxv Exp $");
+__KERNEL_RCSID(0, "$NetBSD: sig_machdep.c,v 1.47 2020/02/20 07:07:02 rin Exp $");
 
 #include "opt_ppcarch.h"
 #include "opt_altivec.h"
@@ -191,6 +191,8 @@
 int
 cpu_mcontext_validate(struct lwp *l, const mcontext_t *mcp)
 {
+
+       KASSERT(PSL_USEROK_P(mcp->__gregs[_REG_MSR]));
        return 0;
 }
 



Home | Main Index | Thread Index | Old Index