author | dholmes |
Sat, 23 Jun 2018 01:32:41 -0400 | |
changeset 50735 | 2f2af62dfac7 |
parent 50152 | b5023063346d |
child 52067 | 2e72562697bf |
permissions | -rw-r--r-- |
1 | 1 |
/* |
49192
6734eeef4283
8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents:
47216
diff
changeset
|
2 |
* Copyright (c) 1997, 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:
5419
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5419
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:
5419
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "classfile/systemDictionary.hpp" |
|
27 |
#include "classfile/vmSymbols.hpp" |
|
28 |
#include "code/nmethod.hpp" |
|
29 |
#include "compiler/compileBroker.hpp" |
|
30 |
#include "interpreter/interpreter.hpp" |
|
31 |
#include "interpreter/linkResolver.hpp" |
|
49359
59f6547e151f
8199264: Remove universe.inline.hpp to simplify include dependencies
stefank
parents:
49340
diff
changeset
|
32 |
#include "memory/universe.hpp" |
49340
4e82736053ae
8191102: Incorrect include file use in classLoader.hpp
hseigel
parents:
49192
diff
changeset
|
33 |
#include "oops/method.inline.hpp" |
7397 | 34 |
#include "oops/oop.inline.hpp" |
35 |
#include "prims/jniCheck.hpp" |
|
36 |
#include "runtime/compilationPolicy.hpp" |
|
37 |
#include "runtime/handles.inline.hpp" |
|
49449
ef5d5d343e2a
8199263: Split interfaceSupport.hpp to not require including .inline.hpp files
coleenp
parents:
49359
diff
changeset
|
38 |
#include "runtime/interfaceSupport.inline.hpp" |
7397 | 39 |
#include "runtime/javaCalls.hpp" |
49192
6734eeef4283
8198474: Move JNIHandles::resolve into jniHandles.inline.hpp
kbarrett
parents:
47216
diff
changeset
|
40 |
#include "runtime/jniHandles.inline.hpp" |
7397 | 41 |
#include "runtime/mutexLocker.hpp" |
25468
5331df506290
8048241: Introduce umbrella header os.inline.hpp and clean up includes
goetz
parents:
22775
diff
changeset
|
42 |
#include "runtime/os.inline.hpp" |
49593
4dd58ecc9912
8200105: Remove cyclic dependency between oop.inline.hpp and collectedHeap.inline.hpp
stefank
parents:
49449
diff
changeset
|
43 |
#include "runtime/sharedRuntime.hpp" |
7397 | 44 |
#include "runtime/signature.hpp" |
45 |
#include "runtime/stubRoutines.hpp" |
|
14583
d70ee55535f4
8003935: Simplify the needed includes for using Thread::current()
stefank
parents:
14391
diff
changeset
|
46 |
#include "runtime/thread.inline.hpp" |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
47 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
48 |
#include "jvmci/jvmciJavaClasses.hpp" |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
49 |
#include "jvmci/jvmciRuntime.hpp" |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
50 |
#endif |
1 | 51 |
|
52 |
// ----------------------------------------------------- |
|
53 |
// Implementation of JavaCallWrapper |
|
54 |
||
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
46630
diff
changeset
|
55 |
JavaCallWrapper::JavaCallWrapper(const methodHandle& callee_method, Handle receiver, JavaValue* result, TRAPS) { |
1 | 56 |
JavaThread* thread = (JavaThread *)THREAD; |
57 |
bool clear_pending_exception = true; |
|
58 |
||
59 |
guarantee(thread->is_Java_thread(), "crucial check - the VM thread cannot and must not escape to Java code"); |
|
60 |
assert(!thread->owns_locks(), "must release all locks when leaving VM"); |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
61 |
guarantee(thread->can_call_java(), "cannot make java calls from the native compiler"); |
1 | 62 |
_result = result; |
63 |
||
64 |
// Allocate handle block for Java code. This must be done before we change thread_state to _thread_in_Java_or_stub, |
|
65 |
// since it can potentially block. |
|
66 |
JNIHandleBlock* new_handles = JNIHandleBlock::allocate_block(thread); |
|
67 |
||
68 |
// After this, we are official in JavaCode. This needs to be done before we change any of the thread local |
|
69 |
// info, since we cannot find oops before the new information is set up completely. |
|
70 |
ThreadStateTransition::transition(thread, _thread_in_vm, _thread_in_Java); |
|
71 |
||
72 |
// Make sure that we handle asynchronous stops and suspends _before_ we clear all thread state |
|
73 |
// in JavaCallWrapper::JavaCallWrapper(). This way, we can decide if we need to do any pd actions |
|
74 |
// to prepare for stop/suspend (flush register windows on sparcs, cache sp, or other state). |
|
75 |
if (thread->has_special_runtime_exit_condition()) { |
|
76 |
thread->handle_special_runtime_exit_condition(); |
|
77 |
if (HAS_PENDING_EXCEPTION) { |
|
78 |
clear_pending_exception = false; |
|
79 |
} |
|
80 |
} |
|
81 |
||
82 |
||
83 |
// Make sure to set the oop's after the thread transition - since we can block there. No one is GC'ing |
|
84 |
// the JavaCallWrapper before the entry frame is on the stack. |
|
85 |
_callee_method = callee_method(); |
|
86 |
_receiver = receiver(); |
|
87 |
||
88 |
#ifdef CHECK_UNHANDLED_OOPS |
|
89 |
THREAD->allow_unhandled_oop(&_receiver); |
|
90 |
#endif // CHECK_UNHANDLED_OOPS |
|
91 |
||
92 |
_thread = (JavaThread *)thread; |
|
93 |
_handles = _thread->active_handles(); // save previous handle block & Java frame linkage |
|
94 |
||
95 |
// For the profiler, the last_Java_frame information in thread must always be in |
|
96 |
// legal state. We have no last Java frame if last_Java_sp == NULL so |
|
97 |
// the valid transition is to clear _last_Java_sp and then reset the rest of |
|
98 |
// the (platform specific) state. |
|
99 |
||
100 |
_anchor.copy(_thread->frame_anchor()); |
|
101 |
_thread->frame_anchor()->clear(); |
|
102 |
||
103 |
debug_only(_thread->inc_java_call_counter()); |
|
104 |
_thread->set_active_handles(new_handles); // install new handle block and reset Java frame linkage |
|
105 |
||
106 |
assert (_thread->thread_state() != _thread_in_native, "cannot set native pc to NULL"); |
|
107 |
||
108 |
// clear any pending exception in thread (native calls start with no exception pending) |
|
109 |
if(clear_pending_exception) { |
|
110 |
_thread->clear_pending_exception(); |
|
111 |
} |
|
112 |
||
113 |
if (_anchor.last_Java_sp() == NULL) { |
|
114 |
_thread->record_base_of_stack_pointer(); |
|
115 |
} |
|
116 |
} |
|
117 |
||
118 |
||
119 |
JavaCallWrapper::~JavaCallWrapper() { |
|
120 |
assert(_thread == JavaThread::current(), "must still be the same thread"); |
|
121 |
||
122 |
// restore previous handle block & Java frame linkage |
|
123 |
JNIHandleBlock *_old_handles = _thread->active_handles(); |
|
124 |
_thread->set_active_handles(_handles); |
|
125 |
||
126 |
_thread->frame_anchor()->zap(); |
|
127 |
||
128 |
debug_only(_thread->dec_java_call_counter()); |
|
129 |
||
130 |
if (_anchor.last_Java_sp() == NULL) { |
|
131 |
_thread->set_base_of_stack_pointer(NULL); |
|
132 |
} |
|
133 |
||
134 |
||
135 |
// Old thread-local info. has been restored. We are not back in the VM. |
|
136 |
ThreadStateTransition::transition_from_java(_thread, _thread_in_vm); |
|
137 |
||
138 |
// State has been restored now make the anchor frame visible for the profiler. |
|
139 |
// Do this after the transition because this allows us to put an assert |
|
140 |
// the Java->vm transition which checks to see that stack is not walkable |
|
141 |
// on sparc/ia64 which will catch violations of the reseting of last_Java_frame |
|
142 |
// invariants (i.e. _flags always cleared on return to Java) |
|
143 |
||
144 |
_thread->frame_anchor()->copy(&_anchor); |
|
145 |
||
146 |
// Release handles after we are marked as being inside the VM again, since this |
|
147 |
// operation might block |
|
148 |
JNIHandleBlock::release_block(_old_handles, _thread); |
|
149 |
} |
|
150 |
||
151 |
||
152 |
void JavaCallWrapper::oops_do(OopClosure* f) { |
|
153 |
f->do_oop((oop*)&_receiver); |
|
154 |
handles()->oops_do(f); |
|
155 |
} |
|
156 |
||
157 |
||
158 |
// Helper methods |
|
159 |
static BasicType runtime_type_from(JavaValue* result) { |
|
160 |
switch (result->get_type()) { |
|
161 |
case T_BOOLEAN: // fall through |
|
162 |
case T_CHAR : // fall through |
|
163 |
case T_SHORT : // fall through |
|
164 |
case T_INT : // fall through |
|
165 |
#ifndef _LP64 |
|
166 |
case T_OBJECT : // fall through |
|
167 |
case T_ARRAY : // fall through |
|
168 |
#endif |
|
169 |
case T_BYTE : // fall through |
|
170 |
case T_VOID : return T_INT; |
|
171 |
case T_LONG : return T_LONG; |
|
172 |
case T_FLOAT : return T_FLOAT; |
|
173 |
case T_DOUBLE : return T_DOUBLE; |
|
174 |
#ifdef _LP64 |
|
175 |
case T_ARRAY : // fall through |
|
176 |
case T_OBJECT: return T_OBJECT; |
|
177 |
#endif |
|
46630
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
46458
diff
changeset
|
178 |
default: |
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
46458
diff
changeset
|
179 |
ShouldNotReachHere(); |
75aa3e39d02c
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents:
46458
diff
changeset
|
180 |
return T_ILLEGAL; |
1 | 181 |
} |
182 |
} |
|
183 |
||
184 |
// ============ Virtual calls ============ |
|
185 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
186 |
void JavaCalls::call_virtual(JavaValue* result, Klass* spec_klass, Symbol* name, Symbol* signature, JavaCallArguments* args, TRAPS) { |
1 | 187 |
CallInfo callinfo; |
188 |
Handle receiver = args->receiver(); |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
189 |
Klass* recvrKlass = receiver.is_null() ? (Klass*)NULL : receiver->klass(); |
38719
133bf85c3f36
8145148: InterfaceMethod CP entry pointing to a class should cause ICCE
coleenp
parents:
38139
diff
changeset
|
190 |
LinkInfo link_info(spec_klass, name, signature); |
1 | 191 |
LinkResolver::resolve_virtual_call( |
31019 | 192 |
callinfo, receiver, recvrKlass, link_info, true, CHECK); |
1 | 193 |
methodHandle method = callinfo.selected_method(); |
194 |
assert(method.not_null(), "should have thrown exception"); |
|
195 |
||
196 |
// Invoke the method |
|
197 |
JavaCalls::call(result, method, args, CHECK); |
|
198 |
} |
|
199 |
||
200 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
201 |
void JavaCalls::call_virtual(JavaValue* result, Handle receiver, Klass* spec_klass, Symbol* name, Symbol* signature, TRAPS) { |
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
202 |
JavaCallArguments args(receiver); |
1 | 203 |
call_virtual(result, spec_klass, name, signature, &args, CHECK); |
204 |
} |
|
205 |
||
206 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
207 |
void JavaCalls::call_virtual(JavaValue* result, Handle receiver, Klass* spec_klass, Symbol* name, Symbol* signature, Handle arg1, TRAPS) { |
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
208 |
JavaCallArguments args(receiver); |
1 | 209 |
args.push_oop(arg1); |
210 |
call_virtual(result, spec_klass, name, signature, &args, CHECK); |
|
211 |
} |
|
212 |
||
213 |
||
214 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
215 |
void JavaCalls::call_virtual(JavaValue* result, Handle receiver, Klass* spec_klass, Symbol* name, Symbol* signature, Handle arg1, Handle arg2, TRAPS) { |
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
216 |
JavaCallArguments args(receiver); |
1 | 217 |
args.push_oop(arg1); |
218 |
args.push_oop(arg2); |
|
219 |
call_virtual(result, spec_klass, name, signature, &args, CHECK); |
|
220 |
} |
|
221 |
||
222 |
||
223 |
// ============ Special calls ============ |
|
224 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
225 |
void JavaCalls::call_special(JavaValue* result, Klass* klass, Symbol* name, Symbol* signature, JavaCallArguments* args, TRAPS) { |
1 | 226 |
CallInfo callinfo; |
38719
133bf85c3f36
8145148: InterfaceMethod CP entry pointing to a class should cause ICCE
coleenp
parents:
38139
diff
changeset
|
227 |
LinkInfo link_info(klass, name, signature); |
44738 | 228 |
LinkResolver::resolve_special_call(callinfo, args->receiver(), link_info, CHECK); |
1 | 229 |
methodHandle method = callinfo.selected_method(); |
230 |
assert(method.not_null(), "should have thrown exception"); |
|
231 |
||
232 |
// Invoke the method |
|
233 |
JavaCalls::call(result, method, args, CHECK); |
|
234 |
} |
|
235 |
||
236 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
237 |
void JavaCalls::call_special(JavaValue* result, Handle receiver, Klass* klass, Symbol* name, Symbol* signature, TRAPS) { |
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
238 |
JavaCallArguments args(receiver); |
1 | 239 |
call_special(result, klass, name, signature, &args, CHECK); |
240 |
} |
|
241 |
||
242 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
243 |
void JavaCalls::call_special(JavaValue* result, Handle receiver, Klass* klass, Symbol* name, Symbol* signature, Handle arg1, TRAPS) { |
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
244 |
JavaCallArguments args(receiver); |
1 | 245 |
args.push_oop(arg1); |
246 |
call_special(result, klass, name, signature, &args, CHECK); |
|
247 |
} |
|
248 |
||
249 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
250 |
void JavaCalls::call_special(JavaValue* result, Handle receiver, Klass* klass, Symbol* name, Symbol* signature, Handle arg1, Handle arg2, TRAPS) { |
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
251 |
JavaCallArguments args(receiver); |
1 | 252 |
args.push_oop(arg1); |
253 |
args.push_oop(arg2); |
|
254 |
call_special(result, klass, name, signature, &args, CHECK); |
|
255 |
} |
|
256 |
||
257 |
||
258 |
// ============ Static calls ============ |
|
259 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
260 |
void JavaCalls::call_static(JavaValue* result, Klass* klass, Symbol* name, Symbol* signature, JavaCallArguments* args, TRAPS) { |
1 | 261 |
CallInfo callinfo; |
38719
133bf85c3f36
8145148: InterfaceMethod CP entry pointing to a class should cause ICCE
coleenp
parents:
38139
diff
changeset
|
262 |
LinkInfo link_info(klass, name, signature); |
31019 | 263 |
LinkResolver::resolve_static_call(callinfo, link_info, true, CHECK); |
1 | 264 |
methodHandle method = callinfo.selected_method(); |
265 |
assert(method.not_null(), "should have thrown exception"); |
|
266 |
||
267 |
// Invoke the method |
|
268 |
JavaCalls::call(result, method, args, CHECK); |
|
269 |
} |
|
270 |
||
271 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
272 |
void JavaCalls::call_static(JavaValue* result, Klass* klass, Symbol* name, Symbol* signature, TRAPS) { |
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
273 |
JavaCallArguments args; |
1 | 274 |
call_static(result, klass, name, signature, &args, CHECK); |
275 |
} |
|
276 |
||
277 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
278 |
void JavaCalls::call_static(JavaValue* result, Klass* klass, Symbol* name, Symbol* signature, Handle arg1, TRAPS) { |
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
279 |
JavaCallArguments args(arg1); |
1 | 280 |
call_static(result, klass, name, signature, &args, CHECK); |
281 |
} |
|
282 |
||
283 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
284 |
void JavaCalls::call_static(JavaValue* result, Klass* klass, Symbol* name, Symbol* signature, Handle arg1, Handle arg2, TRAPS) { |
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
285 |
JavaCallArguments args; |
1 | 286 |
args.push_oop(arg1); |
287 |
args.push_oop(arg2); |
|
288 |
call_static(result, klass, name, signature, &args, CHECK); |
|
289 |
} |
|
290 |
||
291 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
43969
diff
changeset
|
292 |
void JavaCalls::call_static(JavaValue* result, Klass* klass, Symbol* name, Symbol* signature, Handle arg1, Handle arg2, Handle arg3, TRAPS) { |
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
293 |
JavaCallArguments args; |
42307
cefc81dc1d52
8169069: Module system implementation refresh (11/2016)
alanb
parents:
39390
diff
changeset
|
294 |
args.push_oop(arg1); |
cefc81dc1d52
8169069: Module system implementation refresh (11/2016)
alanb
parents:
39390
diff
changeset
|
295 |
args.push_oop(arg2); |
cefc81dc1d52
8169069: Module system implementation refresh (11/2016)
alanb
parents:
39390
diff
changeset
|
296 |
args.push_oop(arg3); |
cefc81dc1d52
8169069: Module system implementation refresh (11/2016)
alanb
parents:
39390
diff
changeset
|
297 |
call_static(result, klass, name, signature, &args, CHECK); |
cefc81dc1d52
8169069: Module system implementation refresh (11/2016)
alanb
parents:
39390
diff
changeset
|
298 |
} |
cefc81dc1d52
8169069: Module system implementation refresh (11/2016)
alanb
parents:
39390
diff
changeset
|
299 |
|
50152
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
300 |
// ============ allocate and initialize new object instance ============ |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
301 |
|
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
302 |
Handle JavaCalls::construct_new_instance(InstanceKlass* klass, Symbol* constructor_signature, JavaCallArguments* args, TRAPS) { |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
303 |
klass->initialize(CHECK_NH); // Quick no-op if already initialized. |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
304 |
Handle obj = klass->allocate_instance_handle(CHECK_NH); |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
305 |
JavaValue void_result(T_VOID); |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
306 |
args->set_receiver(obj); // inserts <obj> as the first argument. |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
307 |
JavaCalls::call_special(&void_result, klass, |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
308 |
vmSymbols::object_initializer_name(), |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
309 |
constructor_signature, args, CHECK_NH); |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
310 |
return obj; |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
311 |
} |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
312 |
|
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
313 |
Handle JavaCalls::construct_new_instance(InstanceKlass* klass, Symbol* constructor_signature, TRAPS) { |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
314 |
JavaCallArguments args; |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
315 |
return JavaCalls::construct_new_instance(klass, constructor_signature, &args, CHECK_NH); |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
316 |
} |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
317 |
|
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
318 |
Handle JavaCalls::construct_new_instance(InstanceKlass* klass, Symbol* constructor_signature, Handle arg1, TRAPS) { |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
319 |
JavaCallArguments args; |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
320 |
args.push_oop(arg1); |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
321 |
return JavaCalls::construct_new_instance(klass, constructor_signature, &args, CHECK_NH); |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
322 |
} |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
323 |
|
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
324 |
Handle JavaCalls::construct_new_instance(InstanceKlass* klass, Symbol* constructor_signature, Handle arg1, Handle arg2, TRAPS) { |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
325 |
JavaCallArguments args; |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
326 |
args.push_oop(arg1); |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
327 |
args.push_oop(arg2); |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
328 |
return JavaCalls::construct_new_instance(klass, constructor_signature, &args, CHECK_NH); |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
329 |
} |
b5023063346d
8189140: SystemDictionaryShared::initialize() should be renamed to be more meaningful
iklam
parents:
49593
diff
changeset
|
330 |
|
1 | 331 |
// ------------------------------------------------- |
332 |
// Implementation of JavaCalls (low level) |
|
333 |
||
334 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
335 |
void JavaCalls::call(JavaValue* result, const methodHandle& method, JavaCallArguments* args, TRAPS) { |
1 | 336 |
// Check if we need to wrap a potential OS exception handler around thread |
337 |
// This is used for e.g. Win32 structured exception handlers |
|
338 |
assert(THREAD->is_Java_thread(), "only JavaThreads can make JavaCalls"); |
|
22551 | 339 |
// Need to wrap each and every time, since there might be native code down the |
1 | 340 |
// stack that has installed its own exception handlers |
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
341 |
os::os_exception_wrapper(call_helper, result, method, args, THREAD); |
1 | 342 |
} |
343 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
344 |
void JavaCalls::call_helper(JavaValue* result, const methodHandle& method, JavaCallArguments* args, TRAPS) { |
26135
82b516c550f7
8046070: Class Data Sharing clean up and refactoring
iklam
parents:
25468
diff
changeset
|
345 |
|
1 | 346 |
JavaThread* thread = (JavaThread*)THREAD; |
347 |
assert(thread->is_Java_thread(), "must be called by a java thread"); |
|
348 |
assert(method.not_null(), "must have a method to call"); |
|
349 |
assert(!SafepointSynchronize::is_at_safepoint(), "call to Java code during VM operation"); |
|
350 |
assert(!thread->handle_area()->no_handle_mark_active(), "cannot call out to Java here"); |
|
351 |
||
352 |
||
353 |
CHECK_UNHANDLED_OOPS_ONLY(thread->clear_unhandled_oops();) |
|
354 |
||
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
355 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
356 |
// Gets the nmethod (if any) that should be called instead of normal target |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
357 |
nmethod* alternative_target = args->alternative_target(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
358 |
if (alternative_target == NULL) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
359 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
360 |
// Verify the arguments |
1618
2d86b9b84aa5
6739363: Xcheck jni doesn't check native function arguments
poonam
parents:
1
diff
changeset
|
361 |
|
2d86b9b84aa5
6739363: Xcheck jni doesn't check native function arguments
poonam
parents:
1
diff
changeset
|
362 |
if (CheckJNICalls) { |
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
363 |
args->verify(method, result->get_type()); |
1618
2d86b9b84aa5
6739363: Xcheck jni doesn't check native function arguments
poonam
parents:
1
diff
changeset
|
364 |
} |
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
365 |
else debug_only(args->verify(method, result->get_type())); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
366 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
367 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
368 |
#else |
1 | 369 |
|
370 |
// Ignore call if method is empty |
|
371 |
if (method->is_empty_method()) { |
|
372 |
assert(result->get_type() == T_VOID, "an empty method must return a void value"); |
|
373 |
return; |
|
374 |
} |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
375 |
#endif |
1 | 376 |
|
377 |
#ifdef ASSERT |
|
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
13728
diff
changeset
|
378 |
{ InstanceKlass* holder = method->method_holder(); |
1 | 379 |
// A klass might not be initialized since JavaCall's might be used during the executing of |
380 |
// the <clinit>. For example, a Thread.start might start executing on an object that is |
|
381 |
// not fully initialized! (bad Java programming style) |
|
22775 | 382 |
assert(holder->is_linked(), "rewriting must have taken place"); |
1 | 383 |
} |
384 |
#endif |
|
385 |
||
38139
cf6f5c1b7205
8152903: [JVMCI] CompilerToVM::resolveMethod should correctly handle private methods in interfaces
never
parents:
35479
diff
changeset
|
386 |
CompilationPolicy::compile_if_required(method, CHECK); |
1 | 387 |
|
388 |
// Since the call stub sets up like the interpreter we call the from_interpreted_entry |
|
389 |
// so we can go compiled via a i2c. Otherwise initial entry method will always |
|
390 |
// run interpreted. |
|
391 |
address entry_point = method->from_interpreted_entry(); |
|
392 |
if (JvmtiExport::can_post_interpreter_events() && thread->is_interp_only_mode()) { |
|
393 |
entry_point = method->interpreter_entry(); |
|
394 |
} |
|
395 |
||
396 |
// Figure out if the result value is an oop or not (Note: This is a different value |
|
397 |
// than result_type. result_type will be T_INT of oops. (it is about size) |
|
398 |
BasicType result_type = runtime_type_from(result); |
|
399 |
bool oop_result_flag = (result->get_type() == T_OBJECT || result->get_type() == T_ARRAY); |
|
400 |
||
401 |
// NOTE: if we move the computation of the result_val_address inside |
|
402 |
// the call to call_stub, the optimizer produces wrong code. |
|
403 |
intptr_t* result_val_address = (intptr_t*)(result->get_value_addr()); |
|
404 |
||
405 |
// Find receiver |
|
406 |
Handle receiver = (!method->is_static()) ? args->receiver() : Handle(); |
|
407 |
||
35071
a0910b1d3e0d
8046936: JEP 270: Reserved Stack Areas for Critical Sections
fparain
parents:
34238
diff
changeset
|
408 |
// When we reenter Java, we need to reenable the reserved/yellow zone which |
1 | 409 |
// might already be disabled when we are in VM. |
35071
a0910b1d3e0d
8046936: JEP 270: Reserved Stack Areas for Critical Sections
fparain
parents:
34238
diff
changeset
|
410 |
if (!thread->stack_guards_enabled()) { |
1 | 411 |
thread->reguard_stack(); |
412 |
} |
|
413 |
||
414 |
// Check that there are shadow pages available before changing thread state |
|
39390
edf6a424a8b7
8159335: Fix problems with stack overflow handling.
goetz
parents:
38719
diff
changeset
|
415 |
// to Java. Calculate current_stack_pointer here to make sure |
edf6a424a8b7
8159335: Fix problems with stack overflow handling.
goetz
parents:
38719
diff
changeset
|
416 |
// stack_shadow_pages_available() and bang_stack_shadow_pages() use the same sp. |
edf6a424a8b7
8159335: Fix problems with stack overflow handling.
goetz
parents:
38719
diff
changeset
|
417 |
address sp = os::current_stack_pointer(); |
edf6a424a8b7
8159335: Fix problems with stack overflow handling.
goetz
parents:
38719
diff
changeset
|
418 |
if (!os::stack_shadow_pages_available(THREAD, method, sp)) { |
1 | 419 |
// Throw stack overflow exception with preinitialized exception. |
9321
c29711c6ae35
7009923: JSR 292: VM crash in JavaThread::last_frame
coleenp
parents:
8921
diff
changeset
|
420 |
Exceptions::throw_stack_overflow_exception(THREAD, __FILE__, __LINE__, method); |
1 | 421 |
return; |
422 |
} else { |
|
423 |
// Touch pages checked if the OS needs them to be touched to be mapped. |
|
39390
edf6a424a8b7
8159335: Fix problems with stack overflow handling.
goetz
parents:
38719
diff
changeset
|
424 |
os::map_stack_shadow_pages(sp); |
1 | 425 |
} |
426 |
||
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
427 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
428 |
if (alternative_target != NULL) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
429 |
if (alternative_target->is_alive()) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
430 |
thread->set_jvmci_alternate_call_target(alternative_target->verified_entry_point()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
431 |
entry_point = method->adapter()->get_i2c_entry(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
432 |
} else { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
433 |
THROW(vmSymbols::jdk_vm_ci_code_InvalidInstalledCodeException()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
434 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
435 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
436 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31019
diff
changeset
|
437 |
|
1 | 438 |
// do call |
439 |
{ JavaCallWrapper link(method, receiver, result, CHECK); |
|
440 |
{ HandleMark hm(thread); // HandleMark used by HandleMarkCleaner |
|
441 |
||
442 |
StubRoutines::call_stub()( |
|
443 |
(address)&link, |
|
444 |
// (intptr_t*)&(result->_value), // see NOTE above (compiler problem) |
|
445 |
result_val_address, // see NOTE above (compiler problem) |
|
446 |
result_type, |
|
447 |
method(), |
|
448 |
entry_point, |
|
449 |
args->parameters(), |
|
450 |
args->size_of_parameters(), |
|
451 |
CHECK |
|
452 |
); |
|
453 |
||
454 |
result = link.result(); // circumvent MS C++ 5.0 compiler bug (result is clobbered across call) |
|
455 |
// Preserve oop return value across possible gc points |
|
456 |
if (oop_result_flag) { |
|
457 |
thread->set_vm_result((oop) result->get_jobject()); |
|
458 |
} |
|
459 |
} |
|
460 |
} // Exit JavaCallWrapper (can block - potential return oop must be preserved) |
|
461 |
||
462 |
// Check if a thread stop or suspend should be executed |
|
463 |
// The following assert was not realistic. Thread.stop can set that bit at any moment. |
|
464 |
//assert(!thread->has_special_runtime_exit_condition(), "no async. exceptions should be installed"); |
|
465 |
||
466 |
// Restore possible oop return |
|
467 |
if (oop_result_flag) { |
|
468 |
result->set_jobject((jobject)thread->vm_result()); |
|
469 |
thread->set_vm_result(NULL); |
|
470 |
} |
|
471 |
} |
|
472 |
||
473 |
||
474 |
//-------------------------------------------------------------------------------------- |
|
475 |
// Implementation of JavaCallArguments |
|
476 |
||
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
477 |
inline bool is_value_state_indirect_oop(uint state) { |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
478 |
assert(state != JavaCallArguments::value_state_oop, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
479 |
"Checking for handles after removal"); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
480 |
assert(state < JavaCallArguments::value_state_limit, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
481 |
"Invalid value state %u", state); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
482 |
return state != JavaCallArguments::value_state_primitive; |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
483 |
} |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
484 |
|
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
485 |
inline oop resolve_indirect_oop(intptr_t value, uint state) { |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
486 |
switch (state) { |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
487 |
case JavaCallArguments::value_state_handle: |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
488 |
{ |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
489 |
oop* ptr = reinterpret_cast<oop*>(value); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
490 |
return Handle::raw_resolve(ptr); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
491 |
} |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
492 |
|
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
493 |
case JavaCallArguments::value_state_jobject: |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
494 |
{ |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
495 |
jobject obj = reinterpret_cast<jobject>(value); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
496 |
return JNIHandles::resolve(obj); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
497 |
} |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
498 |
|
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
499 |
default: |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
500 |
ShouldNotReachHere(); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
501 |
return NULL; |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
502 |
} |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
503 |
} |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
504 |
|
1 | 505 |
intptr_t* JavaCallArguments::parameters() { |
506 |
// First convert all handles to oops |
|
507 |
for(int i = 0; i < _size; i++) { |
|
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
508 |
uint state = _value_state[i]; |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
509 |
assert(state != value_state_oop, "Multiple handle conversions"); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
510 |
if (is_value_state_indirect_oop(state)) { |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
511 |
oop obj = resolve_indirect_oop(_value[i], state); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
512 |
_value[i] = cast_from_oop<intptr_t>(obj); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
513 |
_value_state[i] = value_state_oop; |
1 | 514 |
} |
515 |
} |
|
516 |
// Return argument vector |
|
5419 | 517 |
return _value; |
1 | 518 |
} |
519 |
||
520 |
||
521 |
class SignatureChekker : public SignatureIterator { |
|
522 |
private: |
|
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
523 |
int _pos; |
1 | 524 |
BasicType _return_type; |
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
525 |
u_char* _value_state; |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
526 |
intptr_t* _value; |
1 | 527 |
|
528 |
public: |
|
529 |
bool _is_return; |
|
530 |
||
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
531 |
SignatureChekker(Symbol* signature, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
532 |
BasicType return_type, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
533 |
bool is_static, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
534 |
u_char* value_state, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
535 |
intptr_t* value) : |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
536 |
SignatureIterator(signature), |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
537 |
_pos(0), |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
538 |
_return_type(return_type), |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
539 |
_value_state(value_state), |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
540 |
_value(value), |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
541 |
_is_return(false) |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
542 |
{ |
1 | 543 |
if (!is_static) { |
544 |
check_value(true); // Receiver must be an oop |
|
545 |
} |
|
546 |
} |
|
547 |
||
548 |
void check_value(bool type) { |
|
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
549 |
uint state = _value_state[_pos++]; |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
550 |
if (type) { |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
551 |
guarantee(is_value_state_indirect_oop(state), |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
552 |
"signature does not match pushed arguments: %u at %d", |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
553 |
state, _pos - 1); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
554 |
} else { |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
555 |
guarantee(state == JavaCallArguments::value_state_primitive, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
556 |
"signature does not match pushed arguments: %u at %d", |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
557 |
state, _pos - 1); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
558 |
} |
1 | 559 |
} |
560 |
||
561 |
void check_doing_return(bool state) { _is_return = state; } |
|
562 |
||
563 |
void check_return_type(BasicType t) { |
|
564 |
guarantee(_is_return && t == _return_type, "return type does not match"); |
|
565 |
} |
|
566 |
||
567 |
void check_int(BasicType t) { |
|
568 |
if (_is_return) { |
|
569 |
check_return_type(t); |
|
570 |
return; |
|
571 |
} |
|
572 |
check_value(false); |
|
573 |
} |
|
574 |
||
575 |
void check_double(BasicType t) { check_long(t); } |
|
576 |
||
577 |
void check_long(BasicType t) { |
|
578 |
if (_is_return) { |
|
579 |
check_return_type(t); |
|
580 |
return; |
|
581 |
} |
|
582 |
||
583 |
check_value(false); |
|
584 |
check_value(false); |
|
585 |
} |
|
586 |
||
587 |
void check_obj(BasicType t) { |
|
588 |
if (_is_return) { |
|
589 |
check_return_type(t); |
|
590 |
return; |
|
591 |
} |
|
1618
2d86b9b84aa5
6739363: Xcheck jni doesn't check native function arguments
poonam
parents:
1
diff
changeset
|
592 |
|
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
593 |
intptr_t v = _value[_pos]; |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
594 |
if (v != 0) { |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
595 |
// v is a "handle" referring to an oop, cast to integral type. |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
596 |
// There shouldn't be any handles in very low memory. |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
597 |
guarantee((size_t)v >= (size_t)os::vm_page_size(), |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
598 |
"Bad JNI oop argument %d: " PTR_FORMAT, _pos, v); |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
599 |
// Verify the pointee. |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
600 |
oop vv = resolve_indirect_oop(v, _value_state[_pos]); |
46968
9119841280f4
8160399: is_oop_or_null involves undefined behavior
coleenp
parents:
46727
diff
changeset
|
601 |
guarantee(oopDesc::is_oop_or_null(vv, true), |
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
602 |
"Bad JNI oop argument %d: " PTR_FORMAT " -> " PTR_FORMAT, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
603 |
_pos, v, p2i(vv)); |
1618
2d86b9b84aa5
6739363: Xcheck jni doesn't check native function arguments
poonam
parents:
1
diff
changeset
|
604 |
} |
2d86b9b84aa5
6739363: Xcheck jni doesn't check native function arguments
poonam
parents:
1
diff
changeset
|
605 |
|
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
606 |
check_value(true); // Verify value state. |
1 | 607 |
} |
608 |
||
609 |
void do_bool() { check_int(T_BOOLEAN); } |
|
610 |
void do_char() { check_int(T_CHAR); } |
|
611 |
void do_float() { check_int(T_FLOAT); } |
|
612 |
void do_double() { check_double(T_DOUBLE); } |
|
613 |
void do_byte() { check_int(T_BYTE); } |
|
614 |
void do_short() { check_int(T_SHORT); } |
|
615 |
void do_int() { check_int(T_INT); } |
|
616 |
void do_long() { check_long(T_LONG); } |
|
617 |
void do_void() { check_return_type(T_VOID); } |
|
618 |
void do_object(int begin, int end) { check_obj(T_OBJECT); } |
|
619 |
void do_array(int begin, int end) { check_obj(T_OBJECT); } |
|
620 |
}; |
|
621 |
||
1618
2d86b9b84aa5
6739363: Xcheck jni doesn't check native function arguments
poonam
parents:
1
diff
changeset
|
622 |
|
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
623 |
void JavaCallArguments::verify(const methodHandle& method, BasicType return_type) { |
1 | 624 |
guarantee(method->size_of_parameters() == size_of_parameters(), "wrong no. of arguments pushed"); |
625 |
||
626 |
// Treat T_OBJECT and T_ARRAY as the same |
|
627 |
if (return_type == T_ARRAY) return_type = T_OBJECT; |
|
628 |
||
629 |
// Check that oop information is correct |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
630 |
Symbol* signature = method->signature(); |
1 | 631 |
|
44406
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
632 |
SignatureChekker sc(signature, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
633 |
return_type, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
634 |
method->is_static(), |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
635 |
_value_state, |
a46a6c4d1dd9
8176100: [REDO][REDO] G1 Needs pre barrier on dereference of weak JNI handles
mgerdin
parents:
44093
diff
changeset
|
636 |
_value); |
1 | 637 |
sc.iterate_parameters(); |
638 |
sc.check_doing_return(true); |
|
639 |
sc.iterate_returntype(); |
|
640 |
} |