From 50587316b8c66f2d18756de6bb041bd38af8d17b Mon Sep 17 00:00:00 2001 From: mame Date: Thu, 13 May 2010 16:20:26 +0000 Subject: * vm_core.c (rb_thread_struct): add a field for sigaltstack. * thread_pthread.c (thread_start_func_1): initialize machine stack information. * thread.c (thread_start_func_2): set sigaltstack for each sub thread. [ruby-core:24540] [ruby-core:30207] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@27789 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 10 ++++++++++ signal.c | 19 +++++++------------ thread.c | 5 +++++ thread_pthread.c | 1 + vm_core.h | 7 +++++++ 5 files changed, 30 insertions(+), 12 deletions(-) diff --git a/ChangeLog b/ChangeLog index 84f24c1c08..82587bc36d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,13 @@ +Fri May 14 01:17:10 2010 Yusuke Endoh + + * vm_core.c (rb_thread_struct): add a field for sigaltstack. + + * thread_pthread.c (thread_start_func_1): initialize machine stack + information. + + * thread.c (thread_start_func_2): set sigaltstack for each sub thread. + [ruby-core:24540] [ruby-core:30207] + Thu May 13 21:40:39 2010 Tanaka Akira * missing/ffs.c (ffs): fixed for non-zero values. diff --git a/signal.c b/signal.c index 3c563cf120..9ecf4fc0f2 100644 --- a/signal.c +++ b/signal.c @@ -420,10 +420,6 @@ static struct { #define sighandler_t sh_t #endif -#if defined(SIGSEGV) && defined(HAVE_SIGALTSTACK) && defined(SA_SIGINFO) && !defined(__NetBSD__) -#define USE_SIGALTSTACK -#endif - typedef RETSIGTYPE (*sighandler_t)(int); #ifdef USE_SIGALTSTACK typedef void ruby_sigaction_t(int, siginfo_t*, void*); @@ -442,18 +438,17 @@ typedef RETSIGTYPE ruby_sigaction_t(int); #define ALT_STACK_SIZE (4*1024) #endif /* alternate stack for SIGSEGV */ -static void -register_sigaltstack(void) +void +rb_register_sigaltstack(rb_thread_t *th) { - static void *altstack = 0; stack_t newSS, oldSS; - if (altstack) return; + if (th->altstack) return; - newSS.ss_sp = altstack = malloc(ALT_STACK_SIZE); + newSS.ss_sp = th->altstack = malloc(ALT_STACK_SIZE); if (newSS.ss_sp == NULL) /* should handle error */ - rb_bug("register_sigaltstack. malloc error\n"); + rb_bug("rb_register_sigaltstack. malloc error\n"); newSS.ss_size = ALT_STACK_SIZE; newSS.ss_flags = 0; @@ -737,7 +732,7 @@ default_handler(int sig) case SIGSEGV: func = (sighandler_t)sigsegv; # ifdef USE_SIGALTSTACK - register_sigaltstack(); + rb_register_sigaltstack(GET_THREAD()); # endif break; #endif @@ -1130,7 +1125,7 @@ Init_signal(void) #endif #ifdef SIGSEGV # ifdef USE_SIGALTSTACK - register_sigaltstack(); + rb_register_sigaltstack(GET_THREAD()); # endif install_sighandler(SIGSEGV, (sighandler_t)sigsegv); #endif diff --git a/thread.c b/thread.c index 8fc02ec534..485c1462eb 100644 --- a/thread.c +++ b/thread.c @@ -417,6 +417,11 @@ thread_start_func_2(rb_thread_t *th, VALUE *stack_start, VALUE *register_stack_s rb_thread_t *join_th; rb_thread_t *main_th; VALUE errinfo = Qnil; +# ifdef USE_SIGALTSTACK + void rb_register_sigaltstack(rb_thread_t *th); + + rb_register_sigaltstack(th); +# endif ruby_thread_set_native(th); diff --git a/thread_pthread.c b/thread_pthread.c index af56f01507..285852baa9 100644 --- a/thread_pthread.c +++ b/thread_pthread.c @@ -344,6 +344,7 @@ thread_start_func_1(void *th_ptr) rb_thread_t *th = th_ptr; VALUE stack_start; + native_thread_init_stack(th); /* run */ thread_start_func_2(th, &stack_start, rb_ia64_bsp()); } diff --git a/vm_core.h b/vm_core.h index 39dfe55556..1843567693 100644 --- a/vm_core.h +++ b/vm_core.h @@ -63,6 +63,10 @@ #define va_init_list(a,b) va_start(a) #endif +#if defined(SIGSEGV) && defined(HAVE_SIGALTSTACK) && defined(SA_SIGINFO) && !defined(__NetBSD__) +#define USE_SIGALTSTACK +#endif + /*****************/ /* configuration */ /*****************/ @@ -473,6 +477,9 @@ typedef struct rb_thread_struct /* misc */ int method_missing_reason; int abort_on_exception; +#ifdef USE_SIGALTSTACK + void *altstack; +#endif } rb_thread_t; /* iseq.c */ -- cgit v1.2.3