File 0408-erts-Fix-ERTS_LC_ASSERT-to-use-ERTS_IS_CRA... of Package erlang

Overview Repositories Revisions Requests Users Attributes Meta

File 0408-erts-Fix-ERTS_LC_ASSERT-to-use-ERTS_IS_CRASH__DUMPIN.patch of Package erlang (Revision 8cb5d5fde1cca12e0e8bdc4c50dbf291)

Currently displaying revision 8cb5d5fde1cca12e0e8bdc4c50dbf291 , Show latest

From c12a50bafeec92b62b7536c494a237b484a9fbc0 Mon Sep 17 00:00:00 2001
From: Sverker Eriksson <sverker@erlang.org>
Date: Fri, 5 Feb 2021 17:41:51 +0100
Subject: [PATCH 08/34] erts: Fix ERTS_LC_ASSERT to use ERTS_IS_CRASH__DUMPING

instead of ERTS_SOMEONE_IS_CRASH_DUMPING. Seems more correct
that only the thread doing the crash dump is allowed skip locks.

Change triggered by code checker warning erts_lc_mtx_is_locked
called with prt->port_data_lock as NULL in erts_thr_drvport2port.
---
 erts/emulator/beam/erl_lock_check.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/erts/emulator/beam/erl_lock_check.h b/erts/emulator/beam/erl_lock_check.h
index 92f056bee1..3f3ddcca78 100644
--- a/erts/emulator/beam/erl_lock_check.h
+++ b/erts/emulator/beam/erl_lock_check.h
@@ -98,7 +98,7 @@ int erts_lc_is_emu_thr(void);
 Eterm erts_lc_dump_graph(void);
 
 #define ERTS_LC_ASSERT(A) \
-    ((void) (((A) || ERTS_SOMEONE_IS_CRASH_DUMPING) ? 1 : erts_lc_assert_failed(__FILE__, __LINE__, #A)))
+    ((void) (((A) || ERTS_IS_CRASH_DUMPING) ? 1 : erts_lc_assert_failed(__FILE__, __LINE__, #A)))
 #else /* #ifdef ERTS_ENABLE_LOCK_CHECK */
 #define ERTS_LC_ASSERT(A) ((void) 1)
 #endif /* #ifdef ERTS_ENABLE_LOCK_CHECK */
-- 
2.26.2

Places

File 0408-erts-Fix-ERTS_LC_ASSERT-to-use-ERTS_IS_CRASH__DUMPIN.patch of Package erlang (Revision 8cb5d5fde1cca12e0e8bdc4c50dbf291)

Places