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

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/

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