Mercurial > hg > graal-compiler

comparison src/share/vm/compiler/compileBroker.cpp @ 20589:0c0e68524c17

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
8042570: Excessive number of tests timing out on nightly testing due to fix for 8040798 Summary: Remove unnecessary lock in CompileBroker::mark_on_stack() that causes the deadlock Reviewed-by: kvn, dcubed
author anoll
date Wed, 07 May 2014 22:06:42 +0200
parents 41dcdd636080
children c47fcf523fff
comparison
equal deleted inserted replaced
20588:41dcdd636080 20589:0c0e68524c17
1126 } 1126 }
1127 1127
1128 1128
1129 /** 1129 /**
1130 * Set the methods on the stack as on_stack so that redefine classes doesn't 1130 * Set the methods on the stack as on_stack so that redefine classes doesn't
1131 * reclaim them 1131 * reclaim them. This method is executed at a safepoint.
1132 */ 1132 */
1133 void CompileBroker::mark_on_stack() { 1133 void CompileBroker::mark_on_stack() {
1134 assert(SafepointSynchronize::is_at_safepoint(), "sanity check");
1135 // Since we are at a safepoint, we do not need a lock to access
1136 // the compile queues.
1134 if (_c2_compile_queue != NULL) { 1137 if (_c2_compile_queue != NULL) {
1135 MutexLocker locker(_c2_compile_queue->lock());
1136 _c2_compile_queue->mark_on_stack(); 1138 _c2_compile_queue->mark_on_stack();
1137 } 1139 }
1138 if (_c1_compile_queue != NULL) { 1140 if (_c1_compile_queue != NULL) {
1139 MutexLocker locker(_c1_compile_queue->lock());
1140 _c1_compile_queue->mark_on_stack(); 1141 _c1_compile_queue->mark_on_stack();
1141 } 1142 }
1142 } 1143 }
1143 1144
1144 // ------------------------------------------------------------------ 1145 // ------------------------------------------------------------------