diff options
author | yugui <yugui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-12-29 14:41:22 +0000 |
---|---|---|
committer | yugui <yugui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2008-12-29 14:41:22 +0000 |
commit | b3ff7eb3efed3b3bf5f4cb7eeaf03b96108e13d9 (patch) | |
tree | 32859d8fdc67d86561dc3b31e8c4ef9d43b51d45 | |
parent | c7e236efc17ca06d73fd17ed2edb736ed09129d8 (diff) | |
download | ruby-b3ff7eb3efed3b3bf5f4cb7eeaf03b96108e13d9.tar.gz |
* process.c (rb_waitpid): retries waitpid when EINTR.
[ruby-core:19744].
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@21181 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | process.c | 9 | ||||
-rw-r--r-- | test/ruby/test_process.rb | 12 |
3 files changed, 21 insertions, 5 deletions
@@ -1,3 +1,8 @@ +Mon Dec 29 22:37:17 2008 Yuki Sonoda (Yugui) <yugui@yugui.jp> + + * process.c (rb_waitpid): retries waitpid when EINTR. + [ruby-core:19744]. + Mon Dec 29 23:18:52 2008 Tadayoshi Funaba <tadf@dotrb.org> * bignum.c (rb_cstr_to_inum): changed an error message. @@ -624,18 +624,17 @@ rb_waitpid(rb_pid_t pid, int *st, int flags) #ifndef NO_WAITPID struct waitpid_arg arg; +retry: arg.pid = pid; arg.st = st; arg.flags = flags; result = (rb_pid_t)rb_thread_blocking_region(rb_waitpid_blocking, &arg, RUBY_UBF_PROCESS, 0); if (result < 0) { -#if 0 if (errno == EINTR) { - rb_thread_polling(); - goto retry; - } -#endif + RUBY_VM_CHECK_INTS(); + goto retry; + } return -1; } #else /* NO_WAITPID */ diff --git a/test/ruby/test_process.rb b/test/ruby/test_process.rb index 97b48827b7..1e7b63a47a 100644 --- a/test/ruby/test_process.rb +++ b/test/ruby/test_process.rb @@ -1044,4 +1044,16 @@ class TestProcess < Test::Unit::TestCase def test_pst_inspect assert_nothing_raised { Process::Status.allocate.inspect } end + + def test_wait_and_sigchild + signal_received = [] + Signal.trap(:CHLD) { signal_received << true; puts "child died" } + pid = fork { sleep 1; exit } + Thread.start { raise } + Process.wait pid + sleep 2 + assert_equal [true], signal_received, " [ruby-core:19744]" + ensure + Signal.trap(:CHLD, 'DEFAULT') + end end |