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/