summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--ChangeLog8
-rw-r--r--signal.c6
-rw-r--r--thread_pthread.c13
3 files changed, 23 insertions, 4 deletions
diff --git a/ChangeLog b/ChangeLog
index 145ee91950..d1d03204fb 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+Fri Jan 2 06:43:58 2009 NARUSE, Yui <naruse@ruby-lang.org>
+
+ * signal.c (rb_signal_buff_size): defined for check whether signal
+ is in the buffer or not.
+
+ * thread_pthread.c (thread_timer): don't exit the loop when signal
+ is in the buffer. [ruby-dev:37637]
+
Fri Jan 2 04:40:00 2009 Tanaka Akira <akr@fsij.org>
* ext/socket/mkconstants.rb: check duplicates.
diff --git a/signal.c b/signal.c
index 91aa7c418e..8d75c3bc5c 100644
--- a/signal.c
+++ b/signal.c
@@ -514,6 +514,12 @@ sighandler(int sig)
#endif
}
+int
+rb_signal_buff_size()
+{
+ return signal_buff.size;
+}
+
#if USE_TRAP_MASK
# ifdef HAVE_SIGPROCMASK
static sigset_t trap_last_mask;
diff --git a/thread_pthread.c b/thread_pthread.c
index a23e7727db..72f8a2f288 100644
--- a/thread_pthread.c
+++ b/thread_pthread.c
@@ -741,19 +741,24 @@ get_ts(struct timespec *ts, unsigned long nsec)
return ts;
}
+int rb_signal_buff_size(void);
+
static void *
thread_timer(void *dummy)
{
struct timespec ts;
- int err;
native_mutex_lock(&timer_thread_lock);
native_cond_broadcast(&timer_thread_cond);
#define WAIT_FOR_10MS() native_cond_timedwait(&timer_thread_cond, &timer_thread_lock, get_ts(&ts, PER_NANO/100))
- while (system_working > 0 && (err = WAIT_FOR_10MS()) != 0 && err != EINTR) {
- if (err != ETIMEDOUT) {
- rb_bug("thread_timer/timedwait: %d", err);
+ while (system_working > 0) {
+ int err = WAIT_FOR_10MS();
+ if (err == ETIMEDOUT);
+ else if (err == 0 || err == EINTR) {
+ if (rb_signal_buff_size() == 0) break;
}
+ else rb_bug("thread_timer/timedwait: %d", err);
+
#ifndef __CYGWIN__
if (signal_thread_list_anchor.next) {
FGLOCK(&signal_thread_list_lock, {