author | phh |
Sat, 30 Nov 2019 14:33:05 -0800 | |
changeset 59330 | 5b96c12f909d |
parent 59252 | 623722a6aeb9 |
permissions | -rw-r--r-- |
6975 | 1 |
/* |
54469
8592226f5cd3
8221584: SIGSEGV in os::PlatformEvent::unpark() in JvmtiRawMonitor::raw_exit while posting method exit event
dholmes
parents:
51702
diff
changeset
|
2 |
* Copyright (c) 2003, 2019, Oracle and/or its affiliates. All rights reserved. |
6975 | 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 |
* |
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
20 |
* or visit www.oracle.com if you need additional information or have any |
|
21 |
* questions. |
|
22 |
* |
|
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
49360 | 26 |
#include "memory/allocation.inline.hpp" |
7397 | 27 |
#include "prims/jvmtiRawMonitor.hpp" |
40655
9f644073d3a0
8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents:
25351
diff
changeset
|
28 |
#include "runtime/atomic.hpp" |
49449
ef5d5d343e2a
8199263: Split interfaceSupport.hpp to not require including .inline.hpp files
coleenp
parents:
49360
diff
changeset
|
29 |
#include "runtime/interfaceSupport.inline.hpp" |
50429
83aec1d357d4
8204301: Make OrderAccess functions available to hpp rather than inline.hpp files
coleenp
parents:
49449
diff
changeset
|
30 |
#include "runtime/orderAccess.hpp" |
24351
61b33cc6d3cf
8042195: Introduce umbrella header orderAccess.inline.hpp.
goetz
parents:
13963
diff
changeset
|
31 |
#include "runtime/thread.inline.hpp" |
6975 | 32 |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
33 |
JvmtiRawMonitor::QNode::QNode(Thread* thread) : _next(NULL), _prev(NULL), |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
34 |
_event(thread->_ParkEvent), |
58509 | 35 |
_notified(0), _t_state(TS_RUN) { |
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
36 |
} |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
37 |
|
58509 | 38 |
GrowableArray<JvmtiRawMonitor*>* JvmtiPendingMonitors::_monitors = |
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
39 |
new (ResourceObj::C_HEAP, mtInternal) GrowableArray<JvmtiRawMonitor*>(1, true); |
6975 | 40 |
|
41 |
void JvmtiPendingMonitors::transition_raw_monitors() { |
|
42 |
assert((Threads::number_of_threads()==1), |
|
58509 | 43 |
"Java thread has not been created yet or more than one java thread " |
44 |
"is running. Raw monitor transition will not work"); |
|
45 |
JavaThread* current_java_thread = JavaThread::current(); |
|
6975 | 46 |
assert(current_java_thread->thread_state() == _thread_in_vm, "Must be in vm"); |
58509 | 47 |
for (int i = 0; i < count(); i++) { |
48 |
JvmtiRawMonitor* rmonitor = monitors()->at(i); |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
49 |
rmonitor->raw_enter(current_java_thread); |
6975 | 50 |
} |
51 |
// pending monitors are converted to real monitor so delete them all. |
|
52 |
dispose(); |
|
53 |
} |
|
54 |
||
55 |
// |
|
56 |
// class JvmtiRawMonitor |
|
57 |
// |
|
58 |
||
58509 | 59 |
JvmtiRawMonitor::JvmtiRawMonitor(const char* name) : _owner(NULL), |
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
60 |
_recursions(0), |
58509 | 61 |
_entry_list(NULL), |
62 |
_wait_set(NULL), |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
63 |
_waiters(0), |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
64 |
_magic(JVMTI_RM_MAGIC), |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
65 |
_name(NULL) { |
6975 | 66 |
#ifdef ASSERT |
13195 | 67 |
_name = strcpy(NEW_C_HEAP_ARRAY(char, strlen(name) + 1, mtInternal), name); |
6975 | 68 |
#endif |
69 |
} |
|
70 |
||
71 |
JvmtiRawMonitor::~JvmtiRawMonitor() { |
|
72 |
#ifdef ASSERT |
|
73 |
FreeHeap(_name); |
|
74 |
#endif |
|
75 |
_magic = 0; |
|
76 |
} |
|
77 |
||
78 |
||
79 |
bool |
|
80 |
JvmtiRawMonitor::is_valid() { |
|
81 |
int value = 0; |
|
82 |
||
83 |
// This object might not be a JvmtiRawMonitor so we can't assume |
|
84 |
// the _magic field is properly aligned. Get the value in a safe |
|
85 |
// way and then check against JVMTI_RM_MAGIC. |
|
86 |
||
87 |
switch (sizeof(_magic)) { |
|
88 |
case 2: |
|
89 |
value = Bytes::get_native_u2((address)&_magic); |
|
90 |
break; |
|
91 |
||
92 |
case 4: |
|
93 |
value = Bytes::get_native_u4((address)&_magic); |
|
94 |
break; |
|
95 |
||
96 |
case 8: |
|
97 |
value = Bytes::get_native_u8((address)&_magic); |
|
98 |
break; |
|
99 |
||
100 |
default: |
|
101 |
guarantee(false, "_magic field is an unexpected size"); |
|
102 |
} |
|
103 |
||
104 |
return value == JVMTI_RM_MAGIC; |
|
105 |
} |
|
106 |
||
107 |
// ------------------------------------------------------------------------- |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
108 |
// The JVMTI raw monitor subsystem is entirely distinct from normal |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
109 |
// java-synchronization or jni-synchronization. JVMTI raw monitors are not |
6975 | 110 |
// associated with objects. They can be implemented in any manner |
111 |
// that makes sense. The original implementors decided to piggy-back |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
112 |
// the raw-monitor implementation on the existing Java ObjectMonitor mechanism. |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
113 |
// Now we just use a simplified form of that ObjectMonitor code. |
6975 | 114 |
// |
115 |
// Note that we use the single RawMonitor_lock to protect queue operations for |
|
116 |
// _all_ raw monitors. This is a scalability impediment, but since raw monitor usage |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
117 |
// is fairly rare, this is not of concern. The RawMonitor_lock can not |
6975 | 118 |
// be held indefinitely. The critical sections must be short and bounded. |
119 |
// |
|
120 |
// ------------------------------------------------------------------------- |
|
121 |
||
58509 | 122 |
void JvmtiRawMonitor::simple_enter(Thread* self) { |
6975 | 123 |
for (;;) { |
59252
623722a6aeb9
8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents:
59247
diff
changeset
|
124 |
if (Atomic::replace_if_null(&_owner, self)) { |
58509 | 125 |
return; |
6975 | 126 |
} |
127 |
||
58509 | 128 |
QNode node(self); |
129 |
self->_ParkEvent->reset(); // strictly optional |
|
130 |
node._t_state = QNode::TS_ENTER; |
|
6975 | 131 |
|
58509 | 132 |
RawMonitor_lock->lock_without_safepoint_check(); |
133 |
node._next = _entry_list; |
|
134 |
_entry_list = &node; |
|
135 |
OrderAccess::fence(); |
|
59252
623722a6aeb9
8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents:
59247
diff
changeset
|
136 |
if (_owner == NULL && Atomic::replace_if_null(&_owner, self)) { |
58509 | 137 |
_entry_list = node._next; |
138 |
RawMonitor_lock->unlock(); |
|
139 |
return; |
|
6975 | 140 |
} |
58509 | 141 |
RawMonitor_lock->unlock(); |
142 |
while (node._t_state == QNode::TS_ENTER) { |
|
143 |
self->_ParkEvent->park(); |
|
6975 | 144 |
} |
145 |
} |
|
146 |
} |
|
147 |
||
58509 | 148 |
void JvmtiRawMonitor::simple_exit(Thread* self) { |
149 |
guarantee(_owner == self, "invariant"); |
|
59247
56bf71d64d51
8234562: Move OrderAccess::release_store*/load_acquire to Atomic
stefank
parents:
59105
diff
changeset
|
150 |
Atomic::release_store(&_owner, (Thread*)NULL); |
58509 | 151 |
OrderAccess::fence(); |
152 |
if (_entry_list == NULL) { |
|
153 |
return; |
|
154 |
} |
|
6975 | 155 |
|
58509 | 156 |
RawMonitor_lock->lock_without_safepoint_check(); |
157 |
QNode* w = _entry_list; |
|
6975 | 158 |
if (w != NULL) { |
58509 | 159 |
_entry_list = w->_next; |
6975 | 160 |
} |
58509 | 161 |
RawMonitor_lock->unlock(); |
6975 | 162 |
if (w != NULL) { |
58509 | 163 |
guarantee(w ->_t_state == QNode::TS_ENTER, "invariant"); |
164 |
// Once we set _t_state to TS_RUN the waiting thread can complete |
|
165 |
// simple_enter and 'w' is pointing into random stack space. So we have |
|
166 |
// to ensure we extract the ParkEvent (which is in type-stable memory) |
|
167 |
// before we set the state, and then don't access 'w'. |
|
168 |
ParkEvent* ev = w->_event; |
|
169 |
OrderAccess::loadstore(); |
|
170 |
w->_t_state = QNode::TS_RUN; |
|
171 |
OrderAccess::fence(); |
|
172 |
ev->unpark(); |
|
6975 | 173 |
} |
58509 | 174 |
return; |
6975 | 175 |
} |
176 |
||
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
177 |
inline void JvmtiRawMonitor::enqueue_waiter(QNode& node) { |
58509 | 178 |
node._notified = 0; |
179 |
node._t_state = QNode::TS_WAIT; |
|
180 |
RawMonitor_lock->lock_without_safepoint_check(); |
|
181 |
node._next = _wait_set; |
|
182 |
_wait_set = &node; |
|
183 |
RawMonitor_lock->unlock(); |
|
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
184 |
} |
6975 | 185 |
|
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
186 |
inline void JvmtiRawMonitor::dequeue_waiter(QNode& node) { |
6975 | 187 |
// If thread still resides on the waitset then unlink it. |
188 |
// Double-checked locking -- the usage is safe in this context |
|
58509 | 189 |
// as _t_state is volatile and the lock-unlock operators are |
6975 | 190 |
// serializing (barrier-equivalent). |
191 |
||
58509 | 192 |
if (node._t_state == QNode::TS_WAIT) { |
193 |
RawMonitor_lock->lock_without_safepoint_check(); |
|
194 |
if (node._t_state == QNode::TS_WAIT) { |
|
6975 | 195 |
// Simple O(n) unlink, but performance isn't critical here. |
58509 | 196 |
QNode* p; |
197 |
QNode* q = NULL; |
|
198 |
for (p = _wait_set; p != &node; p = p->_next) { |
|
199 |
q = p; |
|
6975 | 200 |
} |
58509 | 201 |
guarantee(p == &node, "invariant"); |
6975 | 202 |
if (q == NULL) { |
58509 | 203 |
guarantee (p == _wait_set, "invariant"); |
204 |
_wait_set = p->_next; |
|
6975 | 205 |
} else { |
58509 | 206 |
guarantee(p == q->_next, "invariant"); |
207 |
q->_next = p->_next; |
|
6975 | 208 |
} |
58509 | 209 |
node._t_state = QNode::TS_RUN; |
6975 | 210 |
} |
58509 | 211 |
RawMonitor_lock->unlock(); |
6975 | 212 |
} |
213 |
||
58509 | 214 |
guarantee(node._t_state == QNode::TS_RUN, "invariant"); |
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
215 |
} |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
216 |
|
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
217 |
// simple_wait is not quite so simple as we have to deal with the interaction |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
218 |
// with the Thread interrupt state, which resides in the java.lang.Thread object. |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
219 |
// That state must only be accessed while _thread_in_vm and requires proper thread-state |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
220 |
// transitions. However, we cannot perform such transitions whilst we hold the RawMonitor, |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
221 |
// else we can deadlock with the VMThread (which may also use RawMonitors as part of |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
222 |
// executing various callbacks). |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
223 |
// Returns M_OK usually, but M_INTERRUPTED if the thread is a JavaThread and was |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
224 |
// interrupted. |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
225 |
int JvmtiRawMonitor::simple_wait(Thread* self, jlong millis) { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
226 |
guarantee(_owner == self , "invariant"); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
227 |
guarantee(_recursions == 0, "invariant"); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
228 |
|
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
229 |
QNode node(self); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
230 |
enqueue_waiter(node); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
231 |
|
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
232 |
simple_exit(self); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
233 |
guarantee(_owner != self, "invariant"); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
234 |
|
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
235 |
int ret = M_OK; |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
236 |
if (self->is_Java_thread()) { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
237 |
JavaThread* jt = (JavaThread*) self; |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
238 |
// Transition to VM so we can check interrupt state |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
239 |
ThreadInVMfromNative tivm(jt); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
240 |
if (jt->is_interrupted(true)) { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
241 |
ret = M_INTERRUPTED; |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
242 |
} else { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
243 |
ThreadBlockInVM tbivm(jt); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
244 |
jt->set_suspend_equivalent(); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
245 |
if (millis <= 0) { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
246 |
self->_ParkEvent->park(); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
247 |
} else { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
248 |
self->_ParkEvent->park(millis); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
249 |
} |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
250 |
// Return to VM before post-check of interrupt state |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
251 |
} |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
252 |
if (jt->is_interrupted(true)) { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
253 |
ret = M_INTERRUPTED; |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
254 |
} |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
255 |
} else { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
256 |
if (millis <= 0) { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
257 |
self->_ParkEvent->park(); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
258 |
} else { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
259 |
self->_ParkEvent->park(millis); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
260 |
} |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
261 |
} |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
262 |
|
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
263 |
dequeue_waiter(node); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
264 |
|
58509 | 265 |
simple_enter(self); |
266 |
guarantee(_owner == self, "invariant"); |
|
267 |
guarantee(_recursions == 0, "invariant"); |
|
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
268 |
|
58509 | 269 |
return ret; |
6975 | 270 |
} |
271 |
||
58509 | 272 |
void JvmtiRawMonitor::simple_notify(Thread* self, bool all) { |
273 |
guarantee(_owner == self, "invariant"); |
|
274 |
if (_wait_set == NULL) { |
|
275 |
return; |
|
276 |
} |
|
6975 | 277 |
|
278 |
// We have two options: |
|
58509 | 279 |
// A. Transfer the threads from the _wait_set to the _entry_list |
280 |
// B. Remove the thread from the _wait_set and unpark() it. |
|
6975 | 281 |
// |
282 |
// We use (B), which is crude and results in lots of futile |
|
283 |
// context switching. In particular (B) induces lots of contention. |
|
284 |
||
58509 | 285 |
ParkEvent* ev = NULL; // consider using a small auto array ... |
286 |
RawMonitor_lock->lock_without_safepoint_check(); |
|
6975 | 287 |
for (;;) { |
58509 | 288 |
QNode* w = _wait_set; |
289 |
if (w == NULL) break; |
|
290 |
_wait_set = w->_next; |
|
291 |
if (ev != NULL) { |
|
292 |
ev->unpark(); |
|
293 |
ev = NULL; |
|
294 |
} |
|
295 |
ev = w->_event; |
|
296 |
OrderAccess::loadstore(); |
|
297 |
w->_t_state = QNode::TS_RUN; |
|
298 |
OrderAccess::storeload(); |
|
299 |
if (!all) { |
|
300 |
break; |
|
301 |
} |
|
6975 | 302 |
} |
58509 | 303 |
RawMonitor_lock->unlock(); |
304 |
if (ev != NULL) { |
|
305 |
ev->unpark(); |
|
306 |
} |
|
307 |
return; |
|
6975 | 308 |
} |
309 |
||
310 |
// Any JavaThread will enter here with state _thread_blocked |
|
58509 | 311 |
void JvmtiRawMonitor::raw_enter(Thread* self) { |
312 |
void* contended; |
|
313 |
JavaThread* jt = NULL; |
|
6975 | 314 |
// don't enter raw monitor if thread is being externally suspended, it will |
315 |
// surprise the suspender if a "suspended" thread can still enter monitor |
|
58509 | 316 |
if (self->is_Java_thread()) { |
317 |
jt = (JavaThread*)self; |
|
6975 | 318 |
jt->SR_lock()->lock_without_safepoint_check(); |
319 |
while (jt->is_external_suspend()) { |
|
320 |
jt->SR_lock()->unlock(); |
|
321 |
jt->java_suspend_self(); |
|
322 |
jt->SR_lock()->lock_without_safepoint_check(); |
|
323 |
} |
|
324 |
// guarded by SR_lock to avoid racing with new external suspend requests. |
|
59252
623722a6aeb9
8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents:
59247
diff
changeset
|
325 |
contended = Atomic::cmpxchg(&_owner, (Thread*)NULL, jt); |
6975 | 326 |
jt->SR_lock()->unlock(); |
327 |
} else { |
|
59252
623722a6aeb9
8234740: Harmonize parameter order in Atomic - cmpxchg
stefank
parents:
59247
diff
changeset
|
328 |
contended = Atomic::cmpxchg(&_owner, (Thread*)NULL, self); |
6975 | 329 |
} |
330 |
||
58509 | 331 |
if (contended == self) { |
332 |
_recursions++; |
|
333 |
return; |
|
6975 | 334 |
} |
335 |
||
58509 | 336 |
if (contended == NULL) { |
337 |
guarantee(_owner == self, "invariant"); |
|
338 |
guarantee(_recursions == 0, "invariant"); |
|
339 |
return; |
|
6975 | 340 |
} |
341 |
||
58509 | 342 |
self->set_current_pending_raw_monitor(this); |
6975 | 343 |
|
58509 | 344 |
if (!self->is_Java_thread()) { |
345 |
simple_enter(self); |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
346 |
} else { |
58509 | 347 |
guarantee(jt->thread_state() == _thread_blocked, "invariant"); |
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
348 |
for (;;) { |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
349 |
jt->set_suspend_equivalent(); |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
350 |
// cleared by handle_special_suspend_equivalent_condition() or |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
351 |
// java_suspend_self() |
58509 | 352 |
simple_enter(jt); |
6975 | 353 |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
354 |
// were we externally suspended while we were waiting? |
58509 | 355 |
if (!jt->handle_special_suspend_equivalent_condition()) { |
356 |
break; |
|
357 |
} |
|
6975 | 358 |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
359 |
// This thread was externally suspended |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
360 |
// We have reentered the contended monitor, but while we were |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
361 |
// waiting another thread suspended us. We don't want to reenter |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
362 |
// the monitor while suspended because that would surprise the |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
363 |
// thread that suspended us. |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
364 |
// |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
365 |
// Drop the lock |
58509 | 366 |
simple_exit(jt); |
6975 | 367 |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
368 |
jt->java_suspend_self(); |
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
369 |
} |
6975 | 370 |
} |
371 |
||
58509 | 372 |
self->set_current_pending_raw_monitor(NULL); |
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
373 |
|
58509 | 374 |
guarantee(_owner == self, "invariant"); |
375 |
guarantee(_recursions == 0, "invariant"); |
|
6975 | 376 |
} |
377 |
||
58509 | 378 |
int JvmtiRawMonitor::raw_exit(Thread* self) { |
379 |
if (self != _owner) { |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
380 |
return M_ILLEGAL_MONITOR_STATE; |
6975 | 381 |
} |
382 |
if (_recursions > 0) { |
|
58509 | 383 |
_recursions--; |
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
384 |
} else { |
58509 | 385 |
simple_exit(self); |
6975 | 386 |
} |
387 |
||
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
388 |
return M_OK; |
6975 | 389 |
} |
390 |
||
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
391 |
int JvmtiRawMonitor::raw_wait(jlong millis, Thread* self) { |
58509 | 392 |
if (self != _owner) { |
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
393 |
return M_ILLEGAL_MONITOR_STATE; |
6975 | 394 |
} |
395 |
||
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
396 |
int ret = M_OK; |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
397 |
|
58509 | 398 |
// To avoid spurious wakeups we reset the parkevent. This is strictly optional. |
6975 | 399 |
// The caller must be able to tolerate spurious returns from raw_wait(). |
58509 | 400 |
self->_ParkEvent->reset(); |
401 |
OrderAccess::fence(); |
|
6975 | 402 |
|
58509 | 403 |
intptr_t save = _recursions; |
404 |
_recursions = 0; |
|
405 |
_waiters++; |
|
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
406 |
ret = simple_wait(self, millis); |
58509 | 407 |
_recursions = save; |
408 |
_waiters--; |
|
6975 | 409 |
|
58509 | 410 |
guarantee(self == _owner, "invariant"); |
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
411 |
|
58509 | 412 |
if (self->is_Java_thread()) { |
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
413 |
JavaThread* jt = (JavaThread*)self; |
58509 | 414 |
for (;;) { |
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
415 |
jt->set_suspend_equivalent(); |
58509 | 416 |
if (!jt->handle_special_suspend_equivalent_condition()) { |
417 |
break; |
|
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
418 |
} else { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
419 |
// We've been suspended whilst waiting and so we have to |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
420 |
// relinquish the raw monitor until we are resumed. Of course |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
421 |
// after reacquiring we have to re-check for suspension again. |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
422 |
// Suspension requires we are _thread_blocked, and we also have to |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
423 |
// recheck for being interrupted. |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
424 |
simple_exit(jt); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
425 |
{ |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
426 |
ThreadInVMfromNative tivm(jt); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
427 |
{ |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
428 |
ThreadBlockInVM tbivm(jt); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
429 |
jt->java_suspend_self(); |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
430 |
} |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
431 |
if (jt->is_interrupted(true)) { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
432 |
ret = M_INTERRUPTED; |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
433 |
} |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
434 |
} |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
435 |
simple_enter(jt); |
58509 | 436 |
} |
437 |
} |
|
438 |
guarantee(jt == _owner, "invariant"); |
|
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
439 |
} else { |
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
440 |
assert(ret != M_INTERRUPTED, "Only JavaThreads can be interrupted"); |
6975 | 441 |
} |
442 |
||
59105
76ae9aa0e794
8233549: Thread interrupted state must only be accessed when not in a safepoint-safe state
dholmes
parents:
58509
diff
changeset
|
443 |
return ret; |
6975 | 444 |
} |
445 |
||
58509 | 446 |
int JvmtiRawMonitor::raw_notify(Thread* self) { |
447 |
if (self != _owner) { |
|
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
448 |
return M_ILLEGAL_MONITOR_STATE; |
6975 | 449 |
} |
58509 | 450 |
simple_notify(self, false); |
58488
165b193b30dd
8231289: Disentangle JvmtiRawMonitor from ObjectMonitor and clean it up
dholmes
parents:
58196
diff
changeset
|
451 |
return M_OK; |
6975 | 452 |
} |
58509 | 453 |
|
454 |
int JvmtiRawMonitor::raw_notifyAll(Thread* self) { |
|
455 |
if (self != _owner) { |
|
456 |
return M_ILLEGAL_MONITOR_STATE; |
|
457 |
} |
|
458 |
simple_notify(self, true); |
|
459 |
return M_OK; |
|
460 |
} |