Bug #8616: Process.daemon messes up threads

[Aaron Patterson]

=begin
Hi,

When I daemonize a process, it somehow messes up threads from the parent process. Here is a script to reproduce the problem:

r1, w1 = IO.pipe
r2, w2 = IO.pipe

t = Thread.new {
puts "start"
w1.write "x"
IO.select([r2])
puts "alive"
}

IO.select([r1])
Process.daemon true, true # comment this line out and everything works
puts Process.pid
w2.write "x"
t.join
puts "done"

If you run this program, there will be a ruby process in the background that never dies. If you comment out the "Process.daemon" line, the script finishes.
=end

[Akira Tanaka]

2013/7/10 tenderlovemaking (Aaron Patterson) [e-mail]:

Bug #8616: Process.daemon messes up threads
https://bugs.ruby-lang.org/issues/8616

Process.daemon cannot preserve threads because
it uses fork and fork doesn't copy other threads.

Maybe, Process.daemon should raise an exception when
multi threads.

[Evan Phoenix]

No, it shouldn't raise an exception. Process.daemon and fork both use
after_fork() in C. That function should iterate all the threads and mark
them as KILLED since they are all dead.

On Tue, Jul 9, 2013 at 6:14 PM, Tanaka Akira [e-mail] wrote:

2013/7/10 tenderlovemaking (Aaron Patterson) [e-mail]:

Bug #8616: Process.daemon messes up threads
https://bugs.ruby-lang.org/issues/8616

Process.daemon cannot preserve threads because
it uses fork and fork doesn't copy other threads.

Maybe, Process.daemon should raise an exception when
multi threads.

[Ben Weintraub]

I agree with Evan. The behavior he describes would be consistent with how threads are handled when a process forks via Process.fork in Ruby.

See this example for a demonstration: https://gist.github.com/benweint/6692546

[Ben Weintraub]

Actually, it looks like this is fixed in Ruby 2.1.0-preview1 (Thread#alive? returns false for background threads spawned before a call to Process.daemon), but still broken in 2.0.0-p247. Any chance of a backport?

[Ben Weintraub]

FWIW, it looks like the relevant change that fixed it for 2.1.0-preview1 was https://github.com/ruby/ruby/commit/300b7c80e

[Tomoyuki Chikanaga]

• Backport changed from 1.9.3: UNKNOWN, 2.0.0: UNKNOWN to 1.9.3: UNKNOWN, 2.0.0: REQUIRED

I'll check r41910 to be backported later.

This ticket can be closed (for trunk)?

[Tomoyuki Chikanaga]

• Status changed from Open to Closed

I'll backport r41886, r41903 and r41910 to ruby_2_0_0.

Thank you for your notice!

[Tomoyuki Chikanaga]

• Backport changed from 1.9.3: UNKNOWN, 2.0.0: REQUIRED to 1.9.3: UNKNOWN, 2.0.0: DONE

... and r40534 to resolve conflict.
merged r40534,r 41886 r41903, r41910 to ruby_2_0_0 at r43142.

[Usaku NAKAMURA]

• Backport changed from 1.9.3: UNKNOWN, 2.0.0: DONE to 1.9.3: DONE, 2.0.0: DONE

backported into ruby_1_9_3 at r44766.