# HG changeset patch # User sbohne # Date 1207856956 14400 # Node ID c6ff24ceec1c7abce53e5c54b2d9a9923d23707a # Parent ebec5b9731e232f1902927dee1bdb69621b26ff0 6686407: Fix for 6666698 broke -XX:BiasedLockingStartupDelay=0 Summary: Stack allocated VM_EnableBiasedLocking op must be marked as such Reviewed-by: xlu, acorn, never, dholmes diff -r ebec5b9731e2 -r c6ff24ceec1c src/share/vm/runtime/biasedLocking.cpp --- a/src/share/vm/runtime/biasedLocking.cpp Thu Apr 10 12:21:01 2008 -0400 +++ b/src/share/vm/runtime/biasedLocking.cpp Thu Apr 10 15:49:16 2008 -0400 @@ -1,4 +1,3 @@ - /* * Copyright 2005-2007 Sun Microsystems, Inc. All Rights Reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. @@ -37,11 +36,13 @@ } class VM_EnableBiasedLocking: public VM_Operation { + private: + bool _is_cheap_allocated; public: - VM_EnableBiasedLocking() {} + VM_EnableBiasedLocking(bool is_cheap_allocated) { _is_cheap_allocated = is_cheap_allocated; } VMOp_Type type() const { return VMOp_EnableBiasedLocking; } - Mode evaluation_mode() const { return _async_safepoint; } - bool is_cheap_allocated() const { return true; } + Mode evaluation_mode() const { return _is_cheap_allocated ? _async_safepoint : _safepoint; } + bool is_cheap_allocated() const { return _is_cheap_allocated; } void doit() { // Iterate the system dictionary enabling biased locking for all @@ -67,7 +68,7 @@ virtual void task() { // Use async VM operation to avoid blocking the Watcher thread. // VM Thread will free C heap storage. - VM_EnableBiasedLocking *op = new VM_EnableBiasedLocking(); + VM_EnableBiasedLocking *op = new VM_EnableBiasedLocking(true); VMThread::execute(op); // Reclaim our storage and disenroll ourself @@ -89,7 +90,7 @@ EnableBiasedLockingTask* task = new EnableBiasedLockingTask(BiasedLockingStartupDelay); task->enroll(); } else { - VM_EnableBiasedLocking op; + VM_EnableBiasedLocking op(false); VMThread::execute(&op); } }