diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-12-05 01:47:12 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2013-12-05 01:47:12 +0000 |
commit | c06f12031ee1f6a7f51448f0688a20d571aed880 (patch) | |
tree | 871ffb1e4d06c539e17f594d25ebe20d9a2f040b | |
parent | 3e36402e7fd71a7bac2f54fa8617cd7e30aecd9c (diff) | |
download | ruby-c06f12031ee1f6a7f51448f0688a20d571aed880.tar.gz |
gc.c: flush all deferred finalizers
* gc.c (finalize_deferred): flush all deferred finalizers while other
finalizers can get ready to run newly by lazy sweep.
[ruby-core:58833] [Bug #9205]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@43994 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | gc.c | 5 | ||||
-rw-r--r-- | test/ruby/test_gc.rb | 14 |
3 files changed, 22 insertions, 3 deletions
@@ -1,3 +1,9 @@ +Thu Dec 5 10:47:09 2013 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * gc.c (finalize_deferred): flush all deferred finalizers while other + finalizers can get ready to run newly by lazy sweep. + [ruby-core:58833] [Bug #9205] + Thu Dec 5 09:07:59 2013 Aman Gupta <ruby@tmm1.net> * gc.c (ruby_gc_set_params): Accept safe_level argument so GC tuning @@ -2055,10 +2055,9 @@ finalize_list(rb_objspace_t *objspace, RVALUE *p) static void finalize_deferred(rb_objspace_t *objspace) { - RVALUE *p = heap_pages_deferred_final; - heap_pages_deferred_final = 0; + RVALUE *p; - if (p) { + while ((p = ATOMIC_PTR_EXCHANGE(heap_pages_deferred_final, 0)) != 0) { finalize_list(objspace, p); } } diff --git a/test/ruby/test_gc.rb b/test/ruby/test_gc.rb index 034a330c85..41e09b63fb 100644 --- a/test/ruby/test_gc.rb +++ b/test/ruby/test_gc.rb @@ -238,4 +238,18 @@ class TestGc < Test::Unit::TestCase assert_not_nil GC::INTERNAL_CONSTANTS[:HEAP_OBJ_LIMIT] assert_not_nil GC::INTERNAL_CONSTANTS[:RVALUE_SIZE] end + + def test_sweep_in_finalizer + bug9205 = '[ruby-core:58833] [Bug #9205]' + 100.times do + assert_ruby_status([], <<-'end;', bug9205) + raise_proc = proc do |id| + GC.start + end + 1000.times do + ObjectSpace.define_finalizer(Object.new, raise_proc) + end + end; + end + end end |