[前][次][番号順一覧][スレッド一覧]

ruby-changes:63791

From: Takashi <ko1@a...>
Date: Sun, 29 Nov 2020 11:50:46 +0900 (JST)
Subject: [ruby-changes:63791] dbdeb92b68 (master): Do not throttle the workaround for --jit-wait

https://git.ruby-lang.org/ruby.git/commit/?id=dbdeb92b68

From dbdeb92b6832867ef208130d9a8e7d01df0ccb93 Mon Sep 17 00:00:00 2001
From: Takashi Kokubun <takashikkbn@g...>
Date: Sat, 28 Nov 2020 18:49:45 -0800
Subject: Do not throttle the workaround for --jit-wait

--jit-wait CI can be stuck when the workaround is throttled
http://ci.rvm.jp/results/trunk-mjit-wait@phosphorus-docker/3274091

diff --git a/mjit_worker.c b/mjit_worker.c
index b8c5886..725a922 100644
--- a/mjit_worker.c
+++ b/mjit_worker.c
@@ -1369,11 +1369,10 @@ mjit_worker(void) https://github.com/ruby/ruby/blob/trunk/mjit_worker.c#L1369
                 RB_DEBUG_COUNTER_INC(mjit_unload_units);
                 unload_units();
                 unload_requests = 0;
-
-                if (active_units.length == mjit_opts.max_cache_size && mjit_opts.wait) { // Sometimes all methods may be in use
-                    mjit_opts.max_cache_size++; // avoid infinite loop on `rb_mjit_wait_call`. Note that --jit-wait is just for testing.
-                    verbose(1, "No units can be unloaded -- incremented max-cache-size to %d for --jit-wait", mjit_opts.max_cache_size);
-                }
+            }
+            if (active_units.length == mjit_opts.max_cache_size && mjit_opts.wait) { // Sometimes all methods may be in use
+                mjit_opts.max_cache_size++; // avoid infinite loop on `rb_mjit_wait_call`. Note that --jit-wait is just for testing.
+                verbose(1, "No units can be unloaded -- incremented max-cache-size to %d for --jit-wait", mjit_opts.max_cache_size);
             }
         }
         unit = get_from_list(&unit_queue);
-- 
cgit v0.10.2


--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/

[前][次][番号順一覧][スレッド一覧]