author | coleenp |
Tue, 08 Oct 2019 13:18:40 -0400 | |
changeset 58505 | c16f3a24a6fc |
parent 57603 | f9d9bed12d1a |
child 58679 | 9c3209ff7550 |
child 59056 | 15936b142f86 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
57603
f9d9bed12d1a
8228630: Remove always true parameter to NoSafepointVerifier
coleenp
parents:
54786
diff
changeset
|
2 |
* Copyright (c) 1997, 2019, 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:
1
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1
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:
1
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "classfile/systemDictionary.hpp" |
|
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
27 |
#include "code/codeBehaviours.hpp" |
7397 | 28 |
#include "code/codeCache.hpp" |
29 |
#include "code/compiledIC.hpp" |
|
30 |
#include "code/icBuffer.hpp" |
|
31 |
#include "code/nmethod.hpp" |
|
32 |
#include "code/vtableStubs.hpp" |
|
33 |
#include "interpreter/interpreter.hpp" |
|
34 |
#include "interpreter/linkResolver.hpp" |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
35 |
#include "memory/metadataFactory.hpp" |
7397 | 36 |
#include "memory/oopFactory.hpp" |
37248 | 37 |
#include "memory/resourceArea.hpp" |
54786 | 38 |
#include "memory/universe.hpp" |
49340
4e82736053ae
8191102: Incorrect include file use in classLoader.hpp
hseigel
parents:
48858
diff
changeset
|
39 |
#include "oops/method.inline.hpp" |
7397 | 40 |
#include "oops/oop.inline.hpp" |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7397
diff
changeset
|
41 |
#include "oops/symbol.hpp" |
49361
1956d0ec092a
8199319: Remove handles.inline.hpp include from reflectionUtils.hpp
stefank
parents:
49340
diff
changeset
|
42 |
#include "runtime/handles.inline.hpp" |
7397 | 43 |
#include "runtime/icache.hpp" |
44 |
#include "runtime/sharedRuntime.hpp" |
|
45 |
#include "runtime/stubRoutines.hpp" |
|
46 |
#include "utilities/events.hpp" |
|
1 | 47 |
|
48 |
||
49 |
// Every time a compiled IC is changed or its type is being accessed, |
|
50 |
// either the CompiledIC_lock must be set or we must be at a safe point. |
|
51 |
||
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
52 |
CompiledICLocker::CompiledICLocker(CompiledMethod* method) |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
53 |
: _method(method), |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
54 |
_behaviour(CompiledICProtectionBehaviour::current()), |
57603
f9d9bed12d1a
8228630: Remove always true parameter to NoSafepointVerifier
coleenp
parents:
54786
diff
changeset
|
55 |
_locked(_behaviour->lock(_method)) { |
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
56 |
} |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
57 |
|
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
58 |
CompiledICLocker::~CompiledICLocker() { |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
59 |
if (_locked) { |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
60 |
_behaviour->unlock(_method); |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
61 |
} |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
62 |
} |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
63 |
|
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
64 |
bool CompiledICLocker::is_safe(CompiledMethod* method) { |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
65 |
return CompiledICProtectionBehaviour::current()->is_safe(method); |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
66 |
} |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
67 |
|
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
68 |
bool CompiledICLocker::is_safe(address code) { |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
69 |
CodeBlob* cb = CodeCache::find_blob_unsafe(code); |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
70 |
assert(cb != NULL && cb->is_compiled(), "must be compiled"); |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
71 |
CompiledMethod* cm = cb->as_compiled_method(); |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
72 |
return CompiledICProtectionBehaviour::current()->is_safe(cm); |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
73 |
} |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
74 |
|
1 | 75 |
//----------------------------------------------------------------------------- |
76 |
// Low-level access to an inline cache. Private, since they might not be |
|
77 |
// MT-safe to use. |
|
78 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
79 |
void* CompiledIC::cached_value() const { |
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
80 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
81 |
assert (!is_optimized(), "an optimized virtual call does not have a cached metadata"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
82 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
83 |
if (!is_in_transition_state()) { |
42650 | 84 |
void* data = get_data(); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
85 |
// If we let the metadata value here be initialized to zero... |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
86 |
assert(data != NULL || Universe::non_oop_word() == NULL, |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
87 |
"no raw nulls in CompiledIC metadatas, because of patching races"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
88 |
return (data == (void*)Universe::non_oop_word()) ? NULL : data; |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
89 |
} else { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
90 |
return InlineCacheBuffer::cached_value_for((CompiledIC *)this); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
91 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
92 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
93 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
94 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
95 |
void CompiledIC::internal_set_ic_destination(address entry_point, bool is_icstub, void* cache, bool is_icholder) { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
96 |
assert(entry_point != NULL, "must set legal entry point"); |
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
97 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
98 |
assert (!is_optimized() || cache == NULL, "an optimized virtual call does not have a cached metadata"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
99 |
assert (cache == NULL || cache != (Metadata*)badOopVal, "invalid metadata"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
100 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
101 |
assert(!is_icholder || is_icholder_entry(entry_point), "must be"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
102 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
103 |
// Don't use ic_destination for this test since that forwards |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
104 |
// through ICBuffer instead of returning the actual current state of |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
105 |
// the CompiledIC. |
42650 | 106 |
if (is_icholder_entry(_call->destination())) { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
107 |
// When patching for the ICStub case the cached value isn't |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
108 |
// overwritten until the ICStub copied into the CompiledIC during |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
109 |
// the next safepoint. Make sure that the CompiledICHolder* is |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
110 |
// marked for release at this point since it won't be identifiable |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
111 |
// once the entry point is overwritten. |
42650 | 112 |
InlineCacheBuffer::queue_for_release((CompiledICHolder*)get_data()); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
113 |
} |
1 | 114 |
|
115 |
if (TraceCompiledIC) { |
|
116 |
tty->print(" "); |
|
117 |
print_compiled_ic(); |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22223
diff
changeset
|
118 |
tty->print(" changing destination to " INTPTR_FORMAT, p2i(entry_point)); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
119 |
if (!is_optimized()) { |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22223
diff
changeset
|
120 |
tty->print(" changing cached %s to " INTPTR_FORMAT, is_icholder ? "icholder" : "metadata", p2i((address)cache)); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
121 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
122 |
if (is_icstub) { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
123 |
tty->print(" (icstub)"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
124 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
125 |
tty->cr(); |
1 | 126 |
} |
127 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
128 |
{ |
42650 | 129 |
CodeBlob* cb = CodeCache::find_blob_unsafe(_call->instruction_address()); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
130 |
assert(cb != NULL && cb->is_compiled(), "must be compiled"); |
42650 | 131 |
_call->set_destination_mt_safe(entry_point); |
25492
d27050bdfb04
8049421: G1 Class Unloading after completing a concurrent mark cycle
stefank
parents:
25491
diff
changeset
|
132 |
} |
1 | 133 |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
134 |
if (is_optimized() || is_icstub) { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
135 |
// Optimized call sites don't have a cache value and ICStub call |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
136 |
// sites only change the entry point. Changing the value in that |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
137 |
// case could lead to MT safety issues. |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
138 |
assert(cache == NULL, "must be null"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
139 |
return; |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
140 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
141 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
142 |
if (cache == NULL) cache = (void*)Universe::non_oop_word(); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
143 |
|
42650 | 144 |
set_data((intptr_t)cache); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
145 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
146 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
147 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
148 |
void CompiledIC::set_ic_destination(ICStub* stub) { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
149 |
internal_set_ic_destination(stub->code_begin(), true, NULL, false); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
150 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
151 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
152 |
|
1 | 153 |
|
154 |
address CompiledIC::ic_destination() const { |
|
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
155 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
156 |
if (!is_in_transition_state()) { |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
157 |
return _call->destination(); |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
158 |
} else { |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
159 |
return InlineCacheBuffer::ic_destination_for((CompiledIC *)this); |
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
160 |
} |
1 | 161 |
} |
162 |
||
163 |
||
164 |
bool CompiledIC::is_in_transition_state() const { |
|
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
165 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
42650 | 166 |
return InlineCacheBuffer::contains(_call->destination());; |
1 | 167 |
} |
168 |
||
169 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
170 |
bool CompiledIC::is_icholder_call() const { |
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
171 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
172 |
return !_is_optimized && is_icholder_entry(ic_destination()); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
173 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
174 |
|
1 | 175 |
// Returns native address of 'call' instruction in inline-cache. Used by |
176 |
// the InlineCacheBuffer when it needs to find the stub. |
|
177 |
address CompiledIC::stub_address() const { |
|
178 |
assert(is_in_transition_state(), "should only be called when we are in a transition state"); |
|
42650 | 179 |
return _call->destination(); |
1 | 180 |
} |
181 |
||
26912
19021f626ad2
8058737: CodeCache::find_blob fails with 'unsafe access to zombie method'
thartmann
parents:
26580
diff
changeset
|
182 |
// Clears the IC stub if the compiled IC is in transition state |
19021f626ad2
8058737: CodeCache::find_blob fails with 'unsafe access to zombie method'
thartmann
parents:
26580
diff
changeset
|
183 |
void CompiledIC::clear_ic_stub() { |
19021f626ad2
8058737: CodeCache::find_blob fails with 'unsafe access to zombie method'
thartmann
parents:
26580
diff
changeset
|
184 |
if (is_in_transition_state()) { |
19021f626ad2
8058737: CodeCache::find_blob fails with 'unsafe access to zombie method'
thartmann
parents:
26580
diff
changeset
|
185 |
ICStub* stub = ICStub_from_destination_address(stub_address()); |
19021f626ad2
8058737: CodeCache::find_blob fails with 'unsafe access to zombie method'
thartmann
parents:
26580
diff
changeset
|
186 |
stub->clear(); |
19021f626ad2
8058737: CodeCache::find_blob fails with 'unsafe access to zombie method'
thartmann
parents:
26580
diff
changeset
|
187 |
} |
19021f626ad2
8058737: CodeCache::find_blob fails with 'unsafe access to zombie method'
thartmann
parents:
26580
diff
changeset
|
188 |
} |
19021f626ad2
8058737: CodeCache::find_blob fails with 'unsafe access to zombie method'
thartmann
parents:
26580
diff
changeset
|
189 |
|
1 | 190 |
//----------------------------------------------------------------------------- |
191 |
// High-level access to an inline cache. Guaranteed to be MT-safe. |
|
192 |
||
25486
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
193 |
void CompiledIC::initialize_from_iter(RelocIterator* iter) { |
42650 | 194 |
assert(iter->addr() == _call->instruction_address(), "must find ic_call"); |
25486
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
195 |
|
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
196 |
if (iter->type() == relocInfo::virtual_call_type) { |
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
197 |
virtual_call_Relocation* r = iter->virtual_call_reloc(); |
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
198 |
_is_optimized = false; |
42650 | 199 |
_value = _call->get_load_instruction(r); |
25486
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
200 |
} else { |
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
201 |
assert(iter->type() == relocInfo::opt_virtual_call_type, "must be a virtual call"); |
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
202 |
_is_optimized = true; |
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
203 |
_value = NULL; |
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
204 |
} |
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
205 |
} |
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
206 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
207 |
CompiledIC::CompiledIC(CompiledMethod* cm, NativeCall* call) |
42650 | 208 |
: _method(cm) |
25359
8a19ff205f89
8047326: Consolidate all CompiledIC::CompiledIC implementations and move it to compiledIC.cpp
stefank
parents:
24424
diff
changeset
|
209 |
{ |
42650 | 210 |
_call = _method->call_wrapper_at((address) call); |
211 |
address ic_call = _call->instruction_address(); |
|
25359
8a19ff205f89
8047326: Consolidate all CompiledIC::CompiledIC implementations and move it to compiledIC.cpp
stefank
parents:
24424
diff
changeset
|
212 |
|
8a19ff205f89
8047326: Consolidate all CompiledIC::CompiledIC implementations and move it to compiledIC.cpp
stefank
parents:
24424
diff
changeset
|
213 |
assert(ic_call != NULL, "ic_call address must be set"); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
214 |
assert(cm != NULL, "must pass compiled method"); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
215 |
assert(cm->contains(ic_call), "must be in compiled method"); |
25359
8a19ff205f89
8047326: Consolidate all CompiledIC::CompiledIC implementations and move it to compiledIC.cpp
stefank
parents:
24424
diff
changeset
|
216 |
|
8a19ff205f89
8047326: Consolidate all CompiledIC::CompiledIC implementations and move it to compiledIC.cpp
stefank
parents:
24424
diff
changeset
|
217 |
// Search for the ic_call at the given address. |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
218 |
RelocIterator iter(cm, ic_call, ic_call+1); |
25359
8a19ff205f89
8047326: Consolidate all CompiledIC::CompiledIC implementations and move it to compiledIC.cpp
stefank
parents:
24424
diff
changeset
|
219 |
bool ret = iter.next(); |
8a19ff205f89
8047326: Consolidate all CompiledIC::CompiledIC implementations and move it to compiledIC.cpp
stefank
parents:
24424
diff
changeset
|
220 |
assert(ret == true, "relocInfo must exist at this address"); |
8a19ff205f89
8047326: Consolidate all CompiledIC::CompiledIC implementations and move it to compiledIC.cpp
stefank
parents:
24424
diff
changeset
|
221 |
assert(iter.addr() == ic_call, "must find ic_call"); |
25486
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
222 |
|
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
223 |
initialize_from_iter(&iter); |
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
224 |
} |
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
225 |
|
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
226 |
CompiledIC::CompiledIC(RelocIterator* iter) |
42650 | 227 |
: _method(iter->code()) |
25486
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
228 |
{ |
42650 | 229 |
_call = _method->call_wrapper_at(iter->addr()); |
230 |
address ic_call = _call->instruction_address(); |
|
25486
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
231 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
232 |
CompiledMethod* nm = iter->code(); |
25486
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
233 |
assert(ic_call != NULL, "ic_call address must be set"); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
234 |
assert(nm != NULL, "must pass compiled method"); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
235 |
assert(nm->contains(ic_call), "must be in compiled method"); |
25486
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
236 |
|
be44827be5e7
8047362: Add a version of CompiledIC_at that doesn't create a new RelocIterator
stefank
parents:
25359
diff
changeset
|
237 |
initialize_from_iter(iter); |
25359
8a19ff205f89
8047326: Consolidate all CompiledIC::CompiledIC implementations and move it to compiledIC.cpp
stefank
parents:
24424
diff
changeset
|
238 |
} |
1 | 239 |
|
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
240 |
// This function may fail for two reasons: either due to running out of vtable |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
241 |
// stubs, or due to running out of IC stubs in an attempted transition to a |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
242 |
// transitional state. The needs_ic_stub_refill value will be set if the failure |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
243 |
// was due to running out of IC stubs, in which case the caller will refill IC |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
244 |
// stubs and retry. |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
245 |
bool CompiledIC::set_to_megamorphic(CallInfo* call_info, Bytecodes::Code bytecode, |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
246 |
bool& needs_ic_stub_refill, TRAPS) { |
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
247 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
1 | 248 |
assert(!is_optimized(), "cannot set an optimized virtual call to megamorphic"); |
249 |
assert(is_call_to_compiled() || is_call_to_interpreted(), "going directly to megamorphic?"); |
|
250 |
||
251 |
address entry; |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
252 |
if (call_info->call_kind() == CallInfo::itable_call) { |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
253 |
assert(bytecode == Bytecodes::_invokeinterface, ""); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
254 |
int itable_index = call_info->itable_index(); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
255 |
entry = VtableStubs::find_itable_stub(itable_index); |
48824 | 256 |
if (entry == NULL) { |
20072 | 257 |
return false; |
258 |
} |
|
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
259 |
#ifdef ASSERT |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
260 |
int index = call_info->resolved_method()->itable_index(); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
261 |
assert(index == itable_index, "CallInfo pre-computes this"); |
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
13728
diff
changeset
|
262 |
InstanceKlass* k = call_info->resolved_method()->method_holder(); |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
263 |
assert(k->verify_itable_index(itable_index), "sanity check"); |
48557 | 264 |
#endif //ASSERT |
265 |
CompiledICHolder* holder = new CompiledICHolder(call_info->resolved_method()->method_holder(), |
|
49655
d6893a76c554
8199406: Performance drop with Java JDK 1.8.0_162-b32
poonam
parents:
49361
diff
changeset
|
266 |
call_info->resolved_klass(), false); |
48557 | 267 |
holder->claim(); |
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
268 |
if (!InlineCacheBuffer::create_transition_stub(this, holder, entry)) { |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
269 |
delete holder; |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
270 |
needs_ic_stub_refill = true; |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
271 |
return false; |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
272 |
} |
1 | 273 |
} else { |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
274 |
assert(call_info->call_kind() == CallInfo::vtable_call, "either itable or vtable"); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
275 |
// Can be different than selected_method->vtable_index(), due to package-private etc. |
1 | 276 |
int vtable_index = call_info->vtable_index(); |
20017
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
277 |
assert(call_info->resolved_klass()->verify_vtable_index(vtable_index), "sanity check"); |
81eba62e9048
8014013: CallInfo structure no longer accurately reports the result of a LinkResolver operation
drchase
parents:
17094
diff
changeset
|
278 |
entry = VtableStubs::find_vtable_stub(vtable_index); |
20072 | 279 |
if (entry == NULL) { |
280 |
return false; |
|
281 |
} |
|
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
282 |
if (!InlineCacheBuffer::create_transition_stub(this, NULL, entry)) { |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
283 |
needs_ic_stub_refill = true; |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
284 |
return false; |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
285 |
} |
1 | 286 |
} |
287 |
||
288 |
if (TraceICs) { |
|
289 |
ResourceMark rm; |
|
51314
82adcc8ad853
8208604: Metadata::print_value_string() compares 'this' to NULL
hseigel
parents:
50416
diff
changeset
|
290 |
assert(!call_info->selected_method().is_null(), "Unexpected null selected method"); |
1 | 291 |
tty->print_cr ("IC@" INTPTR_FORMAT ": to megamorphic %s entry: " INTPTR_FORMAT, |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22223
diff
changeset
|
292 |
p2i(instruction_address()), call_info->selected_method()->print_value_string(), p2i(entry)); |
1 | 293 |
} |
294 |
||
295 |
// We can't check this anymore. With lazy deopt we could have already |
|
296 |
// cleaned this IC entry before we even return. This is possible if |
|
297 |
// we ran out of space in the inline cache buffer trying to do the |
|
298 |
// set_next and we safepointed to free up space. This is a benign |
|
299 |
// race because the IC entry was complete when we safepointed so |
|
300 |
// cleaning it immediately is harmless. |
|
301 |
// assert(is_megamorphic(), "sanity check"); |
|
20072 | 302 |
return true; |
1 | 303 |
} |
304 |
||
305 |
||
306 |
// true if destination is megamorphic stub |
|
307 |
bool CompiledIC::is_megamorphic() const { |
|
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
308 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
1 | 309 |
assert(!is_optimized(), "an optimized call cannot be megamorphic"); |
310 |
||
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
311 |
// Cannot rely on cached_value. It is either an interface or a method. |
49655
d6893a76c554
8199406: Performance drop with Java JDK 1.8.0_162-b32
poonam
parents:
49361
diff
changeset
|
312 |
return VtableStubs::entry_point(ic_destination()) != NULL; |
1 | 313 |
} |
314 |
||
315 |
bool CompiledIC::is_call_to_compiled() const { |
|
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
316 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
1 | 317 |
|
318 |
// Use unsafe, since an inline cache might point to a zombie method. However, the zombie |
|
319 |
// method is guaranteed to still exist, since we only remove methods after all inline caches |
|
320 |
// has been cleaned up |
|
321 |
CodeBlob* cb = CodeCache::find_blob_unsafe(ic_destination()); |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
322 |
bool is_monomorphic = (cb != NULL && cb->is_compiled()); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
323 |
// Check that the cached_value is a klass for non-optimized monomorphic calls |
1 | 324 |
// This assertion is invalid for compiler1: a call that does not look optimized (no static stub) can be used |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
32466
diff
changeset
|
325 |
// for calling directly to vep without using the inline cache (i.e., cached_value == NULL). |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
32466
diff
changeset
|
326 |
// For JVMCI this occurs because CHA is only used to improve inlining so call sites which could be optimized |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
32466
diff
changeset
|
327 |
// virtuals because there are no currently loaded subclasses of a type are left as virtual call sites. |
1 | 328 |
#ifdef ASSERT |
329 |
CodeBlob* caller = CodeCache::find_blob_unsafe(instruction_address()); |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
32466
diff
changeset
|
330 |
bool is_c1_or_jvmci_method = caller->is_compiled_by_c1() || caller->is_compiled_by_jvmci(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
32466
diff
changeset
|
331 |
assert( is_c1_or_jvmci_method || |
1 | 332 |
!is_monomorphic || |
333 |
is_optimized() || |
|
32466
915d0c3d25aa
8134493: Cleaning inline caches of unloaded nmethods should be done in sweeper
thartmann
parents:
32401
diff
changeset
|
334 |
!caller->is_alive() || |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
335 |
(cached_metadata() != NULL && cached_metadata()->is_klass()), "sanity check"); |
1 | 336 |
#endif // ASSERT |
337 |
return is_monomorphic; |
|
338 |
} |
|
339 |
||
340 |
||
341 |
bool CompiledIC::is_call_to_interpreted() const { |
|
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
342 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
1 | 343 |
// Call to interpreter if destination is either calling to a stub (if it |
344 |
// is optimized), or calling to an I2C blob |
|
345 |
bool is_call_to_interpreted = false; |
|
346 |
if (!is_optimized()) { |
|
347 |
// must use unsafe because the destination can be a zombie (and we're cleaning) |
|
348 |
// and the print_compiled_ic code wants to know if site (in the non-zombie) |
|
349 |
// is to the interpreter. |
|
350 |
CodeBlob* cb = CodeCache::find_blob_unsafe(ic_destination()); |
|
351 |
is_call_to_interpreted = (cb != NULL && cb->is_adapter_blob()); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
352 |
assert(!is_call_to_interpreted || (is_icholder_call() && cached_icholder() != NULL), "sanity check"); |
1 | 353 |
} else { |
354 |
// Check if we are calling into our own codeblob (i.e., to a stub) |
|
355 |
address dest = ic_destination(); |
|
356 |
#ifdef ASSERT |
|
357 |
{ |
|
42650 | 358 |
_call->verify_resolve_call(dest); |
1 | 359 |
} |
360 |
#endif /* ASSERT */ |
|
42650 | 361 |
is_call_to_interpreted = _call->is_call_to_interpreted(dest); |
1 | 362 |
} |
363 |
return is_call_to_interpreted; |
|
364 |
} |
|
365 |
||
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
366 |
bool CompiledIC::set_to_clean(bool in_use) { |
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
367 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
1 | 368 |
if (TraceInlineCacheClearing || TraceICs) { |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22223
diff
changeset
|
369 |
tty->print_cr("IC@" INTPTR_FORMAT ": set to clean", p2i(instruction_address())); |
1 | 370 |
print(); |
371 |
} |
|
372 |
||
42650 | 373 |
address entry = _call->get_resolve_call_stub(is_optimized()); |
1 | 374 |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
375 |
// A zombie transition will always be safe, since the metadata has already been set to NULL, so |
1 | 376 |
// we only need to patch the destination |
52858
dad45affbdaa
8214257: IC cache not clean after cleaning assertion failure
eosterlund
parents:
52857
diff
changeset
|
377 |
bool safe_transition = _call->is_safe_for_patching() || !in_use || is_optimized() || SafepointSynchronize::is_at_safepoint(); |
1 | 378 |
|
379 |
if (safe_transition) { |
|
380 |
// Kill any leftover stub we might have too |
|
26912
19021f626ad2
8058737: CodeCache::find_blob fails with 'unsafe access to zombie method'
thartmann
parents:
26580
diff
changeset
|
381 |
clear_ic_stub(); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
382 |
if (is_optimized()) { |
32401
cc58aeaec340
8075805: Crash while trying to release CompiledICHolder
thartmann
parents:
26912
diff
changeset
|
383 |
set_ic_destination(entry); |
cc58aeaec340
8075805: Crash while trying to release CompiledICHolder
thartmann
parents:
26912
diff
changeset
|
384 |
} else { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
385 |
set_ic_destination_and_value(entry, (void*)NULL); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
386 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
387 |
} else { |
1 | 388 |
// Unsafe transition - create stub. |
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
389 |
if (!InlineCacheBuffer::create_transition_stub(this, NULL, entry)) { |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
390 |
return false; |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
391 |
} |
1 | 392 |
} |
393 |
// We can't check this anymore. With lazy deopt we could have already |
|
394 |
// cleaned this IC entry before we even return. This is possible if |
|
395 |
// we ran out of space in the inline cache buffer trying to do the |
|
396 |
// set_next and we safepointed to free up space. This is a benign |
|
397 |
// race because the IC entry was complete when we safepointed so |
|
398 |
// cleaning it immediately is harmless. |
|
399 |
// assert(is_clean(), "sanity check"); |
|
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
400 |
return true; |
1 | 401 |
} |
402 |
||
403 |
bool CompiledIC::is_clean() const { |
|
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
404 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
1 | 405 |
bool is_clean = false; |
406 |
address dest = ic_destination(); |
|
42650 | 407 |
is_clean = dest == _call->get_resolve_call_stub(is_optimized()); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
408 |
assert(!is_clean || is_optimized() || cached_value() == NULL, "sanity check"); |
1 | 409 |
return is_clean; |
410 |
} |
|
411 |
||
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
412 |
bool CompiledIC::set_to_monomorphic(CompiledICInfo& info) { |
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
413 |
assert(CompiledICLocker::is_safe(_method), "mt unsafe call"); |
1 | 414 |
// Updating a cache to the wrong entry can cause bugs that are very hard |
415 |
// to track down - if cache entry gets invalid - we just clean it. In |
|
416 |
// this way it is always the same code path that is responsible for |
|
417 |
// updating and resolving an inline cache |
|
418 |
// |
|
419 |
// The above is no longer true. SharedRuntime::fixup_callers_callsite will change optimized |
|
420 |
// callsites. In addition ic_miss code will update a site to monomorphic if it determines |
|
421 |
// that an monomorphic call to the interpreter can now be monomorphic to compiled code. |
|
422 |
// |
|
423 |
// In both of these cases the only thing being modifed is the jump/call target and these |
|
424 |
// transitions are mt_safe |
|
425 |
||
426 |
Thread *thread = Thread::current(); |
|
42650 | 427 |
if (info.to_interpreter() || info.to_aot()) { |
1 | 428 |
// Call to interpreter |
429 |
if (info.is_optimized() && is_optimized()) { |
|
52858
dad45affbdaa
8214257: IC cache not clean after cleaning assertion failure
eosterlund
parents:
52857
diff
changeset
|
430 |
assert(is_clean(), "unsafe IC path"); |
1 | 431 |
// the call analysis (callee structure) specifies that the call is optimized |
432 |
// (either because of CHA or the static target is final) |
|
433 |
// At code generation time, this call has been emitted as static call |
|
434 |
// Call via stub |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
435 |
assert(info.cached_metadata() != NULL && info.cached_metadata()->is_method(), "sanity check"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
436 |
methodHandle method (thread, (Method*)info.cached_metadata()); |
42650 | 437 |
_call->set_to_interpreted(method, info); |
438 |
||
1 | 439 |
if (TraceICs) { |
440 |
ResourceMark rm(thread); |
|
42650 | 441 |
tty->print_cr ("IC@" INTPTR_FORMAT ": monomorphic to %s: %s", |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22223
diff
changeset
|
442 |
p2i(instruction_address()), |
42650 | 443 |
(info.to_aot() ? "aot" : "interpreter"), |
1 | 444 |
method->print_value_string()); |
445 |
} |
|
446 |
} else { |
|
447 |
// Call via method-klass-holder |
|
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
448 |
CompiledICHolder* holder = info.claim_cached_icholder(); |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
449 |
if (!InlineCacheBuffer::create_transition_stub(this, holder, info.entry())) { |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
450 |
delete holder; |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
451 |
return false; |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
452 |
} |
1 | 453 |
if (TraceICs) { |
454 |
ResourceMark rm(thread); |
|
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22223
diff
changeset
|
455 |
tty->print_cr ("IC@" INTPTR_FORMAT ": monomorphic to interpreter via icholder ", p2i(instruction_address())); |
1 | 456 |
} |
457 |
} |
|
458 |
} else { |
|
459 |
// Call to compiled code |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
460 |
bool static_bound = info.is_optimized() || (info.cached_metadata() == NULL); |
1 | 461 |
#ifdef ASSERT |
462 |
CodeBlob* cb = CodeCache::find_blob_unsafe(info.entry()); |
|
48858
e50e326a2bfc
8194990: 3 Null pointer dereference defect groups related to CodeCache::find_blob_unsafe()
dlong
parents:
48824
diff
changeset
|
463 |
assert (cb != NULL && cb->is_compiled(), "must be compiled!"); |
1 | 464 |
#endif /* ASSERT */ |
465 |
||
466 |
// This is MT safe if we come from a clean-cache and go through a |
|
467 |
// non-verified entry point |
|
468 |
bool safe = SafepointSynchronize::is_at_safepoint() || |
|
469 |
(!is_in_transition_state() && (info.is_optimized() || static_bound || is_clean())); |
|
470 |
||
471 |
if (!safe) { |
|
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
472 |
if (!InlineCacheBuffer::create_transition_stub(this, info.cached_metadata(), info.entry())) { |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
473 |
return false; |
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
474 |
} |
1 | 475 |
} else { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
476 |
if (is_optimized()) { |
35086
bbf32241d851
8072008: Emit direct call instead of linkTo* for recursive indy/MH.invoke* calls
vlivanov
parents:
33593
diff
changeset
|
477 |
set_ic_destination(info.entry()); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
478 |
} else { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
479 |
set_ic_destination_and_value(info.entry(), info.cached_metadata()); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
480 |
} |
1 | 481 |
} |
482 |
||
483 |
if (TraceICs) { |
|
484 |
ResourceMark rm(thread); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
485 |
assert(info.cached_metadata() == NULL || info.cached_metadata()->is_klass(), "must be"); |
53415 | 486 |
tty->print_cr ("IC@" INTPTR_FORMAT ": monomorphic to compiled (rcvr klass = %s) %s", |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22223
diff
changeset
|
487 |
p2i(instruction_address()), |
53415 | 488 |
(info.cached_metadata() != NULL) ? ((Klass*)info.cached_metadata())->print_value_string() : "NULL", |
489 |
(safe) ? "" : " via stub"); |
|
1 | 490 |
} |
491 |
} |
|
492 |
// We can't check this anymore. With lazy deopt we could have already |
|
493 |
// cleaned this IC entry before we even return. This is possible if |
|
494 |
// we ran out of space in the inline cache buffer trying to do the |
|
495 |
// set_next and we safepointed to free up space. This is a benign |
|
496 |
// race because the IC entry was complete when we safepointed so |
|
497 |
// cleaning it immediately is harmless. |
|
498 |
// assert(is_call_to_compiled() || is_call_to_interpreted(), "sanity check"); |
|
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
499 |
return true; |
1 | 500 |
} |
501 |
||
502 |
||
42544
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
503 |
// is_optimized: Compiler has generated an optimized call (i.e. fixed, no inline cache) |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
504 |
// static_bound: The call can be static bound. If it isn't also optimized, the property |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
505 |
// wasn't provable at time of compilation. An optimized call will have any necessary |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
506 |
// null check, while a static_bound won't. A static_bound (but not optimized) must |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
507 |
// therefore use the unverified entry point. |
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33160
diff
changeset
|
508 |
void CompiledIC::compute_monomorphic_entry(const methodHandle& method, |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
42650
diff
changeset
|
509 |
Klass* receiver_klass, |
1 | 510 |
bool is_optimized, |
511 |
bool static_bound, |
|
42650 | 512 |
bool caller_is_nmethod, |
1 | 513 |
CompiledICInfo& info, |
514 |
TRAPS) { |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
515 |
CompiledMethod* method_code = method->code(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
516 |
|
1 | 517 |
address entry = NULL; |
22223 | 518 |
if (method_code != NULL && method_code->is_in_use()) { |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
519 |
assert(method_code->is_compiled(), "must be compiled"); |
1 | 520 |
// Call to compiled code |
42544
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
521 |
// |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
522 |
// Note: the following problem exists with Compiler1: |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
523 |
// - at compile time we may or may not know if the destination is final |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
524 |
// - if we know that the destination is final (is_optimized), we will emit |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
525 |
// an optimized virtual call (no inline cache), and need a Method* to make |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
526 |
// a call to the interpreter |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
527 |
// - if we don't know if the destination is final, we emit a standard |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
528 |
// virtual call, and use CompiledICHolder to call interpreted code |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
529 |
// (no static call stub has been generated) |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
530 |
// - In the case that we here notice the call is static bound we |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
531 |
// convert the call into what looks to be an optimized virtual call, |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
532 |
// but we must use the unverified entry point (since there will be no |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
533 |
// null check on a call when the target isn't loaded). |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
534 |
// This causes problems when verifying the IC because |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
535 |
// it looks vanilla but is optimized. Code in is_call_to_interpreted |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
536 |
// is aware of this and weakens its asserts. |
58de8aaf9365
8160543: C1: Crash in java.lang.String.indexOf in some java.sql tests
neliasso
parents:
38133
diff
changeset
|
537 |
if (is_optimized) { |
1 | 538 |
entry = method_code->verified_entry_point(); |
539 |
} else { |
|
540 |
entry = method_code->entry_point(); |
|
541 |
} |
|
542 |
} |
|
42650 | 543 |
bool far_c2a = entry != NULL && caller_is_nmethod && method_code->is_far_code(); |
544 |
if (entry != NULL && !far_c2a) { |
|
545 |
// Call to near compiled code (nmethod or aot). |
|
48001
600868bd3a8c
8169766: c1 + Xcomp reresolving call target every invocation
neliasso
parents:
47216
diff
changeset
|
546 |
info.set_compiled_entry(entry, is_optimized ? NULL : receiver_klass, is_optimized); |
1 | 547 |
} else { |
548 |
if (is_optimized) { |
|
42650 | 549 |
if (far_c2a) { |
550 |
// Call to aot code from nmethod. |
|
551 |
info.set_aot_entry(entry, method()); |
|
552 |
} else { |
|
553 |
// Use stub entry |
|
554 |
info.set_interpreter_entry(method()->get_c2i_entry(), method()); |
|
555 |
} |
|
1 | 556 |
} else { |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
557 |
// Use icholder entry |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
558 |
assert(method_code == NULL || method_code->is_compiled(), "must be compiled"); |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
42650
diff
changeset
|
559 |
CompiledICHolder* holder = new CompiledICHolder(method(), receiver_klass); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
560 |
info.set_icholder_entry(method()->get_c2i_unverified_entry(), holder); |
1 | 561 |
} |
562 |
} |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
563 |
assert(info.is_optimized() == is_optimized, "must agree"); |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
564 |
} |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
565 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
566 |
|
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
567 |
bool CompiledIC::is_icholder_entry(address entry) { |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
568 |
CodeBlob* cb = CodeCache::find_blob_unsafe(entry); |
48557 | 569 |
if (cb != NULL && cb->is_adapter_blob()) { |
570 |
return true; |
|
571 |
} |
|
572 |
// itable stubs also use CompiledICHolder |
|
49655
d6893a76c554
8199406: Performance drop with Java JDK 1.8.0_162-b32
poonam
parents:
49361
diff
changeset
|
573 |
if (cb != NULL && cb->is_vtable_blob()) { |
d6893a76c554
8199406: Performance drop with Java JDK 1.8.0_162-b32
poonam
parents:
49361
diff
changeset
|
574 |
VtableStub* s = VtableStubs::entry_point(entry); |
d6893a76c554
8199406: Performance drop with Java JDK 1.8.0_162-b32
poonam
parents:
49361
diff
changeset
|
575 |
return (s != NULL) && s->is_itable_stub(); |
48557 | 576 |
} |
49655
d6893a76c554
8199406: Performance drop with Java JDK 1.8.0_162-b32
poonam
parents:
49361
diff
changeset
|
577 |
|
48557 | 578 |
return false; |
1 | 579 |
} |
580 |
||
42650 | 581 |
bool CompiledIC::is_icholder_call_site(virtual_call_Relocation* call_site, const CompiledMethod* cm) { |
582 |
// This call site might have become stale so inspect it carefully. |
|
583 |
address dest = cm->call_wrapper_at(call_site->addr())->destination(); |
|
584 |
return is_icholder_entry(dest); |
|
585 |
} |
|
586 |
||
1 | 587 |
// ---------------------------------------------------------------------------- |
588 |
||
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
589 |
bool CompiledStaticCall::set_to_clean(bool in_use) { |
50416
ef980b9ac191
8203837: Split nmethod unloading from inline cache cleaning
coleenp
parents:
49655
diff
changeset
|
590 |
// in_use is unused but needed to match template function in CompiledMethod |
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
591 |
assert(CompiledICLocker::is_safe(instruction_address()), "mt unsafe call"); |
1 | 592 |
// Reset call site |
42650 | 593 |
set_destination_mt_safe(resolve_call_stub()); |
1 | 594 |
|
595 |
// Do not reset stub here: It is too expensive to call find_stub. |
|
596 |
// Instead, rely on caller (nmethod::clear_inline_caches) to clear |
|
597 |
// both the call and its stub. |
|
52857
7e268f863ff0
8214338: Move IC stub refilling out of IC cache transitions
eosterlund
parents:
52384
diff
changeset
|
598 |
return true; |
1 | 599 |
} |
600 |
||
601 |
bool CompiledStaticCall::is_clean() const { |
|
42650 | 602 |
return destination() == resolve_call_stub(); |
1 | 603 |
} |
604 |
||
605 |
bool CompiledStaticCall::is_call_to_compiled() const { |
|
606 |
return CodeCache::contains(destination()); |
|
607 |
} |
|
608 |
||
42650 | 609 |
bool CompiledDirectStaticCall::is_call_to_interpreted() const { |
1 | 610 |
// It is a call to interpreted, if it calls to a stub. Hence, the destination |
611 |
// must be in the stub part of the nmethod that contains the call |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
612 |
CompiledMethod* cm = CodeCache::find_compiled(instruction_address()); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
613 |
return cm->stub_contains(destination()); |
1 | 614 |
} |
615 |
||
42650 | 616 |
bool CompiledDirectStaticCall::is_call_to_far() const { |
617 |
// It is a call to aot method, if it calls to a stub. Hence, the destination |
|
618 |
// must be in the stub part of the nmethod that contains the call |
|
619 |
CodeBlob* desc = CodeCache::find_blob(instruction_address()); |
|
620 |
return desc->as_compiled_method()->stub_contains(destination()); |
|
621 |
} |
|
622 |
||
623 |
void CompiledStaticCall::set_to_compiled(address entry) { |
|
624 |
if (TraceICs) { |
|
625 |
ResourceMark rm; |
|
626 |
tty->print_cr("%s@" INTPTR_FORMAT ": set_to_compiled " INTPTR_FORMAT, |
|
627 |
name(), |
|
628 |
p2i(instruction_address()), |
|
629 |
p2i(entry)); |
|
630 |
} |
|
631 |
// Call to compiled code |
|
632 |
assert(CodeCache::contains(entry), "wrong entry point"); |
|
633 |
set_destination_mt_safe(entry); |
|
634 |
} |
|
635 |
||
1 | 636 |
void CompiledStaticCall::set(const StaticCallInfo& info) { |
52384
d6dc479bcdd3
8212681: Refactor IC locking to use a fine grained CompiledICLocker
eosterlund
parents:
51314
diff
changeset
|
637 |
assert(CompiledICLocker::is_safe(instruction_address()), "mt unsafe call"); |
1 | 638 |
// Updating a cache to the wrong entry can cause bugs that are very hard |
639 |
// to track down - if cache entry gets invalid - we just clean it. In |
|
640 |
// this way it is always the same code path that is responsible for |
|
641 |
// updating and resolving an inline cache |
|
642 |
assert(is_clean(), "do not update a call entry - use clean"); |
|
643 |
||
644 |
if (info._to_interpreter) { |
|
645 |
// Call to interpreted code |
|
646 |
set_to_interpreted(info.callee(), info.entry()); |
|
42650 | 647 |
#if INCLUDE_AOT |
648 |
} else if (info._to_aot) { |
|
649 |
// Call to far code |
|
650 |
set_to_far(info.callee(), info.entry()); |
|
651 |
#endif |
|
1 | 652 |
} else { |
42650 | 653 |
set_to_compiled(info.entry()); |
1 | 654 |
} |
655 |
} |
|
656 |
||
657 |
// Compute settings for a CompiledStaticCall. Since we might have to set |
|
658 |
// the stub when calling to the interpreter, we need to return arguments. |
|
42650 | 659 |
void CompiledStaticCall::compute_entry(const methodHandle& m, bool caller_is_nmethod, StaticCallInfo& info) { |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
38035
diff
changeset
|
660 |
CompiledMethod* m_code = m->code(); |
1 | 661 |
info._callee = m; |
22223 | 662 |
if (m_code != NULL && m_code->is_in_use()) { |
42650 | 663 |
if (caller_is_nmethod && m_code->is_far_code()) { |
664 |
// Call to far aot code from nmethod. |
|
665 |
info._to_aot = true; |
|
666 |
} else { |
|
667 |
info._to_aot = false; |
|
668 |
} |
|
1 | 669 |
info._to_interpreter = false; |
670 |
info._entry = m_code->verified_entry_point(); |
|
671 |
} else { |
|
672 |
// Callee is interpreted code. In any case entering the interpreter |
|
673 |
// puts a converter-frame on the stack to save arguments. |
|
26580
8cca7fbe77b8
8056154: JVM crash with EXCEPTION_ACCESS_VIOLATION when there are many threads running
iveresov
parents:
25492
diff
changeset
|
674 |
assert(!m->is_method_handle_intrinsic(), "Compiled code should never call interpreter MH intrinsics"); |
1 | 675 |
info._to_interpreter = true; |
676 |
info._entry = m()->get_c2i_entry(); |
|
677 |
} |
|
678 |
} |
|
679 |
||
42650 | 680 |
address CompiledDirectStaticCall::find_stub_for(address instruction, bool is_aot) { |
1 | 681 |
// Find reloc. information containing this call-site |
42650 | 682 |
RelocIterator iter((nmethod*)NULL, instruction); |
1 | 683 |
while (iter.next()) { |
42650 | 684 |
if (iter.addr() == instruction) { |
1 | 685 |
switch(iter.type()) { |
686 |
case relocInfo::static_call_type: |
|
42650 | 687 |
return iter.static_call_reloc()->static_stub(is_aot); |
1 | 688 |
// We check here for opt_virtual_call_type, since we reuse the code |
689 |
// from the CompiledIC implementation |
|
690 |
case relocInfo::opt_virtual_call_type: |
|
42650 | 691 |
return iter.opt_virtual_call_reloc()->static_stub(is_aot); |
1 | 692 |
case relocInfo::poll_type: |
693 |
case relocInfo::poll_return_type: // A safepoint can't overlap a call. |
|
694 |
default: |
|
695 |
ShouldNotReachHere(); |
|
696 |
} |
|
697 |
} |
|
698 |
} |
|
699 |
return NULL; |
|
700 |
} |
|
701 |
||
42650 | 702 |
address CompiledDirectStaticCall::find_stub(bool is_aot) { |
703 |
return CompiledDirectStaticCall::find_stub_for(instruction_address(), is_aot); |
|
704 |
} |
|
705 |
||
706 |
address CompiledDirectStaticCall::resolve_call_stub() const { |
|
707 |
return SharedRuntime::get_resolve_static_call_stub(); |
|
708 |
} |
|
1 | 709 |
|
710 |
//----------------------------------------------------------------------------- |
|
711 |
// Non-product mode code |
|
712 |
#ifndef PRODUCT |
|
713 |
||
714 |
void CompiledIC::verify() { |
|
42650 | 715 |
_call->verify(); |
1 | 716 |
assert(is_clean() || is_call_to_compiled() || is_call_to_interpreted() |
717 |
|| is_optimized() || is_megamorphic(), "sanity check"); |
|
718 |
} |
|
719 |
||
720 |
void CompiledIC::print() { |
|
721 |
print_compiled_ic(); |
|
722 |
tty->cr(); |
|
723 |
} |
|
724 |
||
725 |
void CompiledIC::print_compiled_ic() { |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
11636
diff
changeset
|
726 |
tty->print("Inline cache at " INTPTR_FORMAT ", calling %s " INTPTR_FORMAT " cached_value " INTPTR_FORMAT, |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22223
diff
changeset
|
727 |
p2i(instruction_address()), is_call_to_interpreted() ? "interpreted " : "", p2i(ic_destination()), p2i(is_optimized() ? NULL : cached_value())); |
1 | 728 |
} |
729 |
||
42650 | 730 |
void CompiledDirectStaticCall::print() { |
24424
2658d7834c6e
8037816: Fix for 8036122 breaks build with Xcode5/clang
drchase
parents:
22223
diff
changeset
|
731 |
tty->print("static call at " INTPTR_FORMAT " -> ", p2i(instruction_address())); |
1 | 732 |
if (is_clean()) { |
733 |
tty->print("clean"); |
|
734 |
} else if (is_call_to_compiled()) { |
|
735 |
tty->print("compiled"); |
|
42650 | 736 |
} else if (is_call_to_far()) { |
737 |
tty->print("far"); |
|
1 | 738 |
} else if (is_call_to_interpreted()) { |
739 |
tty->print("interpreted"); |
|
740 |
} |
|
741 |
tty->cr(); |
|
742 |
} |
|
743 |
||
58505
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
744 |
void CompiledDirectStaticCall::verify_mt_safe(const methodHandle& callee, address entry, |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
745 |
NativeMovConstReg* method_holder, |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
746 |
NativeJump* jump) { |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
747 |
// A generated lambda form might be deleted from the Lambdaform |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
748 |
// cache in MethodTypeForm. If a jit compiled lambdaform method |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
749 |
// becomes not entrant and the cache access returns null, the new |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
750 |
// resolve will lead to a new generated LambdaForm. |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
751 |
Method* old_method = reinterpret_cast<Method*>(method_holder->data()); |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
752 |
assert(old_method == NULL || old_method == callee() || |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
753 |
callee->is_compiled_lambda_form() || |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
754 |
!old_method->method_holder()->is_loader_alive() || |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
755 |
old_method->is_old(), // may be race patching deoptimized nmethod due to redefinition. |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
756 |
"a) MT-unsafe modification of inline cache"); |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
757 |
|
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
758 |
address destination = jump->jump_destination(); |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
759 |
assert(destination == (address)-1 || destination == entry, |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
760 |
"b) MT-unsafe modification of inline cache"); |
c16f3a24a6fc
8225681: vmTestbase/nsk/jvmti/RedefineClasses/StressRedefine fails due a) MT-unsafe modification of inline cache
coleenp
parents:
57603
diff
changeset
|
761 |
} |
17094
29c4955396d2
8003853: specify offset of IC load in java_to_interp stub
dlong
parents:
16377
diff
changeset
|
762 |
#endif // !PRODUCT |