ruby-dev

こんにちは、
バンサンです。

このメールはこの前、ruby-coreに（ruby-core:11584）英語で送ったんですけど、答えがなかったからruby-devに、今回日本語で、送ろうと思ったんです…

Mac OS Tiger(10.4)では、こういう問題があって：
% ruby -ve 'require "timeout" ; Timeout.timeout(5) { puts `df -h` }'
ruby 1.8.6 (2007-06-07 patchlevel 36) [i686-darwin8.10.1]
/usr/local/lib/ruby/1.8/timeout.rb:54: execution expired (Timeout::Error)
        from /usr/local/lib/ruby/1.8/timeout.rb:56:in `timeout'
        from -e:1

patchlevel 39でも同じ問題が起こります。ですけど--disable-pthreadsをついたら、それとも1.8のブランチや1.8.6-p0では、問題は起こらないです。

もっと調べたら、済んでない関数はrb_thread_cancel_timer
(eval.c)。この関数は1.8.6-p36や-p39だけでproc_exec_v (process.h)に呼ばれてます。
そしてpthreadを使わない場合、この関数が空っぽになります

それでrb_thread_cancel_timerで実際に何が起こるのか、そしてどうしてTiger以外のOS（Linux,
Leopard…）でどうして問題が起こらないかを調べたんです。理由がTigerのバグであるみたいです。
実際にRubyのsystem()や``を使ったら起こるのがこういうこと：
＊プロセスがフォルクされて、親が興味あることをしない。でも子は以下のことをする。
＊timer threadをkillするために、rb_thread_cancel_timerが呼ばれている。だけどこのプロセスはフォルクで作られてるから、timer
threadがない（forkは子ではcurrent
thread以外全てのthreadを殺す）。それで、子の変数のtime_threadが親のthreadを参照する。このthreadがこのプロセスのthreadでないから大部分のOSではrb_thread_cancel_timerのpthread_cancelとpthread_joinの両方も失敗する。けど、Tigerではpthread_cancelが成功すてpthread_cancelが永遠に待つ。


どうしてrb_thread_cancel_timerが呼ばれたかを調べたら（ruby-dev:30581）、これは消せないと思います。でもCのforkの子から直接呼ぶのもダメです。

このメールに小さい（pthread_atforkを使ってる）パッチを添付したんです。でもこのパッチは本当に正しい直し方であるかどうか…わからないです。

Attachments (1)

thread_not_alive_anymore.patch (507 Bytes, text/x-diff)

Index: eval.c
===================================================================
--- eval.c	(revision 12679)
+++ eval.c	(working copy)
@@ -11826,6 +11826,12 @@
 {
 }
 
+static void
+rb_child_at_fork(void)
+{
+    time_thread_alive_p = 0;
+}
+
 void
 rb_thread_cancel_timer()
 {
@@ -11920,6 +11926,7 @@
 #ifdef _THREAD_SAFE
 	pthread_create(&time_thread, 0, thread_timer, 0);
         time_thread_alive_p = 1;
+        pthread_atfork(NULL, NULL, rb_child_at_fork);
 #else
 	rb_thread_start_timer();
 #endif

Thread

Prev Next

In This Thread

Prev Next

[#31101] exception when converting block arg — "Yusuke ENDOH" <mame@...>

[#31102] [*0];0 dumps core — "Yusuke ENDOH" <mame@...>

[#31104] v0,(*,v1,) = 0 dumps core — Tanaka Akira <akr@...>

[#31107] *v1,(*) = 0: Stack consistency error — Tanaka Akira <akr@...>

[#31109] assignment syntax — Tanaka Akira <akr@...>

[#31110] ary[]+=1 dumps core — "Yusuke ENDOH" <mame@...>

[#31118] 0..$1=0 dumps core — "Yusuke ENDOH" <mame@...>

[#31119] loop{x+=1&&next} dumps core — "Yusuke ENDOH" <mame@...>

[#31132] (v,) = [[]] — Tanaka Akira <akr@...>

[#31136] *a = *nil — Tanaka Akira <akr@...>

[#31138] nil.to_splat — Tanaka Akira <akr@...>

[#31140] ripper.y:3235: error: syntax error before '}' token — pegacorn <subscriber.jp@...>

[#31141] SEGV by m {|(*,v)|} — Tanaka Akira <akr@...>

[#31142] yield :k=>:v, *[] — Tanaka Akira <akr@...>

[#31143] m {|(*,(*)),|} — Tanaka Akira <akr@...>

[#31145] make test result — SASADA Koichi <ko1@...>

[#31147] Proc from yield — Tanaka Akira <akr@...>

[#31148] lib/irb/completion.rb needs .to_s — "NARUSE, Yui" <naruse@...>

[#31149] net/http: path.empty? was removed when path is nil — "NARUSE, Yui" <naruse@...>

[#31151] trunk: rdoc: format.rb 処理中に Bus error — pegacorn <subscriber.jp@...>

[#31153] syntax error: m {|(*,a,b)|} — Tanaka Akira <akr@...>

[#31160] Stack consistency error: m {|(v,(*))|} — Tanaka Akira <akr@...>

[#31162] rb_eval_string("lambda{}") dumps core — Masahiro Sakai (酒井政裕) <masahiro.sakai@...>

[#31163] Invalid read of size 4: valgrind ./ruby -ve 'print "a"' — Tanaka Akira <akr@...>

[#31164] ruby_set_current_source remains in intern.h — Masahiro Sakai (酒井政裕) <masahiro.sakai@...>

[#31166] is_ruby_native_thread() — Masahiro Sakai (酒井政裕) <masahiro.sakai@...>

[#31168] 構造体オブジェクトのcloneメソッド呼び出しでメモリリーク発生 — m-ohkubo@... (Mitsuhiko OHKUBO)

[#31177] [Fwd: Netrc?] — Shugo Maeda <shugo@...>

[#31189] 0x3fffffff.succ — Tanaka Akira <akr@...>

[#31190] 0x3fffffffffffffff.succ — Tanaka Akira <akr@...>

[#31197] ary.flatten("") — Tanaka Akira <akr@...>

[#31198] (0...0).max — Tanaka Akira <akr@...>

[#31199] Re: [ruby-cvs:19979] Ruby:r12743 (trunk): * range.c (range_max): use FIX2LONG instead of FIX2INT to avoid — "U.Nakamura" <usa@...>

[#31208] mswin32 on cygwin zsh/bash — "NAKAMURA, Hiroshi" <nakahiro@...>

[#31210] (-0x40000000)/(-1) — Tanaka Akira <akr@...>

[#31211] ruby_1_8_6がtest_condvar_wait_not_owner(TC_Thread)で1F — Kazuhiro NISHIYAMA <zn@...>

[#31212] 0.round — Tanaka Akira <akr@...>

[#31213] 24 ** 7 — Tanaka Akira <akr@...>

[#31214] Warning: OpenSSL::PKCS7::PKCS7 is deprecated after Ruby 1.9; use OpenSSL::PKCS7 instead — Kazuhiro NISHIYAMA <zn@...>

[#31215] 65538 ** 2 — Tanaka Akira <akr@...>

[#31216] 1.8 floating point exception: 0 ** 2 — Tanaka Akira <akr@...>

[#31217] trunk: generator.c:150: warning: unsigned int format, long int arg (arg 3) — pegacorn <subscriber.jp@...>

[#31221] Base64.b64encode len bug — "Kazuhiro NISHIYAMA" <zn@...>

[#31222] trunk: バグを指摘している警告 — pegacorn <subscriber.jp@...>

[#31225] Invalid read/write by ~big_zero. — Tanaka Akira <akr@...>

[#31229] ~big_zero after big_zero % big_neg — Tanaka Akira <akr@...>

[#31236] rb_big_pow — Tanaka Akira <akr@...>

[#31242] p(65536**(1<<29)) stalls — "Yusuke ENDOH" <mame@...>

[#31243] ruby -ne ! dumps core — "Yusuke ENDOH" <mame@...>

[#31244] shift — Tanaka Akira <akr@...>

[#31252] [()] dumps core — "Yusuke ENDOH" <mame@...>

[#31253] [PATCH] Problem with ruby 1.8.6-p36 (and p39) on Tiger — "Vincent Isambart" <vincent.isambart@...>

[#31261] [ruby-bugs-12398] Redefining Exception#backtrace and unhandled exception bug — Nobuyoshi Nakada <nobu@...>

[#31263] !_=() and 0..() dump core — "Yusuke ENDOH" <mame@...>

[#31264] +:x doesn't raise exception — "Yusuke ENDOH" <mame@...>

[#31265] "ab".rindex(/ab/, 0) => nil — Tanaka Akira <akr@...>

[#31271] (1 << 0x40000000)[0x40000000] — Tanaka Akira <akr@...>

[#31285] p()#=>[] — eklerni <eklerni@...>

[#31292] ParseDate.parsedate("Tuesday, July 6th, 2007, 18:35:20 UTC") — Tanaka Akira <akr@...>

[#31293] a = -(1) : warning: empty expression — Tanaka Akira <akr@...>

[#31296] ("a".."z")==="g"#=>false — eklerni <eklerni@...>

[#31298] retryの使い方 — eklerni <eklerni@...>

[#31304] rb_thread_statusが未定義? — Kentaro KAWAMOTO <kentaro.kawamoto@...>

[#31312] faster Bignum#to_s — "Yusuke ENDOH" <mame@...>

[#31318] URI and RFC 3986 (Re: URI::merge bug?) — Nobuyoshi Nakada <nobu@...>

[#31319] DelegatorおよびWeakRefのメモリ使用量 — "GO Noguchi" <gonoguti@...>

[ruby-dev:31253] [PATCH] Problem with ruby 1.8.6-p36 (and p39) on Tiger

Attachments (1)

Thread

In This Thread

[#31107] v1,() = 0: Stack consistency error — Tanaka Akira <akr@...>

[#31136] a = nil — Tanaka Akira <akr@...>

[#31143] m {|(,()),|} — Tanaka Akira <akr@...>