ruby-changes:64011
From: Takashi <ko1@a...>
Date: Tue, 8 Dec 2020 08:34:19 +0900 (JST)
Subject: [ruby-changes:64011] 5c2ff88be2 (master): Lock the entire active_units loop
https://git.ruby-lang.org/ruby.git/commit/?id=5c2ff88be2 From 5c2ff88be2e515613dfe54823e8429656f688e9f Mon Sep 17 00:00:00 2001 From: Takashi Kokubun <takashikkbn@g...> Date: Mon, 7 Dec 2020 15:33:29 -0800 Subject: Lock the entire active_units loop The previous fix seems not working. Let me test if this works. diff --git a/mjit_worker.c b/mjit_worker.c index f97bf65..586ac13 100644 --- a/mjit_worker.c +++ b/mjit_worker.c @@ -938,7 +938,6 @@ compile_compact_jit_code(char* c_file) https://github.com/ruby/ruby/blob/trunk/mjit_worker.c#L938 bool success = true; CRITICAL_SECTION_START(3, "before active_units list_for_each"); list_for_each(&active_units.head, child_unit, unode) { - CRITICAL_SECTION_FINISH(3, "after active_units list_for_each"); char funcname[MAXPATHLEN]; sprint_funcname(funcname, child_unit); @@ -952,8 +951,6 @@ compile_compact_jit_code(char* c_file) https://github.com/ruby/ruby/blob/trunk/mjit_worker.c#L951 if (!iseq_label) iseq_label = sep = ""; fprintf(f, "\n/* %s%s%s:%ld */\n", iseq_label, sep, iseq_path, iseq_lineno); success &= mjit_compile(f, child_unit->iseq, funcname, child_unit->id); - - CRITICAL_SECTION_START(3, "before active_units list_for_each"); } CRITICAL_SECTION_FINISH(3, "after active_units list_for_each"); -- cgit v0.10.2 -- ML: ruby-changes@q... Info: http://www.atdot.net/~ko1/quickml/