Merge 'sverker/erts/jit_debug_descriptor-leak' into maint

OTP-17445
erlang · May 26, 2021 · 045ce64 · 045ce64
2 parents f3a1966 + 3508c71
commit 045ce64
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/erts/emulator/beam/erl_lock_check.c b/erts/emulator/beam/erl_lock_check.c
@@ -167,6 +167,7 @@ static erts_lc_lock_order_t erts_lock_order[] = {
     {	"erts_alloc_hard_debug",		NULL			},
     {	"hard_dbg_mseg",		        NULL	                },
     {	"perf", 				NULL			},
+    {	"jit_debug_descriptor",			NULL			},
     {	"erts_mmap",				NULL			}
 };
 

diff --git a/erts/emulator/beam/jit/x86/beam_asm.cpp b/erts/emulator/beam/jit/x86/beam_asm.cpp
@@ -624,6 +624,7 @@ struct jit_descriptor {
     jit_actions action_flag;
     struct jit_code_entry *relevant_entry;
     struct jit_code_entry *first_entry;
+    erts_mtx_t mutex;
 };
 
 extern "C"
@@ -665,6 +666,12 @@ static void beamasm_init_gdb_jit_info(void) {
     __jit_debug_descriptor.first_entry = entry;
     __jit_debug_descriptor.relevant_entry = entry;
     __jit_debug_register_code();
+
+    erts_mtx_init(&__jit_debug_descriptor.mutex,
+                  "jit_debug_descriptor",
+                  NIL,
+                  (ERTS_LOCK_FLAGS_PROPERTY_STATIC |
+                   ERTS_LOCK_FLAGS_CATEGORY_GENERIC));
 }
 
 void BeamAssembler::update_gdb_jit_info(std::string modulename,
@@ -711,6 +718,7 @@ void BeamAssembler::update_gdb_jit_info(std::string modulename,
     ASSERT(symfile_size == (symfile - entry->symfile_addr));
 
     /* Insert into linked list */
+    erts_mtx_lock(&__jit_debug_descriptor.mutex);
     entry->next_entry = __jit_debug_descriptor.first_entry;
     if (entry->next_entry) {
         entry->next_entry->prev_entry = entry;
@@ -723,6 +731,7 @@ void BeamAssembler::update_gdb_jit_info(std::string modulename,
     __jit_debug_descriptor.first_entry = entry;
     __jit_debug_descriptor.relevant_entry = entry;
     __jit_debug_register_code();
+    erts_mtx_unlock(&__jit_debug_descriptor.mutex);
 }
 
 extern "C"