diff options
author | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-12-13 07:13:31 +0000 |
---|---|---|
committer | nobu <nobu@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2011-12-13 07:13:31 +0000 |
commit | ab6c8910f47a2b95f7338a182715ee0bee5ec45d (patch) | |
tree | d7691af9df1fbba665fc00009dea147af3a2eb31 /test | |
parent | ddc15717ccb98d57a5b82b4138b1dc97375e4ac2 (diff) | |
download | ruby-ab6c8910f47a2b95f7338a182715ee0bee5ec45d.tar.gz |
* load.c (load_unlock): all threads requiring one file should
share same loading barrier, so it must be kept alive while those
are waiting on it. [ruby-core:41618] [Bug #5754]
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@34027 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'test')
-rw-r--r-- | test/ruby/test_require.rb | 56 |
1 files changed, 56 insertions, 0 deletions
diff --git a/test/ruby/test_require.rb b/test/ruby/test_require.rb index 96b1551da2..858ea6170a 100644 --- a/test/ruby/test_require.rb +++ b/test/ruby/test_require.rb @@ -339,4 +339,60 @@ class TestRequire < Test::Unit::TestCase [], /\$LOADED_FEATURES is frozen; cannot append feature \(RuntimeError\)$/, bug3756) end + + class << self + attr_accessor :scratch + end + + def test_race_excption + bug5754 = '[ruby-core:41618]' + tmp = Tempfile.new(%w"bug5754 .rb") + path = tmp.path + tmp.print <<-EOS +TestRequire.scratch << :pre +Thread.pass until t2 = TestRequire.scratch[1] +Thread.pass until t2.stop? +open(__FILE__, "w") {|f| f.puts "TestRequire.scratch << :post"} +raise "con1" + EOS + tmp.close + + start = false + fin = false + + TestRequire.scratch = scratch = [] + t1_res = nil + t2_res = nil + + t1 = Thread.new do + begin + require(path) + rescue RuntimeError + end + + t1_res = require(path) + + Thread.pass until fin + scratch << :t1 + end + + t2 = Thread.new do + Thread.pass until scratch[0] + begin + scratch << t2 + t2_res = require(path) + scratch << :t2 + ensure + fin = true + end + end + + assert_nothing_raised(ThreadError, bug5754) {t1.join} + assert_nothing_raised(ThreadError, bug5754) {t2.join} + + assert_equal([false, true], [t1_res, t2_res], bug5754) + assert_equal([:pre, t2, :post, :t2, :t1], scratch, bug5754) + + tmp.close(true) + end end |