author | stefank |
Tue, 06 Aug 2019 10:48:21 +0200 | |
changeset 57777 | 90ead0febf56 |
parent 55625 | f7e8dbb77156 |
child 57811 | 947252a54b98 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
54710
aa29971a17a1
8220724: TestBiasedLockRevocationEvents fails while matching revoke events to VMOperation events
pchilanomate
parents:
52877
diff
changeset
|
2 |
* Copyright (c) 2005, 2019, Oracle and/or its affiliates. All rights reserved. |
1 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
3261
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
3261
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
3261
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
51959
db0c3952de52
8209645: Split ClassLoaderData and ClassLoaderDataGraph into separate files
coleenp
parents:
50113
diff
changeset
|
26 |
#include "classfile/classLoaderDataGraph.hpp" |
50113 | 27 |
#include "jfr/jfrEvents.hpp" |
28 |
#include "jfr/support/jfrThreadId.hpp" |
|
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
29 |
#include "logging/log.hpp" |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
30 |
#include "memory/resourceArea.hpp" |
7397 | 31 |
#include "oops/klass.inline.hpp" |
32 |
#include "oops/markOop.hpp" |
|
29081
c61eb4914428
8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents:
25351
diff
changeset
|
33 |
#include "oops/oop.inline.hpp" |
40655
9f644073d3a0
8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents:
37251
diff
changeset
|
34 |
#include "runtime/atomic.hpp" |
7397 | 35 |
#include "runtime/basicLock.hpp" |
36 |
#include "runtime/biasedLocking.hpp" |
|
49449
ef5d5d343e2a
8199263: Split interfaceSupport.hpp to not require including .inline.hpp files
coleenp
parents:
48105
diff
changeset
|
37 |
#include "runtime/handles.inline.hpp" |
7397 | 38 |
#include "runtime/task.hpp" |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
39 |
#include "runtime/threadSMR.hpp" |
7397 | 40 |
#include "runtime/vframe.hpp" |
41 |
#include "runtime/vmThread.hpp" |
|
52877
9e041366c764
8214850: Rename vm_operations.?pp files to vmOperations.?pp files
tschatzl
parents:
51959
diff
changeset
|
42 |
#include "runtime/vmOperations.hpp" |
50113 | 43 |
|
1 | 44 |
|
45 |
static bool _biased_locking_enabled = false; |
|
46 |
BiasedLockingCounters BiasedLocking::_counters; |
|
47 |
||
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
48 |
static GrowableArray<Handle>* _preserved_oop_stack = NULL; |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
49 |
static GrowableArray<markWord>* _preserved_mark_stack = NULL; |
1 | 50 |
|
46729 | 51 |
static void enable_biased_locking(InstanceKlass* k) { |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
52 |
k->set_prototype_header(markWord::biased_locking_prototype()); |
1 | 53 |
} |
54 |
||
55537
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
55 |
static void enable_biased_locking() { |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
56 |
_biased_locking_enabled = true; |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
57 |
log_info(biasedlocking)("Biased locking enabled"); |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
58 |
} |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
59 |
|
1 | 60 |
class VM_EnableBiasedLocking: public VM_Operation { |
61 |
public: |
|
55537
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
62 |
VM_EnableBiasedLocking() {} |
231
3cfeeae32993
6666698: EnableBiasedLocking with BiasedLockingStartupDelay can block Watcher thread
sbohne
parents:
1
diff
changeset
|
63 |
VMOp_Type type() const { return VMOp_EnableBiasedLocking; } |
55537
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
64 |
Mode evaluation_mode() const { return _async_safepoint; } |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
65 |
bool is_cheap_allocated() const { return true; } |
231
3cfeeae32993
6666698: EnableBiasedLocking with BiasedLockingStartupDelay can block Watcher thread
sbohne
parents:
1
diff
changeset
|
66 |
|
1 | 67 |
void doit() { |
46729 | 68 |
// Iterate the class loader data dictionaries enabling biased locking for all |
69 |
// currently loaded classes. |
|
70 |
ClassLoaderDataGraph::dictionary_classes_do(enable_biased_locking); |
|
1 | 71 |
// Indicate that future instances should enable it as well |
55537
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
72 |
enable_biased_locking(); |
1 | 73 |
} |
74 |
||
75 |
bool allow_nested_vm_operations() const { return false; } |
|
76 |
}; |
|
77 |
||
78 |
||
79 |
// One-shot PeriodicTask subclass for enabling biased locking |
|
80 |
class EnableBiasedLockingTask : public PeriodicTask { |
|
81 |
public: |
|
82 |
EnableBiasedLockingTask(size_t interval_time) : PeriodicTask(interval_time) {} |
|
83 |
||
84 |
virtual void task() { |
|
231
3cfeeae32993
6666698: EnableBiasedLocking with BiasedLockingStartupDelay can block Watcher thread
sbohne
parents:
1
diff
changeset
|
85 |
// Use async VM operation to avoid blocking the Watcher thread. |
3cfeeae32993
6666698: EnableBiasedLocking with BiasedLockingStartupDelay can block Watcher thread
sbohne
parents:
1
diff
changeset
|
86 |
// VM Thread will free C heap storage. |
55537
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
87 |
VM_EnableBiasedLocking *op = new VM_EnableBiasedLocking(); |
231
3cfeeae32993
6666698: EnableBiasedLocking with BiasedLockingStartupDelay can block Watcher thread
sbohne
parents:
1
diff
changeset
|
88 |
VMThread::execute(op); |
1 | 89 |
|
90 |
// Reclaim our storage and disenroll ourself |
|
91 |
delete this; |
|
92 |
} |
|
93 |
}; |
|
94 |
||
95 |
||
96 |
void BiasedLocking::init() { |
|
55537
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
97 |
// If biased locking is enabled and BiasedLockingStartupDelay is set, |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
98 |
// schedule a task to fire after the specified delay which turns on |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
99 |
// biased locking for all currently loaded classes as well as future |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
100 |
// ones. This could be a workaround for startup time regressions |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
101 |
// due to large number of safepoints being taken during VM startup for |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
102 |
// bias revocation. |
1 | 103 |
if (UseBiasedLocking) { |
104 |
if (BiasedLockingStartupDelay > 0) { |
|
105 |
EnableBiasedLockingTask* task = new EnableBiasedLockingTask(BiasedLockingStartupDelay); |
|
106 |
task->enroll(); |
|
107 |
} else { |
|
55537
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
108 |
enable_biased_locking(); |
1 | 109 |
} |
110 |
} |
|
111 |
} |
|
112 |
||
113 |
||
114 |
bool BiasedLocking::enabled() { |
|
55537
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
115 |
assert(UseBiasedLocking, "precondition"); |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
116 |
// We check "BiasedLockingStartupDelay == 0" here to cover the |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
117 |
// possibility of calls to BiasedLocking::enabled() before |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
118 |
// BiasedLocking::init(). |
4309fedaf7dc
8225344: Avoid VM_EnableBiasedLocking VM operation during bootstrap if BiasedLockingStartupDelay is 0
redestad
parents:
55514
diff
changeset
|
119 |
return _biased_locking_enabled || BiasedLockingStartupDelay == 0; |
1 | 120 |
} |
121 |
||
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
122 |
|
1 | 123 |
// Returns MonitorInfos for all objects locked on this thread in youngest to oldest order |
124 |
static GrowableArray<MonitorInfo*>* get_or_compute_monitor_info(JavaThread* thread) { |
|
125 |
GrowableArray<MonitorInfo*>* info = thread->cached_monitor_info(); |
|
126 |
if (info != NULL) { |
|
127 |
return info; |
|
128 |
} |
|
129 |
||
130 |
info = new GrowableArray<MonitorInfo*>(); |
|
131 |
||
132 |
// It's possible for the thread to not have any Java frames on it, |
|
133 |
// i.e., if it's the main thread and it's already returned from main() |
|
134 |
if (thread->has_last_Java_frame()) { |
|
135 |
RegisterMap rm(thread); |
|
136 |
for (javaVFrame* vf = thread->last_java_vframe(&rm); vf != NULL; vf = vf->java_sender()) { |
|
137 |
GrowableArray<MonitorInfo*> *monitors = vf->monitors(); |
|
138 |
if (monitors != NULL) { |
|
139 |
int len = monitors->length(); |
|
140 |
// Walk monitors youngest to oldest |
|
141 |
for (int i = len - 1; i >= 0; i--) { |
|
142 |
MonitorInfo* mon_info = monitors->at(i); |
|
22926
1a4328d3fd0e
8032011: nsk/stress/jck60/jck60022 crashes in src\share\vm\runtime\synchronizer.cpp:239
roland
parents:
22859
diff
changeset
|
143 |
if (mon_info->eliminated()) continue; |
1 | 144 |
oop owner = mon_info->owner(); |
145 |
if (owner != NULL) { |
|
146 |
info->append(mon_info); |
|
147 |
} |
|
148 |
} |
|
149 |
} |
|
150 |
} |
|
151 |
} |
|
152 |
||
153 |
thread->set_cached_monitor_info(info); |
|
154 |
return info; |
|
155 |
} |
|
156 |
||
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
157 |
|
47764
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
158 |
// After the call, *biased_locker will be set to obj->mark()->biased_locker() if biased_locker != NULL, |
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
159 |
// AND it is a living thread. Otherwise it will not be updated, (i.e. the caller is responsible for initialization). |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
160 |
BiasedLocking::Condition BiasedLocking::single_revoke_at_safepoint(oop obj, bool allow_rebias, bool is_bulk, JavaThread* requesting_thread, JavaThread** biased_locker) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
161 |
assert(SafepointSynchronize::is_at_safepoint(), "must be done at safepoint"); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
162 |
assert(Thread::current()->is_VM_thread(), "must be VMThread"); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
163 |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
164 |
markWord mark = obj->mark(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
165 |
if (!mark.has_bias_pattern()) { |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
166 |
if (log_is_enabled(Info, biasedlocking)) { |
1 | 167 |
ResourceMark rm; |
37251
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
168 |
log_info(biasedlocking)(" (Skipping revocation of object " INTPTR_FORMAT |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
169 |
", mark " INTPTR_FORMAT ", type %s" |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
170 |
", requesting thread " INTPTR_FORMAT |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
171 |
" because it's no longer biased)", |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
172 |
p2i((void *)obj), mark.value(), |
37251
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
173 |
obj->klass()->external_name(), |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
174 |
(intptr_t) requesting_thread); |
1 | 175 |
} |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
176 |
return NOT_BIASED; |
1 | 177 |
} |
178 |
||
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
179 |
uint age = mark.age(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
180 |
markWord biased_prototype = markWord::biased_locking_prototype().set_age(age); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
181 |
markWord unbiased_prototype = markWord::prototype().set_age(age); |
1 | 182 |
|
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
183 |
// Log at "info" level if not bulk, else "trace" level |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
184 |
if (!is_bulk) { |
1 | 185 |
ResourceMark rm; |
37251
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
186 |
log_info(biasedlocking)("Revoking bias of object " INTPTR_FORMAT ", mark " |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
187 |
INTPTR_FORMAT ", type %s, prototype header " INTPTR_FORMAT |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
188 |
", allow rebias %d, requesting thread " INTPTR_FORMAT, |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
189 |
p2i((void *)obj), |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
190 |
mark.value(), |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
191 |
obj->klass()->external_name(), |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
192 |
obj->klass()->prototype_header().value(), |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
193 |
(allow_rebias ? 1 : 0), |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
194 |
(intptr_t) requesting_thread); |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
195 |
} else { |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
196 |
ResourceMark rm; |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
197 |
log_trace(biasedlocking)("Revoking bias of object " INTPTR_FORMAT " , mark " |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
198 |
INTPTR_FORMAT " , type %s , prototype header " INTPTR_FORMAT |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
199 |
" , allow rebias %d , requesting thread " INTPTR_FORMAT, |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
200 |
p2i((void *)obj), |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
201 |
mark.value(), |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
202 |
obj->klass()->external_name(), |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
203 |
obj->klass()->prototype_header().value(), |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
204 |
(allow_rebias ? 1 : 0), |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
205 |
(intptr_t) requesting_thread); |
1 | 206 |
} |
207 |
||
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
208 |
JavaThread* biased_thread = mark.biased_locker(); |
1 | 209 |
if (biased_thread == NULL) { |
210 |
// Object is anonymously biased. We can get here if, for |
|
211 |
// example, we revoke the bias due to an identity hash code |
|
212 |
// being computed for an object. |
|
213 |
if (!allow_rebias) { |
|
214 |
obj->set_mark(unbiased_prototype); |
|
215 |
} |
|
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
216 |
// Log at "info" level if not bulk, else "trace" level |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
217 |
if (!is_bulk) { |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
218 |
log_info(biasedlocking)(" Revoked bias of anonymously-biased object"); |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
219 |
} else { |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
220 |
log_trace(biasedlocking)(" Revoked bias of anonymously-biased object"); |
1 | 221 |
} |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
222 |
return BIAS_REVOKED; |
1 | 223 |
} |
224 |
||
225 |
// Handle case where the thread toward which the object was biased has exited |
|
226 |
bool thread_is_alive = false; |
|
227 |
if (requesting_thread == biased_thread) { |
|
228 |
thread_is_alive = true; |
|
229 |
} else { |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
230 |
ThreadsListHandle tlh; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
231 |
thread_is_alive = tlh.includes(biased_thread); |
1 | 232 |
} |
233 |
if (!thread_is_alive) { |
|
234 |
if (allow_rebias) { |
|
235 |
obj->set_mark(biased_prototype); |
|
236 |
} else { |
|
237 |
obj->set_mark(unbiased_prototype); |
|
238 |
} |
|
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
239 |
// Log at "info" level if not bulk, else "trace" level |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
240 |
if (!is_bulk) { |
37251
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
241 |
log_info(biasedlocking)(" Revoked bias of object biased toward dead thread (" |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
242 |
PTR_FORMAT ")", p2i(biased_thread)); |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
243 |
} else { |
37251
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
244 |
log_trace(biasedlocking)(" Revoked bias of object biased toward dead thread (" |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
245 |
PTR_FORMAT ")", p2i(biased_thread)); |
1 | 246 |
} |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
247 |
return BIAS_REVOKED; |
1 | 248 |
} |
249 |
||
37251
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
250 |
// Log at "info" level if not bulk, else "trace" level |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
251 |
if (!is_bulk) { |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
252 |
log_info(biasedlocking)(" Revoked bias of object biased toward live thread (" |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
253 |
PTR_FORMAT ")", p2i(biased_thread)); |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
254 |
} else { |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
255 |
log_trace(biasedlocking)(" Revoked bias of object biased toward live thread (" |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
256 |
PTR_FORMAT ")", p2i(biased_thread)); |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
257 |
} |
9fc139ad74b5
8152358: code and comment cleanups found during the hunt for 8077392
dcubed
parents:
36186
diff
changeset
|
258 |
|
1 | 259 |
// Thread owning bias is alive. |
260 |
// Check to see whether it currently owns the lock and, if so, |
|
261 |
// write down the needed displaced headers to the thread's stack. |
|
262 |
// Otherwise, restore the object's header either to the unlocked |
|
263 |
// or unbiased state. |
|
264 |
GrowableArray<MonitorInfo*>* cached_monitor_info = get_or_compute_monitor_info(biased_thread); |
|
265 |
BasicLock* highest_lock = NULL; |
|
266 |
for (int i = 0; i < cached_monitor_info->length(); i++) { |
|
267 |
MonitorInfo* mon_info = cached_monitor_info->at(i); |
|
49658 | 268 |
if (oopDesc::equals(mon_info->owner(), obj)) { |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
269 |
log_trace(biasedlocking)(" mon_info->owner (" PTR_FORMAT ") == obj (" PTR_FORMAT ")", |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
270 |
p2i((void *) mon_info->owner()), |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
271 |
p2i((void *) obj)); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
272 |
// Assume recursive case and fix up highest lock below |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
273 |
markWord mark = markWord::encode((BasicLock*) NULL); |
1 | 274 |
highest_lock = mon_info->lock(); |
275 |
highest_lock->set_displaced_header(mark); |
|
276 |
} else { |
|
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
277 |
log_trace(biasedlocking)(" mon_info->owner (" PTR_FORMAT ") != obj (" PTR_FORMAT ")", |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
278 |
p2i((void *) mon_info->owner()), |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
279 |
p2i((void *) obj)); |
1 | 280 |
} |
281 |
} |
|
282 |
if (highest_lock != NULL) { |
|
283 |
// Fix up highest lock to contain displaced header and point |
|
284 |
// object at it |
|
285 |
highest_lock->set_displaced_header(unbiased_prototype); |
|
22859
7b88983393b7
8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents:
20282
diff
changeset
|
286 |
// Reset object header to point to displaced mark. |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
287 |
// Must release store the lock address for platforms without TSO |
22859
7b88983393b7
8029396: PPC64 (part 212): Several memory ordering fixes in C-code.
goetz
parents:
20282
diff
changeset
|
288 |
// ordering (e.g. ppc). |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
289 |
obj->release_set_mark(markWord::encode(highest_lock)); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
290 |
assert(!obj->mark().has_bias_pattern(), "illegal mark state: stack lock used bias bit"); |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
291 |
// Log at "info" level if not bulk, else "trace" level |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
292 |
if (!is_bulk) { |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
293 |
log_info(biasedlocking)(" Revoked bias of currently-locked object"); |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
294 |
} else { |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
295 |
log_trace(biasedlocking)(" Revoked bias of currently-locked object"); |
1 | 296 |
} |
297 |
} else { |
|
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
298 |
// Log at "info" level if not bulk, else "trace" level |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
299 |
if (!is_bulk) { |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
300 |
log_info(biasedlocking)(" Revoked bias of currently-unlocked object"); |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
301 |
} else { |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
302 |
log_trace(biasedlocking)(" Revoked bias of currently-unlocked object"); |
1 | 303 |
} |
304 |
if (allow_rebias) { |
|
305 |
obj->set_mark(biased_prototype); |
|
306 |
} else { |
|
307 |
// Store the unlocked value into the object's header. |
|
308 |
obj->set_mark(unbiased_prototype); |
|
309 |
} |
|
310 |
} |
|
311 |
||
47764
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
312 |
// If requested, return information on which thread held the bias |
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
313 |
if (biased_locker != NULL) { |
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
314 |
*biased_locker = biased_thread; |
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
315 |
} |
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
316 |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
317 |
return BIAS_REVOKED; |
1 | 318 |
} |
319 |
||
320 |
||
321 |
enum HeuristicsResult { |
|
322 |
HR_NOT_BIASED = 1, |
|
323 |
HR_SINGLE_REVOKE = 2, |
|
324 |
HR_BULK_REBIAS = 3, |
|
325 |
HR_BULK_REVOKE = 4 |
|
326 |
}; |
|
327 |
||
328 |
||
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
329 |
static HeuristicsResult update_heuristics(oop o) { |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
330 |
markWord mark = o->mark(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
331 |
if (!mark.has_bias_pattern()) { |
1 | 332 |
return HR_NOT_BIASED; |
333 |
} |
|
334 |
||
335 |
// Heuristics to attempt to throttle the number of revocations. |
|
336 |
// Stages: |
|
337 |
// 1. Revoke the biases of all objects in the heap of this type, |
|
338 |
// but allow rebiasing of those objects if unlocked. |
|
339 |
// 2. Revoke the biases of all objects in the heap of this type |
|
340 |
// and don't allow rebiasing of these objects. Disable |
|
341 |
// allocation of objects of that type with the bias bit set. |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
342 |
Klass* k = o->klass(); |
1 | 343 |
jlong cur_time = os::javaTimeMillis(); |
344 |
jlong last_bulk_revocation_time = k->last_biased_lock_bulk_revocation_time(); |
|
345 |
int revocation_count = k->biased_lock_revocation_count(); |
|
346 |
if ((revocation_count >= BiasedLockingBulkRebiasThreshold) && |
|
347 |
(revocation_count < BiasedLockingBulkRevokeThreshold) && |
|
348 |
(last_bulk_revocation_time != 0) && |
|
349 |
(cur_time - last_bulk_revocation_time >= BiasedLockingDecayTime)) { |
|
350 |
// This is the first revocation we've seen in a while of an |
|
351 |
// object of this type since the last time we performed a bulk |
|
352 |
// rebiasing operation. The application is allocating objects in |
|
353 |
// bulk which are biased toward a thread and then handing them |
|
354 |
// off to another thread. We can cope with this allocation |
|
355 |
// pattern via the bulk rebiasing mechanism so we reset the |
|
356 |
// klass's revocation count rather than allow it to increase |
|
357 |
// monotonically. If we see the need to perform another bulk |
|
358 |
// rebias operation later, we will, and if subsequently we see |
|
359 |
// many more revocation operations in a short period of time we |
|
360 |
// will completely disable biasing for this type. |
|
361 |
k->set_biased_lock_revocation_count(0); |
|
362 |
revocation_count = 0; |
|
363 |
} |
|
364 |
||
365 |
// Make revocation count saturate just beyond BiasedLockingBulkRevokeThreshold |
|
366 |
if (revocation_count <= BiasedLockingBulkRevokeThreshold) { |
|
367 |
revocation_count = k->atomic_incr_biased_lock_revocation_count(); |
|
368 |
} |
|
369 |
||
370 |
if (revocation_count == BiasedLockingBulkRevokeThreshold) { |
|
371 |
return HR_BULK_REVOKE; |
|
372 |
} |
|
373 |
||
374 |
if (revocation_count == BiasedLockingBulkRebiasThreshold) { |
|
375 |
return HR_BULK_REBIAS; |
|
376 |
} |
|
377 |
||
378 |
return HR_SINGLE_REVOKE; |
|
379 |
} |
|
380 |
||
381 |
||
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
382 |
BiasedLocking::Condition BiasedLocking::bulk_revoke_or_rebias_at_safepoint(oop o, |
1 | 383 |
bool bulk_rebias, |
384 |
bool attempt_rebias_of_object, |
|
385 |
JavaThread* requesting_thread) { |
|
386 |
assert(SafepointSynchronize::is_at_safepoint(), "must be done at safepoint"); |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
387 |
assert(Thread::current()->is_VM_thread(), "must be VMThread"); |
1 | 388 |
|
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
389 |
log_info(biasedlocking)("* Beginning bulk revocation (kind == %s) because of object " |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
390 |
INTPTR_FORMAT " , mark " INTPTR_FORMAT " , type %s", |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
391 |
(bulk_rebias ? "rebias" : "revoke"), |
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
392 |
p2i((void *) o), |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
393 |
o->mark().value(), |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
394 |
o->klass()->external_name()); |
1 | 395 |
|
396 |
jlong cur_time = os::javaTimeMillis(); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
397 |
o->klass()->set_last_biased_lock_bulk_revocation_time(cur_time); |
1 | 398 |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
399 |
Klass* k_o = o->klass(); |
14488 | 400 |
Klass* klass = k_o; |
1 | 401 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
402 |
{ |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
403 |
JavaThreadIteratorWithHandle jtiwh; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
404 |
|
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
405 |
if (bulk_rebias) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
406 |
// Use the epoch in the klass of the object to implicitly revoke |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
407 |
// all biases of objects of this data type and force them to be |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
408 |
// reacquired. However, we also need to walk the stacks of all |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
409 |
// threads and update the headers of lightweight locked objects |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
410 |
// with biases to have the current epoch. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
411 |
|
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
412 |
// If the prototype header doesn't have the bias pattern, don't |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
413 |
// try to update the epoch -- assume another VM operation came in |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
414 |
// and reset the header to the unbiased state, which will |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
415 |
// implicitly cause all existing biases to be revoked |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
416 |
if (klass->prototype_header().has_bias_pattern()) { |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
417 |
int prev_epoch = klass->prototype_header().bias_epoch(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
418 |
klass->set_prototype_header(klass->prototype_header().incr_bias_epoch()); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
419 |
int cur_epoch = klass->prototype_header().bias_epoch(); |
1 | 420 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
421 |
// Now walk all threads' stacks and adjust epochs of any biased |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
422 |
// and locked objects of this data type we encounter |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
423 |
for (; JavaThread *thr = jtiwh.next(); ) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
424 |
GrowableArray<MonitorInfo*>* cached_monitor_info = get_or_compute_monitor_info(thr); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
425 |
for (int i = 0; i < cached_monitor_info->length(); i++) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
426 |
MonitorInfo* mon_info = cached_monitor_info->at(i); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
427 |
oop owner = mon_info->owner(); |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
428 |
markWord mark = owner->mark(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
429 |
if ((owner->klass() == k_o) && mark.has_bias_pattern()) { |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
430 |
// We might have encountered this object already in the case of recursive locking |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
431 |
assert(mark.bias_epoch() == prev_epoch || mark.bias_epoch() == cur_epoch, "error in bias epoch adjustment"); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
432 |
owner->set_mark(mark.set_bias_epoch(cur_epoch)); |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
433 |
} |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
434 |
} |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
435 |
} |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
436 |
} |
1 | 437 |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
438 |
// At this point we're done. All we have to do is potentially |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
439 |
// adjust the header of the given object to revoke its bias. |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
440 |
single_revoke_at_safepoint(o, attempt_rebias_of_object && klass->prototype_header().has_bias_pattern(), true, requesting_thread, NULL); |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
441 |
} else { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
442 |
if (log_is_enabled(Info, biasedlocking)) { |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
443 |
ResourceMark rm; |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
444 |
log_info(biasedlocking)("* Disabling biased locking for type %s", klass->external_name()); |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
445 |
} |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
446 |
|
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
447 |
// Disable biased locking for this data type. Not only will this |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
448 |
// cause future instances to not be biased, but existing biased |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
449 |
// instances will notice that this implicitly caused their biases |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
450 |
// to be revoked. |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
451 |
klass->set_prototype_header(markWord::prototype()); |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
452 |
|
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
453 |
// Now walk all threads' stacks and forcibly revoke the biases of |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
454 |
// any locked and biased objects of this data type we encounter. |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
455 |
for (; JavaThread *thr = jtiwh.next(); ) { |
1 | 456 |
GrowableArray<MonitorInfo*>* cached_monitor_info = get_or_compute_monitor_info(thr); |
457 |
for (int i = 0; i < cached_monitor_info->length(); i++) { |
|
458 |
MonitorInfo* mon_info = cached_monitor_info->at(i); |
|
459 |
oop owner = mon_info->owner(); |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
460 |
markWord mark = owner->mark(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
461 |
if ((owner->klass() == k_o) && mark.has_bias_pattern()) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
462 |
single_revoke_at_safepoint(owner, false, true, requesting_thread, NULL); |
1 | 463 |
} |
464 |
} |
|
465 |
} |
|
466 |
||
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
467 |
// Must force the bias of the passed object to be forcibly revoked |
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
468 |
// as well to ensure guarantees to callers |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
469 |
single_revoke_at_safepoint(o, false, true, requesting_thread, NULL); |
1 | 470 |
} |
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
471 |
} // ThreadsListHandle is destroyed here. |
1 | 472 |
|
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
473 |
log_info(biasedlocking)("* Ending bulk revocation"); |
1 | 474 |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
475 |
BiasedLocking::Condition status_code = BIAS_REVOKED; |
1 | 476 |
|
477 |
if (attempt_rebias_of_object && |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
478 |
o->mark().has_bias_pattern() && |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
479 |
klass->prototype_header().has_bias_pattern()) { |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
480 |
markWord new_mark = markWord::encode(requesting_thread, o->mark().age(), |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
481 |
klass->prototype_header().bias_epoch()); |
1 | 482 |
o->set_mark(new_mark); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
483 |
status_code = BIAS_REVOKED_AND_REBIASED; |
36186
06763de0d7ad
8149383: Convert TraceBiasedLocking to Unified Logging
rprotacio
parents:
29081
diff
changeset
|
484 |
log_info(biasedlocking)(" Rebiased object toward thread " INTPTR_FORMAT, (intptr_t) requesting_thread); |
1 | 485 |
} |
486 |
||
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
487 |
assert(!o->mark().has_bias_pattern() || |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
488 |
(attempt_rebias_of_object && (o->mark().biased_locker() == requesting_thread)), |
1 | 489 |
"bug in bulk bias revocation"); |
490 |
||
491 |
return status_code; |
|
492 |
} |
|
493 |
||
494 |
||
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
495 |
static void clean_up_cached_monitor_info(JavaThread* thread = NULL) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
496 |
if (thread != NULL) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
497 |
thread->set_cached_monitor_info(NULL); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
498 |
} else { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
499 |
// Walk the thread list clearing out the cached monitors |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
500 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *thr = jtiwh.next(); ) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
501 |
thr->set_cached_monitor_info(NULL); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
502 |
} |
1 | 503 |
} |
504 |
} |
|
505 |
||
506 |
||
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
507 |
class VM_BulkRevokeBias : public VM_Operation { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
508 |
private: |
1 | 509 |
Handle* _obj; |
510 |
JavaThread* _requesting_thread; |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
511 |
bool _bulk_rebias; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
512 |
bool _attempt_rebias_of_object; |
1 | 513 |
BiasedLocking::Condition _status_code; |
54710
aa29971a17a1
8220724: TestBiasedLockRevocationEvents fails while matching revoke events to VMOperation events
pchilanomate
parents:
52877
diff
changeset
|
514 |
uint64_t _safepoint_id; |
1 | 515 |
|
516 |
public: |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
517 |
VM_BulkRevokeBias(Handle* obj, JavaThread* requesting_thread, |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
518 |
bool bulk_rebias, |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
519 |
bool attempt_rebias_of_object) |
1 | 520 |
: _obj(obj) |
521 |
, _requesting_thread(requesting_thread) |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
522 |
, _bulk_rebias(bulk_rebias) |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
523 |
, _attempt_rebias_of_object(attempt_rebias_of_object) |
47764
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
524 |
, _status_code(BiasedLocking::NOT_BIASED) |
54710
aa29971a17a1
8220724: TestBiasedLockRevocationEvents fails while matching revoke events to VMOperation events
pchilanomate
parents:
52877
diff
changeset
|
525 |
, _safepoint_id(0) {} |
1 | 526 |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
527 |
virtual VMOp_Type type() const { return VMOp_BulkRevokeBias; } |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
528 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
529 |
virtual void doit() { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
530 |
_status_code = BiasedLocking::bulk_revoke_or_rebias_at_safepoint((*_obj)(), _bulk_rebias, _attempt_rebias_of_object, _requesting_thread); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
531 |
_safepoint_id = SafepointSynchronize::safepoint_id(); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
532 |
clean_up_cached_monitor_info(); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
533 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
534 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
535 |
bool is_bulk_rebias() const { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
536 |
return _bulk_rebias; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
537 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
538 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
539 |
BiasedLocking::Condition status_code() const { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
540 |
return _status_code; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
541 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
542 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
543 |
uint64_t safepoint_id() const { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
544 |
return _safepoint_id; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
545 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
546 |
}; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
547 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
548 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
549 |
class RevokeOneBias : public ThreadClosure { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
550 |
protected: |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
551 |
Handle _obj; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
552 |
JavaThread* _requesting_thread; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
553 |
JavaThread* _biased_locker; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
554 |
BiasedLocking::Condition _status_code; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
555 |
traceid _biased_locker_id; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
556 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
557 |
public: |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
558 |
RevokeOneBias(Handle obj, JavaThread* requesting_thread, JavaThread* biased_locker) |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
559 |
: _obj(obj) |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
560 |
, _requesting_thread(requesting_thread) |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
561 |
, _biased_locker(biased_locker) |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
562 |
, _status_code(BiasedLocking::NOT_BIASED) |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
563 |
, _biased_locker_id(0) {} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
564 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
565 |
void do_thread(Thread* target) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
566 |
assert(target == _biased_locker, "Wrong thread"); |
1 | 567 |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
568 |
oop o = _obj(); |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
569 |
markWord mark = o->mark(); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
570 |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
571 |
if (!mark.has_bias_pattern()) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
572 |
return; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
573 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
574 |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
575 |
markWord prototype = o->klass()->prototype_header(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
576 |
if (!prototype.has_bias_pattern()) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
577 |
// This object has a stale bias from before the handshake |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
578 |
// was requested. If we fail this race, the object's bias |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
579 |
// has been revoked by another thread so we simply return. |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
580 |
markWord biased_value = mark; |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
581 |
mark = o->cas_set_mark(markWord::prototype().set_age(mark.age()), mark); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
582 |
assert(!o->mark().has_bias_pattern(), "even if we raced, should still be revoked"); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
583 |
if (biased_value == mark) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
584 |
_status_code = BiasedLocking::BIAS_REVOKED; |
1 | 585 |
} |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
586 |
return; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
587 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
588 |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
589 |
if (_biased_locker == mark.biased_locker()) { |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
590 |
if (mark.bias_epoch() == prototype.bias_epoch()) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
591 |
// Epoch is still valid. This means biaser could be currently |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
592 |
// synchronized on this object. We must walk its stack looking |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
593 |
// for monitor records associated with this object and change |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
594 |
// them to be stack locks if any are found. |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
595 |
ResourceMark rm; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
596 |
BiasedLocking::walk_stack_and_revoke(o, _biased_locker); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
597 |
_biased_locker->set_cached_monitor_info(NULL); |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
598 |
assert(!o->mark().has_bias_pattern(), "invariant"); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
599 |
_biased_locker_id = JFR_THREAD_ID(_biased_locker); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
600 |
_status_code = BiasedLocking::BIAS_REVOKED; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
601 |
return; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
602 |
} else { |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
603 |
markWord biased_value = mark; |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
604 |
mark = o->cas_set_mark(markWord::prototype().set_age(mark.age()), mark); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
605 |
if (mark == biased_value || !mark.has_bias_pattern()) { |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
606 |
assert(!o->mark().has_bias_pattern(), "should be revoked"); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
607 |
_status_code = (biased_value == mark) ? BiasedLocking::BIAS_REVOKED : BiasedLocking::NOT_BIASED; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
608 |
return; |
1 | 609 |
} |
610 |
} |
|
611 |
} |
|
612 |
||
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
613 |
_status_code = BiasedLocking::NOT_REVOKED; |
1 | 614 |
} |
615 |
||
616 |
BiasedLocking::Condition status_code() const { |
|
617 |
return _status_code; |
|
618 |
} |
|
47764
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
619 |
|
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
620 |
traceid biased_locker() const { |
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
621 |
return _biased_locker_id; |
029d5efaaa6c
8189368: Add information on current bias holder for BiasedLockRevocation event
egahlin
parents:
47632
diff
changeset
|
622 |
} |
1 | 623 |
}; |
624 |
||
625 |
||
50113 | 626 |
static void post_self_revocation_event(EventBiasedLockSelfRevocation* event, Klass* k) { |
627 |
assert(event != NULL, "invariant"); |
|
628 |
assert(k != NULL, "invariant"); |
|
629 |
assert(event->should_commit(), "invariant"); |
|
630 |
event->set_lockClass(k); |
|
631 |
event->commit(); |
|
632 |
} |
|
633 |
||
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
634 |
static void post_revocation_event(EventBiasedLockRevocation* event, Klass* k, RevokeOneBias* op) { |
50113 | 635 |
assert(event != NULL, "invariant"); |
636 |
assert(k != NULL, "invariant"); |
|
54710
aa29971a17a1
8220724: TestBiasedLockRevocationEvents fails while matching revoke events to VMOperation events
pchilanomate
parents:
52877
diff
changeset
|
637 |
assert(op != NULL, "invariant"); |
50113 | 638 |
assert(event->should_commit(), "invariant"); |
639 |
event->set_lockClass(k); |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
640 |
event->set_safepointId(0); |
54710
aa29971a17a1
8220724: TestBiasedLockRevocationEvents fails while matching revoke events to VMOperation events
pchilanomate
parents:
52877
diff
changeset
|
641 |
event->set_previousOwner(op->biased_locker()); |
50113 | 642 |
event->commit(); |
643 |
} |
|
644 |
||
54710
aa29971a17a1
8220724: TestBiasedLockRevocationEvents fails while matching revoke events to VMOperation events
pchilanomate
parents:
52877
diff
changeset
|
645 |
static void post_class_revocation_event(EventBiasedLockClassRevocation* event, Klass* k, VM_BulkRevokeBias* op) { |
50113 | 646 |
assert(event != NULL, "invariant"); |
647 |
assert(k != NULL, "invariant"); |
|
54710
aa29971a17a1
8220724: TestBiasedLockRevocationEvents fails while matching revoke events to VMOperation events
pchilanomate
parents:
52877
diff
changeset
|
648 |
assert(op != NULL, "invariant"); |
50113 | 649 |
assert(event->should_commit(), "invariant"); |
650 |
event->set_revokedClass(k); |
|
54710
aa29971a17a1
8220724: TestBiasedLockRevocationEvents fails while matching revoke events to VMOperation events
pchilanomate
parents:
52877
diff
changeset
|
651 |
event->set_disableBiasing(!op->is_bulk_rebias()); |
aa29971a17a1
8220724: TestBiasedLockRevocationEvents fails while matching revoke events to VMOperation events
pchilanomate
parents:
52877
diff
changeset
|
652 |
event->set_safepointId(op->safepoint_id()); |
50113 | 653 |
event->commit(); |
654 |
} |
|
1 | 655 |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
656 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
657 |
BiasedLocking::Condition BiasedLocking::single_revoke_with_handshake(Handle obj, JavaThread *requester, JavaThread *biaser) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
658 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
659 |
EventBiasedLockRevocation event; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
660 |
if (PrintBiasedLockingStatistics) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
661 |
Atomic::inc(handshakes_count_addr()); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
662 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
663 |
log_info(biasedlocking, handshake)("JavaThread " INTPTR_FORMAT " handshaking JavaThread " |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
664 |
INTPTR_FORMAT " to revoke object " INTPTR_FORMAT, p2i(requester), |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
665 |
p2i(biaser), p2i(obj())); |
1 | 666 |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
667 |
RevokeOneBias revoke(obj, requester, biaser); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
668 |
bool executed = Handshake::execute(&revoke, biaser); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
669 |
if (revoke.status_code() == NOT_REVOKED) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
670 |
return NOT_REVOKED; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
671 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
672 |
if (executed) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
673 |
log_info(biasedlocking, handshake)("Handshake revocation for object " INTPTR_FORMAT " succeeded. Bias was %srevoked", |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
674 |
p2i(obj()), (revoke.status_code() == BIAS_REVOKED ? "" : "already ")); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
675 |
if (event.should_commit() && revoke.status_code() == BIAS_REVOKED) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
676 |
post_revocation_event(&event, obj->klass(), &revoke); |
1 | 677 |
} |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
678 |
assert(!obj->mark().has_bias_pattern(), "invariant"); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
679 |
return revoke.status_code(); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
680 |
} else { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
681 |
// Thread was not alive. |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
682 |
// Grab Threads_lock before manually trying to revoke bias. This avoids race with a newly |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
683 |
// created JavaThread (that happens to get the same memory address as biaser) synchronizing |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
684 |
// on this object. |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
685 |
{ |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
686 |
MutexLocker ml(Threads_lock); |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
687 |
markWord mark = obj->mark(); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
688 |
// Check if somebody else was able to revoke it before biased thread exited. |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
689 |
if (!mark.has_bias_pattern()) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
690 |
return NOT_BIASED; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
691 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
692 |
ThreadsListHandle tlh; |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
693 |
markWord prototype = obj->klass()->prototype_header(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
694 |
if (!prototype.has_bias_pattern() || (!tlh.includes(biaser) && biaser == mark.biased_locker() && |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
695 |
prototype.bias_epoch() == mark.bias_epoch())) { |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
696 |
obj->cas_set_mark(markWord::prototype().set_age(mark.age()), mark); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
697 |
if (event.should_commit()) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
698 |
post_revocation_event(&event, obj->klass(), &revoke); |
1 | 699 |
} |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
700 |
assert(!obj->mark().has_bias_pattern(), "bias should be revoked by now"); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
701 |
return BIAS_REVOKED; |
1 | 702 |
} |
703 |
} |
|
704 |
} |
|
705 |
||
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
706 |
return NOT_REVOKED; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
707 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
708 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
709 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
710 |
// Caller should have instantiated a ResourceMark object before calling this method |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
711 |
void BiasedLocking::walk_stack_and_revoke(oop obj, JavaThread* biased_locker) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
712 |
assert(!SafepointSynchronize::is_at_safepoint() || !ThreadLocalHandshakes, |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
713 |
"if ThreadLocalHandshakes is enabled this should always be executed outside safepoints"); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
714 |
assert(Thread::current() == biased_locker || Thread::current()->is_VM_thread(), "wrong thread"); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
715 |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
716 |
markWord mark = obj->mark(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
717 |
assert(mark.biased_locker() == biased_locker && |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
718 |
obj->klass()->prototype_header().bias_epoch() == mark.bias_epoch(), "invariant"); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
719 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
720 |
log_trace(biasedlocking)("%s(" INTPTR_FORMAT ") revoking object " INTPTR_FORMAT ", mark " |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
721 |
INTPTR_FORMAT ", type %s, prototype header " INTPTR_FORMAT |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
722 |
", biaser " INTPTR_FORMAT " %s", |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
723 |
Thread::current()->is_VM_thread() ? "VMThread" : "JavaThread", |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
724 |
p2i(Thread::current()), |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
725 |
p2i(obj), |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
726 |
mark.value(), |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
727 |
obj->klass()->external_name(), |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
728 |
obj->klass()->prototype_header().value(), |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
729 |
p2i(biased_locker), |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
730 |
Thread::current()->is_VM_thread() ? "" : "(walking own stack)"); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
731 |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
732 |
markWord unbiased_prototype = markWord::prototype().set_age(obj->mark().age()); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
733 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
734 |
GrowableArray<MonitorInfo*>* cached_monitor_info = get_or_compute_monitor_info(biased_locker); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
735 |
BasicLock* highest_lock = NULL; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
736 |
for (int i = 0; i < cached_monitor_info->length(); i++) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
737 |
MonitorInfo* mon_info = cached_monitor_info->at(i); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
738 |
if (oopDesc::equals(mon_info->owner(), obj)) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
739 |
log_trace(biasedlocking)(" mon_info->owner (" PTR_FORMAT ") == obj (" PTR_FORMAT ")", |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
740 |
p2i(mon_info->owner()), |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
741 |
p2i(obj)); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
742 |
// Assume recursive case and fix up highest lock below |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
743 |
markWord mark = markWord::encode((BasicLock*) NULL); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
744 |
highest_lock = mon_info->lock(); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
745 |
highest_lock->set_displaced_header(mark); |
1 | 746 |
} else { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
747 |
log_trace(biasedlocking)(" mon_info->owner (" PTR_FORMAT ") != obj (" PTR_FORMAT ")", |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
748 |
p2i(mon_info->owner()), |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
749 |
p2i(obj)); |
1 | 750 |
} |
751 |
} |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
752 |
if (highest_lock != NULL) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
753 |
// Fix up highest lock to contain displaced header and point |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
754 |
// object at it |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
755 |
highest_lock->set_displaced_header(unbiased_prototype); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
756 |
// Reset object header to point to displaced mark. |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
757 |
// Must release store the lock address for platforms without TSO |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
758 |
// ordering (e.g. ppc). |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
759 |
obj->release_set_mark(markWord::encode(highest_lock)); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
760 |
assert(!obj->mark().has_bias_pattern(), "illegal mark state: stack lock used bias bit"); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
761 |
log_info(biasedlocking)(" Revoked bias of currently-locked object"); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
762 |
} else { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
763 |
log_info(biasedlocking)(" Revoked bias of currently-unlocked object"); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
764 |
// Store the unlocked value into the object's header. |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
765 |
obj->set_mark(unbiased_prototype); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
766 |
} |
1 | 767 |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
768 |
assert(!obj->mark().has_bias_pattern(), "must not be biased"); |
1 | 769 |
} |
770 |
||
771 |
||
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
772 |
BiasedLocking::Condition BiasedLocking::revoke_and_rebias(Handle obj, bool attempt_rebias, TRAPS) { |
1 | 773 |
assert(!SafepointSynchronize::is_at_safepoint(), "must not be called while at safepoint"); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
774 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
775 |
while (true) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
776 |
// We can revoke the biases of anonymously-biased objects |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
777 |
// efficiently enough that we should not cause these revocations to |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
778 |
// update the heuristics because doing so may cause unwanted bulk |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
779 |
// revocations (which are expensive) to occur. |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
780 |
markWord mark = obj->mark(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
781 |
if (mark.is_biased_anonymously() && !attempt_rebias) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
782 |
// We are probably trying to revoke the bias of this object due to |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
783 |
// an identity hash code computation. Try to revoke the bias |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
784 |
// without a safepoint. This is possible if we can successfully |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
785 |
// compare-and-exchange an unbiased header into the mark word of |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
786 |
// the object, meaning that no other thread has raced to acquire |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
787 |
// the bias of the object. |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
788 |
markWord biased_value = mark; |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
789 |
markWord unbiased_prototype = markWord::prototype().set_age(mark.age()); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
790 |
markWord res_mark = obj->cas_set_mark(unbiased_prototype, mark); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
791 |
if (res_mark == biased_value) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
792 |
return BIAS_REVOKED; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
793 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
794 |
mark = res_mark; // Refresh mark with the latest value. |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
795 |
} else if (mark.has_bias_pattern()) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
796 |
Klass* k = obj->klass(); |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
797 |
markWord prototype_header = k->prototype_header(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
798 |
if (!prototype_header.has_bias_pattern()) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
799 |
// This object has a stale bias from before the bulk revocation |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
800 |
// for this data type occurred. It's pointless to update the |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
801 |
// heuristics at this point so simply update the header with a |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
802 |
// CAS. If we fail this race, the object's bias has been revoked |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
803 |
// by another thread so we simply return and let the caller deal |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
804 |
// with it. |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
805 |
obj->cas_set_mark(prototype_header.set_age(mark.age()), mark); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
806 |
assert(!obj->mark().has_bias_pattern(), "even if we raced, should still be revoked"); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
807 |
return BIAS_REVOKED; |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
808 |
} else if (prototype_header.bias_epoch() != mark.bias_epoch()) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
809 |
// The epoch of this biasing has expired indicating that the |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
810 |
// object is effectively unbiased. Depending on whether we need |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
811 |
// to rebias or revoke the bias of this object we can do it |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
812 |
// efficiently enough with a CAS that we shouldn't update the |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
813 |
// heuristics. This is normally done in the assembly code but we |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
814 |
// can reach this point due to various points in the runtime |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
815 |
// needing to revoke biases. |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
816 |
markWord res_mark; |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
817 |
if (attempt_rebias) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
818 |
assert(THREAD->is_Java_thread(), ""); |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
819 |
markWord biased_value = mark; |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
820 |
markWord rebiased_prototype = markWord::encode((JavaThread*) THREAD, mark.age(), prototype_header.bias_epoch()); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
821 |
res_mark = obj->cas_set_mark(rebiased_prototype, mark); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
822 |
if (res_mark == biased_value) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
823 |
return BIAS_REVOKED_AND_REBIASED; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
824 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
825 |
} else { |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
826 |
markWord biased_value = mark; |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
827 |
markWord unbiased_prototype = markWord::prototype().set_age(mark.age()); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
828 |
res_mark = obj->cas_set_mark(unbiased_prototype, mark); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
829 |
if (res_mark == biased_value) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
830 |
return BIAS_REVOKED; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
831 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
832 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
833 |
mark = res_mark; // Refresh mark with the latest value. |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
834 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
835 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
836 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
837 |
HeuristicsResult heuristics = update_heuristics(obj()); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
838 |
if (heuristics == HR_NOT_BIASED) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
839 |
return NOT_BIASED; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
840 |
} else if (heuristics == HR_SINGLE_REVOKE) { |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
841 |
JavaThread *blt = mark.biased_locker(); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
842 |
assert(blt != NULL, "invariant"); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
843 |
if (blt == THREAD) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
844 |
// A thread is trying to revoke the bias of an object biased |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
845 |
// toward it, again likely due to an identity hash code |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
846 |
// computation. We can again avoid a safepoint/handshake in this case |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
847 |
// since we are only going to walk our own stack. There are no |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
848 |
// races with revocations occurring in other threads because we |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
849 |
// reach no safepoints in the revocation path. |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
850 |
EventBiasedLockSelfRevocation event; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
851 |
ResourceMark rm; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
852 |
walk_stack_and_revoke(obj(), blt); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
853 |
blt->set_cached_monitor_info(NULL); |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
854 |
assert(!obj->mark().has_bias_pattern(), "invariant"); |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
855 |
if (event.should_commit()) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
856 |
post_self_revocation_event(&event, obj->klass()); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
857 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
858 |
return BIAS_REVOKED; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
859 |
} else { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
860 |
BiasedLocking::Condition cond = single_revoke_with_handshake(obj, (JavaThread*)THREAD, blt); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
861 |
if (cond != NOT_REVOKED) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
862 |
return cond; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
863 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
864 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
865 |
} else { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
866 |
assert((heuristics == HR_BULK_REVOKE) || |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
867 |
(heuristics == HR_BULK_REBIAS), "?"); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
868 |
EventBiasedLockClassRevocation event; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
869 |
VM_BulkRevokeBias bulk_revoke(&obj, (JavaThread*)THREAD, |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
870 |
(heuristics == HR_BULK_REBIAS), |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
871 |
attempt_rebias); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
872 |
VMThread::execute(&bulk_revoke); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
873 |
if (event.should_commit()) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
874 |
post_class_revocation_event(&event, obj->klass(), &bulk_revoke); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
875 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
876 |
return bulk_revoke.status_code(); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
877 |
} |
1 | 878 |
} |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
879 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
880 |
|
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
881 |
// All objects in objs should be locked by biaser |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
882 |
void BiasedLocking::revoke(GrowableArray<Handle>* objs, JavaThread *biaser) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
883 |
bool clean_my_cache = false; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
884 |
for (int i = 0; i < objs->length(); i++) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
885 |
oop obj = (objs->at(i))(); |
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
886 |
markWord mark = obj->mark(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
887 |
if (mark.has_bias_pattern()) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
888 |
walk_stack_and_revoke(obj, biaser); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
889 |
clean_my_cache = true; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
890 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
891 |
} |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
892 |
if (clean_my_cache) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
893 |
clean_up_cached_monitor_info(biaser); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
894 |
} |
1 | 895 |
} |
896 |
||
897 |
||
898 |
void BiasedLocking::revoke_at_safepoint(Handle h_obj) { |
|
899 |
assert(SafepointSynchronize::is_at_safepoint(), "must only be called while at safepoint"); |
|
900 |
oop obj = h_obj(); |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
901 |
HeuristicsResult heuristics = update_heuristics(obj); |
1 | 902 |
if (heuristics == HR_SINGLE_REVOKE) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
903 |
JavaThread* biased_locker = NULL; |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
904 |
single_revoke_at_safepoint(obj, false, false, NULL, &biased_locker); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
905 |
if (biased_locker) { |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
906 |
clean_up_cached_monitor_info(biased_locker); |
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
907 |
} |
1 | 908 |
} else if ((heuristics == HR_BULK_REBIAS) || |
909 |
(heuristics == HR_BULK_REVOKE)) { |
|
910 |
bulk_revoke_or_rebias_at_safepoint(obj, (heuristics == HR_BULK_REBIAS), false, NULL); |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
911 |
clean_up_cached_monitor_info(); |
1 | 912 |
} |
913 |
} |
|
914 |
||
915 |
||
916 |
void BiasedLocking::revoke_at_safepoint(GrowableArray<Handle>* objs) { |
|
917 |
assert(SafepointSynchronize::is_at_safepoint(), "must only be called while at safepoint"); |
|
918 |
int len = objs->length(); |
|
919 |
for (int i = 0; i < len; i++) { |
|
920 |
oop obj = (objs->at(i))(); |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
921 |
HeuristicsResult heuristics = update_heuristics(obj); |
1 | 922 |
if (heuristics == HR_SINGLE_REVOKE) { |
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
923 |
single_revoke_at_safepoint(obj, false, false, NULL, NULL); |
1 | 924 |
} else if ((heuristics == HR_BULK_REBIAS) || |
925 |
(heuristics == HR_BULK_REVOKE)) { |
|
926 |
bulk_revoke_or_rebias_at_safepoint(obj, (heuristics == HR_BULK_REBIAS), false, NULL); |
|
927 |
} |
|
928 |
} |
|
929 |
clean_up_cached_monitor_info(); |
|
930 |
} |
|
931 |
||
932 |
||
933 |
void BiasedLocking::preserve_marks() { |
|
934 |
if (!UseBiasedLocking) |
|
935 |
return; |
|
936 |
||
937 |
assert(SafepointSynchronize::is_at_safepoint(), "must only be called while at safepoint"); |
|
938 |
||
939 |
assert(_preserved_oop_stack == NULL, "double initialization"); |
|
940 |
assert(_preserved_mark_stack == NULL, "double initialization"); |
|
941 |
||
942 |
// In order to reduce the number of mark words preserved during GC |
|
943 |
// due to the presence of biased locking, we reinitialize most mark |
|
944 |
// words to the class's prototype during GC -- even those which have |
|
945 |
// a currently valid bias owner. One important situation where we |
|
946 |
// must not clobber a bias is when a biased object is currently |
|
947 |
// locked. To handle this case we iterate over the currently-locked |
|
948 |
// monitors in a prepass and, if they are biased, preserve their |
|
949 |
// mark words here. This should be a relatively small set of objects |
|
950 |
// especially compared to the number of objects in the heap. |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
951 |
_preserved_mark_stack = new (ResourceObj::C_HEAP, mtInternal) GrowableArray<markWord>(10, true); |
13195 | 952 |
_preserved_oop_stack = new (ResourceObj::C_HEAP, mtInternal) GrowableArray<Handle>(10, true); |
1 | 953 |
|
954 |
ResourceMark rm; |
|
955 |
Thread* cur = Thread::current(); |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47764
diff
changeset
|
956 |
for (JavaThreadIteratorWithHandle jtiwh; JavaThread *thread = jtiwh.next(); ) { |
1 | 957 |
if (thread->has_last_Java_frame()) { |
958 |
RegisterMap rm(thread); |
|
959 |
for (javaVFrame* vf = thread->last_java_vframe(&rm); vf != NULL; vf = vf->java_sender()) { |
|
960 |
GrowableArray<MonitorInfo*> *monitors = vf->monitors(); |
|
961 |
if (monitors != NULL) { |
|
962 |
int len = monitors->length(); |
|
963 |
// Walk monitors youngest to oldest |
|
964 |
for (int i = len - 1; i >= 0; i--) { |
|
965 |
MonitorInfo* mon_info = monitors->at(i); |
|
3171
aa289b22b577
6837472: com/sun/jdi/MonitorFrameInfo.java fails with AggressiveOpts in 6u14
kvn
parents:
1600
diff
changeset
|
966 |
if (mon_info->owner_is_scalar_replaced()) continue; |
1 | 967 |
oop owner = mon_info->owner(); |
968 |
if (owner != NULL) { |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
969 |
markWord mark = owner->mark(); |
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
970 |
if (mark.has_bias_pattern()) { |
1 | 971 |
_preserved_oop_stack->push(Handle(cur, owner)); |
972 |
_preserved_mark_stack->push(mark); |
|
973 |
} |
|
974 |
} |
|
975 |
} |
|
976 |
} |
|
977 |
} |
|
978 |
} |
|
979 |
} |
|
980 |
} |
|
981 |
||
982 |
||
983 |
void BiasedLocking::restore_marks() { |
|
984 |
if (!UseBiasedLocking) |
|
985 |
return; |
|
986 |
||
987 |
assert(_preserved_oop_stack != NULL, "double free"); |
|
988 |
assert(_preserved_mark_stack != NULL, "double free"); |
|
989 |
||
990 |
int len = _preserved_oop_stack->length(); |
|
991 |
for (int i = 0; i < len; i++) { |
|
992 |
Handle owner = _preserved_oop_stack->at(i); |
|
57777
90ead0febf56
8229258: Rework markOop and markOopDesc into a simpler mark word value carrier
stefank
parents:
55625
diff
changeset
|
993 |
markWord mark = _preserved_mark_stack->at(i); |
1 | 994 |
owner->set_mark(mark); |
995 |
} |
|
996 |
||
997 |
delete _preserved_oop_stack; |
|
998 |
_preserved_oop_stack = NULL; |
|
999 |
delete _preserved_mark_stack; |
|
1000 |
_preserved_mark_stack = NULL; |
|
1001 |
} |
|
1002 |
||
1003 |
||
1004 |
int* BiasedLocking::total_entry_count_addr() { return _counters.total_entry_count_addr(); } |
|
1005 |
int* BiasedLocking::biased_lock_entry_count_addr() { return _counters.biased_lock_entry_count_addr(); } |
|
1006 |
int* BiasedLocking::anonymously_biased_lock_entry_count_addr() { return _counters.anonymously_biased_lock_entry_count_addr(); } |
|
1007 |
int* BiasedLocking::rebiased_lock_entry_count_addr() { return _counters.rebiased_lock_entry_count_addr(); } |
|
1008 |
int* BiasedLocking::revoked_lock_entry_count_addr() { return _counters.revoked_lock_entry_count_addr(); } |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
1009 |
int* BiasedLocking::handshakes_count_addr() { return _counters.handshakes_count_addr(); } |
1 | 1010 |
int* BiasedLocking::fast_path_entry_count_addr() { return _counters.fast_path_entry_count_addr(); } |
1011 |
int* BiasedLocking::slow_path_entry_count_addr() { return _counters.slow_path_entry_count_addr(); } |
|
1012 |
||
1013 |
||
1014 |
// BiasedLockingCounters |
|
1015 |
||
54807 | 1016 |
int BiasedLockingCounters::slow_path_entry_count() const { |
1 | 1017 |
if (_slow_path_entry_count != 0) { |
1018 |
return _slow_path_entry_count; |
|
1019 |
} |
|
1020 |
int sum = _biased_lock_entry_count + _anonymously_biased_lock_entry_count + |
|
1021 |
_rebiased_lock_entry_count + _revoked_lock_entry_count + |
|
1022 |
_fast_path_entry_count; |
|
1023 |
||
1024 |
return _total_entry_count - sum; |
|
1025 |
} |
|
1026 |
||
54807 | 1027 |
void BiasedLockingCounters::print_on(outputStream* st) const { |
1 | 1028 |
tty->print_cr("# total entries: %d", _total_entry_count); |
1029 |
tty->print_cr("# biased lock entries: %d", _biased_lock_entry_count); |
|
1030 |
tty->print_cr("# anonymously biased lock entries: %d", _anonymously_biased_lock_entry_count); |
|
1031 |
tty->print_cr("# rebiased lock entries: %d", _rebiased_lock_entry_count); |
|
1032 |
tty->print_cr("# revoked lock entries: %d", _revoked_lock_entry_count); |
|
55625
f7e8dbb77156
8191890: Biased locking still uses the inferior stop the world safepoint for revocation
pchilanomate
parents:
55537
diff
changeset
|
1033 |
tty->print_cr("# handshakes entries: %d", _handshakes_count); |
1 | 1034 |
tty->print_cr("# fast path lock entries: %d", _fast_path_entry_count); |
1035 |
tty->print_cr("# slow path lock entries: %d", slow_path_entry_count()); |
|
1036 |
} |
|
54807 | 1037 |
|
1038 |
void BiasedLockingCounters::print() const { print_on(tty); } |