Project

General

Profile

Actions

Bug #4920

closed

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

Added by kosaki (Motohiro KOSAKI) almost 13 years ago. Updated almost 13 years ago.

Status:
Closed
Target version:
ruby -v:
ruby 1.9.3dev (2011-06-19 trunk 32170) [x86_64-linux]
Backport:
[ruby-dev:43873]

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)ではなく、rb_daemon()を使用し、
rb_daemonは内部的にrb_fork_err()を使用するため結果的に

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

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

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

Updated by kosaki (Motohiro KOSAKI) almost 13 years ago

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

commit afbd5661a0ca041ac818c9e5732911c5db21c9f9
Author: nobu
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(HAVE_DAEMON) || (defined(HAVE_FORK) && defined(HAVE_SETSID))
+#ifndef HAVE_DAEMON
+static int rb_daemon(int nochdir, int noclose);
+#define daemon(nochdir, noclose) rb_daemon(nochdir, noclose)
+#endif
+
/*

  • call-seq:
  • Process.daemon()                        -> 0
    

@@ -4577,14 +4582,20 @@ proc_daemon(int argc, VALUE *argv)
rb_secure(2);
rb_scan_args(argc, argv, "02", &nochdir, &noclose);

-#if defined(HAVE_DAEMON)
prefork();
before_fork();
n = daemon(RTEST(nochdir), RTEST(noclose));
after_fork();
if (n < 0) rb_sys_fail("daemon");
return INT2FIX(n);
-#elif defined(HAVE_FORK)
+}
+
+#ifndef HAVE_DAEMON
+static int
+rb_daemon(int nochdir, int noclose)
+{

  • int n, err = 0;
  • switch (rb_fork(0, 0, 0, Qnil)) {
    case -1:
    rb_sys_fail("daemon");
    @@ -4599,26 +4610,26 @@ proc_daemon(int argc, VALUE argv)
    /
    must not be process-leader */
    switch (rb_fork(0, 0, 0, Qnil)) {
    case -1:
  •   rb_sys_fail("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 proc_daemon rb_f_notimplement
    #endif

Updated by kosaki (Motohiro KOSAKI) almost 13 years ago

  • Status changed from Assigned to Closed
  • Assignee changed from kosaki (Motohiro KOSAKI) to nobu (Nobuyoshi Nakada)

fixed by r32221.

Actions

Also available in: Atom PDF

Like0
Like0Like0