Bug #4920

Process.daemon()呼び出しによりタイマースレッドが2つ出来てしまう

Added by Motohiro KOSAKI almost 3 years ago. Updated almost 3 years ago.

[ruby-dev:43873]
Status:Closed
Priority:High
Assignee:Nobuyoshi Nakada
Category:core
Target version:1.9.3
ruby -v:ruby 1.9.3dev (2011-06-19 trunk 32170) [x86_64-linux] Backport:

Description

ささださんとakrさんが発見してくださったのですが、現在 trunkで

$ ./ruby -e 'Process.daemon(true,true); p Dir.entries("/proc/self/task")'

と実行すると、スレッドが3匹います(Linuxで実行してください)。何故かというと Process.daemon()
を契機にタイマースレッドが2つ出来てしまうからです。

原因は以下のコードです


static VALUE
proc_daemon(int argc, VALUE *argv)
{
VALUE nochdir, noclose;
int n;

rb_secure(2);
rb_scan_args(argc, argv, "02", &nochdir, &noclose);

prefork();
before_fork();
n = daemon(RTEST(nochdir), RTEST(noclose));
after_fork();
if (n < 0) rb_sys_fail("daemon");
return INT2FIX(n);

}

BSD以外の環境ではdaemon()はOSのdaemon(3)ではなく、rbdaemon()を使用し、
rb
daemonは内部的にrbforkerr()を使用するため結果的に

before_fork();
before_fork();
fork();
after_fork();
after_fork();

という呼び出し順序になり、after_fork()のたびにタイマースレッドをつくるので合計2スレッド。

もちろん、1.9.2ではちゃんと動くのでこれはリグレッションです。
さすがに、これはダメだろうと思うので時期が時期だけに恐縮ですがターゲットを1.9.3に
設定させていただきます。

History

#1 Updated by Motohiro KOSAKI almost 3 years ago

直接の原因はこのコミットのようです。

commit afbd5661a0ca041ac818c9e5732911c5db21c9f9
Author: nobu nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Date: Tue Jul 13 12:31:17 2010 +0000

* process.c (rb_daemon): split from proc_daemon.


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@28630 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

process.c | 27 +++++++++++++++++++--------
1 files changed, 19 insertions(+), 8 deletions(-)

diff --git a/process.c b/process.c
index 8e601ba..84fed60 100644
--- a/process.c
+++ b/process.c
@@ -4554,6 +4554,11 @@ proc_setmaxgroups(VALUE obj, VALUE val)
}

#if defined(HAVEDAEMON) || (defined(HAVEFORK) && defined(HAVESETSID))
+#ifndef HAVE
DAEMON
+static int rbdaemon(int nochdir, int noclose);
+#define daemon(nochdir, noclose) rb
daemon(nochdir, noclose)
+#endif
+
/*
* call-seq:
* Process.daemon() -> 0
@@ -4577,14 +4582,20 @@ procdaemon(int argc, VALUE *argv)
rb
secure(2);
rbscanargs(argc, argv, "02", &nochdir, &noclose);

-#if defined(HAVEDAEMON)
prefork();
before
fork();
n = daemon(RTEST(nochdir), RTEST(noclose));
afterfork();
if (n < 0) rb
sysfail("daemon");
return INT2FIX(n);
-#elif defined(HAVE
FORK)
+}
+
+#ifndef HAVEDAEMON
+static int
+rb
daemon(int nochdir, int noclose)
+{
+ int n, err = 0;
+
switch (rbfork(0, 0, 0, Qnil)) {
case -1:
rb
sysfail("daemon");
@@ -4599,26 +4610,26 @@ proc
daemon(int argc, VALUE argv)
/
must not be process-leader */
switch (rbfork(0, 0, 0, Qnil)) {
case -1:
- rb
sysfail("daemon");
+ return -1;
case 0:
break;
default:
_exit(EXIT
SUCCESS);
}

  • if (!RTEST(nochdir))
  • (void)chdir("/");
  • if (!nochdir)
  •   err = chdir("/");
    
  • if (!RTEST(noclose) && (n = open("/dev/null", O_RDWR, 0)) != -1) {

  • if (!noclose && (n = open("/dev/null", O_RDWR, 0)) != -1) {
    (void)dup2(n, 0);
    (void)dup2(n, 1);
    (void)dup2(n, 2);
    if (n > 2)
    (void)close (n);
    }

  • return INT2FIX(0);
    -#endif

  • return err;
    }
    +#endif
    #else
    #define procdaemon rbf_notimplement
    #endif

#2 Updated by Motohiro KOSAKI almost 3 years ago

  • Status changed from Assigned to Closed
  • Assignee changed from Motohiro KOSAKI to Nobuyoshi Nakada

fixed by r32221.

Also available in: Atom PDF