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