[ruby-dev:12914] Re: recursive malloc / fork deadlock / thread deadlock

ごとうゆうぞうです。

えっと、make world し直してから ruby を作り直すと、差
が出なくなってしまいました。なんかフに落ちませんが、こ
の話は一旦打ち切ったほうがいいかなあと思います。
お騒がせしてしまってすみません。

In message <[e-mail]>,
 `Jun Adachi <[e-mail]>' wrote:
> ファイル構造体の示している実体がttyの場合または文字列などを疑似的にファ
> イルに見せている場合にはBUFSIZで、実体がファイルの場合には、ファイルシ
> ステムのブロックサイズなのだろうと思います。

そうですね。ウソを書いていました(__;;

神戸さん:
> > NetBSDのsetvbuf(3)によると、
(略)
> > とあって、sizeをゼロでない値としてbufにNULLを指定することで、即座にバッ
> > ファの割り当てを受けると期待することの方が拡張だという記述があります。
> > 従って、sizeをゼロで、bufはNULLの方が移植性という面では良さそうに思え
> > ます。

わたしもそう思います。というわけで、基本はそのままでパッ
チを作り直しました。1.6 にもそのままあたると思います。
malloc の再入は、

  ruby -e 'while true; Thread.start{`ls`} end'
  ruby -e 'while true; Thread.start{open("/dev/zero"){|io| io.read(100)}}end'

のような感じで放っておけば、起きると思いますが、このパッ
チで起こらなくなると思います。

--
ごとうゆうぞう

--- io.c.orig	Fri Apr 13 04:57:38 2001
+++ io.c	Fri Apr 13 06:19:29 2001
@@ -1379,6 +1379,10 @@
 	    rb_sys_fail(fname);
 	}
     }
+#if defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__) || defined(sun)
+    if(setvbuf(file, NULL, _IOFBF, 0) != 0)
+	rb_warn("setvbuf() can't be honered for %s.", fname);
+#endif
 #ifdef __human68k__
     fmode(file, _IOTEXT);
 #endif
@@ -1402,6 +1406,11 @@
 	    rb_sys_fail(0);
 	}
     }
+#if defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__) || defined(sun)
+    if(setvbuf(file, NULL, _IOFBF, 0) != 0)
+	rb_warn("setvbuf() can't be honered (fd=%d).", fd);
+#endif
+
     return file;
 }
 
@@ -1925,10 +1934,19 @@
     if (freopen(RSTRING(fname)->ptr, mode, fptr->f) == 0) {
 	rb_sys_fail(fptr->path);
     }
+#if defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__) || defined(sun)
+    if (setvbuf(fptr->f, NULL, _IOFBF, 0) != 0)
+	rb_warn("setvbuf() can't be honered for fptr->f.");
+#endif
+
     if (fptr->f2) {
 	if (freopen(RSTRING(fname)->ptr, "w", fptr->f2) == 0) {
 	    rb_sys_fail(fptr->path);
 	}
+#if defined(__FreeBSD__) || defined(__NetBSD__) || defined(__OpenBSD__) || defined(sun)
+	if (setvbuf(fptr->f2, NULL, _IOFBF, 0) != 0)
+	    rb_warn("setvbuf() can't be honered for fptr->f2.");
+#endif
     }
 
     return file;