Backport #7720
closedProcess.daemon does not clean up background Ruby threads, leading to hang at exit
Description
Process.fork appears to make special considerations for Ruby background threads so that they are cleaned up in the child process after fork (since fork() only duplicates the calling thread). Process.daemon also relies on fork internally, but does not make the same considerations, leading to a hang at exit in a daemonized process that had multiple Ruby threads prior to calling Process.daemon.
The attached daemon.rb demonstrates the issue. It will call either Process.daemon (default) or Process.fork (if you pass 'fork' as the first arg).
If you pass nothing, causing it to call Process.daemon, the daemonized Ruby process will still think it has two threads, and will hang indefinitely on exit, in a loop in rb_thread_terminate_all. If you pass 'fork' as the first argument, the forked child does not hang, and appears to have only one Ruby thread (the one that called Process.fork).
It looks like rb_thread_atfork() is the relevant call in the fork case that makes this work, but it doesn't appear to be called in the daemon case.
Files
Updated by samg (Sam Goldstein) over 11 years ago
This seems to be the underlying cause of the bad interaction between resque and new relic described at https://github.com/defunkt/resque/issues/578
Updated by ko1 (Koichi Sasada) over 11 years ago
Does this issue remain on trunk?
(Should we modify trunk before 2.0?)
Updated by kosaki (Motohiro KOSAKI) over 11 years ago
trunk doesn't have an issue. fork and daemon make the same result.
Updated by usa (Usaku NAKAMURA) over 11 years ago
- Status changed from Open to Assigned
- Assignee set to usa (Usaku NAKAMURA)
So, what's the difference?
Updated by usa (Usaku NAKAMURA) over 11 years ago
- Tracker changed from Bug to Backport
- Project changed from Ruby master to Backport193
Updated by usa (Usaku NAKAMURA) almost 7 years ago
- Status changed from Assigned to Rejected
1.9.3 is out of date