From 039868965d6679379f1b87b7336986605229430b Mon Sep 17 00:00:00 2001 From: Ian Lance Taylor Date: Sat, 22 Jan 2011 02:15:01 +0000 Subject: libgo: Adjust deadlock avoidance. From-SVN: r169120 --- libgo/runtime/go-go.c | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/libgo/runtime/go-go.c b/libgo/runtime/go-go.c index 7c5f40d..3146268 100644 --- a/libgo/runtime/go-go.c +++ b/libgo/runtime/go-go.c @@ -297,6 +297,15 @@ gc_stop_handler (int sig __attribute__ ((unused))) { struct M *pm = m; + if (__sync_bool_compare_and_swap (&pm->holds_finlock, 1, 1)) + { + /* We can't interrupt the thread while it holds the finalizer + lock. Otherwise we can get into a deadlock when mark calls + runtime_walkfintab. */ + __sync_bool_compare_and_swap (&pm->gcing_for_finlock, 0, 1); + return; + } + if (__sync_bool_compare_and_swap (&pm->mallocing, 1, 1)) { /* m->mallocing was already non-zero. We can't interrupt the @@ -315,15 +324,6 @@ gc_stop_handler (int sig __attribute__ ((unused))) return; } - if (__sync_bool_compare_and_swap (&pm->holds_finlock, 1, 1)) - { - /* Similarly, we can't interrupt the thread while it holds the - finalizer lock. Otherwise we can get into a deadlock when - mark calls runtime_walkfintab. */ - __sync_bool_compare_and_swap (&pm->gcing_for_finlock, 0, 1); - return; - } - stop_for_gc (); } -- cgit v1.1