um: start switching the references to host mcontext_t to its userland type
authorAl Viro <viro@ftp.linux.org.uk>
Thu, 18 Aug 2011 19:05:09 +0000 (20:05 +0100)
committerRichard Weinberger <richard@nod.at>
Wed, 2 Nov 2011 13:14:59 +0000 (14:14 +0100)
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Richard Weinberger <richard@nod.at>
arch/um/include/shared/process.h
arch/um/os-Linux/internal.h [new file with mode: 0644]
arch/um/os-Linux/signal.c
arch/um/os-Linux/time.c
arch/um/sys-x86/shared/sysdep/sigcontext_32.h
arch/um/sys-x86/shared/sysdep/sigcontext_64.h
arch/um/sys-x86/stub_segv.c

index 28215300fc6985f0b5baa44bc4a27daded3e3979..2ba310e1b6560365a3f509ee1b66528baf47bcc1 100644 (file)
@@ -6,9 +6,4 @@
 #ifndef __PROCESS_H__
 #define __PROCESS_H__
 
-#include <signal.h>
-
-extern void sig_handler(int sig, struct sigcontext *sc);
-extern void alarm_handler(int sig, struct sigcontext *sc);
-
 #endif
diff --git a/arch/um/os-Linux/internal.h b/arch/um/os-Linux/internal.h
new file mode 100644 (file)
index 0000000..2c3c3ec
--- /dev/null
@@ -0,0 +1 @@
+void alarm_handler(int, mcontext_t *);
index 08d30617699589f7becf333381d0643c468cef25..07d9905e44ef0184682097417e72dbf100c7c25b 100644 (file)
@@ -26,7 +26,7 @@ void (*sig_info[NSIG])(int, struct uml_pt_regs *) = {
        [SIGIO]         = sigio_handler,
        [SIGVTALRM]     = timer_handler };
 
-static void sig_handler_common(int sig, struct sigcontext *sc)
+static void sig_handler_common(int sig, mcontext_t *mc)
 {
        struct uml_pt_regs r;
        int save_errno = errno;
@@ -34,8 +34,8 @@ static void sig_handler_common(int sig, struct sigcontext *sc)
        r.is_user = 0;
        if (sig == SIGSEGV) {
                /* For segfaults, we want the data from the sigcontext. */
-               copy_sc(&r, sc);
-               GET_FAULTINFO_FROM_SC(r.faultinfo, sc);
+               copy_sc(&r, (struct sigcontext *)mc);
+               GET_FAULTINFO_FROM_MC(r.faultinfo, mc);
        }
 
        /* enable signals if sig isn't IRQ signal */
@@ -62,7 +62,7 @@ static void sig_handler_common(int sig, struct sigcontext *sc)
 static int signals_enabled;
 static unsigned int signals_pending;
 
-void sig_handler(int sig, struct sigcontext *sc)
+void sig_handler(int sig, mcontext_t *mc)
 {
        int enabled;
 
@@ -74,23 +74,23 @@ void sig_handler(int sig, struct sigcontext *sc)
 
        block_signals();
 
-       sig_handler_common(sig, sc);
+       sig_handler_common(sig, mc);
 
        set_signals(enabled);
 }
 
-static void real_alarm_handler(struct sigcontext *sc)
+static void real_alarm_handler(mcontext_t *mc)
 {
        struct uml_pt_regs regs;
 
-       if (sc != NULL)
-               copy_sc(&regs, sc);
+       if (mc != NULL)
+               copy_sc(&regs, (struct sigcontext *)mc);
        regs.is_user = 0;
        unblock_signals();
        timer_handler(SIGVTALRM, &regs);
 }
 
-void alarm_handler(int sig, struct sigcontext *sc)
+void alarm_handler(int sig, mcontext_t *mc)
 {
        int enabled;
 
@@ -102,7 +102,7 @@ void alarm_handler(int sig, struct sigcontext *sc)
 
        block_signals();
 
-       real_alarm_handler(sc);
+       real_alarm_handler(mc);
        set_signals(enabled);
 }
 
@@ -121,7 +121,7 @@ void set_sigstack(void *sig_stack, int size)
                panic("enabling signal stack failed, errno = %d\n", errno);
 }
 
-static void (*handlers[_NSIG])(int sig, struct sigcontext *sc) = {
+static void (*handlers[_NSIG])(int sig, mcontext_t *mc) = {
        [SIGSEGV] = sig_handler,
        [SIGBUS] = sig_handler,
        [SIGILL] = sig_handler,
@@ -133,8 +133,11 @@ static void (*handlers[_NSIG])(int sig, struct sigcontext *sc) = {
        [SIGVTALRM] = alarm_handler
 };
 
-static void handle_signal(int sig, struct sigcontext *sc)
+
+static void hard_handler(int sig, siginfo_t *info, void *p)
 {
+       struct ucontext *uc = p;
+       mcontext_t *mc = &uc->uc_mcontext;
        unsigned long pending = 1UL << sig;
 
        do {
@@ -160,7 +163,7 @@ static void handle_signal(int sig, struct sigcontext *sc)
                while ((sig = ffs(pending)) != 0){
                        sig--;
                        pending &= ~(1 << sig);
-                       (*handlers[sig])(sig, sc);
+                       (*handlers[sig])(sig, mc);
                }
 
                /*
@@ -174,12 +177,6 @@ static void handle_signal(int sig, struct sigcontext *sc)
        } while (pending);
 }
 
-static void hard_handler(int sig, siginfo_t *info, void *p)
-{
-       struct ucontext *uc = p;
-       handle_signal(sig, (struct sigcontext *) &uc->uc_mcontext);
-}
-
 void set_handler(int sig)
 {
        struct sigaction action;
index 995ca304f1c9e11f2a35a27c827f69786d1e6635..910499d76a678a1830f5ea634c86acfd67276493 100644 (file)
@@ -10,7 +10,7 @@
 #include <sys/time.h>
 #include "kern_util.h"
 #include "os.h"
-#include "process.h"
+#include "internal.h"
 
 int set_interval(void)
 {
index aebc2dd61f0dc86bcd6de52b558b0be3900e0922..548c3cdb743b637aab1e1a1ca545a5a1e68eca62 100644 (file)
                (fi).trap_no = SC_TRAPNO(sc); \
        }
 
+#define GET_FAULTINFO_FROM_MC(fi, mc) \
+       { \
+               (fi).cr2 = (mc)->cr2; \
+               (fi).error_code = (mc)->gregs[REG_ERR]; \
+               (fi).trap_no = (mc)->gregs[REG_TRAPNO]; \
+       }
+
 #endif
index 60d89a2fba062b4659af23341d3f8a8bd13564f7..5c4a6a359141a2b7f94080cf6acc3724a67b106b 100644 (file)
                (fi).trap_no = SC_TRAPNO(sc); \
        }
 
+#define GET_FAULTINFO_FROM_MC(fi, mc) \
+       { \
+               (fi).cr2 = (mc)->gregs[REG_CR2]; \
+               (fi).error_code = (mc)->gregs[REG_ERR]; \
+               (fi).trap_no = (mc)->gregs[REG_TRAPNO]; \
+       }
+
 #endif
index f62771cf236eb611abb3fa443951693740ae4eb5..bd2eaf69103f90f14770dacaff4a2686c603c694 100644 (file)
@@ -12,7 +12,7 @@ stub_segv_handler(int sig, siginfo_t *info, void *p)
 {
        struct ucontext *uc = p;
 
-       GET_FAULTINFO_FROM_SC(*((struct faultinfo *) STUB_DATA),
+       GET_FAULTINFO_FROM_MC(*((struct faultinfo *) STUB_DATA),
                              &uc->uc_mcontext);
        trap_myself();
 }