aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--ChangeLog5
-rw-r--r--thread_pthread.c21
2 files changed, 19 insertions, 7 deletions
diff --git a/ChangeLog b/ChangeLog
index 44b58bfc36..a52c8bf5c1 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+Sun Mar 3 02:42:29 2013 KOSAKI Motohiro <kosaki.motohiro@gmail.com>
+
+ * thread_pthread.c (consume_communication_pipe): retry when
+ read returned CCP_READ_BUFF_SIZE.
+
Wed Mar 6 21:31:35 2013 KOSAKI Motohiro <kosaki.motohiro@gmail.com>
* thread_pthread.c (timer_thread_sleep): use poll() instead of
diff --git a/thread_pthread.c b/thread_pthread.c
index cd0aef7095..4ae7e2abe4 100644
--- a/thread_pthread.c
+++ b/thread_pthread.c
@@ -1186,13 +1186,20 @@ consume_communication_pipe(void)
static char buff[CCP_READ_BUFF_SIZE];
ssize_t result;
- retry:
- result = read(timer_thread_pipe[0], buff, CCP_READ_BUFF_SIZE);
- if (result < 0) {
- switch (errno) {
- case EINTR: goto retry;
- default:
- rb_async_bug_errno("consume_communication_pipe: read\n", errno);
+ while (1) {
+ result = read(timer_thread_pipe[0], buff, sizeof(buff));
+ if (result == 0) {
+ return;
+ }
+ else if (result < 0) {
+ switch (errno) {
+ case EINTR:
+ continue; /* retry */
+ case EAGAIN:
+ return;
+ default:
+ rb_async_bug_errno("consume_communication_pipe: read\n", errno);
+ }
}
}
}