Browse Source

um: merge hard_handler() instances, switch to use of SA_SIGINFO

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Richard Weinberger <richard@nod.at>
Al Viro 14 years ago
parent
commit
7eb122555c

+ 8 - 4
arch/um/os-Linux/signal.c

@@ -127,7 +127,7 @@ void set_sigstack(void *sig_stack, int size)
 
 
 static void (*handlers[_NSIG])(int sig, struct sigcontext *sc);
 static void (*handlers[_NSIG])(int sig, struct sigcontext *sc);
 
 
-void handle_signal(int sig, struct sigcontext *sc)
+static void handle_signal(int sig, struct sigcontext *sc)
 {
 {
 	unsigned long pending = 1UL << sig;
 	unsigned long pending = 1UL << sig;
 
 
@@ -168,7 +168,11 @@ void handle_signal(int sig, struct sigcontext *sc)
 	} while (pending);
 	} while (pending);
 }
 }
 
 
-extern void hard_handler(int sig);
+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, void (*handler)(int), int flags, ...)
 void set_handler(int sig, void (*handler)(int), int flags, ...)
 {
 {
@@ -178,7 +182,7 @@ void set_handler(int sig, void (*handler)(int), int flags, ...)
 	int mask;
 	int mask;
 
 
 	handlers[sig] = (void (*)(int, struct sigcontext *)) handler;
 	handlers[sig] = (void (*)(int, struct sigcontext *)) handler;
-	action.sa_handler = hard_handler;
+	action.sa_sigaction = hard_handler;
 
 
 	sigemptyset(&action.sa_mask);
 	sigemptyset(&action.sa_mask);
 
 
@@ -190,7 +194,7 @@ void set_handler(int sig, void (*handler)(int), int flags, ...)
 	if (sig == SIGSEGV)
 	if (sig == SIGSEGV)
 		flags |= SA_NODEFER;
 		flags |= SA_NODEFER;
 
 
-	action.sa_flags = flags;
+	action.sa_flags = flags | SA_SIGINFO;
 	action.sa_restorer = NULL;
 	action.sa_restorer = NULL;
 	if (sigaction(sig, &action, NULL) < 0)
 	if (sigaction(sig, &action, NULL) < 0)
 		panic("sigaction failed - errno = %d\n", errno);
 		panic("sigaction failed - errno = %d\n", errno);

+ 1 - 1
arch/um/os-Linux/sys-i386/Makefile

@@ -3,7 +3,7 @@
 # Licensed under the GPL
 # Licensed under the GPL
 #
 #
 
 
-obj-y = registers.o signal.o task_size.o tls.o
+obj-y = registers.o task_size.o tls.o
 
 
 USER_OBJS := $(obj-y)
 USER_OBJS := $(obj-y)
 
 

+ 0 - 13
arch/um/os-Linux/sys-i386/signal.c

@@ -1,13 +0,0 @@
-/*
- * Copyright (C) 2006 Jeff Dike (jdike@{addtoit,linux.intel}.com)
- * Licensed under the GPL
- */
-
-#include <signal.h>
-
-extern void handle_signal(int sig, struct sigcontext *sc);
-
-void hard_handler(int sig)
-{
-	handle_signal(sig, (struct sigcontext *) (&sig + 1));
-}

+ 1 - 1
arch/um/os-Linux/sys-x86_64/Makefile

@@ -3,7 +3,7 @@
 # Licensed under the GPL
 # Licensed under the GPL
 #
 #
 
 
-obj-y = registers.o prctl.o signal.o task_size.o
+obj-y = registers.o prctl.o task_size.o
 
 
 USER_OBJS := $(obj-y)
 USER_OBJS := $(obj-y)
 
 

+ 0 - 16
arch/um/os-Linux/sys-x86_64/signal.c

@@ -1,16 +0,0 @@
-/*
- * Copyright (C) 2006 Jeff Dike (jdike@{addtoit,linux.intel}.com)
- * Licensed under the GPL
- */
-
-#include <signal.h>
-
-extern void handle_signal(int sig, struct sigcontext *sc);
-
-void hard_handler(int sig)
-{
-	struct ucontext *uc;
-	asm("movq %%rdx, %0" : "=r" (uc));
-
-	handle_signal(sig, (struct sigcontext *) &uc->uc_mcontext);
-}