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