author | coleenp |
Wed, 15 Feb 2017 22:59:57 -0500 | |
changeset 46271 | 979ebd346ecf |
parent 40655 | 9f644073d3a0 |
child 46329 | 53ccc37bda19 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
46271
979ebd346ecf
8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents:
40655
diff
changeset
|
2 |
* Copyright (c) 2003, 2017, 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:
2137
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
2137
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:
2137
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "classfile/systemDictionary.hpp" |
|
27 |
#include "interpreter/interpreter.hpp" |
|
25626
362d0c3a7651
8013942: JSR 292: assert(type() == T_OBJECT) failed: type check
sspitsyn
parents:
25351
diff
changeset
|
28 |
#include "interpreter/oopMapCache.hpp" |
7397 | 29 |
#include "jvmtifiles/jvmtiEnv.hpp" |
37992 | 30 |
#include "logging/log.hpp" |
31 |
#include "logging/logStream.hpp" |
|
7397 | 32 |
#include "memory/resourceArea.hpp" |
33 |
#include "oops/instanceKlass.hpp" |
|
29081
c61eb4914428
8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents:
27880
diff
changeset
|
34 |
#include "oops/oop.inline.hpp" |
7397 | 35 |
#include "prims/jvmtiAgentThread.hpp" |
36 |
#include "prims/jvmtiEventController.inline.hpp" |
|
37 |
#include "prims/jvmtiImpl.hpp" |
|
38 |
#include "prims/jvmtiRedefineClasses.hpp" |
|
40655
9f644073d3a0
8157907: Incorrect inclusion of atomic.hpp instead of atomic.inline.hpp
dholmes
parents:
38259
diff
changeset
|
39 |
#include "runtime/atomic.hpp" |
7397 | 40 |
#include "runtime/deoptimization.hpp" |
41 |
#include "runtime/handles.hpp" |
|
42 |
#include "runtime/handles.inline.hpp" |
|
43 |
#include "runtime/interfaceSupport.hpp" |
|
44 |
#include "runtime/javaCalls.hpp" |
|
9405
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
45 |
#include "runtime/os.hpp" |
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
46 |
#include "runtime/serviceThread.hpp" |
7397 | 47 |
#include "runtime/signature.hpp" |
14583
d70ee55535f4
8003935: Simplify the needed includes for using Thread::current()
stefank
parents:
14488
diff
changeset
|
48 |
#include "runtime/thread.inline.hpp" |
7397 | 49 |
#include "runtime/vframe.hpp" |
50 |
#include "runtime/vframe_hp.hpp" |
|
51 |
#include "runtime/vm_operations.hpp" |
|
52 |
#include "utilities/exceptions.hpp" |
|
1 | 53 |
|
54 |
// |
|
55 |
// class JvmtiAgentThread |
|
56 |
// |
|
57 |
// JavaThread used to wrap a thread started by an agent |
|
58 |
// using the JVMTI method RunAgentThread. |
|
59 |
// |
|
60 |
||
61 |
JvmtiAgentThread::JvmtiAgentThread(JvmtiEnv* env, jvmtiStartFunction start_fn, const void *start_arg) |
|
62 |
: JavaThread(start_function_wrapper) { |
|
63 |
_env = env; |
|
64 |
_start_fn = start_fn; |
|
65 |
_start_arg = start_arg; |
|
66 |
} |
|
67 |
||
68 |
void |
|
69 |
JvmtiAgentThread::start_function_wrapper(JavaThread *thread, TRAPS) { |
|
70 |
// It is expected that any Agent threads will be created as |
|
71 |
// Java Threads. If this is the case, notification of the creation |
|
72 |
// of the thread is given in JavaThread::thread_main(). |
|
73 |
assert(thread->is_Java_thread(), "debugger thread should be a Java Thread"); |
|
74 |
assert(thread == JavaThread::current(), "sanity check"); |
|
75 |
||
76 |
JvmtiAgentThread *dthread = (JvmtiAgentThread *)thread; |
|
77 |
dthread->call_start_function(); |
|
78 |
} |
|
79 |
||
80 |
void |
|
81 |
JvmtiAgentThread::call_start_function() { |
|
82 |
ThreadToNativeFromVM transition(this); |
|
83 |
_start_fn(_env->jvmti_external(), jni_environment(), (void*)_start_arg); |
|
84 |
} |
|
85 |
||
86 |
||
87 |
// |
|
88 |
// class GrowableCache - private methods |
|
89 |
// |
|
90 |
||
91 |
void GrowableCache::recache() { |
|
92 |
int len = _elements->length(); |
|
93 |
||
27880
afb974a04396
8060074: os::free() takes MemoryTrackingLevel but doesn't need it
coleenp
parents:
26558
diff
changeset
|
94 |
FREE_C_HEAP_ARRAY(address, _cache); |
13195 | 95 |
_cache = NEW_C_HEAP_ARRAY(address,len+1, mtInternal); |
1 | 96 |
|
97 |
for (int i=0; i<len; i++) { |
|
98 |
_cache[i] = _elements->at(i)->getCacheValue(); |
|
99 |
// |
|
100 |
// The cache entry has gone bad. Without a valid frame pointer |
|
101 |
// value, the entry is useless so we simply delete it in product |
|
102 |
// mode. The call to remove() will rebuild the cache again |
|
103 |
// without the bad entry. |
|
104 |
// |
|
105 |
if (_cache[i] == NULL) { |
|
106 |
assert(false, "cannot recache NULL elements"); |
|
107 |
remove(i); |
|
108 |
return; |
|
109 |
} |
|
110 |
} |
|
111 |
_cache[len] = NULL; |
|
112 |
||
113 |
_listener_fun(_this_obj,_cache); |
|
114 |
} |
|
115 |
||
116 |
bool GrowableCache::equals(void* v, GrowableElement *e2) { |
|
117 |
GrowableElement *e1 = (GrowableElement *) v; |
|
118 |
assert(e1 != NULL, "e1 != NULL"); |
|
119 |
assert(e2 != NULL, "e2 != NULL"); |
|
120 |
||
121 |
return e1->equals(e2); |
|
122 |
} |
|
123 |
||
124 |
// |
|
125 |
// class GrowableCache - public methods |
|
126 |
// |
|
127 |
||
128 |
GrowableCache::GrowableCache() { |
|
129 |
_this_obj = NULL; |
|
130 |
_listener_fun = NULL; |
|
131 |
_elements = NULL; |
|
132 |
_cache = NULL; |
|
133 |
} |
|
134 |
||
135 |
GrowableCache::~GrowableCache() { |
|
136 |
clear(); |
|
137 |
delete _elements; |
|
27880
afb974a04396
8060074: os::free() takes MemoryTrackingLevel but doesn't need it
coleenp
parents:
26558
diff
changeset
|
138 |
FREE_C_HEAP_ARRAY(address, _cache); |
1 | 139 |
} |
140 |
||
141 |
void GrowableCache::initialize(void *this_obj, void listener_fun(void *, address*) ) { |
|
142 |
_this_obj = this_obj; |
|
143 |
_listener_fun = listener_fun; |
|
13195 | 144 |
_elements = new (ResourceObj::C_HEAP, mtInternal) GrowableArray<GrowableElement*>(5,true); |
1 | 145 |
recache(); |
146 |
} |
|
147 |
||
148 |
// number of elements in the collection |
|
149 |
int GrowableCache::length() { |
|
150 |
return _elements->length(); |
|
151 |
} |
|
152 |
||
153 |
// get the value of the index element in the collection |
|
154 |
GrowableElement* GrowableCache::at(int index) { |
|
155 |
GrowableElement *e = (GrowableElement *) _elements->at(index); |
|
156 |
assert(e != NULL, "e != NULL"); |
|
157 |
return e; |
|
158 |
} |
|
159 |
||
160 |
int GrowableCache::find(GrowableElement* e) { |
|
161 |
return _elements->find(e, GrowableCache::equals); |
|
162 |
} |
|
163 |
||
164 |
// append a copy of the element to the end of the collection |
|
165 |
void GrowableCache::append(GrowableElement* e) { |
|
166 |
GrowableElement *new_e = e->clone(); |
|
167 |
_elements->append(new_e); |
|
168 |
recache(); |
|
169 |
} |
|
170 |
||
171 |
// insert a copy of the element using lessthan() |
|
172 |
void GrowableCache::insert(GrowableElement* e) { |
|
173 |
GrowableElement *new_e = e->clone(); |
|
174 |
_elements->append(new_e); |
|
175 |
||
176 |
int n = length()-2; |
|
177 |
for (int i=n; i>=0; i--) { |
|
178 |
GrowableElement *e1 = _elements->at(i); |
|
179 |
GrowableElement *e2 = _elements->at(i+1); |
|
180 |
if (e2->lessThan(e1)) { |
|
181 |
_elements->at_put(i+1, e1); |
|
182 |
_elements->at_put(i, e2); |
|
183 |
} |
|
184 |
} |
|
185 |
||
186 |
recache(); |
|
187 |
} |
|
188 |
||
189 |
// remove the element at index |
|
190 |
void GrowableCache::remove (int index) { |
|
191 |
GrowableElement *e = _elements->at(index); |
|
192 |
assert(e != NULL, "e != NULL"); |
|
193 |
_elements->remove(e); |
|
194 |
delete e; |
|
195 |
recache(); |
|
196 |
} |
|
197 |
||
198 |
// clear out all elements, release all heap space and |
|
199 |
// let our listener know that things have changed. |
|
200 |
void GrowableCache::clear() { |
|
201 |
int len = _elements->length(); |
|
202 |
for (int i=0; i<len; i++) { |
|
203 |
delete _elements->at(i); |
|
204 |
} |
|
205 |
_elements->clear(); |
|
206 |
recache(); |
|
207 |
} |
|
208 |
||
209 |
void GrowableCache::oops_do(OopClosure* f) { |
|
210 |
int len = _elements->length(); |
|
211 |
for (int i=0; i<len; i++) { |
|
212 |
GrowableElement *e = _elements->at(i); |
|
213 |
e->oops_do(f); |
|
7918
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
214 |
} |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
215 |
} |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
216 |
|
21519
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
217 |
void GrowableCache::metadata_do(void f(Metadata*)) { |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
218 |
int len = _elements->length(); |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
219 |
for (int i=0; i<len; i++) { |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
220 |
GrowableElement *e = _elements->at(i); |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
221 |
e->metadata_do(f); |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
222 |
} |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
223 |
} |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
224 |
|
7918
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
225 |
void GrowableCache::gc_epilogue() { |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
226 |
int len = _elements->length(); |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
227 |
for (int i=0; i<len; i++) { |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
228 |
_cache[i] = _elements->at(i)->getCacheValue(); |
1 | 229 |
} |
230 |
} |
|
231 |
||
232 |
// |
|
233 |
// class JvmtiBreakpoint |
|
234 |
// |
|
235 |
||
236 |
JvmtiBreakpoint::JvmtiBreakpoint() { |
|
237 |
_method = NULL; |
|
238 |
_bci = 0; |
|
21519
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
239 |
_class_holder = NULL; |
1 | 240 |
} |
241 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
242 |
JvmtiBreakpoint::JvmtiBreakpoint(Method* m_method, jlocation location) { |
1 | 243 |
_method = m_method; |
21519
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
244 |
_class_holder = _method->method_holder()->klass_holder(); |
21116
6e8fb0940359
8025834: NPE in Parallel Scavenge with -XX:+CheckUnhandledOops
ehelin
parents:
20059
diff
changeset
|
245 |
#ifdef CHECK_UNHANDLED_OOPS |
21519
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
246 |
// _class_holder can't be wrapped in a Handle, because JvmtiBreakpoints are |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
247 |
// sometimes allocated on the heap. |
21116
6e8fb0940359
8025834: NPE in Parallel Scavenge with -XX:+CheckUnhandledOops
ehelin
parents:
20059
diff
changeset
|
248 |
// |
21519
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
249 |
// The code handling JvmtiBreakpoints allocated on the stack can't be |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
250 |
// interrupted by a GC until _class_holder is reachable by the GC via the |
21116
6e8fb0940359
8025834: NPE in Parallel Scavenge with -XX:+CheckUnhandledOops
ehelin
parents:
20059
diff
changeset
|
251 |
// oops_do method. |
21519
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
252 |
Thread::current()->allow_unhandled_oop(&_class_holder); |
21116
6e8fb0940359
8025834: NPE in Parallel Scavenge with -XX:+CheckUnhandledOops
ehelin
parents:
20059
diff
changeset
|
253 |
#endif // CHECK_UNHANDLED_OOPS |
1 | 254 |
assert(_method != NULL, "_method != NULL"); |
255 |
_bci = (int) location; |
|
256 |
assert(_bci >= 0, "_bci >= 0"); |
|
257 |
} |
|
258 |
||
259 |
void JvmtiBreakpoint::copy(JvmtiBreakpoint& bp) { |
|
260 |
_method = bp._method; |
|
261 |
_bci = bp._bci; |
|
21519
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
262 |
_class_holder = bp._class_holder; |
1 | 263 |
} |
264 |
||
265 |
bool JvmtiBreakpoint::lessThan(JvmtiBreakpoint& bp) { |
|
266 |
Unimplemented(); |
|
267 |
return false; |
|
268 |
} |
|
269 |
||
270 |
bool JvmtiBreakpoint::equals(JvmtiBreakpoint& bp) { |
|
271 |
return _method == bp._method |
|
272 |
&& _bci == bp._bci; |
|
273 |
} |
|
274 |
||
275 |
bool JvmtiBreakpoint::is_valid() { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
276 |
// class loader can be NULL |
1 | 277 |
return _method != NULL && |
278 |
_bci >= 0; |
|
279 |
} |
|
280 |
||
38091
d7e51f40ba2d
8154719: JvmtiBreakpoint rename method print() to print_on()
rehn
parents:
37992
diff
changeset
|
281 |
address JvmtiBreakpoint::getBcp() const { |
1 | 282 |
return _method->bcp_from(_bci); |
283 |
} |
|
284 |
||
285 |
void JvmtiBreakpoint::each_method_version_do(method_action meth_act) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
286 |
((Method*)_method->*meth_act)(_bci); |
1 | 287 |
|
26558
b7df27df6384
8055008: Clean up code that saves the previous versions of redefined classes
coleenp
parents:
25626
diff
changeset
|
288 |
// add/remove breakpoint to/from versions of the method that are EMCP. |
1 | 289 |
Thread *thread = Thread::current(); |
290 |
instanceKlassHandle ikh = instanceKlassHandle(thread, _method->method_holder()); |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7918
diff
changeset
|
291 |
Symbol* m_name = _method->name(); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7918
diff
changeset
|
292 |
Symbol* m_signature = _method->signature(); |
1 | 293 |
|
20059
c26474fd5ac0
8022887: Assertion hit while using class and redefining it with RedefineClasses simultaneously
coleenp
parents:
18025
diff
changeset
|
294 |
// search previous versions if they exist |
26558
b7df27df6384
8055008: Clean up code that saves the previous versions of redefined classes
coleenp
parents:
25626
diff
changeset
|
295 |
for (InstanceKlass* pv_node = ikh->previous_versions(); |
b7df27df6384
8055008: Clean up code that saves the previous versions of redefined classes
coleenp
parents:
25626
diff
changeset
|
296 |
pv_node != NULL; |
b7df27df6384
8055008: Clean up code that saves the previous versions of redefined classes
coleenp
parents:
25626
diff
changeset
|
297 |
pv_node = pv_node->previous_versions()) { |
b7df27df6384
8055008: Clean up code that saves the previous versions of redefined classes
coleenp
parents:
25626
diff
changeset
|
298 |
Array<Method*>* methods = pv_node->methods(); |
1 | 299 |
|
20059
c26474fd5ac0
8022887: Assertion hit while using class and redefining it with RedefineClasses simultaneously
coleenp
parents:
18025
diff
changeset
|
300 |
for (int i = methods->length() - 1; i >= 0; i--) { |
c26474fd5ac0
8022887: Assertion hit while using class and redefining it with RedefineClasses simultaneously
coleenp
parents:
18025
diff
changeset
|
301 |
Method* method = methods->at(i); |
26558
b7df27df6384
8055008: Clean up code that saves the previous versions of redefined classes
coleenp
parents:
25626
diff
changeset
|
302 |
// Only set breakpoints in running EMCP methods. |
b7df27df6384
8055008: Clean up code that saves the previous versions of redefined classes
coleenp
parents:
25626
diff
changeset
|
303 |
if (method->is_running_emcp() && |
20059
c26474fd5ac0
8022887: Assertion hit while using class and redefining it with RedefineClasses simultaneously
coleenp
parents:
18025
diff
changeset
|
304 |
method->name() == m_name && |
c26474fd5ac0
8022887: Assertion hit while using class and redefining it with RedefineClasses simultaneously
coleenp
parents:
18025
diff
changeset
|
305 |
method->signature() == m_signature) { |
38259
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
38091
diff
changeset
|
306 |
ResourceMark rm; |
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
38091
diff
changeset
|
307 |
log_debug(redefine, class, breakpoint) |
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
38091
diff
changeset
|
308 |
("%sing breakpoint in %s(%s)", meth_act == &Method::set_breakpoint ? "sett" : "clear", |
b495d1cfe673
8153535: Convert TraceRedefineClasses to Unified Logging
rehn
parents:
38091
diff
changeset
|
309 |
method->name()->as_C_string(), method->signature()->as_C_string()); |
20059
c26474fd5ac0
8022887: Assertion hit while using class and redefining it with RedefineClasses simultaneously
coleenp
parents:
18025
diff
changeset
|
310 |
(method->*meth_act)(_bci); |
c26474fd5ac0
8022887: Assertion hit while using class and redefining it with RedefineClasses simultaneously
coleenp
parents:
18025
diff
changeset
|
311 |
break; |
1 | 312 |
} |
20059
c26474fd5ac0
8022887: Assertion hit while using class and redefining it with RedefineClasses simultaneously
coleenp
parents:
18025
diff
changeset
|
313 |
} |
c26474fd5ac0
8022887: Assertion hit while using class and redefining it with RedefineClasses simultaneously
coleenp
parents:
18025
diff
changeset
|
314 |
} |
1 | 315 |
} |
316 |
||
317 |
void JvmtiBreakpoint::set() { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
318 |
each_method_version_do(&Method::set_breakpoint); |
1 | 319 |
} |
320 |
||
321 |
void JvmtiBreakpoint::clear() { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
322 |
each_method_version_do(&Method::clear_breakpoint); |
1 | 323 |
} |
324 |
||
38091
d7e51f40ba2d
8154719: JvmtiBreakpoint rename method print() to print_on()
rehn
parents:
37992
diff
changeset
|
325 |
void JvmtiBreakpoint::print_on(outputStream* out) const { |
1 | 326 |
#ifndef PRODUCT |
37992 | 327 |
ResourceMark rm; |
1 | 328 |
const char *class_name = (_method == NULL) ? "NULL" : _method->klass_name()->as_C_string(); |
329 |
const char *method_name = (_method == NULL) ? "NULL" : _method->name()->as_C_string(); |
|
37992 | 330 |
out->print("Breakpoint(%s,%s,%d,%p)", class_name, method_name, _bci, getBcp()); |
1 | 331 |
#endif |
332 |
} |
|
333 |
||
334 |
||
335 |
// |
|
336 |
// class VM_ChangeBreakpoints |
|
337 |
// |
|
338 |
// Modify the Breakpoints data structure at a safepoint |
|
339 |
// |
|
340 |
||
341 |
void VM_ChangeBreakpoints::doit() { |
|
342 |
switch (_operation) { |
|
343 |
case SET_BREAKPOINT: |
|
344 |
_breakpoints->set_at_safepoint(*_bp); |
|
345 |
break; |
|
346 |
case CLEAR_BREAKPOINT: |
|
347 |
_breakpoints->clear_at_safepoint(*_bp); |
|
348 |
break; |
|
349 |
default: |
|
350 |
assert(false, "Unknown operation"); |
|
351 |
} |
|
352 |
} |
|
353 |
||
354 |
void VM_ChangeBreakpoints::oops_do(OopClosure* f) { |
|
18025 | 355 |
// The JvmtiBreakpoints in _breakpoints will be visited via |
356 |
// JvmtiExport::oops_do. |
|
1 | 357 |
if (_bp != NULL) { |
358 |
_bp->oops_do(f); |
|
359 |
} |
|
360 |
} |
|
361 |
||
21519
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
362 |
void VM_ChangeBreakpoints::metadata_do(void f(Metadata*)) { |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
363 |
// Walk metadata in breakpoints to keep from being deallocated with RedefineClasses |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
364 |
if (_bp != NULL) { |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
365 |
_bp->metadata_do(f); |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
366 |
} |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
367 |
} |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
368 |
|
1 | 369 |
// |
370 |
// class JvmtiBreakpoints |
|
371 |
// |
|
372 |
// a JVMTI internal collection of JvmtiBreakpoint |
|
373 |
// |
|
374 |
||
375 |
JvmtiBreakpoints::JvmtiBreakpoints(void listener_fun(void *,address *)) { |
|
376 |
_bps.initialize(this,listener_fun); |
|
377 |
} |
|
378 |
||
379 |
JvmtiBreakpoints:: ~JvmtiBreakpoints() {} |
|
380 |
||
381 |
void JvmtiBreakpoints::oops_do(OopClosure* f) { |
|
382 |
_bps.oops_do(f); |
|
383 |
} |
|
384 |
||
21519
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
385 |
void JvmtiBreakpoints::metadata_do(void f(Metadata*)) { |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
386 |
_bps.metadata_do(f); |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
387 |
} |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
388 |
|
7918
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
389 |
void JvmtiBreakpoints::gc_epilogue() { |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
390 |
_bps.gc_epilogue(); |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
391 |
} |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
392 |
|
37992 | 393 |
void JvmtiBreakpoints::print() { |
1 | 394 |
#ifndef PRODUCT |
37992 | 395 |
LogTarget(Trace, jvmti) log; |
396 |
LogStreamCHeap log_stream(log); |
|
1 | 397 |
|
398 |
int n = _bps.length(); |
|
399 |
for (int i=0; i<n; i++) { |
|
400 |
JvmtiBreakpoint& bp = _bps.at(i); |
|
37992 | 401 |
log_stream.print("%d: ", i); |
38091
d7e51f40ba2d
8154719: JvmtiBreakpoint rename method print() to print_on()
rehn
parents:
37992
diff
changeset
|
402 |
bp.print_on(&log_stream); |
37992 | 403 |
log_stream.cr(); |
1 | 404 |
} |
405 |
#endif |
|
406 |
} |
|
407 |
||
408 |
||
409 |
void JvmtiBreakpoints::set_at_safepoint(JvmtiBreakpoint& bp) { |
|
410 |
assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint"); |
|
411 |
||
412 |
int i = _bps.find(bp); |
|
413 |
if (i == -1) { |
|
414 |
_bps.append(bp); |
|
415 |
bp.set(); |
|
416 |
} |
|
417 |
} |
|
418 |
||
419 |
void JvmtiBreakpoints::clear_at_safepoint(JvmtiBreakpoint& bp) { |
|
420 |
assert(SafepointSynchronize::is_at_safepoint(), "must be at safepoint"); |
|
421 |
||
422 |
int i = _bps.find(bp); |
|
423 |
if (i != -1) { |
|
424 |
_bps.remove(i); |
|
425 |
bp.clear(); |
|
426 |
} |
|
427 |
} |
|
428 |
||
429 |
int JvmtiBreakpoints::length() { return _bps.length(); } |
|
430 |
||
431 |
int JvmtiBreakpoints::set(JvmtiBreakpoint& bp) { |
|
432 |
if ( _bps.find(bp) != -1) { |
|
433 |
return JVMTI_ERROR_DUPLICATE; |
|
434 |
} |
|
18025 | 435 |
VM_ChangeBreakpoints set_breakpoint(VM_ChangeBreakpoints::SET_BREAKPOINT, &bp); |
1 | 436 |
VMThread::execute(&set_breakpoint); |
437 |
return JVMTI_ERROR_NONE; |
|
438 |
} |
|
439 |
||
440 |
int JvmtiBreakpoints::clear(JvmtiBreakpoint& bp) { |
|
441 |
if ( _bps.find(bp) == -1) { |
|
442 |
return JVMTI_ERROR_NOT_FOUND; |
|
443 |
} |
|
444 |
||
18025 | 445 |
VM_ChangeBreakpoints clear_breakpoint(VM_ChangeBreakpoints::CLEAR_BREAKPOINT, &bp); |
1 | 446 |
VMThread::execute(&clear_breakpoint); |
447 |
return JVMTI_ERROR_NONE; |
|
448 |
} |
|
449 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
450 |
void JvmtiBreakpoints::clearall_in_class_at_safepoint(Klass* klass) { |
1 | 451 |
bool changed = true; |
452 |
// We are going to run thru the list of bkpts |
|
453 |
// and delete some. This deletion probably alters |
|
454 |
// the list in some implementation defined way such |
|
455 |
// that when we delete entry i, the next entry might |
|
456 |
// no longer be at i+1. To be safe, each time we delete |
|
457 |
// an entry, we'll just start again from the beginning. |
|
458 |
// We'll stop when we make a pass thru the whole list without |
|
459 |
// deleting anything. |
|
460 |
while (changed) { |
|
461 |
int len = _bps.length(); |
|
462 |
changed = false; |
|
463 |
for (int i = 0; i < len; i++) { |
|
464 |
JvmtiBreakpoint& bp = _bps.at(i); |
|
465 |
if (bp.method()->method_holder() == klass) { |
|
466 |
bp.clear(); |
|
467 |
_bps.remove(i); |
|
468 |
// This changed 'i' so we have to start over. |
|
469 |
changed = true; |
|
470 |
break; |
|
471 |
} |
|
472 |
} |
|
473 |
} |
|
474 |
} |
|
475 |
||
476 |
// |
|
477 |
// class JvmtiCurrentBreakpoints |
|
478 |
// |
|
479 |
||
480 |
JvmtiBreakpoints *JvmtiCurrentBreakpoints::_jvmti_breakpoints = NULL; |
|
481 |
address * JvmtiCurrentBreakpoints::_breakpoint_list = NULL; |
|
482 |
||
483 |
||
484 |
JvmtiBreakpoints& JvmtiCurrentBreakpoints::get_jvmti_breakpoints() { |
|
485 |
if (_jvmti_breakpoints != NULL) return (*_jvmti_breakpoints); |
|
486 |
_jvmti_breakpoints = new JvmtiBreakpoints(listener_fun); |
|
487 |
assert(_jvmti_breakpoints != NULL, "_jvmti_breakpoints != NULL"); |
|
488 |
return (*_jvmti_breakpoints); |
|
489 |
} |
|
490 |
||
491 |
void JvmtiCurrentBreakpoints::listener_fun(void *this_obj, address *cache) { |
|
492 |
JvmtiBreakpoints *this_jvmti = (JvmtiBreakpoints *) this_obj; |
|
493 |
assert(this_jvmti != NULL, "this_jvmti != NULL"); |
|
494 |
||
495 |
debug_only(int n = this_jvmti->length();); |
|
496 |
assert(cache[n] == NULL, "cache must be NULL terminated"); |
|
497 |
||
498 |
set_breakpoint_list(cache); |
|
499 |
} |
|
500 |
||
501 |
||
502 |
void JvmtiCurrentBreakpoints::oops_do(OopClosure* f) { |
|
503 |
if (_jvmti_breakpoints != NULL) { |
|
504 |
_jvmti_breakpoints->oops_do(f); |
|
505 |
} |
|
506 |
} |
|
507 |
||
21519
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
508 |
void JvmtiCurrentBreakpoints::metadata_do(void f(Metadata*)) { |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
509 |
if (_jvmti_breakpoints != NULL) { |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
510 |
_jvmti_breakpoints->metadata_do(f); |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
511 |
} |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
512 |
} |
72e63a0dbda2
8026946: JvmtiEnv::SetBreakpoint and JvmtiEnv::ClearBreakpoint should use MethodHandle
coleenp
parents:
21116
diff
changeset
|
513 |
|
7918
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
514 |
void JvmtiCurrentBreakpoints::gc_epilogue() { |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
515 |
if (_jvmti_breakpoints != NULL) { |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
516 |
_jvmti_breakpoints->gc_epilogue(); |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
517 |
} |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
518 |
} |
ce1e4ae77aea
7012505: BreakpointWithFullGC.sh fails with Internal Error (src/share/vm/oops/methodOop.cpp:220)
kamg
parents:
7896
diff
changeset
|
519 |
|
1 | 520 |
/////////////////////////////////////////////////////////////// |
521 |
// |
|
522 |
// class VM_GetOrSetLocal |
|
523 |
// |
|
524 |
||
525 |
// Constructor for non-object getter |
|
526 |
VM_GetOrSetLocal::VM_GetOrSetLocal(JavaThread* thread, jint depth, int index, BasicType type) |
|
527 |
: _thread(thread) |
|
528 |
, _calling_thread(NULL) |
|
529 |
, _depth(depth) |
|
530 |
, _index(index) |
|
531 |
, _type(type) |
|
532 |
, _set(false) |
|
533 |
, _jvf(NULL) |
|
534 |
, _result(JVMTI_ERROR_NONE) |
|
535 |
{ |
|
536 |
} |
|
537 |
||
538 |
// Constructor for object or non-object setter |
|
539 |
VM_GetOrSetLocal::VM_GetOrSetLocal(JavaThread* thread, jint depth, int index, BasicType type, jvalue value) |
|
540 |
: _thread(thread) |
|
541 |
, _calling_thread(NULL) |
|
542 |
, _depth(depth) |
|
543 |
, _index(index) |
|
544 |
, _type(type) |
|
545 |
, _value(value) |
|
546 |
, _set(true) |
|
547 |
, _jvf(NULL) |
|
548 |
, _result(JVMTI_ERROR_NONE) |
|
549 |
{ |
|
550 |
} |
|
551 |
||
552 |
// Constructor for object getter |
|
553 |
VM_GetOrSetLocal::VM_GetOrSetLocal(JavaThread* thread, JavaThread* calling_thread, jint depth, int index) |
|
554 |
: _thread(thread) |
|
555 |
, _calling_thread(calling_thread) |
|
556 |
, _depth(depth) |
|
557 |
, _index(index) |
|
558 |
, _type(T_OBJECT) |
|
559 |
, _set(false) |
|
560 |
, _jvf(NULL) |
|
561 |
, _result(JVMTI_ERROR_NONE) |
|
562 |
{ |
|
563 |
} |
|
564 |
||
565 |
vframe *VM_GetOrSetLocal::get_vframe() { |
|
566 |
if (!_thread->has_last_Java_frame()) { |
|
567 |
return NULL; |
|
568 |
} |
|
569 |
RegisterMap reg_map(_thread); |
|
570 |
vframe *vf = _thread->last_java_vframe(®_map); |
|
571 |
int d = 0; |
|
572 |
while ((vf != NULL) && (d < _depth)) { |
|
573 |
vf = vf->java_sender(); |
|
574 |
d++; |
|
575 |
} |
|
576 |
return vf; |
|
577 |
} |
|
578 |
||
579 |
javaVFrame *VM_GetOrSetLocal::get_java_vframe() { |
|
580 |
vframe* vf = get_vframe(); |
|
581 |
if (vf == NULL) { |
|
582 |
_result = JVMTI_ERROR_NO_MORE_FRAMES; |
|
583 |
return NULL; |
|
584 |
} |
|
585 |
javaVFrame *jvf = (javaVFrame*)vf; |
|
586 |
||
7444 | 587 |
if (!vf->is_java_frame()) { |
1 | 588 |
_result = JVMTI_ERROR_OPAQUE_FRAME; |
589 |
return NULL; |
|
590 |
} |
|
591 |
return jvf; |
|
592 |
} |
|
593 |
||
594 |
// Check that the klass is assignable to a type with the given signature. |
|
595 |
// Another solution could be to use the function Klass::is_subtype_of(type). |
|
596 |
// But the type class can be forced to load/initialize eagerly in such a case. |
|
597 |
// This may cause unexpected consequences like CFLH or class-init JVMTI events. |
|
598 |
// It is better to avoid such a behavior. |
|
599 |
bool VM_GetOrSetLocal::is_assignable(const char* ty_sign, Klass* klass, Thread* thread) { |
|
600 |
assert(ty_sign != NULL, "type signature must not be NULL"); |
|
601 |
assert(thread != NULL, "thread must not be NULL"); |
|
602 |
assert(klass != NULL, "klass must not be NULL"); |
|
603 |
||
604 |
int len = (int) strlen(ty_sign); |
|
605 |
if (ty_sign[0] == 'L' && ty_sign[len-1] == ';') { // Need pure class/interface name |
|
606 |
ty_sign++; |
|
607 |
len -= 2; |
|
608 |
} |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7918
diff
changeset
|
609 |
TempNewSymbol ty_sym = SymbolTable::new_symbol(ty_sign, len, thread); |
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7918
diff
changeset
|
610 |
if (klass->name() == ty_sym) { |
1 | 611 |
return true; |
612 |
} |
|
613 |
// Compare primary supers |
|
614 |
int super_depth = klass->super_depth(); |
|
615 |
int idx; |
|
616 |
for (idx = 0; idx < super_depth; idx++) { |
|
14488 | 617 |
if (klass->primary_super_of_depth(idx)->name() == ty_sym) { |
1 | 618 |
return true; |
619 |
} |
|
620 |
} |
|
621 |
// Compare secondary supers |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
622 |
Array<Klass*>* sec_supers = klass->secondary_supers(); |
1 | 623 |
for (idx = 0; idx < sec_supers->length(); idx++) { |
14488 | 624 |
if (((Klass*) sec_supers->at(idx))->name() == ty_sym) { |
1 | 625 |
return true; |
626 |
} |
|
627 |
} |
|
628 |
return false; |
|
629 |
} |
|
630 |
||
631 |
// Checks error conditions: |
|
632 |
// JVMTI_ERROR_INVALID_SLOT |
|
633 |
// JVMTI_ERROR_TYPE_MISMATCH |
|
634 |
// Returns: 'true' - everything is Ok, 'false' - error code |
|
635 |
||
636 |
bool VM_GetOrSetLocal::check_slot_type(javaVFrame* jvf) { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13195
diff
changeset
|
637 |
Method* method_oop = jvf->method(); |
1 | 638 |
if (!method_oop->has_localvariable_table()) { |
639 |
// Just to check index boundaries |
|
640 |
jint extra_slot = (_type == T_LONG || _type == T_DOUBLE) ? 1 : 0; |
|
641 |
if (_index < 0 || _index + extra_slot >= method_oop->max_locals()) { |
|
642 |
_result = JVMTI_ERROR_INVALID_SLOT; |
|
643 |
return false; |
|
644 |
} |
|
645 |
return true; |
|
646 |
} |
|
647 |
||
648 |
jint num_entries = method_oop->localvariable_table_length(); |
|
649 |
if (num_entries == 0) { |
|
650 |
_result = JVMTI_ERROR_INVALID_SLOT; |
|
651 |
return false; // There are no slots |
|
652 |
} |
|
653 |
int signature_idx = -1; |
|
654 |
int vf_bci = jvf->bci(); |
|
655 |
LocalVariableTableElement* table = method_oop->localvariable_table_start(); |
|
656 |
for (int i = 0; i < num_entries; i++) { |
|
657 |
int start_bci = table[i].start_bci; |
|
658 |
int end_bci = start_bci + table[i].length; |
|
659 |
||
660 |
// Here we assume that locations of LVT entries |
|
661 |
// with the same slot number cannot be overlapped |
|
662 |
if (_index == (jint) table[i].slot && start_bci <= vf_bci && vf_bci <= end_bci) { |
|
663 |
signature_idx = (int) table[i].descriptor_cp_index; |
|
664 |
break; |
|
665 |
} |
|
666 |
} |
|
667 |
if (signature_idx == -1) { |
|
668 |
_result = JVMTI_ERROR_INVALID_SLOT; |
|
669 |
return false; // Incorrect slot index |
|
670 |
} |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7918
diff
changeset
|
671 |
Symbol* sign_sym = method_oop->constants()->symbol_at(signature_idx); |
1 | 672 |
const char* signature = (const char *) sign_sym->as_utf8(); |
673 |
BasicType slot_type = char2type(signature[0]); |
|
674 |
||
675 |
switch (slot_type) { |
|
676 |
case T_BYTE: |
|
677 |
case T_SHORT: |
|
678 |
case T_CHAR: |
|
679 |
case T_BOOLEAN: |
|
680 |
slot_type = T_INT; |
|
681 |
break; |
|
682 |
case T_ARRAY: |
|
683 |
slot_type = T_OBJECT; |
|
684 |
break; |
|
685 |
}; |
|
686 |
if (_type != slot_type) { |
|
687 |
_result = JVMTI_ERROR_TYPE_MISMATCH; |
|
688 |
return false; |
|
689 |
} |
|
690 |
||
691 |
jobject jobj = _value.l; |
|
692 |
if (_set && slot_type == T_OBJECT && jobj != NULL) { // NULL reference is allowed |
|
693 |
// Check that the jobject class matches the return type signature. |
|
694 |
JavaThread* cur_thread = JavaThread::current(); |
|
695 |
HandleMark hm(cur_thread); |
|
696 |
||
46271
979ebd346ecf
8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents:
40655
diff
changeset
|
697 |
Handle obj(cur_thread, JNIHandles::resolve_external_guard(jobj)); |
1 | 698 |
NULL_CHECK(obj, (_result = JVMTI_ERROR_INVALID_OBJECT, false)); |
699 |
KlassHandle ob_kh = KlassHandle(cur_thread, obj->klass()); |
|
700 |
NULL_CHECK(ob_kh, (_result = JVMTI_ERROR_INVALID_OBJECT, false)); |
|
701 |
||
14488 | 702 |
if (!is_assignable(signature, ob_kh(), cur_thread)) { |
1 | 703 |
_result = JVMTI_ERROR_TYPE_MISMATCH; |
704 |
return false; |
|
705 |
} |
|
706 |
} |
|
707 |
return true; |
|
708 |
} |
|
709 |
||
710 |
static bool can_be_deoptimized(vframe* vf) { |
|
711 |
return (vf->is_compiled_frame() && vf->fr().can_be_deoptimized()); |
|
712 |
} |
|
713 |
||
714 |
bool VM_GetOrSetLocal::doit_prologue() { |
|
715 |
_jvf = get_java_vframe(); |
|
716 |
NULL_CHECK(_jvf, false); |
|
717 |
||
7444 | 718 |
if (_jvf->method()->is_native()) { |
719 |
if (getting_receiver() && !_jvf->method()->is_static()) { |
|
720 |
return true; |
|
721 |
} else { |
|
722 |
_result = JVMTI_ERROR_OPAQUE_FRAME; |
|
723 |
return false; |
|
724 |
} |
|
725 |
} |
|
726 |
||
1 | 727 |
if (!check_slot_type(_jvf)) { |
728 |
return false; |
|
729 |
} |
|
730 |
return true; |
|
731 |
} |
|
732 |
||
733 |
void VM_GetOrSetLocal::doit() { |
|
25626
362d0c3a7651
8013942: JSR 292: assert(type() == T_OBJECT) failed: type check
sspitsyn
parents:
25351
diff
changeset
|
734 |
InterpreterOopMap oop_mask; |
362d0c3a7651
8013942: JSR 292: assert(type() == T_OBJECT) failed: type check
sspitsyn
parents:
25351
diff
changeset
|
735 |
_jvf->method()->mask_for(_jvf->bci(), &oop_mask); |
362d0c3a7651
8013942: JSR 292: assert(type() == T_OBJECT) failed: type check
sspitsyn
parents:
25351
diff
changeset
|
736 |
if (oop_mask.is_dead(_index)) { |
362d0c3a7651
8013942: JSR 292: assert(type() == T_OBJECT) failed: type check
sspitsyn
parents:
25351
diff
changeset
|
737 |
// The local can be invalid and uninitialized in the scope of current bci |
362d0c3a7651
8013942: JSR 292: assert(type() == T_OBJECT) failed: type check
sspitsyn
parents:
25351
diff
changeset
|
738 |
_result = JVMTI_ERROR_INVALID_SLOT; |
362d0c3a7651
8013942: JSR 292: assert(type() == T_OBJECT) failed: type check
sspitsyn
parents:
25351
diff
changeset
|
739 |
return; |
362d0c3a7651
8013942: JSR 292: assert(type() == T_OBJECT) failed: type check
sspitsyn
parents:
25351
diff
changeset
|
740 |
} |
1 | 741 |
if (_set) { |
742 |
// Force deoptimization of frame if compiled because it's |
|
743 |
// possible the compiler emitted some locals as constant values, |
|
744 |
// meaning they are not mutable. |
|
745 |
if (can_be_deoptimized(_jvf)) { |
|
746 |
||
747 |
// Schedule deoptimization so that eventually the local |
|
748 |
// update will be written to an interpreter frame. |
|
7106
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
5547
diff
changeset
|
749 |
Deoptimization::deoptimize_frame(_jvf->thread(), _jvf->fr().id()); |
1 | 750 |
|
751 |
// Now store a new value for the local which will be applied |
|
752 |
// once deoptimization occurs. Note however that while this |
|
753 |
// write is deferred until deoptimization actually happens |
|
754 |
// can vframe created after this point will have its locals |
|
755 |
// reflecting this update so as far as anyone can see the |
|
756 |
// write has already taken place. |
|
757 |
||
758 |
// If we are updating an oop then get the oop from the handle |
|
759 |
// since the handle will be long gone by the time the deopt |
|
760 |
// happens. The oop stored in the deferred local will be |
|
761 |
// gc'd on its own. |
|
762 |
if (_type == T_OBJECT) { |
|
763 |
_value.l = (jobject) (JNIHandles::resolve_external_guard(_value.l)); |
|
764 |
} |
|
765 |
// Re-read the vframe so we can see that it is deoptimized |
|
766 |
// [ Only need because of assert in update_local() ] |
|
767 |
_jvf = get_java_vframe(); |
|
768 |
((compiledVFrame*)_jvf)->update_local(_type, _index, _value); |
|
769 |
return; |
|
770 |
} |
|
771 |
StackValueCollection *locals = _jvf->locals(); |
|
772 |
HandleMark hm; |
|
773 |
||
774 |
switch (_type) { |
|
7444 | 775 |
case T_INT: locals->set_int_at (_index, _value.i); break; |
776 |
case T_LONG: locals->set_long_at (_index, _value.j); break; |
|
777 |
case T_FLOAT: locals->set_float_at (_index, _value.f); break; |
|
778 |
case T_DOUBLE: locals->set_double_at(_index, _value.d); break; |
|
779 |
case T_OBJECT: { |
|
46271
979ebd346ecf
8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents:
40655
diff
changeset
|
780 |
Handle ob_h(Thread::current(), JNIHandles::resolve_external_guard(_value.l)); |
7444 | 781 |
locals->set_obj_at (_index, ob_h); |
782 |
break; |
|
783 |
} |
|
784 |
default: ShouldNotReachHere(); |
|
1 | 785 |
} |
786 |
_jvf->set_locals(locals); |
|
787 |
} else { |
|
7444 | 788 |
if (_jvf->method()->is_native() && _jvf->is_compiled_frame()) { |
789 |
assert(getting_receiver(), "Can only get here when getting receiver"); |
|
790 |
oop receiver = _jvf->fr().get_native_receiver(); |
|
791 |
_value.l = JNIHandles::make_local(_calling_thread, receiver); |
|
792 |
} else { |
|
793 |
StackValueCollection *locals = _jvf->locals(); |
|
1 | 794 |
|
7444 | 795 |
if (locals->at(_index)->type() == T_CONFLICT) { |
796 |
memset(&_value, 0, sizeof(_value)); |
|
797 |
_value.l = NULL; |
|
798 |
return; |
|
799 |
} |
|
1 | 800 |
|
7444 | 801 |
switch (_type) { |
802 |
case T_INT: _value.i = locals->int_at (_index); break; |
|
803 |
case T_LONG: _value.j = locals->long_at (_index); break; |
|
804 |
case T_FLOAT: _value.f = locals->float_at (_index); break; |
|
805 |
case T_DOUBLE: _value.d = locals->double_at(_index); break; |
|
806 |
case T_OBJECT: { |
|
807 |
// Wrap the oop to be returned in a local JNI handle since |
|
808 |
// oops_do() no longer applies after doit() is finished. |
|
809 |
oop obj = locals->obj_at(_index)(); |
|
810 |
_value.l = JNIHandles::make_local(_calling_thread, obj); |
|
811 |
break; |
|
812 |
} |
|
813 |
default: ShouldNotReachHere(); |
|
814 |
} |
|
1 | 815 |
} |
816 |
} |
|
817 |
} |
|
818 |
||
819 |
||
820 |
bool VM_GetOrSetLocal::allow_nested_vm_operations() const { |
|
821 |
return true; // May need to deoptimize |
|
822 |
} |
|
823 |
||
824 |
||
7444 | 825 |
VM_GetReceiver::VM_GetReceiver( |
826 |
JavaThread* thread, JavaThread* caller_thread, jint depth) |
|
827 |
: VM_GetOrSetLocal(thread, caller_thread, depth, 0) {} |
|
828 |
||
1 | 829 |
///////////////////////////////////////////////////////////////////////////////////////// |
830 |
||
831 |
// |
|
832 |
// class JvmtiSuspendControl - see comments in jvmtiImpl.hpp |
|
833 |
// |
|
834 |
||
835 |
bool JvmtiSuspendControl::suspend(JavaThread *java_thread) { |
|
836 |
// external suspend should have caught suspending a thread twice |
|
837 |
||
838 |
// Immediate suspension required for JPDA back-end so JVMTI agent threads do |
|
839 |
// not deadlock due to later suspension on transitions while holding |
|
840 |
// raw monitors. Passing true causes the immediate suspension. |
|
841 |
// java_suspend() will catch threads in the process of exiting |
|
842 |
// and will ignore them. |
|
843 |
java_thread->java_suspend(); |
|
844 |
||
845 |
// It would be nice to have the following assertion in all the time, |
|
846 |
// but it is possible for a racing resume request to have resumed |
|
847 |
// this thread right after we suspended it. Temporarily enable this |
|
848 |
// assertion if you are chasing a different kind of bug. |
|
849 |
// |
|
850 |
// assert(java_lang_Thread::thread(java_thread->threadObj()) == NULL || |
|
851 |
// java_thread->is_being_ext_suspended(), "thread is not suspended"); |
|
852 |
||
853 |
if (java_lang_Thread::thread(java_thread->threadObj()) == NULL) { |
|
854 |
// check again because we can get delayed in java_suspend(): |
|
855 |
// the thread is in process of exiting. |
|
856 |
return false; |
|
857 |
} |
|
858 |
||
859 |
return true; |
|
860 |
} |
|
861 |
||
862 |
bool JvmtiSuspendControl::resume(JavaThread *java_thread) { |
|
863 |
// external suspend should have caught resuming a thread twice |
|
864 |
assert(java_thread->is_being_ext_suspended(), "thread should be suspended"); |
|
865 |
||
866 |
// resume thread |
|
867 |
{ |
|
868 |
// must always grab Threads_lock, see JVM_SuspendThread |
|
869 |
MutexLocker ml(Threads_lock); |
|
870 |
java_thread->java_resume(); |
|
871 |
} |
|
872 |
||
873 |
return true; |
|
874 |
} |
|
875 |
||
876 |
||
877 |
void JvmtiSuspendControl::print() { |
|
878 |
#ifndef PRODUCT |
|
879 |
MutexLocker mu(Threads_lock); |
|
37992 | 880 |
LogStreamHandle(Trace, jvmti) log_stream; |
881 |
log_stream.print("Suspended Threads: ["); |
|
1 | 882 |
for (JavaThread *thread = Threads::first(); thread != NULL; thread = thread->next()) { |
15937 | 883 |
#ifdef JVMTI_TRACE |
1 | 884 |
const char *name = JvmtiTrace::safe_get_thread_name(thread); |
885 |
#else |
|
886 |
const char *name = ""; |
|
887 |
#endif /*JVMTI_TRACE */ |
|
37992 | 888 |
log_stream.print("%s(%c ", name, thread->is_being_ext_suspended() ? 'S' : '_'); |
1 | 889 |
if (!thread->has_last_Java_frame()) { |
37992 | 890 |
log_stream.print("no stack"); |
1 | 891 |
} |
37992 | 892 |
log_stream.print(") "); |
1 | 893 |
} |
37992 | 894 |
log_stream.print_cr("]"); |
1 | 895 |
#endif |
896 |
} |
|
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
897 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
898 |
JvmtiDeferredEvent JvmtiDeferredEvent::compiled_method_load_event( |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
899 |
nmethod* nm) { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
900 |
JvmtiDeferredEvent event = JvmtiDeferredEvent(TYPE_COMPILED_METHOD_LOAD); |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
901 |
event._event_data.compiled_method_load = nm; |
8661
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
902 |
// Keep the nmethod alive until the ServiceThread can process |
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
903 |
// this deferred event. |
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
904 |
nmethodLocker::lock_nmethod(nm); |
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
905 |
return event; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
906 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
907 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
908 |
JvmtiDeferredEvent JvmtiDeferredEvent::compiled_method_unload_event( |
8661
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
909 |
nmethod* nm, jmethodID id, const void* code) { |
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
910 |
JvmtiDeferredEvent event = JvmtiDeferredEvent(TYPE_COMPILED_METHOD_UNLOAD); |
8661
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
911 |
event._event_data.compiled_method_unload.nm = nm; |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
912 |
event._event_data.compiled_method_unload.method_id = id; |
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
913 |
event._event_data.compiled_method_unload.code_begin = code; |
8661
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
914 |
// Keep the nmethod alive until the ServiceThread can process |
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
915 |
// this deferred event. This will keep the memory for the |
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
916 |
// generated code from being reused too early. We pass |
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
917 |
// zombie_ok == true here so that our nmethod that was just |
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
918 |
// made into a zombie can be locked. |
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
919 |
nmethodLocker::lock_nmethod(nm, true /* zombie_ok */); |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
920 |
return event; |
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
921 |
} |
9405
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
922 |
|
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
923 |
JvmtiDeferredEvent JvmtiDeferredEvent::dynamic_code_generated_event( |
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
924 |
const char* name, const void* code_begin, const void* code_end) { |
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
925 |
JvmtiDeferredEvent event = JvmtiDeferredEvent(TYPE_DYNAMIC_CODE_GENERATED); |
9405
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
926 |
// Need to make a copy of the name since we don't know how long |
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
927 |
// the event poster will keep it around after we enqueue the |
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
928 |
// deferred event and return. strdup() failure is handled in |
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
929 |
// the post() routine below. |
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
930 |
event._event_data.dynamic_code_generated.name = os::strdup(name); |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
931 |
event._event_data.dynamic_code_generated.code_begin = code_begin; |
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
932 |
event._event_data.dynamic_code_generated.code_end = code_end; |
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
933 |
return event; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
934 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
935 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
936 |
void JvmtiDeferredEvent::post() { |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
937 |
assert(ServiceThread::is_service_thread(Thread::current()), |
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
938 |
"Service thread must post enqueued events"); |
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
939 |
switch(_type) { |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
940 |
case TYPE_COMPILED_METHOD_LOAD: { |
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
941 |
nmethod* nm = _event_data.compiled_method_load; |
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
942 |
JvmtiExport::post_compiled_method_load(nm); |
8661
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
943 |
// done with the deferred event so unlock the nmethod |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
944 |
nmethodLocker::unlock_nmethod(nm); |
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
945 |
break; |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
946 |
} |
8661
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
947 |
case TYPE_COMPILED_METHOD_UNLOAD: { |
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
948 |
nmethod* nm = _event_data.compiled_method_unload.nm; |
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
949 |
JvmtiExport::post_compiled_method_unload( |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
950 |
_event_data.compiled_method_unload.method_id, |
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
951 |
_event_data.compiled_method_unload.code_begin); |
8661
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
952 |
// done with the deferred event so unlock the nmethod |
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
953 |
nmethodLocker::unlock_nmethod(nm); |
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
954 |
break; |
8661
3dc8a18ac563
7024970: 2/3 assert(ServiceThread::is_service_thread(Thread::current())) failed: Service thread must post enqueue
dcubed
parents:
8475
diff
changeset
|
955 |
} |
9405
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
956 |
case TYPE_DYNAMIC_CODE_GENERATED: { |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
957 |
JvmtiExport::post_dynamic_code_generated_internal( |
9405
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
958 |
// if strdup failed give the event a default name |
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
959 |
(_event_data.dynamic_code_generated.name == NULL) |
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
960 |
? "unknown_code" : _event_data.dynamic_code_generated.name, |
8475
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
961 |
_event_data.dynamic_code_generated.code_begin, |
c2b97a27943a
7017640: Fix for 6766644 deadlocks on some NSK tests when running with -Xcomp
kamg
parents:
8110
diff
changeset
|
962 |
_event_data.dynamic_code_generated.code_end); |
9405
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
963 |
if (_event_data.dynamic_code_generated.name != NULL) { |
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
964 |
// release our copy |
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
965 |
os::free((void *)_event_data.dynamic_code_generated.name); |
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
966 |
} |
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
967 |
break; |
9405
d2e9ee78c3c1
7039447: 2/1 java profiling is broken in build 139 (garbage in function name)
dcubed
parents:
8661
diff
changeset
|
968 |
} |
8110
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
969 |
default: |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
970 |
ShouldNotReachHere(); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
971 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
972 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
973 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
974 |
JvmtiDeferredEventQueue::QueueNode* JvmtiDeferredEventQueue::_queue_tail = NULL; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
975 |
JvmtiDeferredEventQueue::QueueNode* JvmtiDeferredEventQueue::_queue_head = NULL; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
976 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
977 |
volatile JvmtiDeferredEventQueue::QueueNode* |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
978 |
JvmtiDeferredEventQueue::_pending_list = NULL; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
979 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
980 |
bool JvmtiDeferredEventQueue::has_events() { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
981 |
assert(Service_lock->owned_by_self(), "Must own Service_lock"); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
982 |
return _queue_head != NULL || _pending_list != NULL; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
983 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
984 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
985 |
void JvmtiDeferredEventQueue::enqueue(const JvmtiDeferredEvent& event) { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
986 |
assert(Service_lock->owned_by_self(), "Must own Service_lock"); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
987 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
988 |
process_pending_events(); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
989 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
990 |
// Events get added to the end of the queue (and are pulled off the front). |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
991 |
QueueNode* node = new QueueNode(event); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
992 |
if (_queue_tail == NULL) { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
993 |
_queue_tail = _queue_head = node; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
994 |
} else { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
995 |
assert(_queue_tail->next() == NULL, "Must be the last element in the list"); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
996 |
_queue_tail->set_next(node); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
997 |
_queue_tail = node; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
998 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
999 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1000 |
Service_lock->notify_all(); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1001 |
assert((_queue_head == NULL) == (_queue_tail == NULL), |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1002 |
"Inconsistent queue markers"); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1003 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1004 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1005 |
JvmtiDeferredEvent JvmtiDeferredEventQueue::dequeue() { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1006 |
assert(Service_lock->owned_by_self(), "Must own Service_lock"); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1007 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1008 |
process_pending_events(); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1009 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1010 |
assert(_queue_head != NULL, "Nothing to dequeue"); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1011 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1012 |
if (_queue_head == NULL) { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1013 |
// Just in case this happens in product; it shouldn't but let's not crash |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1014 |
return JvmtiDeferredEvent(); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1015 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1016 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1017 |
QueueNode* node = _queue_head; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1018 |
_queue_head = _queue_head->next(); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1019 |
if (_queue_head == NULL) { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1020 |
_queue_tail = NULL; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1021 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1022 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1023 |
assert((_queue_head == NULL) == (_queue_tail == NULL), |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1024 |
"Inconsistent queue markers"); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1025 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1026 |
JvmtiDeferredEvent event = node->event(); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1027 |
delete node; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1028 |
return event; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1029 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1030 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1031 |
void JvmtiDeferredEventQueue::add_pending_event( |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1032 |
const JvmtiDeferredEvent& event) { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1033 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1034 |
QueueNode* node = new QueueNode(event); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1035 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1036 |
bool success = false; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1037 |
QueueNode* prev_value = (QueueNode*)_pending_list; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1038 |
do { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1039 |
node->set_next(prev_value); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1040 |
prev_value = (QueueNode*)Atomic::cmpxchg_ptr( |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1041 |
(void*)node, (volatile void*)&_pending_list, (void*)node->next()); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1042 |
} while (prev_value != node->next()); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1043 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1044 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1045 |
// This method transfers any events that were added by someone NOT holding |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1046 |
// the lock into the mainline queue. |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1047 |
void JvmtiDeferredEventQueue::process_pending_events() { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1048 |
assert(Service_lock->owned_by_self(), "Must own Service_lock"); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1049 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1050 |
if (_pending_list != NULL) { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1051 |
QueueNode* head = |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1052 |
(QueueNode*)Atomic::xchg_ptr(NULL, (volatile void*)&_pending_list); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1053 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1054 |
assert((_queue_head == NULL) == (_queue_tail == NULL), |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1055 |
"Inconsistent queue markers"); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1056 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1057 |
if (head != NULL) { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1058 |
// Since we've treated the pending list as a stack (with newer |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1059 |
// events at the beginning), we need to join the bottom of the stack |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1060 |
// with the 'tail' of the queue in order to get the events in the |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1061 |
// right order. We do this by reversing the pending list and appending |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1062 |
// it to the queue. |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1063 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1064 |
QueueNode* new_tail = head; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1065 |
QueueNode* new_head = NULL; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1066 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1067 |
// This reverses the list |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1068 |
QueueNode* prev = new_tail; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1069 |
QueueNode* node = new_tail->next(); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1070 |
new_tail->set_next(NULL); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1071 |
while (node != NULL) { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1072 |
QueueNode* next = node->next(); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1073 |
node->set_next(prev); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1074 |
prev = node; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1075 |
node = next; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1076 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1077 |
new_head = prev; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1078 |
|
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1079 |
// Now append the new list to the queue |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1080 |
if (_queue_tail != NULL) { |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1081 |
_queue_tail->set_next(new_head); |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1082 |
} else { // _queue_head == NULL |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1083 |
_queue_head = new_head; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1084 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1085 |
_queue_tail = new_tail; |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1086 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1087 |
} |
c992c8d52344
6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"
kamg
parents:
8076
diff
changeset
|
1088 |
} |