From e20bfbd18e64b8ab9428aa518b5d7b74a73461d9 Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Sat, 29 Dec 2018 00:07:06 +0000 Subject: runtime: prevent deadlock when profiling signal arrives in stack scan Precise stack scan needs to unwind the stack. When it is unwinding the stack, if a profiling signal arrives, which also does a traceback, it may deadlock in dl_iterate_phdr. Prevent this deadlock by setting up runtime_in_callers before traceback. Reviewed-on: https://go-review.googlesource.com/c/155766 From-SVN: r267457 --- libgo/runtime/go-callers.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'libgo/runtime/go-callers.c') diff --git a/libgo/runtime/go-callers.c b/libgo/runtime/go-callers.c index b16ae0c..78ada7a 100644 --- a/libgo/runtime/go-callers.c +++ b/libgo/runtime/go-callers.c @@ -16,7 +16,7 @@ older versions of glibc when a SIGPROF signal arrives while collecting a backtrace. */ -static uint32 runtime_in_callers; +uint32 __go_runtime_in_callers; /* Argument passed to callback function. */ @@ -185,7 +185,7 @@ bool alreadyInCallers(void) bool alreadyInCallers() { - return runtime_atomicload(&runtime_in_callers) > 0; + return runtime_atomicload(&__go_runtime_in_callers) > 0; } /* Gather caller PC's. */ @@ -203,9 +203,9 @@ runtime_callers (int32 skip, Location *locbuf, int32 m, bool keep_thunks) data.max = m; data.keep_thunks = keep_thunks; state = __go_get_backtrace_state (); - runtime_xadd (&runtime_in_callers, 1); + runtime_xadd (&__go_runtime_in_callers, 1); backtrace_full (state, 0, callback, error_callback, &data); - runtime_xadd (&runtime_in_callers, -1); + runtime_xadd (&__go_runtime_in_callers, -1); /* For some reason GCC sometimes loses the name of a thunk function at the top of the stack. If we are skipping thunks, skip that -- cgit v1.1