ruby-dev

Issue #595 has been updated by wanabe (_ wanabe).

File ensure_fiber2.patch added

ワナベと申します。

(1) GC で mark と sweep の間に、mark されていない Fiber を対象に
(2) ruby_cleanup 中に、メインスレッドに所属するすべての Fiber を対象に
(3) 子スレッド終了時（vm->living_threads から外されるとき）、所属するすべての Fiber を対象に
の 3 つのタイミングで、throw/catch により ensure 節を実行するパッチを書きました。

[ruby-dev:41035] で遠藤さんがおっしゃっているような「yield 中の Fiber は GC しない」
という手法もやってみたのですが、test/ruby/test_fiber.rb がとても終わりそうにないことや
Fiber のマーク処理の重さや Fiber 自体のメモリ消費量などにより、断念しました。
そのため上記(1)のように、rb_gc_marked_p() という関数が必要になるなど強引な手段を使っています。

また、以下のようにして速度低下を計ってみました。

require "benchmark"
GC.start
Benchmark.bm(4) do |x|
  tms = Benchmark::Tms.new
  10.times do |i|
    tms += x.report("   #{i}:") do
      30000.times do
        Fiber.new{Fiber.yield}.resume
      end
    end
  end
  puts " sum:#{tms}"
end

素の r36623 ：2.980000   3.120000   6.100000 (  6.107164)
パッチ適用後：3.580000   3.480000   7.060000 (  7.061093)

と、無視できない程度に（約 14%）速度低下してしまいました。
とはいえこれ以上の方法は思いつかないのですが、いかがでしょうか。
----------------------------------------
Bug #595: Fiber ignores ensure clause
https://bugs.ruby-lang.org/issues/595#change-28655

Author: ko1 (Koichi Sasada)
Status: Assigned
Priority: Normal
Assignee: ko1 (Koichi Sasada)
Category: core
Target version: 3.0
ruby -v: -


=begin
 Ruby プロセス終了時，Fiber が ensure を無視します．
 これは，前から直そうと思って手がついていなかった問題です．
 10月末までには直そうと思います．結構複雑なので，後回しにしていましました．
 
 fib = Fiber.new{
   begin
     Fiber.yield :ok
   ensure
     puts "should be print out"
   end
 }
 p fib.resume
=end



-- 
http://bugs.ruby-lang.org/

Thread

Prev Next

In This Thread

Prev Next

[#46026] Re: [ruby-cvs:43788] shugo:r36612 (trunk): * insns.def (invokesuper): don't skip the same class. instead, use — Urabe Shyouhei <shyouhei@...>

[#46030] [ruby-trunk - Bug #595] Fiber ignores ensure clause — "wanabe (_ wanabe)" <s.wanabe@...>

[#46032] 「Ruby安定版保守委託事業」の提案募集について — Shugo Maeda <shugo@...>

[#46037] Re: [ruby-core:47138] [ruby-trunk - Bug #6861][Open] ERB::Util.escape_html is not escaping single quotes — Shugo Maeda <shugo@...>

[#46049] [ruby-trunk - Bug #6873][Open] Failed to build ruby for NaCl i686 version — "ikeay (Ayaka Ikezawa)" <ikeay@...>

[#46051] [ruby-trunk - Feature #6875][Open] Make test/unit default gem — "kou (Kouhei Sutou)" <kou@...>

[#46053] [ruby-trunk - Bug #6881][Assigned] 2 Failures in test/-ext-/test_printf.rb — "usa (Usaku NAKAMURA)" <usa@...>

[#46054] [ruby-trunk - Bug #6882][Assigned] parallel test crashes when unknown exception is occured in a test — "usa (Usaku NAKAMURA)" <usa@...>

[#46055] Fwd: 『RubyKaja』登録のお知らせ — SASADA Koichi <ko1@...>

[#46061] [ruby-trunk - Bug #6898][Open] lambda周辺でSegmentation fault — uy (西行寺 うゆ) <redmine@...>

[#46064] [ruby-trunk - Bug #6900][Open] execinfo ライブラリ（およびそのヘッダ）の探索に --with-opt-dir の指定が利かない — "metanest (Makoto Kishimoto)" <redmine@...>

[#46065] [ruby-trunk - Bug #6901][Assigned] SEGV with tail call optimization — "shugo (Shugo Maeda)" <redmine@...>

[#46067] [ruby-trunk - Bug #6904][Assigned] make -j all fails (sometimes) — "shyouhei (Shyouhei Urabe)" <shyouhei@...>

[#46073] [ruby-trunk - Bug #2402][Open] super in instance_eval — "shugo (Shugo Maeda)" <redmine@...>

[#46074] [ruby-trunk - Bug #6911][Open] Sync_m#sync_unlock で ThreadError が発生する場合がある — "mrkn (Kenta Murata)" <muraken@...>

[#46077] [ruby-trunk - Bug #6927][Open] 日本語ドキュメントにおける Math.#atan2 について — "n.arita (Naoki Arita)" <ToNaokiArita@...>

[#46081] [ruby-trunk - Feature #6936][Assigned] Forbid singleton class and instance variabls for float — "naruse (Yui NARUSE)" <naruse@...>

[#46097] [ruby-trunk - Bug #6956][Assigned] cannot build with nmake — "usa (Usaku NAKAMURA)" <usa@...>

[ruby-dev:46030] [ruby-trunk - Bug #595] Fiber ignores ensure clause

Thread

In This Thread

[#46061] [ruby-trunk - Bug #6898][Open] lambda周辺でSegmentation fault — uy (西行寺うゆ) <redmine@...>