diff options
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | thread_pthread.c | 12 |
2 files changed, 11 insertions, 7 deletions
@@ -1,3 +1,9 @@ +Tue Jan 21 20:14:55 2014 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * thread_pthread.c (rb_thread_create_timer_thread): fix for platforms + where PTHREAD_STACK_MIN is a dynamic value and not a compile-time + constant. [ruby-dev:47911] [Bug #9436] + Tue Jan 21 17:55:09 2014 Zachary Scott <e@zzak.io> * lib/uri/common.rb: [DOC] Use static w3.org uri [ci skip] diff --git a/thread_pthread.c b/thread_pthread.c index 7f2aae4482..cf0e3705af 100644 --- a/thread_pthread.c +++ b/thread_pthread.c @@ -1480,17 +1480,15 @@ rb_thread_create_timer_thread(void) } # ifdef PTHREAD_STACK_MIN { -# define TIMER_THREAD_STACK_MIN_SIZE (4096 * 4) + const size_t min_size = (4096 * 4); /* Allocate the machine stack for the timer thread * at least 16KB (4 pages). FreeBSD 8.2 AMD64 causes * machine stack overflow only with PTHREAD_STACK_MIN. */ -# if TIMER_THREAD_STACK_MIN_SIZE < PTHREAD_STACK_MIN -# undef TIMER_THREAD_STACK_MIN_SIZE -# define TIMER_THREAD_STACK_MIN_SIZE PTHREAD_STACK_MIN -# endif - pthread_attr_setstacksize(&attr, - TIMER_THREAD_STACK_MIN_SIZE + (THREAD_DEBUG ? BUFSIZ : 0)); + size_t stack_size = PTHREAD_STACK_MIN; /* may be dynamic, get only once */ + if (stack_size < min_size) stack_size = min_size; + if (THREAD_DEBUG) stack_size += BUFSIZ; + pthread_attr_setstacksize(&attr, stack_size); } # endif #endif |