commit 8f59fbc43f77510863e5a8269f055dea4f25faf7
author Albert Noll <anoll@openjdk.org> Fri Jun 20 09:25:13 2014 +0200
committer Albert Noll <anoll@openjdk.org> Fri Jun 20 09:25:13 2014 +0200
tree af13a668833a38d9246f540db34e888b844fe9ac
parent 0e22f68776efde37b40e2e57386ecdfdccb68ca9

8043158: Crash in CodeSweeperSweepNoFlushTest in CompileQueue::free_all()

Acquire lock before calling notify()

Reviewed-by: twisti, thartmann

hotspot/src/share/vm/compiler/compileBroker.cpp

diff --git a/hotspot/src/share/vm/compiler/compileBroker.cpp b/hotspot/src/share/vm/compiler/compileBroker.cpp
index be98488..6edb5c5 100644
--- a/hotspot/src/share/vm/compiler/compileBroker.cpp
+++ b/hotspot/src/share/vm/compiler/compileBroker.cpp
@@ -678,8 +678,11 @@
   while (next != NULL) {
     CompileTask* current = next;
     next = current->next();
-    // Wake up thread that blocks on the compile task.
-    current->lock()->notify();
+    {
+      // Wake up thread that blocks on the compile task.
+      MutexLocker ct_lock(current->lock());
+      current->lock()->notify();
+    }
     // Put the task back on the freelist.
     CompileTask::free(current);
   }