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