author | redestad |
Tue, 29 Jan 2019 14:34:26 +0100 | |
changeset 53546 | 63eb7e38ce84 |
parent 51997 | 9ce37fa2e179 |
child 53582 | 881c5fbeb849 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
53546 | 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:
4896
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
4896
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:
4896
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
47765
b7c7428eaab9
8189610: Reconcile jvm.h and all jvm_md.h between java.base and hotspot
coleenp
parents:
47692
diff
changeset
|
26 |
#include "jvm.h" |
7397 | 27 |
#include "classfile/systemDictionary.hpp" |
25715
d5a8dbdc5150
8049325: Introduce and clean up umbrella headers for the files in the cpu subdirectories.
goetz
parents:
24934
diff
changeset
|
28 |
#include "code/codeCache.hpp" |
7397 | 29 |
#include "code/debugInfoRec.hpp" |
30 |
#include "code/nmethod.hpp" |
|
31 |
#include "code/pcDesc.hpp" |
|
32 |
#include "code/scopeDesc.hpp" |
|
33 |
#include "interpreter/bytecode.hpp" |
|
34 |
#include "interpreter/interpreter.hpp" |
|
35 |
#include "interpreter/oopMapCache.hpp" |
|
36 |
#include "memory/allocation.inline.hpp" |
|
37 |
#include "memory/oopFactory.hpp" |
|
38 |
#include "memory/resourceArea.hpp" |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
39 |
#include "oops/method.hpp" |
35498
392b50de06c6
8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents:
35071
diff
changeset
|
40 |
#include "oops/objArrayOop.inline.hpp" |
7397 | 41 |
#include "oops/oop.inline.hpp" |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
42 |
#include "oops/fieldStreams.hpp" |
49036
bc92debe57e4
8197999: Accessors in typeArrayOopDesc should use new Access API
rkennke
parents:
48105
diff
changeset
|
43 |
#include "oops/typeArrayOop.inline.hpp" |
29084
1b732f2836ce
8073387: Move VerifyOopClosures out from genOopClosures.hpp
stefank
parents:
28047
diff
changeset
|
44 |
#include "oops/verifyOopClosure.hpp" |
7397 | 45 |
#include "prims/jvmtiThreadState.hpp" |
46 |
#include "runtime/biasedLocking.hpp" |
|
47 |
#include "runtime/compilationPolicy.hpp" |
|
48 |
#include "runtime/deoptimization.hpp" |
|
49480
d7df2dd501ce
8199809: Don't include frame.inline.hpp and other.inline.hpp from .hpp files
coleenp
parents:
49449
diff
changeset
|
49 |
#include "runtime/frame.inline.hpp" |
49449
ef5d5d343e2a
8199263: Split interfaceSupport.hpp to not require including .inline.hpp files
coleenp
parents:
49324
diff
changeset
|
50 |
#include "runtime/interfaceSupport.inline.hpp" |
49594
898ef81cbc0e
8200106: Move NoSafepointVerifier out from gcLocker.hpp
stefank
parents:
49480
diff
changeset
|
51 |
#include "runtime/safepointVerifiers.hpp" |
7397 | 52 |
#include "runtime/sharedRuntime.hpp" |
53 |
#include "runtime/signature.hpp" |
|
54 |
#include "runtime/stubRoutines.hpp" |
|
55 |
#include "runtime/thread.hpp" |
|
48105
8d15b1369c7a
8167108: inconsistent handling of SR_lock can lead to crashes
dcubed
parents:
47799
diff
changeset
|
56 |
#include "runtime/threadSMR.hpp" |
7397 | 57 |
#include "runtime/vframe.hpp" |
58 |
#include "runtime/vframeArray.hpp" |
|
59 |
#include "runtime/vframe_hp.hpp" |
|
60 |
#include "utilities/events.hpp" |
|
49324
e6fbaf07192f
8198826: -XX:+VerifyStack fails with fatal error: ExceptionMark constructor expects no pending exceptions
thartmann
parents:
49189
diff
changeset
|
61 |
#include "utilities/preserveException.hpp" |
7397 | 62 |
#include "utilities/xmlstream.hpp" |
1 | 63 |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
64 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
65 |
#include "jvmci/jvmciRuntime.hpp" |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
66 |
#include "jvmci/jvmciJavaClasses.hpp" |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
67 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
68 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
69 |
|
1 | 70 |
bool DeoptimizationMarker::_is_active = false; |
71 |
||
72 |
Deoptimization::UnrollBlock::UnrollBlock(int size_of_deoptimized_frame, |
|
73 |
int caller_adjustment, |
|
9636
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
74 |
int caller_actual_parameters, |
1 | 75 |
int number_of_frames, |
76 |
intptr_t* frame_sizes, |
|
77 |
address* frame_pcs, |
|
34173
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
78 |
BasicType return_type, |
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
79 |
int exec_mode) { |
1 | 80 |
_size_of_deoptimized_frame = size_of_deoptimized_frame; |
81 |
_caller_adjustment = caller_adjustment; |
|
9636
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
82 |
_caller_actual_parameters = caller_actual_parameters; |
1 | 83 |
_number_of_frames = number_of_frames; |
84 |
_frame_sizes = frame_sizes; |
|
85 |
_frame_pcs = frame_pcs; |
|
13195 | 86 |
_register_block = NEW_C_HEAP_ARRAY(intptr_t, RegisterMap::reg_count * 2, mtCompiler); |
1 | 87 |
_return_type = return_type; |
10539
f87cedf7983c
7087445: Improve platform independence of JSR292 shared code
bdelsart
parents:
9636
diff
changeset
|
88 |
_initial_info = 0; |
1 | 89 |
// PD (x86 only) |
90 |
_counter_temp = 0; |
|
34173
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
91 |
_unpack_kind = exec_mode; |
1 | 92 |
_sender_sp_temp = 0; |
93 |
||
94 |
_total_frame_sizes = size_of_frames(); |
|
34173
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
95 |
assert(exec_mode >= 0 && exec_mode < Unpack_LIMIT, "Unexpected exec_mode"); |
1 | 96 |
} |
97 |
||
98 |
||
99 |
Deoptimization::UnrollBlock::~UnrollBlock() { |
|
27880
afb974a04396
8060074: os::free() takes MemoryTrackingLevel but doesn't need it
coleenp
parents:
27408
diff
changeset
|
100 |
FREE_C_HEAP_ARRAY(intptr_t, _frame_sizes); |
afb974a04396
8060074: os::free() takes MemoryTrackingLevel but doesn't need it
coleenp
parents:
27408
diff
changeset
|
101 |
FREE_C_HEAP_ARRAY(intptr_t, _frame_pcs); |
afb974a04396
8060074: os::free() takes MemoryTrackingLevel but doesn't need it
coleenp
parents:
27408
diff
changeset
|
102 |
FREE_C_HEAP_ARRAY(intptr_t, _register_block); |
1 | 103 |
} |
104 |
||
105 |
||
106 |
intptr_t* Deoptimization::UnrollBlock::value_addr_at(int register_number) const { |
|
107 |
assert(register_number < RegisterMap::reg_count, "checking register number"); |
|
108 |
return &_register_block[register_number * 2]; |
|
109 |
} |
|
110 |
||
111 |
||
112 |
||
113 |
int Deoptimization::UnrollBlock::size_of_frames() const { |
|
114 |
// Acount first for the adjustment of the initial frame |
|
115 |
int result = _caller_adjustment; |
|
116 |
for (int index = 0; index < number_of_frames(); index++) { |
|
117 |
result += frame_sizes()[index]; |
|
118 |
} |
|
119 |
return result; |
|
120 |
} |
|
121 |
||
122 |
||
123 |
void Deoptimization::UnrollBlock::print() { |
|
124 |
ttyLocker ttyl; |
|
125 |
tty->print_cr("UnrollBlock"); |
|
126 |
tty->print_cr(" size_of_deoptimized_frame = %d", _size_of_deoptimized_frame); |
|
127 |
tty->print( " frame_sizes: "); |
|
128 |
for (int index = 0; index < number_of_frames(); index++) { |
|
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
129 |
tty->print(INTX_FORMAT " ", frame_sizes()[index]); |
1 | 130 |
} |
131 |
tty->cr(); |
|
132 |
} |
|
133 |
||
134 |
||
135 |
// In order to make fetch_unroll_info work properly with escape |
|
136 |
// analysis, The method was changed from JRT_LEAF to JRT_BLOCK_ENTRY and |
|
137 |
// ResetNoHandleMark and HandleMark were removed from it. The actual reallocation |
|
138 |
// of previously eliminated objects occurs in realloc_objects, which is |
|
139 |
// called from the method fetch_unroll_info_helper below. |
|
34173
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
140 |
JRT_BLOCK_ENTRY(Deoptimization::UnrollBlock*, Deoptimization::fetch_unroll_info(JavaThread* thread, int exec_mode)) |
1 | 141 |
// It is actually ok to allocate handles in a leaf method. It causes no safepoints, |
142 |
// but makes the entry a little slower. There is however a little dance we have to |
|
143 |
// do in debug mode to get around the NoHandleMark code in the JRT_LEAF macro |
|
144 |
||
145 |
// fetch_unroll_info() is called at the beginning of the deoptimization |
|
146 |
// handler. Note this fact before we start generating temporary frames |
|
147 |
// that can confuse an asynchronous stack walker. This counter is |
|
148 |
// decremented at the end of unpack_frames(). |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
149 |
if (TraceDeoptimization) { |
33198 | 150 |
tty->print_cr("Deoptimizing thread " INTPTR_FORMAT, p2i(thread)); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
151 |
} |
1 | 152 |
thread->inc_in_deopt_handler(); |
153 |
||
34173
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
154 |
return fetch_unroll_info_helper(thread, exec_mode); |
1 | 155 |
JRT_END |
156 |
||
157 |
||
158 |
// This is factored, since it is both called from a JRT_LEAF (deoptimization) and a JRT_ENTRY (uncommon_trap) |
|
34173
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
159 |
Deoptimization::UnrollBlock* Deoptimization::fetch_unroll_info_helper(JavaThread* thread, int exec_mode) { |
1 | 160 |
|
161 |
// Note: there is a safepoint safety issue here. No matter whether we enter |
|
162 |
// via vanilla deopt or uncommon trap we MUST NOT stop at a safepoint once |
|
163 |
// the vframeArray is created. |
|
164 |
// |
|
165 |
||
166 |
// Allocate our special deoptimization ResourceMark |
|
167 |
DeoptResourceMark* dmark = new DeoptResourceMark(thread); |
|
168 |
assert(thread->deopt_mark() == NULL, "Pending deopt!"); |
|
169 |
thread->set_deopt_mark(dmark); |
|
170 |
||
171 |
frame stub_frame = thread->last_frame(); // Makes stack walkable as side effect |
|
172 |
RegisterMap map(thread, true); |
|
173 |
RegisterMap dummy_map(thread, false); |
|
174 |
// Now get the deoptee with a valid map |
|
175 |
frame deoptee = stub_frame.sender(&map); |
|
6740
2bc601284215
6986270: guarantee(*bcp != Bytecodes::_monitorenter || exec_mode != Deoptimization::Unpack_exception) fails
iveresov
parents:
6453
diff
changeset
|
176 |
// Set the deoptee nmethod |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
177 |
assert(thread->deopt_compiled_method() == NULL, "Pending deopt!"); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
178 |
CompiledMethod* cm = deoptee.cb()->as_compiled_method_or_null(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
179 |
thread->set_deopt_compiled_method(cm); |
1 | 180 |
|
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
8657
diff
changeset
|
181 |
if (VerifyStack) { |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
8657
diff
changeset
|
182 |
thread->validate_frame_layout(); |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
8657
diff
changeset
|
183 |
} |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
8657
diff
changeset
|
184 |
|
1 | 185 |
// Create a growable array of VFrames where each VFrame represents an inlined |
186 |
// Java frame. This storage is allocated with the usual system arena. |
|
187 |
assert(deoptee.is_compiled_frame(), "Wrong frame type"); |
|
188 |
GrowableArray<compiledVFrame*>* chunk = new GrowableArray<compiledVFrame*>(10); |
|
189 |
vframe* vf = vframe::new_vframe(&deoptee, &map, thread); |
|
190 |
while (!vf->is_top()) { |
|
191 |
assert(vf->is_compiled_frame(), "Wrong frame type"); |
|
192 |
chunk->push(compiledVFrame::cast(vf)); |
|
193 |
vf = vf->sender(); |
|
194 |
} |
|
195 |
assert(vf->is_compiled_frame(), "Wrong frame type"); |
|
196 |
chunk->push(compiledVFrame::cast(vf)); |
|
197 |
||
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
198 |
bool realloc_failures = false; |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
199 |
|
47799 | 200 |
#if COMPILER2_OR_JVMCI |
1 | 201 |
// Reallocate the non-escaping objects and restore their fields. Then |
202 |
// relock objects if synchronization on them was eliminated. |
|
49972
37b2446d7f86
8202552: [AOT][JVMCI] Incorrect usage of INCLUDE_JVMCI and INCLUDE_AOT
kvn
parents:
49594
diff
changeset
|
203 |
#if !INCLUDE_JVMCI |
11445 | 204 |
if (DoEscapeAnalysis || EliminateNestedLocks) { |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
205 |
if (EliminateAllocations) { |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
206 |
#endif // INCLUDE_JVMCI |
256
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
207 |
assert (chunk->at(0)->scope() != NULL,"expect only compiled java frames"); |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
208 |
GrowableArray<ScopeValue*>* objects = chunk->at(0)->scope()->objects(); |
4894
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
209 |
|
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
210 |
// The flag return_oop() indicates call sites which return oop |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
211 |
// in compiled code. Such sites include java method calls, |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
212 |
// runtime calls (for example, used to allocate new objects/arrays |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
213 |
// on slow code path) and any other calls generated in compiled code. |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
214 |
// It is not guaranteed that we can get such information here only |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
215 |
// by analyzing bytecode in deoptimized frames. This is why this flag |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
216 |
// is set during method compilation (see Compile::Process_OopMap_Node()). |
43931
b31dd7e27325
8173699: Crash during deoptimization with "assert(result == __null || result->is_oop()) failed: must be oop"
thartmann
parents:
43461
diff
changeset
|
217 |
// If the previous frame was popped or if we are dispatching an exception, |
b31dd7e27325
8173699: Crash during deoptimization with "assert(result == __null || result->is_oop()) failed: must be oop"
thartmann
parents:
43461
diff
changeset
|
218 |
// we don't have an oop result. |
b31dd7e27325
8173699: Crash during deoptimization with "assert(result == __null || result->is_oop()) failed: must be oop"
thartmann
parents:
43461
diff
changeset
|
219 |
bool save_oop_result = chunk->at(0)->scope()->return_oop() && !thread->popframe_forcing_deopt_reexecution() && (exec_mode == Unpack_deopt); |
4894
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
220 |
Handle return_value; |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
221 |
if (save_oop_result) { |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
222 |
// Reallocation may trigger GC. If deoptimization happened on return from |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
223 |
// call which returns oop we need to save it since it is not in oopmap. |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
224 |
oop result = deoptee.saved_oop_result(&map); |
46968
9119841280f4
8160399: is_oop_or_null involves undefined behavior
coleenp
parents:
46727
diff
changeset
|
225 |
assert(oopDesc::is_oop_or_null(result), "must be oop"); |
4894
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
226 |
return_value = Handle(thread, result); |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
227 |
assert(Universe::heap()->is_in_or_null(result), "must be heap pointer"); |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
228 |
if (TraceDeoptimization) { |
13964 | 229 |
ttyLocker ttyl; |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
230 |
tty->print_cr("SAVED OOP RESULT " INTPTR_FORMAT " in thread " INTPTR_FORMAT, p2i(result), p2i(thread)); |
4894
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
231 |
} |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
232 |
} |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
233 |
if (objects != NULL) { |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
234 |
JRT_BLOCK |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
235 |
realloc_failures = realloc_objects(thread, &deoptee, objects, THREAD); |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
236 |
JRT_END |
42040
70ec5a09a0d5
8166377: is_compiled_by_jvmci hot in some profiles - improve nmethod compiler type detection
neliasso
parents:
39261
diff
changeset
|
237 |
bool skip_internal = (cm != NULL) && !cm->is_compiled_by_jvmci(); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
238 |
reassign_fields(&deoptee, &map, objects, realloc_failures, skip_internal); |
28047
c0e7a15c458f
8067144: SIGSEGV with +TraceDeoptimization in Deoptimization::print_objects
roland
parents:
28039
diff
changeset
|
239 |
#ifndef PRODUCT |
c0e7a15c458f
8067144: SIGSEGV with +TraceDeoptimization in Deoptimization::print_objects
roland
parents:
28039
diff
changeset
|
240 |
if (TraceDeoptimization) { |
c0e7a15c458f
8067144: SIGSEGV with +TraceDeoptimization in Deoptimization::print_objects
roland
parents:
28039
diff
changeset
|
241 |
ttyLocker ttyl; |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
242 |
tty->print_cr("REALLOC OBJECTS in thread " INTPTR_FORMAT, p2i(thread)); |
28047
c0e7a15c458f
8067144: SIGSEGV with +TraceDeoptimization in Deoptimization::print_objects
roland
parents:
28039
diff
changeset
|
243 |
print_objects(objects, realloc_failures); |
c0e7a15c458f
8067144: SIGSEGV with +TraceDeoptimization in Deoptimization::print_objects
roland
parents:
28039
diff
changeset
|
244 |
} |
c0e7a15c458f
8067144: SIGSEGV with +TraceDeoptimization in Deoptimization::print_objects
roland
parents:
28039
diff
changeset
|
245 |
#endif |
1 | 246 |
} |
4894
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
247 |
if (save_oop_result) { |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
248 |
// Restore result. |
8a76fd3d098d
6910618: C2: Error: assert(d->is_oop(),"JVM_ArrayCopy: dst not an oop")
kvn
parents:
4754
diff
changeset
|
249 |
deoptee.set_saved_oop_result(&map, return_value()); |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
250 |
} |
49972
37b2446d7f86
8202552: [AOT][JVMCI] Incorrect usage of INCLUDE_JVMCI and INCLUDE_AOT
kvn
parents:
49594
diff
changeset
|
251 |
#if !INCLUDE_JVMCI |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
252 |
} |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
253 |
if (EliminateLocks) { |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
254 |
#endif // INCLUDE_JVMCI |
256
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
255 |
#ifndef PRODUCT |
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
256 |
bool first = true; |
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
257 |
#endif |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
258 |
for (int i = 0; i < chunk->length(); i++) { |
256
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
259 |
compiledVFrame* cvf = chunk->at(i); |
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
260 |
assert (cvf->scope() != NULL,"expect only compiled java frames"); |
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
261 |
GrowableArray<MonitorInfo*>* monitors = cvf->monitors(); |
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
262 |
if (monitors->is_nonempty()) { |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
263 |
relock_objects(monitors, thread, realloc_failures); |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
264 |
#ifndef PRODUCT |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
265 |
if (PrintDeoptimizationDetails) { |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
266 |
ttyLocker ttyl; |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
267 |
for (int j = 0; j < monitors->length(); j++) { |
256
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
268 |
MonitorInfo* mi = monitors->at(j); |
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
269 |
if (mi->eliminated()) { |
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
270 |
if (first) { |
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
271 |
first = false; |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
272 |
tty->print_cr("RELOCK OBJECTS in thread " INTPTR_FORMAT, p2i(thread)); |
256
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
273 |
} |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
274 |
if (mi->owner_is_scalar_replaced()) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
275 |
Klass* k = java_lang_Class::as_Klass(mi->owner_klass()); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
276 |
tty->print_cr(" failed reallocation for klass %s", k->external_name()); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
277 |
} else { |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
278 |
tty->print_cr(" object <" INTPTR_FORMAT "> locked", p2i(mi->owner())); |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
279 |
} |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
280 |
} |
1 | 281 |
} |
282 |
} |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
283 |
#endif // !PRODUCT |
1 | 284 |
} |
285 |
} |
|
49972
37b2446d7f86
8202552: [AOT][JVMCI] Incorrect usage of INCLUDE_JVMCI and INCLUDE_AOT
kvn
parents:
49594
diff
changeset
|
286 |
#if !INCLUDE_JVMCI |
1 | 287 |
} |
288 |
} |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
289 |
#endif // INCLUDE_JVMCI |
47799 | 290 |
#endif // COMPILER2_OR_JVMCI |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
291 |
|
44632
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
292 |
ScopeDesc* trap_scope = chunk->at(0)->scope(); |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
293 |
Handle exceptionObject; |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
294 |
if (trap_scope->rethrow_exception()) { |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
295 |
if (PrintDeoptimizationDetails) { |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
296 |
tty->print_cr("Exception to be rethrown in the interpreter for method %s::%s at bci %d", trap_scope->method()->method_holder()->name()->as_C_string(), trap_scope->method()->name()->as_C_string(), trap_scope->bci()); |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
297 |
} |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
298 |
GrowableArray<ScopeValue*>* expressions = trap_scope->expressions(); |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
299 |
guarantee(expressions != NULL && expressions->length() > 0, "must have exception to throw"); |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
300 |
ScopeValue* topOfStack = expressions->top(); |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
301 |
exceptionObject = StackValue::create_stack_value(&deoptee, &map, topOfStack)->get_obj(); |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
302 |
guarantee(exceptionObject() != NULL, "exception oop can not be null"); |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
303 |
} |
1164efa943eb
8178119: [JVMCI] when rethrowing exceptions at deopt the exception must be fetched after materialization
never
parents:
43942
diff
changeset
|
304 |
|
1 | 305 |
// Ensure that no safepoint is taken after pointers have been stored |
306 |
// in fields of rematerialized objects. If a safepoint occurs from here on |
|
307 |
// out the java state residing in the vframeArray will be missed. |
|
35492
c8c0273e6b91
8146690: Make all classes in GC follow the naming convention.
david
parents:
35071
diff
changeset
|
308 |
NoSafepointVerifier no_safepoint; |
1 | 309 |
|
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
310 |
vframeArray* array = create_vframeArray(thread, deoptee, &map, chunk, realloc_failures); |
47799 | 311 |
#if COMPILER2_OR_JVMCI |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
312 |
if (realloc_failures) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
313 |
pop_frames_failed_reallocs(thread, array); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
314 |
} |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
315 |
#endif |
1 | 316 |
|
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
317 |
assert(thread->vframe_array_head() == NULL, "Pending deopt!"); |
1 | 318 |
thread->set_vframe_array_head(array); |
319 |
||
320 |
// Now that the vframeArray has been created if we have any deferred local writes |
|
321 |
// added by jvmti then we can free up that structure as the data is now in the |
|
322 |
// vframeArray |
|
323 |
||
324 |
if (thread->deferred_locals() != NULL) { |
|
325 |
GrowableArray<jvmtiDeferredLocalVariableSet*>* list = thread->deferred_locals(); |
|
326 |
int i = 0; |
|
327 |
do { |
|
328 |
// Because of inlining we could have multiple vframes for a single frame |
|
329 |
// and several of the vframes could have deferred writes. Find them all. |
|
330 |
if (list->at(i)->id() == array->original().id()) { |
|
331 |
jvmtiDeferredLocalVariableSet* dlv = list->at(i); |
|
332 |
list->remove_at(i); |
|
333 |
// individual jvmtiDeferredLocalVariableSet are CHeapObj's |
|
334 |
delete dlv; |
|
335 |
} else { |
|
336 |
i++; |
|
337 |
} |
|
338 |
} while ( i < list->length() ); |
|
339 |
if (list->length() == 0) { |
|
340 |
thread->set_deferred_locals(NULL); |
|
341 |
// free the list and elements back to C heap. |
|
342 |
delete list; |
|
343 |
} |
|
344 |
||
345 |
} |
|
346 |
||
347 |
// Compute the caller frame based on the sender sp of stub_frame and stored frame sizes info. |
|
348 |
CodeBlob* cb = stub_frame.cb(); |
|
349 |
// Verify we have the right vframeArray |
|
350 |
assert(cb->frame_size() >= 0, "Unexpected frame size"); |
|
351 |
intptr_t* unpack_sp = stub_frame.sp() + cb->frame_size(); |
|
352 |
||
4752 | 353 |
// If the deopt call site is a MethodHandle invoke call site we have |
354 |
// to adjust the unpack_sp. |
|
355 |
nmethod* deoptee_nm = deoptee.cb()->as_nmethod_or_null(); |
|
356 |
if (deoptee_nm != NULL && deoptee_nm->is_method_handle_return(deoptee.pc())) |
|
357 |
unpack_sp = deoptee.unextended_sp(); |
|
358 |
||
1 | 359 |
#ifdef ASSERT |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
360 |
assert(cb->is_deoptimization_stub() || |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
361 |
cb->is_uncommon_trap_stub() || |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
362 |
strcmp("Stub<DeoptimizationStub.deoptimizationHandler>", cb->name()) == 0 || |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
363 |
strcmp("Stub<UncommonTrapStub.uncommonTrapHandler>", cb->name()) == 0, |
33198 | 364 |
"unexpected code blob: %s", cb->name()); |
1 | 365 |
#endif |
6187 | 366 |
|
1 | 367 |
// This is a guarantee instead of an assert because if vframe doesn't match |
368 |
// we will unpack the wrong deoptimized frame and wind up in strange places |
|
369 |
// where it will be very difficult to figure out what went wrong. Better |
|
370 |
// to die an early death here than some very obscure death later when the |
|
371 |
// trail is cold. |
|
372 |
// Note: on ia64 this guarantee can be fooled by frames with no memory stack |
|
373 |
// in that it will fail to detect a problem when there is one. This needs |
|
374 |
// more work in tiger timeframe. |
|
375 |
guarantee(array->unextended_sp() == unpack_sp, "vframe_array_head must contain the vframeArray to unpack"); |
|
376 |
||
377 |
int number_of_frames = array->frames(); |
|
378 |
||
379 |
// Compute the vframes' sizes. Note that frame_sizes[] entries are ordered from outermost to innermost |
|
380 |
// virtual activation, which is the reverse of the elements in the vframes array. |
|
13195 | 381 |
intptr_t* frame_sizes = NEW_C_HEAP_ARRAY(intptr_t, number_of_frames, mtCompiler); |
1 | 382 |
// +1 because we always have an interpreter return address for the final slot. |
13195 | 383 |
address* frame_pcs = NEW_C_HEAP_ARRAY(address, number_of_frames + 1, mtCompiler); |
1 | 384 |
int popframe_extra_args = 0; |
385 |
// Create an interpreter return address for the stub to use as its return |
|
386 |
// address so the skeletal frames are perfectly walkable |
|
387 |
frame_pcs[number_of_frames] = Interpreter::deopt_entry(vtos, 0); |
|
388 |
||
389 |
// PopFrame requires that the preserved incoming arguments from the recently-popped topmost |
|
390 |
// activation be put back on the expression stack of the caller for reexecution |
|
391 |
if (JvmtiExport::can_pop_frame() && thread->popframe_forcing_deopt_reexecution()) { |
|
392 |
popframe_extra_args = in_words(thread->popframe_preserved_args_size_in_words()); |
|
393 |
} |
|
394 |
||
9636
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
395 |
// Find the current pc for sender of the deoptee. Since the sender may have been deoptimized |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
396 |
// itself since the deoptee vframeArray was created we must get a fresh value of the pc rather |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
397 |
// than simply use array->sender.pc(). This requires us to walk the current set of frames |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
398 |
// |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
399 |
frame deopt_sender = stub_frame.sender(&dummy_map); // First is the deoptee frame |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
400 |
deopt_sender = deopt_sender.sender(&dummy_map); // Now deoptee caller |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
401 |
|
22551 | 402 |
// It's possible that the number of parameters at the call site is |
9636
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
403 |
// different than number of arguments in the callee when method |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
404 |
// handles are used. If the caller is interpreted get the real |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
405 |
// value so that the proper amount of space can be added to it's |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
406 |
// frame. |
10966
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
407 |
bool caller_was_method_handle = false; |
9636
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
408 |
if (deopt_sender.is_interpreted_frame()) { |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
409 |
methodHandle method = deopt_sender.interpreter_frame_method(); |
10979
c25dd872c62f
7106774: JSR 292: nightly test inlineMHTarget fails with wrong result
twisti
parents:
10966
diff
changeset
|
410 |
Bytecode_invoke cur = Bytecode_invoke_check(method, deopt_sender.interpreter_frame_bci()); |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
13195
diff
changeset
|
411 |
if (cur.is_invokedynamic() || cur.is_invokehandle()) { |
10966
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
412 |
// Method handle invokes may involve fairly arbitrary chains of |
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
413 |
// calls so it's impossible to know how much actual space the |
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
414 |
// caller has for locals. |
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
415 |
caller_was_method_handle = true; |
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
416 |
} |
9636
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
417 |
} |
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
418 |
|
1 | 419 |
// |
420 |
// frame_sizes/frame_pcs[0] oldest frame (int or c2i) |
|
421 |
// frame_sizes/frame_pcs[1] next oldest frame (int) |
|
422 |
// frame_sizes/frame_pcs[n] youngest frame (int) |
|
423 |
// |
|
424 |
// Now a pc in frame_pcs is actually the return address to the frame's caller (a frame |
|
425 |
// owns the space for the return address to it's caller). Confusing ain't it. |
|
426 |
// |
|
427 |
// The vframe array can address vframes with indices running from |
|
428 |
// 0.._frames-1. Index 0 is the youngest frame and _frame - 1 is the oldest (root) frame. |
|
429 |
// When we create the skeletal frames we need the oldest frame to be in the zero slot |
|
430 |
// in the frame_sizes/frame_pcs so the assembly code can do a trivial walk. |
|
431 |
// so things look a little strange in this loop. |
|
432 |
// |
|
10966
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
433 |
int callee_parameters = 0; |
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
434 |
int callee_locals = 0; |
1 | 435 |
for (int index = 0; index < array->frames(); index++ ) { |
436 |
// frame[number_of_frames - 1 ] = on_stack_size(youngest) |
|
437 |
// frame[number_of_frames - 2 ] = on_stack_size(sender(youngest)) |
|
438 |
// frame[number_of_frames - 3 ] = on_stack_size(sender(sender(youngest))) |
|
24018
77b156916bab
8032410: compiler/uncommontrap/TestStackBangRbp.java times out on Solaris-Sparc V9
roland
parents:
23525
diff
changeset
|
439 |
frame_sizes[number_of_frames - 1 - index] = BytesPerWord * array->element(index)->on_stack_size(callee_parameters, |
1 | 440 |
callee_locals, |
441 |
index == 0, |
|
442 |
popframe_extra_args); |
|
443 |
// This pc doesn't have to be perfect just good enough to identify the frame |
|
444 |
// as interpreted so the skeleton frame will be walkable |
|
445 |
// The correct pc will be set when the skeleton frame is completely filled out |
|
446 |
// The final pc we store in the loop is wrong and will be overwritten below |
|
447 |
frame_pcs[number_of_frames - 1 - index ] = Interpreter::deopt_entry(vtos, 0) - frame::pc_return_offset; |
|
448 |
||
449 |
callee_parameters = array->element(index)->method()->size_of_parameters(); |
|
450 |
callee_locals = array->element(index)->method()->max_locals(); |
|
451 |
popframe_extra_args = 0; |
|
452 |
} |
|
453 |
||
454 |
// Compute whether the root vframe returns a float or double value. |
|
455 |
BasicType return_type; |
|
456 |
{ |
|
457 |
methodHandle method(thread, array->element(0)->method()); |
|
7913 | 458 |
Bytecode_invoke invoke = Bytecode_invoke_check(method, array->element(0)->bci()); |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7913
diff
changeset
|
459 |
return_type = invoke.is_valid() ? invoke.result_type() : T_ILLEGAL; |
1 | 460 |
} |
461 |
||
462 |
// Compute information for handling adapters and adjusting the frame size of the caller. |
|
463 |
int caller_adjustment = 0; |
|
464 |
||
465 |
// Compute the amount the oldest interpreter frame will have to adjust |
|
466 |
// its caller's stack by. If the caller is a compiled frame then |
|
467 |
// we pretend that the callee has no parameters so that the |
|
468 |
// extension counts for the full amount of locals and not just |
|
469 |
// locals-parms. This is because without a c2i adapter the parm |
|
470 |
// area as created by the compiled frame will not be usable by |
|
471 |
// the interpreter. (Depending on the calling convention there |
|
472 |
// may not even be enough space). |
|
473 |
||
474 |
// QQQ I'd rather see this pushed down into last_frame_adjust |
|
475 |
// and have it take the sender (aka caller). |
|
476 |
||
10966
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
477 |
if (deopt_sender.is_compiled_frame() || caller_was_method_handle) { |
1 | 478 |
caller_adjustment = last_frame_adjust(0, callee_locals); |
10966
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
479 |
} else if (callee_locals > callee_parameters) { |
1 | 480 |
// The caller frame may need extending to accommodate |
481 |
// non-parameter locals of the first unpacked interpreted frame. |
|
482 |
// Compute that adjustment. |
|
10966
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
483 |
caller_adjustment = last_frame_adjust(callee_parameters, callee_locals); |
1 | 484 |
} |
485 |
||
486 |
// If the sender is deoptimized the we must retrieve the address of the handler |
|
487 |
// since the frame will "magically" show the original pc before the deopt |
|
488 |
// and we'd undo the deopt. |
|
489 |
||
490 |
frame_pcs[0] = deopt_sender.raw_pc(); |
|
491 |
||
492 |
assert(CodeCache::find_blob_unsafe(frame_pcs[0]) != NULL, "bad pc"); |
|
493 |
||
49972
37b2446d7f86
8202552: [AOT][JVMCI] Incorrect usage of INCLUDE_JVMCI and INCLUDE_AOT
kvn
parents:
49594
diff
changeset
|
494 |
#if INCLUDE_JVMCI |
34173
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
495 |
if (exceptionObject() != NULL) { |
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
496 |
thread->set_exception_oop(exceptionObject()); |
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
497 |
exec_mode = Unpack_exception; |
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
498 |
} |
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
499 |
#endif |
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
500 |
|
39261
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
501 |
if (thread->frames_to_pop_failed_realloc() > 0 && exec_mode != Unpack_uncommon_trap) { |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
502 |
assert(thread->has_pending_exception(), "should have thrown OOME"); |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
503 |
thread->set_exception_oop(thread->pending_exception()); |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
504 |
thread->clear_pending_exception(); |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
505 |
exec_mode = Unpack_exception; |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
506 |
} |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
507 |
|
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
508 |
#if INCLUDE_JVMCI |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
509 |
if (thread->frames_to_pop_failed_realloc() > 0) { |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
510 |
thread->set_pending_monitorenter(false); |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
511 |
} |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
512 |
#endif |
18f007610de6
8146416: java.lang.OutOfMemoryError triggers: assert(current_bci == 0) failed: bci isn't zero for do_not_unlock_if_synchronized
jcm
parents:
38242
diff
changeset
|
513 |
|
1 | 514 |
UnrollBlock* info = new UnrollBlock(array->frame_size() * BytesPerWord, |
515 |
caller_adjustment * BytesPerWord, |
|
10966
0c9ed2dfc6a2
7090904: JSR 292: JRuby junit test crashes in PSScavengeRootsClosure::do_oop
twisti
parents:
10539
diff
changeset
|
516 |
caller_was_method_handle ? 0 : callee_parameters, |
1 | 517 |
number_of_frames, |
518 |
frame_sizes, |
|
519 |
frame_pcs, |
|
34173
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
520 |
return_type, |
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
521 |
exec_mode); |
10539
f87cedf7983c
7087445: Improve platform independence of JSR292 shared code
bdelsart
parents:
9636
diff
changeset
|
522 |
// On some platforms, we need a way to pass some platform dependent |
f87cedf7983c
7087445: Improve platform independence of JSR292 shared code
bdelsart
parents:
9636
diff
changeset
|
523 |
// information to the unpacking code so the skeletal frames come out |
f87cedf7983c
7087445: Improve platform independence of JSR292 shared code
bdelsart
parents:
9636
diff
changeset
|
524 |
// correct (initial fp value, unextended sp, ...) |
f87cedf7983c
7087445: Improve platform independence of JSR292 shared code
bdelsart
parents:
9636
diff
changeset
|
525 |
info->set_initial_info((intptr_t) array->sender().initial_deoptimization_info()); |
1 | 526 |
|
527 |
if (array->frames() > 1) { |
|
528 |
if (VerifyStack && TraceDeoptimization) { |
|
13964 | 529 |
ttyLocker ttyl; |
1 | 530 |
tty->print_cr("Deoptimizing method containing inlining"); |
531 |
} |
|
532 |
} |
|
533 |
||
534 |
array->set_unroll_block(info); |
|
535 |
return info; |
|
536 |
} |
|
537 |
||
538 |
// Called to cleanup deoptimization data structures in normal case |
|
539 |
// after unpacking to stack and when stack overflow error occurs |
|
540 |
void Deoptimization::cleanup_deopt_info(JavaThread *thread, |
|
541 |
vframeArray *array) { |
|
542 |
||
543 |
// Get array if coming from exception |
|
544 |
if (array == NULL) { |
|
545 |
array = thread->vframe_array_head(); |
|
546 |
} |
|
547 |
thread->set_vframe_array_head(NULL); |
|
548 |
||
549 |
// Free the previous UnrollBlock |
|
550 |
vframeArray* old_array = thread->vframe_array_last(); |
|
551 |
thread->set_vframe_array_last(array); |
|
552 |
||
553 |
if (old_array != NULL) { |
|
554 |
UnrollBlock* old_info = old_array->unroll_block(); |
|
555 |
old_array->set_unroll_block(NULL); |
|
556 |
delete old_info; |
|
557 |
delete old_array; |
|
558 |
} |
|
559 |
||
560 |
// Deallocate any resource creating in this routine and any ResourceObjs allocated |
|
561 |
// inside the vframeArray (StackValueCollections) |
|
562 |
||
563 |
delete thread->deopt_mark(); |
|
564 |
thread->set_deopt_mark(NULL); |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
565 |
thread->set_deopt_compiled_method(NULL); |
1 | 566 |
|
567 |
||
568 |
if (JvmtiExport::can_pop_frame()) { |
|
569 |
#ifndef CC_INTERP |
|
570 |
// Regardless of whether we entered this routine with the pending |
|
571 |
// popframe condition bit set, we should always clear it now |
|
572 |
thread->clear_popframe_condition(); |
|
573 |
#else |
|
22551 | 574 |
// C++ interpreter will clear has_pending_popframe when it enters |
1 | 575 |
// with method_resume. For deopt_resume2 we clear it now. |
576 |
if (thread->popframe_forcing_deopt_reexecution()) |
|
577 |
thread->clear_popframe_condition(); |
|
578 |
#endif /* CC_INTERP */ |
|
579 |
} |
|
580 |
||
581 |
// unpack_frames() is called at the end of the deoptimization handler |
|
582 |
// and (in C2) at the end of the uncommon trap handler. Note this fact |
|
583 |
// so that an asynchronous stack walker can work again. This counter is |
|
584 |
// incremented at the beginning of fetch_unroll_info() and (in C2) at |
|
585 |
// the beginning of uncommon_trap(). |
|
586 |
thread->dec_in_deopt_handler(); |
|
587 |
} |
|
588 |
||
34651
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
589 |
// Moved from cpu directories because none of the cpus has callee save values. |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
590 |
// If a cpu implements callee save values, move this to deoptimization_<cpu>.cpp. |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
591 |
void Deoptimization::unwind_callee_save_values(frame* f, vframeArray* vframe_array) { |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
592 |
|
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
593 |
// This code is sort of the equivalent of C2IAdapter::setup_stack_frame back in |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
594 |
// the days we had adapter frames. When we deoptimize a situation where a |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
595 |
// compiled caller calls a compiled caller will have registers it expects |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
596 |
// to survive the call to the callee. If we deoptimize the callee the only |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
597 |
// way we can restore these registers is to have the oldest interpreter |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
598 |
// frame that we create restore these values. That is what this routine |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
599 |
// will accomplish. |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
600 |
|
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
601 |
// At the moment we have modified c2 to not have any callee save registers |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
602 |
// so this problem does not exist and this routine is just a place holder. |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
603 |
|
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
604 |
assert(f->is_interpreted_frame(), "must be interpreted"); |
07b1cc0f6040
8144534: Refactor templateInterpreter and templateInterpreterGenerator functions
coleenp
parents:
34185
diff
changeset
|
605 |
} |
1 | 606 |
|
607 |
// Return BasicType of value being returned |
|
608 |
JRT_LEAF(BasicType, Deoptimization::unpack_frames(JavaThread* thread, int exec_mode)) |
|
609 |
||
49189
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
610 |
// We are already active in the special DeoptResourceMark any ResourceObj's we |
1 | 611 |
// allocate will be freed at the end of the routine. |
612 |
||
613 |
// It is actually ok to allocate handles in a leaf method. It causes no safepoints, |
|
614 |
// but makes the entry a little slower. There is however a little dance we have to |
|
615 |
// do in debug mode to get around the NoHandleMark code in the JRT_LEAF macro |
|
616 |
ResetNoHandleMark rnhm; // No-op in release/product versions |
|
617 |
HandleMark hm; |
|
618 |
||
619 |
frame stub_frame = thread->last_frame(); |
|
620 |
||
621 |
// Since the frame to unpack is the top frame of this thread, the vframe_array_head |
|
622 |
// must point to the vframeArray for the unpack frame. |
|
623 |
vframeArray* array = thread->vframe_array_head(); |
|
624 |
||
625 |
#ifndef PRODUCT |
|
626 |
if (TraceDeoptimization) { |
|
13964 | 627 |
ttyLocker ttyl; |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
628 |
tty->print_cr("DEOPT UNPACKING thread " INTPTR_FORMAT " vframeArray " INTPTR_FORMAT " mode %d", |
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
629 |
p2i(thread), p2i(array), exec_mode); |
1 | 630 |
} |
631 |
#endif |
|
11636
3c07b54482a5
7141200: log some interesting information in ring buffers for crashes
never
parents:
11445
diff
changeset
|
632 |
Events::log(thread, "DEOPT UNPACKING pc=" INTPTR_FORMAT " sp=" INTPTR_FORMAT " mode %d", |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
633 |
p2i(stub_frame.pc()), p2i(stub_frame.sp()), exec_mode); |
1 | 634 |
|
635 |
UnrollBlock* info = array->unroll_block(); |
|
636 |
||
637 |
// Unpack the interpreter frames and any adapter frame (c2 only) we might create. |
|
9636
363ca5579aff
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
9447
diff
changeset
|
638 |
array->unpack_to_stack(stub_frame, exec_mode, info->caller_actual_parameters()); |
1 | 639 |
|
640 |
BasicType bt = info->return_type(); |
|
641 |
||
642 |
// If we have an exception pending, claim that the return type is an oop |
|
643 |
// so the deopt_blob does not overwrite the exception_oop. |
|
644 |
||
645 |
if (exec_mode == Unpack_exception) |
|
646 |
bt = T_OBJECT; |
|
647 |
||
648 |
// Cleanup thread deopt data |
|
649 |
cleanup_deopt_info(thread, array); |
|
650 |
||
651 |
#ifndef PRODUCT |
|
652 |
if (VerifyStack) { |
|
653 |
ResourceMark res_mark; |
|
49324
e6fbaf07192f
8198826: -XX:+VerifyStack fails with fatal error: ExceptionMark constructor expects no pending exceptions
thartmann
parents:
49189
diff
changeset
|
654 |
// Clear pending exception to not break verification code (restored afterwards) |
e6fbaf07192f
8198826: -XX:+VerifyStack fails with fatal error: ExceptionMark constructor expects no pending exceptions
thartmann
parents:
49189
diff
changeset
|
655 |
PRESERVE_EXCEPTION_MARK; |
1 | 656 |
|
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
8657
diff
changeset
|
657 |
thread->validate_frame_layout(); |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
8657
diff
changeset
|
658 |
|
1 | 659 |
// Verify that the just-unpacked frames match the interpreter's |
660 |
// notions of expression stack and locals |
|
661 |
vframeArray* cur_array = thread->vframe_array_last(); |
|
662 |
RegisterMap rm(thread, false); |
|
663 |
rm.set_include_argument_oops(false); |
|
664 |
bool is_top_frame = true; |
|
665 |
int callee_size_of_parameters = 0; |
|
666 |
int callee_max_locals = 0; |
|
667 |
for (int i = 0; i < cur_array->frames(); i++) { |
|
668 |
vframeArrayElement* el = cur_array->element(i); |
|
669 |
frame* iframe = el->iframe(); |
|
670 |
guarantee(iframe->is_interpreted_frame(), "Wrong frame type"); |
|
671 |
||
672 |
// Get the oop map for this bci |
|
673 |
InterpreterOopMap mask; |
|
674 |
int cur_invoke_parameter_size = 0; |
|
675 |
bool try_next_mask = false; |
|
676 |
int next_mask_expression_stack_size = -1; |
|
677 |
int top_frame_expression_stack_adjustment = 0; |
|
678 |
methodHandle mh(thread, iframe->interpreter_frame_method()); |
|
679 |
OopMapCache::compute_one_oop_map(mh, iframe->interpreter_frame_bci(), &mask); |
|
680 |
BytecodeStream str(mh); |
|
681 |
str.set_start(iframe->interpreter_frame_bci()); |
|
682 |
int max_bci = mh->code_size(); |
|
683 |
// Get to the next bytecode if possible |
|
684 |
assert(str.bci() < max_bci, "bci in interpreter frame out of bounds"); |
|
685 |
// Check to see if we can grab the number of outgoing arguments |
|
686 |
// at an uncommon trap for an invoke (where the compiler |
|
687 |
// generates debug info before the invoke has executed) |
|
688 |
Bytecodes::Code cur_code = str.next(); |
|
49189
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
689 |
if (Bytecodes::is_invoke(cur_code)) { |
7913 | 690 |
Bytecode_invoke invoke(mh, iframe->interpreter_frame_bci()); |
49189
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
691 |
cur_invoke_parameter_size = invoke.size_of_parameters(); |
17872 | 692 |
if (i != 0 && !invoke.is_invokedynamic() && MethodHandles::has_member_arg(invoke.klass(), invoke.name())) { |
693 |
callee_size_of_parameters++; |
|
694 |
} |
|
1 | 695 |
} |
696 |
if (str.bci() < max_bci) { |
|
49189
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
697 |
Bytecodes::Code next_code = str.next(); |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
698 |
if (next_code >= 0) { |
1 | 699 |
// The interpreter oop map generator reports results before |
700 |
// the current bytecode has executed except in the case of |
|
701 |
// calls. It seems to be hard to tell whether the compiler |
|
702 |
// has emitted debug information matching the "state before" |
|
703 |
// a given bytecode or the state after, so we try both |
|
49189
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
704 |
if (!Bytecodes::is_invoke(cur_code) && cur_code != Bytecodes::_athrow) { |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
705 |
// Get expression stack size for the next bytecode |
51522
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
706 |
InterpreterOopMap next_mask; |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
707 |
OopMapCache::compute_one_oop_map(mh, str.bci(), &next_mask); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
708 |
next_mask_expression_stack_size = next_mask.expression_stack_size(); |
49189
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
709 |
if (Bytecodes::is_invoke(next_code)) { |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
710 |
Bytecode_invoke invoke(mh, str.bci()); |
51522
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
711 |
next_mask_expression_stack_size += invoke.size_of_parameters(); |
1 | 712 |
} |
49189
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
713 |
// Need to subtract off the size of the result type of |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
714 |
// the bytecode because this is not described in the |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
715 |
// debug info but returned to the interpreter in the TOS |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
716 |
// caching register |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
717 |
BasicType bytecode_result_type = Bytecodes::result_type(cur_code); |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
718 |
if (bytecode_result_type != T_ILLEGAL) { |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
719 |
top_frame_expression_stack_adjustment = type2size[bytecode_result_type]; |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
720 |
} |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
721 |
assert(top_frame_expression_stack_adjustment >= 0, "stack adjustment must be positive"); |
41e570d862b4
8148871: Possible wrong expression stack depth at deopt point
thartmann
parents:
49036
diff
changeset
|
722 |
try_next_mask = true; |
1 | 723 |
} |
724 |
} |
|
725 |
} |
|
726 |
||
727 |
// Verify stack depth and oops in frame |
|
728 |
// This assertion may be dependent on the platform we're running on and may need modification (tested on x86 and sparc) |
|
729 |
if (!( |
|
730 |
/* SPARC */ |
|
731 |
(iframe->interpreter_frame_expression_stack_size() == mask.expression_stack_size() + callee_size_of_parameters) || |
|
732 |
/* x86 */ |
|
733 |
(iframe->interpreter_frame_expression_stack_size() == mask.expression_stack_size() + callee_max_locals) || |
|
734 |
(try_next_mask && |
|
735 |
(iframe->interpreter_frame_expression_stack_size() == (next_mask_expression_stack_size - |
|
736 |
top_frame_expression_stack_adjustment))) || |
|
737 |
(is_top_frame && (exec_mode == Unpack_exception) && iframe->interpreter_frame_expression_stack_size() == 0) || |
|
25920
6f9fa66be5df
8049252: VerifyStack logic in Deoptimization::unpack_frames does not expect to see invoke bc at the top frame during normal deoptimization
kvn
parents:
25715
diff
changeset
|
738 |
(is_top_frame && (exec_mode == Unpack_uncommon_trap || exec_mode == Unpack_reexecute || el->should_reexecute()) && |
1 | 739 |
(iframe->interpreter_frame_expression_stack_size() == mask.expression_stack_size() + cur_invoke_parameter_size)) |
740 |
)) { |
|
51522
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
741 |
{ |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
742 |
ttyLocker ttyl; |
1 | 743 |
|
51522
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
744 |
// Print out some information that will help us debug the problem |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
745 |
tty->print_cr("Wrong number of expression stack elements during deoptimization"); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
746 |
tty->print_cr(" Error occurred while verifying frame %d (0..%d, 0 is topmost)", i, cur_array->frames() - 1); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
747 |
tty->print_cr(" Fabricated interpreter frame had %d expression stack elements", |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
748 |
iframe->interpreter_frame_expression_stack_size()); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
749 |
tty->print_cr(" Interpreter oop map had %d expression stack elements", mask.expression_stack_size()); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
750 |
tty->print_cr(" try_next_mask = %d", try_next_mask); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
751 |
tty->print_cr(" next_mask_expression_stack_size = %d", next_mask_expression_stack_size); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
752 |
tty->print_cr(" callee_size_of_parameters = %d", callee_size_of_parameters); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
753 |
tty->print_cr(" callee_max_locals = %d", callee_max_locals); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
754 |
tty->print_cr(" top_frame_expression_stack_adjustment = %d", top_frame_expression_stack_adjustment); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
755 |
tty->print_cr(" exec_mode = %d", exec_mode); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
756 |
tty->print_cr(" cur_invoke_parameter_size = %d", cur_invoke_parameter_size); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
757 |
tty->print_cr(" Thread = " INTPTR_FORMAT ", thread ID = %d", p2i(thread), thread->osthread()->thread_id()); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
758 |
tty->print_cr(" Interpreted frames:"); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
759 |
for (int k = 0; k < cur_array->frames(); k++) { |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
760 |
vframeArrayElement* el = cur_array->element(k); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
761 |
tty->print_cr(" %s (bci %d)", el->method()->name_and_sig_as_C_string(), el->bci()); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
762 |
} |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
763 |
cur_array->print_on_2(tty); |
b426c75da4b9
8209825: guarantee(false) failed: wrong number of expression stack elements during deopt
dlong
parents:
51073
diff
changeset
|
764 |
} // release tty lock before calling guarantee |
1 | 765 |
guarantee(false, "wrong number of expression stack elements during deopt"); |
766 |
} |
|
767 |
VerifyOopClosure verify; |
|
38074
8475fdc6dcc3
8154580: Save mirror in interpreter frame to enable cleanups of CLDClosure
coleenp
parents:
37484
diff
changeset
|
768 |
iframe->oops_interpreted_do(&verify, &rm, false); |
1 | 769 |
callee_size_of_parameters = mh->size_of_parameters(); |
770 |
callee_max_locals = mh->max_locals(); |
|
771 |
is_top_frame = false; |
|
772 |
} |
|
773 |
} |
|
774 |
#endif /* !PRODUCT */ |
|
775 |
||
776 |
||
777 |
return bt; |
|
778 |
JRT_END |
|
779 |
||
780 |
||
781 |
int Deoptimization::deoptimize_dependents() { |
|
782 |
Threads::deoptimized_wrt_marked_nmethods(); |
|
783 |
return 0; |
|
784 |
} |
|
785 |
||
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
786 |
Deoptimization::DeoptAction Deoptimization::_unloaded_action |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
787 |
= Deoptimization::Action_reinterpret; |
1 | 788 |
|
47799 | 789 |
#if COMPILER2_OR_JVMCI |
1 | 790 |
bool Deoptimization::realloc_objects(JavaThread* thread, frame* fr, GrowableArray<ScopeValue*>* objects, TRAPS) { |
46271
979ebd346ecf
8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents:
42040
diff
changeset
|
791 |
Handle pending_exception(THREAD, thread->pending_exception()); |
1 | 792 |
const char* exception_file = thread->exception_file(); |
793 |
int exception_line = thread->exception_line(); |
|
794 |
thread->clear_pending_exception(); |
|
795 |
||
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
796 |
bool failures = false; |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
797 |
|
1 | 798 |
for (int i = 0; i < objects->length(); i++) { |
799 |
assert(objects->at(i)->is_object(), "invalid debug information"); |
|
800 |
ObjectValue* sv = (ObjectValue*) objects->at(i); |
|
801 |
||
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
802 |
Klass* k = java_lang_Class::as_Klass(sv->klass()->as_ConstantOopReadValue()->value()()); |
1 | 803 |
oop obj = NULL; |
804 |
||
33611
9abd65805e19
8139203: Consistent naming for klass type predicates
coleenp
parents:
33593
diff
changeset
|
805 |
if (k->is_instance_klass()) { |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
806 |
InstanceKlass* ik = InstanceKlass::cast(k); |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
807 |
obj = ik->allocate_instance(THREAD); |
33611
9abd65805e19
8139203: Consistent naming for klass type predicates
coleenp
parents:
33593
diff
changeset
|
808 |
} else if (k->is_typeArray_klass()) { |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
809 |
TypeArrayKlass* ak = TypeArrayKlass::cast(k); |
1 | 810 |
assert(sv->field_size() % type2size[ak->element_type()] == 0, "non-integral array length"); |
811 |
int len = sv->field_size() / type2size[ak->element_type()]; |
|
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
812 |
obj = ak->allocate(len, THREAD); |
33611
9abd65805e19
8139203: Consistent naming for klass type predicates
coleenp
parents:
33593
diff
changeset
|
813 |
} else if (k->is_objArray_klass()) { |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
814 |
ObjArrayKlass* ak = ObjArrayKlass::cast(k); |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
815 |
obj = ak->allocate(sv->field_size(), THREAD); |
1 | 816 |
} |
817 |
||
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
818 |
if (obj == NULL) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
819 |
failures = true; |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
820 |
} |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
821 |
|
1 | 822 |
assert(sv->value().is_null(), "redundant reallocation"); |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
823 |
assert(obj != NULL || HAS_PENDING_EXCEPTION, "allocation should succeed or we should get an exception"); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
824 |
CLEAR_PENDING_EXCEPTION; |
1 | 825 |
sv->set_value(obj); |
826 |
} |
|
827 |
||
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
828 |
if (failures) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
829 |
THROW_OOP_(Universe::out_of_memory_error_realloc_objects(), failures); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
830 |
} else if (pending_exception.not_null()) { |
1 | 831 |
thread->set_pending_exception(pending_exception(), exception_file, exception_line); |
832 |
} |
|
833 |
||
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
834 |
return failures; |
1 | 835 |
} |
836 |
||
837 |
// restore elements of an eliminated type array |
|
838 |
void Deoptimization::reassign_type_array_elements(frame* fr, RegisterMap* reg_map, ObjectValue* sv, typeArrayOop obj, BasicType type) { |
|
839 |
int index = 0; |
|
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
840 |
intptr_t val; |
1 | 841 |
|
842 |
for (int i = 0; i < sv->field_size(); i++) { |
|
843 |
StackValue* value = StackValue::create_stack_value(fr, reg_map, sv->field_at(i)); |
|
844 |
switch(type) { |
|
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
845 |
case T_LONG: case T_DOUBLE: { |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
846 |
assert(value->type() == T_INT, "Agreement."); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
847 |
StackValue* low = |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
848 |
StackValue::create_stack_value(fr, reg_map, sv->field_at(++i)); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
849 |
#ifdef _LP64 |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
850 |
jlong res = (jlong)low->get_int(); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
851 |
#else |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
852 |
#ifdef SPARC |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
853 |
// For SPARC we have to swap high and low words. |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
854 |
jlong res = jlong_from((jint)low->get_int(), (jint)value->get_int()); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
855 |
#else |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
856 |
jlong res = jlong_from((jint)value->get_int(), (jint)low->get_int()); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
857 |
#endif //SPARC |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
858 |
#endif |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
859 |
obj->long_at_put(index, res); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
860 |
break; |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
861 |
} |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
862 |
|
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
863 |
// Have to cast to INT (32 bits) pointer to avoid little/big-endian problem. |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
864 |
case T_INT: case T_FLOAT: { // 4 bytes. |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
865 |
assert(value->type() == T_INT, "Agreement."); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
866 |
bool big_value = false; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
867 |
if (i + 1 < sv->field_size() && type == T_INT) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
868 |
if (sv->field_at(i)->is_location()) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
869 |
Location::Type type = ((LocationValue*) sv->field_at(i))->location().type(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
870 |
if (type == Location::dbl || type == Location::lng) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
871 |
big_value = true; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
872 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
873 |
} else if (sv->field_at(i)->is_constant_int()) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
874 |
ScopeValue* next_scope_field = sv->field_at(i + 1); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
875 |
if (next_scope_field->is_constant_long() || next_scope_field->is_constant_double()) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
876 |
big_value = true; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
877 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
878 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
879 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
880 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
881 |
if (big_value) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
882 |
StackValue* low = StackValue::create_stack_value(fr, reg_map, sv->field_at(++i)); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
883 |
#ifdef _LP64 |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
884 |
jlong res = (jlong)low->get_int(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
885 |
#else |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
886 |
#ifdef SPARC |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
887 |
// For SPARC we have to swap high and low words. |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
888 |
jlong res = jlong_from((jint)low->get_int(), (jint)value->get_int()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
889 |
#else |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
890 |
jlong res = jlong_from((jint)value->get_int(), (jint)low->get_int()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
891 |
#endif //SPARC |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
892 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
893 |
obj->int_at_put(index, (jint)*((jint*)&res)); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
894 |
obj->int_at_put(++index, (jint)*(((jint*)&res) + 1)); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
895 |
} else { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
896 |
val = value->get_int(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
897 |
obj->int_at_put(index, (jint)*((jint*)&val)); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
898 |
} |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
899 |
break; |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
900 |
} |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
901 |
|
37484
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
902 |
case T_SHORT: |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
903 |
assert(value->type() == T_INT, "Agreement."); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
904 |
val = value->get_int(); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
905 |
obj->short_at_put(index, (jshort)*((jint*)&val)); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
906 |
break; |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
907 |
|
37484
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
908 |
case T_CHAR: |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
909 |
assert(value->type() == T_INT, "Agreement."); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
910 |
val = value->get_int(); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
911 |
obj->char_at_put(index, (jchar)*((jint*)&val)); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
912 |
break; |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
913 |
|
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
914 |
case T_BYTE: |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
915 |
assert(value->type() == T_INT, "Agreement."); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
916 |
val = value->get_int(); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
917 |
obj->byte_at_put(index, (jbyte)*((jint*)&val)); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
918 |
break; |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
919 |
|
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
920 |
case T_BOOLEAN: |
217
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
921 |
assert(value->type() == T_INT, "Agreement."); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
922 |
val = value->get_int(); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
923 |
obj->bool_at_put(index, (jboolean)*((jint*)&val)); |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
924 |
break; |
c646ef2f5d58
6667620: (Escape Analysis) fix deoptimization for scalar replaced objects
kvn
parents:
1
diff
changeset
|
925 |
|
1 | 926 |
default: |
927 |
ShouldNotReachHere(); |
|
928 |
} |
|
929 |
index++; |
|
930 |
} |
|
931 |
} |
|
932 |
||
933 |
||
934 |
// restore fields of an eliminated object array |
|
935 |
void Deoptimization::reassign_object_array_elements(frame* fr, RegisterMap* reg_map, ObjectValue* sv, objArrayOop obj) { |
|
936 |
for (int i = 0; i < sv->field_size(); i++) { |
|
937 |
StackValue* value = StackValue::create_stack_value(fr, reg_map, sv->field_at(i)); |
|
938 |
assert(value->type() == T_OBJECT, "object element expected"); |
|
939 |
obj->obj_at_put(i, value->get_obj()()); |
|
940 |
} |
|
941 |
} |
|
942 |
||
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
943 |
class ReassignedField { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
944 |
public: |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
945 |
int _offset; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
946 |
BasicType _type; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
947 |
public: |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
948 |
ReassignedField() { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
949 |
_offset = 0; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
950 |
_type = T_ILLEGAL; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
951 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
952 |
}; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
953 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
954 |
int compare(ReassignedField* left, ReassignedField* right) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
955 |
return left->_offset - right->_offset; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
956 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
957 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
958 |
// Restore fields of an eliminated instance object using the same field order |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
959 |
// returned by HotSpotResolvedObjectTypeImpl.getInstanceFields(true) |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
960 |
static int reassign_fields_by_klass(InstanceKlass* klass, frame* fr, RegisterMap* reg_map, ObjectValue* sv, int svIndex, oop obj, bool skip_internal) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
961 |
if (klass->superklass() != NULL) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
962 |
svIndex = reassign_fields_by_klass(klass->superklass(), fr, reg_map, sv, svIndex, obj, skip_internal); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
963 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
964 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
965 |
GrowableArray<ReassignedField>* fields = new GrowableArray<ReassignedField>(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
966 |
for (AllFieldStream fs(klass); !fs.done(); fs.next()) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
967 |
if (!fs.access_flags().is_static() && (!skip_internal || !fs.access_flags().is_internal())) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
968 |
ReassignedField field; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
969 |
field._offset = fs.offset(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
970 |
field._type = FieldType::basic_type(fs.signature()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
971 |
fields->append(field); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
972 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
973 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
974 |
fields->sort(compare); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
975 |
for (int i = 0; i < fields->length(); i++) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
976 |
intptr_t val; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
977 |
ScopeValue* scope_field = sv->field_at(svIndex); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
978 |
StackValue* value = StackValue::create_stack_value(fr, reg_map, scope_field); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
979 |
int offset = fields->at(i)._offset; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
980 |
BasicType type = fields->at(i)._type; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
981 |
switch (type) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
982 |
case T_OBJECT: case T_ARRAY: |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
983 |
assert(value->type() == T_OBJECT, "Agreement."); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
984 |
obj->obj_field_put(offset, value->get_obj()()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
985 |
break; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
986 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
987 |
// Have to cast to INT (32 bits) pointer to avoid little/big-endian problem. |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
988 |
case T_INT: case T_FLOAT: { // 4 bytes. |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
989 |
assert(value->type() == T_INT, "Agreement."); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
990 |
bool big_value = false; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
991 |
if (i+1 < fields->length() && fields->at(i+1)._type == T_INT) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
992 |
if (scope_field->is_location()) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
993 |
Location::Type type = ((LocationValue*) scope_field)->location().type(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
994 |
if (type == Location::dbl || type == Location::lng) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
995 |
big_value = true; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
996 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
997 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
998 |
if (scope_field->is_constant_int()) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
999 |
ScopeValue* next_scope_field = sv->field_at(svIndex + 1); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1000 |
if (next_scope_field->is_constant_long() || next_scope_field->is_constant_double()) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1001 |
big_value = true; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1002 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1003 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1004 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1005 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1006 |
if (big_value) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1007 |
i++; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1008 |
assert(i < fields->length(), "second T_INT field needed"); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1009 |
assert(fields->at(i)._type == T_INT, "T_INT field needed"); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1010 |
} else { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1011 |
val = value->get_int(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1012 |
obj->int_field_put(offset, (jint)*((jint*)&val)); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1013 |
break; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1014 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1015 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1016 |
/* no break */ |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1017 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1018 |
case T_LONG: case T_DOUBLE: { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1019 |
assert(value->type() == T_INT, "Agreement."); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1020 |
StackValue* low = StackValue::create_stack_value(fr, reg_map, sv->field_at(++svIndex)); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1021 |
#ifdef _LP64 |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1022 |
jlong res = (jlong)low->get_int(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1023 |
#else |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1024 |
#ifdef SPARC |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1025 |
// For SPARC we have to swap high and low words. |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1026 |
jlong res = jlong_from((jint)low->get_int(), (jint)value->get_int()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1027 |
#else |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1028 |
jlong res = jlong_from((jint)value->get_int(), (jint)low->get_int()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1029 |
#endif //SPARC |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1030 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1031 |
obj->long_field_put(offset, res); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1032 |
break; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1033 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1034 |
|
37484
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1035 |
case T_SHORT: |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1036 |
assert(value->type() == T_INT, "Agreement."); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1037 |
val = value->get_int(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1038 |
obj->short_field_put(offset, (jshort)*((jint*)&val)); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1039 |
break; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1040 |
|
37484
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1041 |
case T_CHAR: |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1042 |
assert(value->type() == T_INT, "Agreement."); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1043 |
val = value->get_int(); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1044 |
obj->char_field_put(offset, (jchar)*((jint*)&val)); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1045 |
break; |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1046 |
|
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1047 |
case T_BYTE: |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1048 |
assert(value->type() == T_INT, "Agreement."); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1049 |
val = value->get_int(); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1050 |
obj->byte_field_put(offset, (jbyte)*((jint*)&val)); |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1051 |
break; |
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1052 |
|
444a8c40cf95
8149367: PolicyQualifierInfo/index_Ctor JCk test fails with IOE: Invalid encoding for PolicyQualifierInfo
kvn
parents:
35499
diff
changeset
|
1053 |
case T_BOOLEAN: |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1054 |
assert(value->type() == T_INT, "Agreement."); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1055 |
val = value->get_int(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1056 |
obj->bool_field_put(offset, (jboolean)*((jint*)&val)); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1057 |
break; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1058 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1059 |
default: |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1060 |
ShouldNotReachHere(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1061 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1062 |
svIndex++; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1063 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1064 |
return svIndex; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1065 |
} |
1 | 1066 |
|
1067 |
// restore fields of all eliminated objects and arrays |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1068 |
void Deoptimization::reassign_fields(frame* fr, RegisterMap* reg_map, GrowableArray<ScopeValue*>* objects, bool realloc_failures, bool skip_internal) { |
1 | 1069 |
for (int i = 0; i < objects->length(); i++) { |
1070 |
ObjectValue* sv = (ObjectValue*) objects->at(i); |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
1071 |
Klass* k = java_lang_Class::as_Klass(sv->klass()->as_ConstantOopReadValue()->value()()); |
1 | 1072 |
Handle obj = sv->value(); |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1073 |
assert(obj.not_null() || realloc_failures, "reallocation was missed"); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1074 |
if (PrintDeoptimizationDetails) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1075 |
tty->print_cr("reassign fields for object of type %s!", k->name()->as_C_string()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1076 |
} |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1077 |
if (obj.is_null()) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1078 |
continue; |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1079 |
} |
1 | 1080 |
|
33611
9abd65805e19
8139203: Consistent naming for klass type predicates
coleenp
parents:
33593
diff
changeset
|
1081 |
if (k->is_instance_klass()) { |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
1082 |
InstanceKlass* ik = InstanceKlass::cast(k); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1083 |
reassign_fields_by_klass(ik, fr, reg_map, sv, 0, obj(), skip_internal); |
33611
9abd65805e19
8139203: Consistent naming for klass type predicates
coleenp
parents:
33593
diff
changeset
|
1084 |
} else if (k->is_typeArray_klass()) { |
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
1085 |
TypeArrayKlass* ak = TypeArrayKlass::cast(k); |
1 | 1086 |
reassign_type_array_elements(fr, reg_map, sv, (typeArrayOop) obj(), ak->element_type()); |
33611
9abd65805e19
8139203: Consistent naming for klass type predicates
coleenp
parents:
33593
diff
changeset
|
1087 |
} else if (k->is_objArray_klass()) { |
1 | 1088 |
reassign_object_array_elements(fr, reg_map, sv, (objArrayOop) obj()); |
1089 |
} |
|
1090 |
} |
|
1091 |
} |
|
1092 |
||
1093 |
||
1094 |
// relock objects for which synchronization was eliminated |
|
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1095 |
void Deoptimization::relock_objects(GrowableArray<MonitorInfo*>* monitors, JavaThread* thread, bool realloc_failures) { |
1 | 1096 |
for (int i = 0; i < monitors->length(); i++) { |
256
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
1097 |
MonitorInfo* mon_info = monitors->at(i); |
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
1098 |
if (mon_info->eliminated()) { |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1099 |
assert(!mon_info->owner_is_scalar_replaced() || realloc_failures, "reallocation was missed"); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1100 |
if (!mon_info->owner_is_scalar_replaced()) { |
46271
979ebd346ecf
8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents:
42040
diff
changeset
|
1101 |
Handle obj(thread, mon_info->owner()); |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1102 |
markOop mark = obj->mark(); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1103 |
if (UseBiasedLocking && mark->has_bias_pattern()) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1104 |
// New allocated objects may have the mark set to anonymously biased. |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1105 |
// Also the deoptimized method may called methods with synchronization |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1106 |
// where the thread-local object is bias locked to the current thread. |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1107 |
assert(mark->is_biased_anonymously() || |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1108 |
mark->biased_locker() == thread, "should be locked to current thread"); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1109 |
// Reset mark word to unbiased prototype. |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1110 |
markOop unbiased_prototype = markOopDesc::prototype()->set_age(mark->age()); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1111 |
obj->set_mark(unbiased_prototype); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1112 |
} |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1113 |
BasicLock* lock = mon_info->lock(); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1114 |
ObjectSynchronizer::slow_enter(obj, lock, thread); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1115 |
assert(mon_info->owner()->is_locked(), "object must be locked now"); |
256
70fdc3927a4e
6681646: Relocking of a scalar replaced object during deoptimization is broken
kvn
parents:
217
diff
changeset
|
1116 |
} |
1 | 1117 |
} |
1118 |
} |
|
1119 |
} |
|
1120 |
||
1121 |
||
1122 |
#ifndef PRODUCT |
|
1123 |
// print information about reallocated objects |
|
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1124 |
void Deoptimization::print_objects(GrowableArray<ScopeValue*>* objects, bool realloc_failures) { |
1 | 1125 |
fieldDescriptor fd; |
1126 |
||
1127 |
for (int i = 0; i < objects->length(); i++) { |
|
1128 |
ObjectValue* sv = (ObjectValue*) objects->at(i); |
|
46329
53ccc37bda19
8155672: Remove instanceKlassHandles and KlassHandles
coleenp
parents:
46327
diff
changeset
|
1129 |
Klass* k = java_lang_Class::as_Klass(sv->klass()->as_ConstantOopReadValue()->value()()); |
1 | 1130 |
Handle obj = sv->value(); |
1131 |
||
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
1132 |
tty->print(" object <" INTPTR_FORMAT "> of type ", p2i(sv->value()())); |
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
1133 |
k->print_value(); |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1134 |
assert(obj.not_null() || realloc_failures, "reallocation was missed"); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1135 |
if (obj.is_null()) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1136 |
tty->print(" allocation failed"); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1137 |
} else { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1138 |
tty->print(" allocated (%d bytes)", obj->size() * HeapWordSize); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1139 |
} |
1 | 1140 |
tty->cr(); |
1141 |
||
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1142 |
if (Verbose && !obj.is_null()) { |
1 | 1143 |
k->oop_print_on(obj(), tty); |
1144 |
} |
|
1145 |
} |
|
1146 |
} |
|
1147 |
#endif |
|
47799 | 1148 |
#endif // COMPILER2_OR_JVMCI |
1 | 1149 |
|
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1150 |
vframeArray* Deoptimization::create_vframeArray(JavaThread* thread, frame fr, RegisterMap *reg_map, GrowableArray<compiledVFrame*>* chunk, bool realloc_failures) { |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
1151 |
Events::log(thread, "DEOPT PACKING pc=" INTPTR_FORMAT " sp=" INTPTR_FORMAT, p2i(fr.pc()), p2i(fr.sp())); |
1 | 1152 |
|
1153 |
#ifndef PRODUCT |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1154 |
if (PrintDeoptimizationDetails) { |
1 | 1155 |
ttyLocker ttyl; |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
1156 |
tty->print("DEOPT PACKING thread " INTPTR_FORMAT " ", p2i(thread)); |
1 | 1157 |
fr.print_on(tty); |
1158 |
tty->print_cr(" Virtual frames (innermost first):"); |
|
1159 |
for (int index = 0; index < chunk->length(); index++) { |
|
1160 |
compiledVFrame* vf = chunk->at(index); |
|
1161 |
tty->print(" %2d - ", index); |
|
1162 |
vf->print_value(); |
|
1163 |
int bci = chunk->at(index)->raw_bci(); |
|
1164 |
const char* code_name; |
|
1165 |
if (bci == SynchronizationEntryBCI) { |
|
1166 |
code_name = "sync entry"; |
|
1167 |
} else { |
|
7913 | 1168 |
Bytecodes::Code code = vf->method()->code_at(bci); |
1 | 1169 |
code_name = Bytecodes::name(code); |
1170 |
} |
|
1171 |
tty->print(" - %s", code_name); |
|
1172 |
tty->print_cr(" @ bci %d ", bci); |
|
1173 |
if (Verbose) { |
|
1174 |
vf->print(); |
|
1175 |
tty->cr(); |
|
1176 |
} |
|
1177 |
} |
|
1178 |
} |
|
1179 |
#endif |
|
1180 |
||
1181 |
// Register map for next frame (used for stack crawl). We capture |
|
1182 |
// the state of the deopt'ing frame's caller. Thus if we need to |
|
1183 |
// stuff a C2I adapter we can properly fill in the callee-save |
|
1184 |
// register locations. |
|
1185 |
frame caller = fr.sender(reg_map); |
|
1186 |
int frame_size = caller.sp() - fr.sp(); |
|
1187 |
||
1188 |
frame sender = caller; |
|
1189 |
||
1190 |
// Since the Java thread being deoptimized will eventually adjust it's own stack, |
|
1191 |
// the vframeArray containing the unpacking information is allocated in the C heap. |
|
1192 |
// For Compiler1, the caller of the deoptimized frame is saved for use by unpack_frames(). |
|
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1193 |
vframeArray* array = vframeArray::allocate(thread, frame_size, chunk, reg_map, sender, caller, fr, realloc_failures); |
1 | 1194 |
|
1195 |
// Compare the vframeArray to the collected vframes |
|
1196 |
assert(array->structural_compare(thread, chunk), "just checking"); |
|
1197 |
||
1198 |
#ifndef PRODUCT |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1199 |
if (PrintDeoptimizationDetails) { |
1 | 1200 |
ttyLocker ttyl; |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
1201 |
tty->print_cr(" Created vframeArray " INTPTR_FORMAT, p2i(array)); |
1 | 1202 |
} |
1203 |
#endif // PRODUCT |
|
1204 |
||
1205 |
return array; |
|
1206 |
} |
|
1207 |
||
47799 | 1208 |
#if COMPILER2_OR_JVMCI |
28039
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1209 |
void Deoptimization::pop_frames_failed_reallocs(JavaThread* thread, vframeArray* array) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1210 |
// Reallocation of some scalar replaced objects failed. Record |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1211 |
// that we need to pop all the interpreter frames for the |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1212 |
// deoptimized compiled frame. |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1213 |
assert(thread->frames_to_pop_failed_realloc() == 0, "missed frames to pop?"); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1214 |
thread->set_frames_to_pop_failed_realloc(array->frames()); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1215 |
// Unlock all monitors here otherwise the interpreter will see a |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1216 |
// mix of locked and unlocked monitors (because of failed |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1217 |
// reallocations of synchronized objects) and be confused. |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1218 |
for (int i = 0; i < array->frames(); i++) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1219 |
MonitorChunk* monitors = array->element(i)->monitors(); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1220 |
if (monitors != NULL) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1221 |
for (int j = 0; j < monitors->number_of_monitors(); j++) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1222 |
BasicObjectLock* src = monitors->at(j); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1223 |
if (src->obj() != NULL) { |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1224 |
ObjectSynchronizer::fast_exit(src->obj(), src->lock(), thread); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1225 |
} |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1226 |
} |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1227 |
array->element(i)->free_monitors(thread); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1228 |
#ifdef ASSERT |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1229 |
array->element(i)->set_removed_monitors(); |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1230 |
#endif |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1231 |
} |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1232 |
} |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1233 |
} |
bf5a8340bf8a
6898462: The escape analysis with G1 cause crash assertion src/share/vm/runtime/vframeArray.cpp:94
roland
parents:
27880
diff
changeset
|
1234 |
#endif |
1 | 1235 |
|
1236 |
static void collect_monitors(compiledVFrame* cvf, GrowableArray<Handle>* objects_to_revoke) { |
|
1237 |
GrowableArray<MonitorInfo*>* monitors = cvf->monitors(); |
|
46271
979ebd346ecf
8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents:
42040
diff
changeset
|
1238 |
Thread* thread = Thread::current(); |
1 | 1239 |
for (int i = 0; i < monitors->length(); i++) { |
1240 |
MonitorInfo* mon_info = monitors->at(i); |
|
3171
aa289b22b577
6837472: com/sun/jdi/MonitorFrameInfo.java fails with AggressiveOpts in 6u14
kvn
parents:
670
diff
changeset
|
1241 |
if (!mon_info->eliminated() && mon_info->owner() != NULL) { |
46271
979ebd346ecf
8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents:
42040
diff
changeset
|
1242 |
objects_to_revoke->append(Handle(thread, mon_info->owner())); |
1 | 1243 |
} |
1244 |
} |
|
1245 |
} |
|
1246 |
||
1247 |
||
1248 |
void Deoptimization::revoke_biases_of_monitors(JavaThread* thread, frame fr, RegisterMap* map) { |
|
1249 |
if (!UseBiasedLocking) { |
|
1250 |
return; |
|
1251 |
} |
|
1252 |
||
1253 |
GrowableArray<Handle>* objects_to_revoke = new GrowableArray<Handle>(); |
|
1254 |
||
1255 |
// Unfortunately we don't have a RegisterMap available in most of |
|
1256 |
// the places we want to call this routine so we need to walk the |
|
1257 |
// stack again to update the register map. |
|
1258 |
if (map == NULL || !map->update_map()) { |
|
1259 |
StackFrameStream sfs(thread, true); |
|
1260 |
bool found = false; |
|
1261 |
while (!found && !sfs.is_done()) { |
|
1262 |
frame* cur = sfs.current(); |
|
1263 |
sfs.next(); |
|
1264 |
found = cur->id() == fr.id(); |
|
1265 |
} |
|
1266 |
assert(found, "frame to be deoptimized not found on target thread's stack"); |
|
1267 |
map = sfs.register_map(); |
|
1268 |
} |
|
1269 |
||
1270 |
vframe* vf = vframe::new_vframe(&fr, map, thread); |
|
1271 |
compiledVFrame* cvf = compiledVFrame::cast(vf); |
|
1272 |
// Revoke monitors' biases in all scopes |
|
1273 |
while (!cvf->is_top()) { |
|
1274 |
collect_monitors(cvf, objects_to_revoke); |
|
1275 |
cvf = compiledVFrame::cast(cvf->sender()); |
|
1276 |
} |
|
1277 |
collect_monitors(cvf, objects_to_revoke); |
|
1278 |
||
1279 |
if (SafepointSynchronize::is_at_safepoint()) { |
|
1280 |
BiasedLocking::revoke_at_safepoint(objects_to_revoke); |
|
1281 |
} else { |
|
1282 |
BiasedLocking::revoke(objects_to_revoke); |
|
1283 |
} |
|
1284 |
} |
|
1285 |
||
1286 |
||
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1287 |
void Deoptimization::deoptimize_single_frame(JavaThread* thread, frame fr, Deoptimization::DeoptReason reason) { |
1 | 1288 |
assert(fr.can_be_deoptimized(), "checking frame type"); |
1289 |
||
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1290 |
gather_statistics(reason, Action_none, Bytecodes::_illegal); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1291 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1292 |
if (LogCompilation && xtty != NULL) { |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
1293 |
CompiledMethod* cm = fr.cb()->as_compiled_method_or_null(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
1294 |
assert(cm != NULL, "only compiled methods can deopt"); |
1 | 1295 |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1296 |
ttyLocker ttyl; |
38242
c9013a328fe2
8156126: LogCompilation: Dump additional info about deoptimization events
jcm
parents:
38144
diff
changeset
|
1297 |
xtty->begin_head("deoptimized thread='" UINTX_FORMAT "' reason='%s' pc='" INTPTR_FORMAT "'",(uintx)thread->osthread()->thread_id(), trap_reason_name(reason), p2i(fr.pc())); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
1298 |
cm->log_identity(xtty); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1299 |
xtty->end_head(); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
1300 |
for (ScopeDesc* sd = cm->scope_desc_at(fr.pc()); ; sd = sd->sender()) { |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1301 |
xtty->begin_elem("jvms bci='%d'", sd->bci()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1302 |
xtty->method(sd->method()); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1303 |
xtty->end_elem(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1304 |
if (sd->is_top()) break; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1305 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1306 |
xtty->tail("deoptimized"); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1307 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1308 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1309 |
// Patch the compiled method so that when execution returns to it we will |
1 | 1310 |
// deopt the execution state and return to the interpreter. |
1311 |
fr.deoptimize(thread); |
|
1312 |
} |
|
1313 |
||
1314 |
void Deoptimization::deoptimize(JavaThread* thread, frame fr, RegisterMap *map) { |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1315 |
deoptimize(thread, fr, map, Reason_constraint); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1316 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1317 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1318 |
void Deoptimization::deoptimize(JavaThread* thread, frame fr, RegisterMap *map, DeoptReason reason) { |
1 | 1319 |
// Deoptimize only if the frame comes from compile code. |
1320 |
// Do not deoptimize the frame which is already patched |
|
1321 |
// during the execution of the loops below. |
|
1322 |
if (!fr.is_compiled_frame() || fr.is_deoptimized_frame()) { |
|
1323 |
return; |
|
1324 |
} |
|
1325 |
ResourceMark rm; |
|
1326 |
DeoptimizationMarker dm; |
|
1327 |
if (UseBiasedLocking) { |
|
1328 |
revoke_biases_of_monitors(thread, fr, map); |
|
1329 |
} |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1330 |
deoptimize_single_frame(thread, fr, reason); |
1 | 1331 |
|
1332 |
} |
|
1333 |
||
46998
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1334 |
#if INCLUDE_JVMCI |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1335 |
address Deoptimization::deoptimize_for_missing_exception_handler(CompiledMethod* cm) { |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1336 |
// there is no exception handler for this pc => deoptimize |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1337 |
cm->make_not_entrant(); |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1338 |
|
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1339 |
// Use Deoptimization::deoptimize for all of its side-effects: |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1340 |
// revoking biases of monitors, gathering traps statistics, logging... |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1341 |
// it also patches the return pc but we do not care about that |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1342 |
// since we return a continuation to the deopt_blob below. |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1343 |
JavaThread* thread = JavaThread::current(); |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1344 |
RegisterMap reg_map(thread, UseBiasedLocking); |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1345 |
frame runtime_frame = thread->last_frame(); |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1346 |
frame caller_frame = runtime_frame.sender(®_map); |
47692
0d2d80c0d425
8189649: AOT: assert(caller_frame.cb()->as_nmethod_or_null() == cm) failed: expect top frame nmethod
dlong
parents:
47687
diff
changeset
|
1347 |
assert(caller_frame.cb()->as_compiled_method_or_null() == cm, "expect top frame compiled method"); |
46998
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1348 |
Deoptimization::deoptimize(thread, caller_frame, ®_map, Deoptimization::Reason_not_compiled_exception_handler); |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1349 |
|
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1350 |
MethodData* trap_mdo = get_method_data(thread, cm->method(), true); |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1351 |
if (trap_mdo != NULL) { |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1352 |
trap_mdo->inc_trap_count(Deoptimization::Reason_not_compiled_exception_handler); |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1353 |
} |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1354 |
|
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1355 |
return SharedRuntime::deopt_blob()->unpack_with_exception_in_tls(); |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1356 |
} |
efb404beeefb
8181858: [JVMCI] JVMCI should update the trap counters when invalidating for Reason_not_compiled_exception_handler
never
parents:
46968
diff
changeset
|
1357 |
#endif |
1 | 1358 |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1359 |
void Deoptimization::deoptimize_frame_internal(JavaThread* thread, intptr_t* id, DeoptReason reason) { |
7106
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
6740
diff
changeset
|
1360 |
assert(thread == Thread::current() || SafepointSynchronize::is_at_safepoint(), |
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
6740
diff
changeset
|
1361 |
"can only deoptimize other thread at a safepoint"); |
1 | 1362 |
// Compute frame and register map based on thread and sp. |
1363 |
RegisterMap reg_map(thread, UseBiasedLocking); |
|
1364 |
frame fr = thread->last_frame(); |
|
1365 |
while (fr.id() != id) { |
|
1366 |
fr = fr.sender(®_map); |
|
1367 |
} |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1368 |
deoptimize(thread, fr, ®_map, reason); |
1 | 1369 |
} |
1370 |
||
1371 |
||
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1372 |
void Deoptimization::deoptimize_frame(JavaThread* thread, intptr_t* id, DeoptReason reason) { |
7106
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
6740
diff
changeset
|
1373 |
if (thread == Thread::current()) { |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1374 |
Deoptimization::deoptimize_frame_internal(thread, id, reason); |
7106
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
6740
diff
changeset
|
1375 |
} else { |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1376 |
VM_DeoptimizeFrame deopt(thread, id, reason); |
7106
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
6740
diff
changeset
|
1377 |
VMThread::execute(&deopt); |
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
6740
diff
changeset
|
1378 |
} |
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
6740
diff
changeset
|
1379 |
} |
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
6740
diff
changeset
|
1380 |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1381 |
void Deoptimization::deoptimize_frame(JavaThread* thread, intptr_t* id) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1382 |
deoptimize_frame(thread, id, Reason_constraint); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1383 |
} |
7106
867c9d296c6b
6968367: can_post_on_exceptions is still using VM_DeoptimizeFrame in some places
never
parents:
6740
diff
changeset
|
1384 |
|
1 | 1385 |
// JVMTI PopFrame support |
1386 |
JRT_LEAF(void, Deoptimization::popframe_preserve_args(JavaThread* thread, int bytes_to_save, void* start_address)) |
|
1387 |
{ |
|
1388 |
thread->popframe_preserve_args(in_ByteSize(bytes_to_save), start_address); |
|
1389 |
} |
|
1390 |
JRT_END |
|
1391 |
||
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1392 |
MethodData* |
46727
6e4a84748e2c
8183039: Re-examine methodHandle methods uninlined by 8144256
coleenp
parents:
46560
diff
changeset
|
1393 |
Deoptimization::get_method_data(JavaThread* thread, const methodHandle& m, |
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1394 |
bool create_if_missing) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1395 |
Thread* THREAD = thread; |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1396 |
MethodData* mdo = m()->method_data(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1397 |
if (mdo == NULL && create_if_missing && !HAS_PENDING_EXCEPTION) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1398 |
// Build an MDO. Ignore errors like OutOfMemory; |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1399 |
// that simply means we won't have an MDO to update. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1400 |
Method::build_interpreter_method_data(m, THREAD); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1401 |
if (HAS_PENDING_EXCEPTION) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1402 |
assert((PENDING_EXCEPTION->is_a(SystemDictionary::OutOfMemoryError_klass())), "we expect only an OOM error here"); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1403 |
CLEAR_PENDING_EXCEPTION; |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1404 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1405 |
mdo = m()->method_data(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1406 |
} |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1407 |
return mdo; |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1408 |
} |
1 | 1409 |
|
47799 | 1410 |
#if COMPILER2_OR_JVMCI |
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33198
diff
changeset
|
1411 |
void Deoptimization::load_class_by_index(const constantPoolHandle& constant_pool, int index, TRAPS) { |
1 | 1412 |
// in case of an unresolved klass entry, load the class. |
1413 |
if (constant_pool->tag_at(index).is_unresolved_klass()) { |
|
27408
9a8090dd6ec3
8058093: Test nsk/stress/jck60/jck60014: assert in src/share/vm/oops/constantPool.cpp: should not be resolved otherwise
coleenp
parents:
26436
diff
changeset
|
1414 |
Klass* tk = constant_pool->klass_at_ignore_error(index, CHECK); |
1 | 1415 |
return; |
1416 |
} |
|
1417 |
||
1418 |
if (!constant_pool->tag_at(index).is_symbol()) return; |
|
1419 |
||
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
13974
diff
changeset
|
1420 |
Handle class_loader (THREAD, constant_pool->pool_holder()->class_loader()); |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7913
diff
changeset
|
1421 |
Symbol* symbol = constant_pool->symbol_at(index); |
1 | 1422 |
|
1423 |
// class name? |
|
51997
9ce37fa2e179
8209138: Symbol constructor uses u1 as the element type of its name argument
hseigel
parents:
51522
diff
changeset
|
1424 |
if (symbol->char_at(0) != '(') { |
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
13974
diff
changeset
|
1425 |
Handle protection_domain (THREAD, constant_pool->pool_holder()->protection_domain()); |
1 | 1426 |
SystemDictionary::resolve_or_null(symbol, class_loader, protection_domain, CHECK); |
1427 |
return; |
|
1428 |
} |
|
1429 |
||
1430 |
// then it must be a signature! |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7913
diff
changeset
|
1431 |
ResourceMark rm(THREAD); |
1 | 1432 |
for (SignatureStream ss(symbol); !ss.is_done(); ss.next()) { |
1433 |
if (ss.is_object()) { |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7913
diff
changeset
|
1434 |
Symbol* class_name = ss.as_symbol(CHECK); |
14391
df0a1573d5bd
8000725: NPG: method_holder() and pool_holder() and pool_holder field should be InstanceKlass
coleenp
parents:
13974
diff
changeset
|
1435 |
Handle protection_domain (THREAD, constant_pool->pool_holder()->protection_domain()); |
1 | 1436 |
SystemDictionary::resolve_or_null(class_name, class_loader, protection_domain, CHECK); |
1437 |
} |
|
1438 |
} |
|
1439 |
} |
|
1440 |
||
1441 |
||
33593
60764a78fa5c
8140274: methodHandles and constantPoolHandles should be passed as const references
coleenp
parents:
33198
diff
changeset
|
1442 |
void Deoptimization::load_class_by_index(const constantPoolHandle& constant_pool, int index) { |
1 | 1443 |
EXCEPTION_MARK; |
1444 |
load_class_by_index(constant_pool, index, THREAD); |
|
1445 |
if (HAS_PENDING_EXCEPTION) { |
|
1446 |
// Exception happened during classloading. We ignore the exception here, since it |
|
22241
58579a498f3a
8029873: compiler/uncommontrap/TestStackBangRbp.java crashes with SIGSEGV
roland
parents:
20282
diff
changeset
|
1447 |
// is going to be rethrown since the current activation is going to be deoptimized and |
1 | 1448 |
// the interpreter will re-execute the bytecode. |
1449 |
CLEAR_PENDING_EXCEPTION; |
|
22241
58579a498f3a
8029873: compiler/uncommontrap/TestStackBangRbp.java crashes with SIGSEGV
roland
parents:
20282
diff
changeset
|
1450 |
// Class loading called java code which may have caused a stack |
58579a498f3a
8029873: compiler/uncommontrap/TestStackBangRbp.java crashes with SIGSEGV
roland
parents:
20282
diff
changeset
|
1451 |
// overflow. If the exception was thrown right before the return |
58579a498f3a
8029873: compiler/uncommontrap/TestStackBangRbp.java crashes with SIGSEGV
roland
parents:
20282
diff
changeset
|
1452 |
// to the runtime the stack is no longer guarded. Reguard the |
58579a498f3a
8029873: compiler/uncommontrap/TestStackBangRbp.java crashes with SIGSEGV
roland
parents:
20282
diff
changeset
|
1453 |
// stack otherwise if we return to the uncommon trap blob and the |
58579a498f3a
8029873: compiler/uncommontrap/TestStackBangRbp.java crashes with SIGSEGV
roland
parents:
20282
diff
changeset
|
1454 |
// stack bang causes a stack overflow we crash. |
58579a498f3a
8029873: compiler/uncommontrap/TestStackBangRbp.java crashes with SIGSEGV
roland
parents:
20282
diff
changeset
|
1455 |
assert(THREAD->is_Java_thread(), "only a java thread can be here"); |
58579a498f3a
8029873: compiler/uncommontrap/TestStackBangRbp.java crashes with SIGSEGV
roland
parents:
20282
diff
changeset
|
1456 |
JavaThread* thread = (JavaThread*)THREAD; |
35071
a0910b1d3e0d
8046936: JEP 270: Reserved Stack Areas for Critical Sections
fparain
parents:
34651
diff
changeset
|
1457 |
bool guard_pages_enabled = thread->stack_guards_enabled(); |
22241
58579a498f3a
8029873: compiler/uncommontrap/TestStackBangRbp.java crashes with SIGSEGV
roland
parents:
20282
diff
changeset
|
1458 |
if (!guard_pages_enabled) guard_pages_enabled = thread->reguard_stack(); |
58579a498f3a
8029873: compiler/uncommontrap/TestStackBangRbp.java crashes with SIGSEGV
roland
parents:
20282
diff
changeset
|
1459 |
assert(guard_pages_enabled, "stack banging in uncommon trap blob may cause crash"); |
1 | 1460 |
} |
1461 |
} |
|
1462 |
||
1463 |
JRT_ENTRY(void, Deoptimization::uncommon_trap_inner(JavaThread* thread, jint trap_request)) { |
|
1464 |
HandleMark hm; |
|
1465 |
||
1466 |
// uncommon_trap() is called at the beginning of the uncommon trap |
|
1467 |
// handler. Note this fact before we start generating temporary frames |
|
1468 |
// that can confuse an asynchronous stack walker. This counter is |
|
1469 |
// decremented at the end of unpack_frames(). |
|
1470 |
thread->inc_in_deopt_handler(); |
|
1471 |
||
1472 |
// We need to update the map if we have biased locking. |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1473 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1474 |
// JVMCI might need to get an exception from the stack, which in turn requires the register map to be valid |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1475 |
RegisterMap reg_map(thread, true); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1476 |
#else |
1 | 1477 |
RegisterMap reg_map(thread, UseBiasedLocking); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1478 |
#endif |
1 | 1479 |
frame stub_frame = thread->last_frame(); |
1480 |
frame fr = stub_frame.sender(®_map); |
|
1481 |
// Make sure the calling nmethod is not getting deoptimized and removed |
|
1482 |
// before we are done with it. |
|
1483 |
nmethodLocker nl(fr.pc()); |
|
1484 |
||
11636
3c07b54482a5
7141200: log some interesting information in ring buffers for crashes
never
parents:
11445
diff
changeset
|
1485 |
// Log a message |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1486 |
Events::log(thread, "Uncommon trap: trap_request=" PTR32_FORMAT " fr.pc=" INTPTR_FORMAT " relative=" INTPTR_FORMAT, |
33198 | 1487 |
trap_request, p2i(fr.pc()), fr.pc() - fr.cb()->code_begin()); |
11636
3c07b54482a5
7141200: log some interesting information in ring buffers for crashes
never
parents:
11445
diff
changeset
|
1488 |
|
1 | 1489 |
{ |
1490 |
ResourceMark rm; |
|
1491 |
||
1492 |
// Revoke biases of any monitors in the frame to ensure we can migrate them |
|
1493 |
revoke_biases_of_monitors(thread, fr, ®_map); |
|
1494 |
||
1495 |
DeoptReason reason = trap_request_reason(trap_request); |
|
1496 |
DeoptAction action = trap_request_action(trap_request); |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1497 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1498 |
int debug_id = trap_request_debug_id(trap_request); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1499 |
#endif |
1 | 1500 |
jint unloaded_class_index = trap_request_index(trap_request); // CP idx or -1 |
1501 |
||
1502 |
vframe* vf = vframe::new_vframe(&fr, ®_map, thread); |
|
1503 |
compiledVFrame* cvf = compiledVFrame::cast(vf); |
|
1504 |
||
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
1505 |
CompiledMethod* nm = cvf->code(); |
1 | 1506 |
|
1507 |
ScopeDesc* trap_scope = cvf->scope(); |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1508 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1509 |
if (TraceDeoptimization) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1510 |
ttyLocker ttyl; |
33198 | 1511 |
tty->print_cr(" bci=%d pc=" INTPTR_FORMAT ", relative_pc=" INTPTR_FORMAT ", method=%s" JVMCI_ONLY(", debug_id=%d"), trap_scope->bci(), p2i(fr.pc()), fr.pc() - nm->code_begin(), trap_scope->method()->name_and_sig_as_C_string() |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1512 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1513 |
, debug_id |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1514 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1515 |
); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1516 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1517 |
|
1 | 1518 |
methodHandle trap_method = trap_scope->method(); |
1519 |
int trap_bci = trap_scope->bci(); |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1520 |
#if INCLUDE_JVMCI |
50858 | 1521 |
long speculation = thread->pending_failed_speculation(); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1522 |
if (nm->is_compiled_by_jvmci()) { |
50858 | 1523 |
if (speculation != 0) { |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
1524 |
oop speculation_log = nm->as_nmethod()->speculation_log(); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1525 |
if (speculation_log != NULL) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1526 |
if (TraceDeoptimization || TraceUncollectedSpeculations) { |
50858 | 1527 |
if (HotSpotSpeculationLog::lastFailed(speculation_log) != 0) { |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1528 |
tty->print_cr("A speculation that was not collected by the compiler is being overwritten"); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1529 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1530 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1531 |
if (TraceDeoptimization) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1532 |
tty->print_cr("Saving speculation to speculation log"); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1533 |
} |
33632 | 1534 |
HotSpotSpeculationLog::set_lastFailed(speculation_log, speculation); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1535 |
} else { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1536 |
if (TraceDeoptimization) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1537 |
tty->print_cr("Speculation present but no speculation log"); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1538 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1539 |
} |
50858 | 1540 |
thread->set_pending_failed_speculation(0); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1541 |
} else { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1542 |
if (TraceDeoptimization) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1543 |
tty->print_cr("No speculation"); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1544 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1545 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1546 |
} else { |
50858 | 1547 |
assert(speculation == 0, "There should not be a speculation for method compiled by non-JVMCI compilers"); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1548 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1549 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1550 |
if (trap_bci == SynchronizationEntryBCI) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1551 |
trap_bci = 0; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1552 |
thread->set_pending_monitorenter(true); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1553 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1554 |
|
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1555 |
if (reason == Deoptimization::Reason_transfer_to_interpreter) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1556 |
thread->set_pending_transfer_to_interpreter(true); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1557 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1558 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1559 |
|
7913 | 1560 |
Bytecodes::Code trap_bc = trap_method->java_code_at(trap_bci); |
1 | 1561 |
// Record this event in the histogram. |
1562 |
gather_statistics(reason, action, trap_bc); |
|
1563 |
||
1564 |
// Ensure that we can record deopt. history: |
|
23491 | 1565 |
// Need MDO to record RTM code generation state. |
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1566 |
bool create_if_missing = ProfileTraps || UseCodeAging RTM_OPT_ONLY( || UseRTMLocking ); |
1 | 1567 |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1568 |
methodHandle profiled_method; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1569 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1570 |
if (nm->is_compiled_by_jvmci()) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1571 |
profiled_method = nm->method(); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1572 |
} else { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1573 |
profiled_method = trap_method; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1574 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1575 |
#else |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1576 |
profiled_method = trap_method; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1577 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1578 |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
1579 |
MethodData* trap_mdo = |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1580 |
get_method_data(thread, profiled_method, create_if_missing); |
1 | 1581 |
|
14620
45167d2bc15a
8001538: hs_err file does not list anymore compiled methods in compilation events
vlivanov
parents:
14391
diff
changeset
|
1582 |
// Log a message |
43942
fff6b7b5611f
8173794: [REDO] [AOT] Missing GC scan of _metaspace_got array containing Klass*
kvn
parents:
43936
diff
changeset
|
1583 |
Events::log_deopt_message(thread, "Uncommon trap: reason=%s action=%s pc=" INTPTR_FORMAT " method=%s @ %d %s", |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
1584 |
trap_reason_name(reason), trap_action_name(action), p2i(fr.pc()), |
43942
fff6b7b5611f
8173794: [REDO] [AOT] Missing GC scan of _metaspace_got array containing Klass*
kvn
parents:
43936
diff
changeset
|
1585 |
trap_method->name_and_sig_as_C_string(), trap_bci, nm->compiler_name()); |
14620
45167d2bc15a
8001538: hs_err file does not list anymore compiled methods in compilation events
vlivanov
parents:
14391
diff
changeset
|
1586 |
|
1 | 1587 |
// Print a bunch of diagnostics, if requested. |
1588 |
if (TraceDeoptimization || LogCompilation) { |
|
1589 |
ResourceMark rm; |
|
1590 |
ttyLocker ttyl; |
|
1591 |
char buf[100]; |
|
1592 |
if (xtty != NULL) { |
|
31592
43f48e165466
8081202: Hotspot compile warning: "Invalid suffix on literal; C++11 requires a space between literal and identifier"
bpittore
parents:
30183
diff
changeset
|
1593 |
xtty->begin_head("uncommon_trap thread='" UINTX_FORMAT "' %s", |
1 | 1594 |
os::current_thread_id(), |
1595 |
format_trap_request(buf, sizeof(buf), trap_request)); |
|
1596 |
nm->log_identity(xtty); |
|
1597 |
} |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7913
diff
changeset
|
1598 |
Symbol* class_name = NULL; |
1 | 1599 |
bool unresolved = false; |
1600 |
if (unloaded_class_index >= 0) { |
|
1601 |
constantPoolHandle constants (THREAD, trap_method->constants()); |
|
1602 |
if (constants->tag_at(unloaded_class_index).is_unresolved_klass()) { |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7913
diff
changeset
|
1603 |
class_name = constants->klass_name_at(unloaded_class_index); |
1 | 1604 |
unresolved = true; |
1605 |
if (xtty != NULL) |
|
1606 |
xtty->print(" unresolved='1'"); |
|
1607 |
} else if (constants->tag_at(unloaded_class_index).is_symbol()) { |
|
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7913
diff
changeset
|
1608 |
class_name = constants->symbol_at(unloaded_class_index); |
1 | 1609 |
} |
1610 |
if (xtty != NULL) |
|
1611 |
xtty->name(class_name); |
|
1612 |
} |
|
24477 | 1613 |
if (xtty != NULL && trap_mdo != NULL && (int)reason < (int)MethodData::_trap_hist_limit) { |
1 | 1614 |
// Dump the relevant MDO state. |
1615 |
// This is the deopt count for the current reason, any previous |
|
1616 |
// reasons or recompiles seen at this point. |
|
1617 |
int dcnt = trap_mdo->trap_count(reason); |
|
1618 |
if (dcnt != 0) |
|
1619 |
xtty->print(" count='%d'", dcnt); |
|
1620 |
ProfileData* pdata = trap_mdo->bci_to_data(trap_bci); |
|
1621 |
int dos = (pdata == NULL)? 0: pdata->trap_state(); |
|
1622 |
if (dos != 0) { |
|
1623 |
xtty->print(" state='%s'", format_trap_state(buf, sizeof(buf), dos)); |
|
1624 |
if (trap_state_is_recompiled(dos)) { |
|
1625 |
int recnt2 = trap_mdo->overflow_recompile_count(); |
|
1626 |
if (recnt2 != 0) |
|
1627 |
xtty->print(" recompiles2='%d'", recnt2); |
|
1628 |
} |
|
1629 |
} |
|
1630 |
} |
|
1631 |
if (xtty != NULL) { |
|
1632 |
xtty->stamp(); |
|
1633 |
xtty->end_head(); |
|
1634 |
} |
|
1635 |
if (TraceDeoptimization) { // make noise on the tty |
|
1636 |
tty->print("Uncommon trap occurred in"); |
|
1637 |
nm->method()->print_short_name(tty); |
|
42040
70ec5a09a0d5
8166377: is_compiled_by_jvmci hot in some profiles - improve nmethod compiler type detection
neliasso
parents:
39261
diff
changeset
|
1638 |
tty->print(" compiler=%s compile_id=%d", nm->compiler_name(), nm->compile_id()); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1639 |
#if INCLUDE_JVMCI |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
1640 |
if (nm->is_nmethod()) { |
47794
e84aa2c71241
8188102: [JVMCI] Convert special JVMCI oops in nmethod to jweak values
dnsimon
parents:
47765
diff
changeset
|
1641 |
char* installed_code_name = nm->as_nmethod()->jvmci_installed_code_name(buf, sizeof(buf)); |
e84aa2c71241
8188102: [JVMCI] Convert special JVMCI oops in nmethod to jweak values
dnsimon
parents:
47765
diff
changeset
|
1642 |
if (installed_code_name != NULL) { |
e84aa2c71241
8188102: [JVMCI] Convert special JVMCI oops in nmethod to jweak values
dnsimon
parents:
47765
diff
changeset
|
1643 |
tty->print(" (JVMCI: installed code name=%s) ", installed_code_name); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1644 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1645 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1646 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1647 |
tty->print(" (@" INTPTR_FORMAT ") thread=" UINTX_FORMAT " reason=%s action=%s unloaded_class_index=%d" JVMCI_ONLY(" debug_id=%d"), |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31593
diff
changeset
|
1648 |
p2i(fr.pc()), |
13964 | 1649 |
os::current_thread_id(), |
1 | 1650 |
trap_reason_name(reason), |
1651 |
trap_action_name(action), |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1652 |
unloaded_class_index |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1653 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1654 |
, debug_id |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1655 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1656 |
); |
8076
96d498ec7ae1
6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents:
7913
diff
changeset
|
1657 |
if (class_name != NULL) { |
1 | 1658 |
tty->print(unresolved ? " unresolved class: " : " symbol: "); |
1659 |
class_name->print_symbol_on(tty); |
|
1660 |
} |
|
1661 |
tty->cr(); |
|
1662 |
} |
|
1663 |
if (xtty != NULL) { |
|
1664 |
// Log the precise location of the trap. |
|
1665 |
for (ScopeDesc* sd = trap_scope; ; sd = sd->sender()) { |
|
1666 |
xtty->begin_elem("jvms bci='%d'", sd->bci()); |
|
1667 |
xtty->method(sd->method()); |
|
1668 |
xtty->end_elem(); |
|
1669 |
if (sd->is_top()) break; |
|
1670 |
} |
|
1671 |
xtty->tail("uncommon_trap"); |
|
1672 |
} |
|
1673 |
} |
|
1674 |
// (End diagnostic printout.) |
|
1675 |
||
1676 |
// Load class if necessary |
|
1677 |
if (unloaded_class_index >= 0) { |
|
1678 |
constantPoolHandle constants(THREAD, trap_method->constants()); |
|
1679 |
load_class_by_index(constants, unloaded_class_index); |
|
1680 |
} |
|
1681 |
||
1682 |
// Flush the nmethod if necessary and desirable. |
|
1683 |
// |
|
1684 |
// We need to avoid situations where we are re-flushing the nmethod |
|
1685 |
// because of a hot deoptimization site. Repeated flushes at the same |
|
1686 |
// point need to be detected by the compiler and avoided. If the compiler |
|
1687 |
// cannot avoid them (or has a bug and "refuses" to avoid them), this |
|
1688 |
// module must take measures to avoid an infinite cycle of recompilation |
|
1689 |
// and deoptimization. There are several such measures: |
|
1690 |
// |
|
1691 |
// 1. If a recompilation is ordered a second time at some site X |
|
1692 |
// and for the same reason R, the action is adjusted to 'reinterpret', |
|
1693 |
// to give the interpreter time to exercise the method more thoroughly. |
|
1694 |
// If this happens, the method's overflow_recompile_count is incremented. |
|
1695 |
// |
|
1696 |
// 2. If the compiler fails to reduce the deoptimization rate, then |
|
1697 |
// the method's overflow_recompile_count will begin to exceed the set |
|
1698 |
// limit PerBytecodeRecompilationCutoff. If this happens, the action |
|
1699 |
// is adjusted to 'make_not_compilable', and the method is abandoned |
|
1700 |
// to the interpreter. This is a performance hit for hot methods, |
|
1701 |
// but is better than a disastrous infinite cycle of recompilations. |
|
1702 |
// (Actually, only the method containing the site X is abandoned.) |
|
1703 |
// |
|
1704 |
// 3. In parallel with the previous measures, if the total number of |
|
1705 |
// recompilations of a method exceeds the much larger set limit |
|
1706 |
// PerMethodRecompilationCutoff, the method is abandoned. |
|
1707 |
// This should only happen if the method is very large and has |
|
1708 |
// many "lukewarm" deoptimizations. The code which enforces this |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
1709 |
// limit is elsewhere (class nmethod, class Method). |
1 | 1710 |
// |
1711 |
// Note that the per-BCI 'is_recompiled' bit gives the compiler one chance |
|
1712 |
// to recompile at each bytecode independently of the per-BCI cutoff. |
|
1713 |
// |
|
1714 |
// The decision to update code is up to the compiler, and is encoded |
|
1715 |
// in the Action_xxx code. If the compiler requests Action_none |
|
1716 |
// no trap state is changed, no compiled code is changed, and the |
|
1717 |
// computation suffers along in the interpreter. |
|
1718 |
// |
|
1719 |
// The other action codes specify various tactics for decompilation |
|
1720 |
// and recompilation. Action_maybe_recompile is the loosest, and |
|
1721 |
// allows the compiled code to stay around until enough traps are seen, |
|
1722 |
// and until the compiler gets around to recompiling the trapping method. |
|
1723 |
// |
|
1724 |
// The other actions cause immediate removal of the present code. |
|
1725 |
||
31228
8e427370cdd1
8074551: GWT can be marked non-compilable due to deopt count pollution
vlivanov
parents:
30183
diff
changeset
|
1726 |
// Traps caused by injected profile shouldn't pollute trap counts. |
8e427370cdd1
8074551: GWT can be marked non-compilable due to deopt count pollution
vlivanov
parents:
30183
diff
changeset
|
1727 |
bool injected_profile_trap = trap_method->has_injected_profile() && |
8e427370cdd1
8074551: GWT can be marked non-compilable due to deopt count pollution
vlivanov
parents:
30183
diff
changeset
|
1728 |
(reason == Reason_intrinsic || reason == Reason_unreached); |
8e427370cdd1
8074551: GWT can be marked non-compilable due to deopt count pollution
vlivanov
parents:
30183
diff
changeset
|
1729 |
|
8e427370cdd1
8074551: GWT can be marked non-compilable due to deopt count pollution
vlivanov
parents:
30183
diff
changeset
|
1730 |
bool update_trap_state = (reason != Reason_tenured) && !injected_profile_trap; |
1 | 1731 |
bool make_not_entrant = false; |
1732 |
bool make_not_compilable = false; |
|
6453 | 1733 |
bool reprofile = false; |
1 | 1734 |
switch (action) { |
1735 |
case Action_none: |
|
1736 |
// Keep the old code. |
|
1737 |
update_trap_state = false; |
|
1738 |
break; |
|
1739 |
case Action_maybe_recompile: |
|
1740 |
// Do not need to invalidate the present code, but we can |
|
1741 |
// initiate another |
|
1742 |
// Start compiler without (necessarily) invalidating the nmethod. |
|
1743 |
// The system will tolerate the old code, but new code should be |
|
1744 |
// generated when possible. |
|
1745 |
break; |
|
1746 |
case Action_reinterpret: |
|
1747 |
// Go back into the interpreter for a while, and then consider |
|
1748 |
// recompiling form scratch. |
|
1749 |
make_not_entrant = true; |
|
1750 |
// Reset invocation counter for outer most method. |
|
1751 |
// This will allow the interpreter to exercise the bytecodes |
|
1752 |
// for a while before recompiling. |
|
1753 |
// By contrast, Action_make_not_entrant is immediate. |
|
1754 |
// |
|
1755 |
// Note that the compiler will track null_check, null_assert, |
|
1756 |
// range_check, and class_check events and log them as if they |
|
1757 |
// had been traps taken from compiled code. This will update |
|
1758 |
// the MDO trap history so that the next compilation will |
|
1759 |
// properly detect hot trap sites. |
|
6453 | 1760 |
reprofile = true; |
1 | 1761 |
break; |
1762 |
case Action_make_not_entrant: |
|
1763 |
// Request immediate recompilation, and get rid of the old code. |
|
1764 |
// Make them not entrant, so next time they are called they get |
|
1765 |
// recompiled. Unloaded classes are loaded now so recompile before next |
|
1766 |
// time they are called. Same for uninitialized. The interpreter will |
|
1767 |
// link the missing class, if any. |
|
1768 |
make_not_entrant = true; |
|
1769 |
break; |
|
1770 |
case Action_make_not_compilable: |
|
1771 |
// Give up on compiling this method at all. |
|
1772 |
make_not_entrant = true; |
|
1773 |
make_not_compilable = true; |
|
1774 |
break; |
|
1775 |
default: |
|
1776 |
ShouldNotReachHere(); |
|
1777 |
} |
|
1778 |
||
1779 |
// Setting +ProfileTraps fixes the following, on all platforms: |
|
1780 |
// 4852688: ProfileInterpreter is off by default for ia64. The result is |
|
1781 |
// infinite heroic-opt-uncommon-trap/deopt/recompile cycles, since the |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
1782 |
// recompile relies on a MethodData* to record heroic opt failures. |
1 | 1783 |
|
1784 |
// Whether the interpreter is producing MDO data or not, we also need |
|
1785 |
// to use the MDO to detect hot deoptimization points and control |
|
1786 |
// aggressive optimization. |
|
4754
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1787 |
bool inc_recompile_count = false; |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1788 |
ProfileData* pdata = NULL; |
43936
093cd5bea2e2
8173679: Disable ProfileTrap code and UseRTMLocking in emulated client Win32
jcm
parents:
43931
diff
changeset
|
1789 |
if (ProfileTraps && !is_client_compilation_mode_vm() && update_trap_state && trap_mdo != NULL) { |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1790 |
assert(trap_mdo == get_method_data(thread, profiled_method, false), "sanity"); |
1 | 1791 |
uint this_trap_count = 0; |
1792 |
bool maybe_prior_trap = false; |
|
1793 |
bool maybe_prior_recompile = false; |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1794 |
pdata = query_update_method_data(trap_mdo, trap_bci, reason, true, |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1795 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1796 |
nm->is_compiled_by_jvmci() && nm->is_osr_method(), |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1797 |
#endif |
22916
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1798 |
nm->method(), |
1 | 1799 |
//outputs: |
1800 |
this_trap_count, |
|
1801 |
maybe_prior_trap, |
|
1802 |
maybe_prior_recompile); |
|
1803 |
// Because the interpreter also counts null, div0, range, and class |
|
1804 |
// checks, these traps from compiled code are double-counted. |
|
1805 |
// This is harmless; it just means that the PerXTrapLimit values |
|
1806 |
// are in effect a little smaller than they look. |
|
1807 |
||
1808 |
DeoptReason per_bc_reason = reason_recorded_per_bytecode_if_any(reason); |
|
1809 |
if (per_bc_reason != Reason_none) { |
|
1810 |
// Now take action based on the partially known per-BCI history. |
|
1811 |
if (maybe_prior_trap |
|
1812 |
&& this_trap_count >= (uint)PerBytecodeTrapLimit) { |
|
1813 |
// If there are too many traps at this BCI, force a recompile. |
|
1814 |
// This will allow the compiler to see the limit overflow, and |
|
1815 |
// take corrective action, if possible. The compiler generally |
|
1816 |
// does not use the exact PerBytecodeTrapLimit value, but instead |
|
1817 |
// changes its tactics if it sees any traps at all. This provides |
|
1818 |
// a little hysteresis, delaying a recompile until a trap happens |
|
1819 |
// several times. |
|
1820 |
// |
|
1821 |
// Actually, since there is only one bit of counter per BCI, |
|
1822 |
// the possible per-BCI counts are {0,1,(per-method count)}. |
|
1823 |
// This produces accurate results if in fact there is only |
|
1824 |
// one hot trap site, but begins to get fuzzy if there are |
|
1825 |
// many sites. For example, if there are ten sites each |
|
1826 |
// trapping two or more times, they each get the blame for |
|
1827 |
// all of their traps. |
|
1828 |
make_not_entrant = true; |
|
1829 |
} |
|
1830 |
||
1831 |
// Detect repeated recompilation at the same BCI, and enforce a limit. |
|
1832 |
if (make_not_entrant && maybe_prior_recompile) { |
|
1833 |
// More than one recompile at this point. |
|
4754
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1834 |
inc_recompile_count = maybe_prior_trap; |
1 | 1835 |
} |
1836 |
} else { |
|
1837 |
// For reasons which are not recorded per-bytecode, we simply |
|
1838 |
// force recompiles unconditionally. |
|
1839 |
// (Note that PerMethodRecompilationCutoff is enforced elsewhere.) |
|
1840 |
make_not_entrant = true; |
|
1841 |
} |
|
1842 |
||
1843 |
// Go back to the compiler if there are too many traps in this method. |
|
22916
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1844 |
if (this_trap_count >= per_method_trap_limit(reason)) { |
1 | 1845 |
// If there are too many traps in this method, force a recompile. |
1846 |
// This will allow the compiler to see the limit overflow, and |
|
1847 |
// take corrective action, if possible. |
|
1848 |
// (This condition is an unlikely backstop only, because the |
|
1849 |
// PerBytecodeTrapLimit is more likely to take effect first, |
|
1850 |
// if it is applicable.) |
|
1851 |
make_not_entrant = true; |
|
1852 |
} |
|
1853 |
||
1854 |
// Here's more hysteresis: If there has been a recompile at |
|
1855 |
// this trap point already, run the method in the interpreter |
|
1856 |
// for a while to exercise it more thoroughly. |
|
1857 |
if (make_not_entrant && maybe_prior_recompile && maybe_prior_trap) { |
|
6453 | 1858 |
reprofile = true; |
1 | 1859 |
} |
4754
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1860 |
} |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1861 |
|
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1862 |
// Take requested actions on the method: |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1863 |
|
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1864 |
// Recompile |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1865 |
if (make_not_entrant) { |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1866 |
if (!nm->make_not_entrant()) { |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1867 |
return; // the call did not change nmethod's state |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1868 |
} |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1869 |
|
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1870 |
if (pdata != NULL) { |
1 | 1871 |
// Record the recompilation event, if any. |
1872 |
int tstate0 = pdata->trap_state(); |
|
1873 |
int tstate1 = trap_state_set_recompiled(tstate0, true); |
|
1874 |
if (tstate1 != tstate0) |
|
1875 |
pdata->set_trap_state(tstate1); |
|
1876 |
} |
|
23491 | 1877 |
|
1878 |
#if INCLUDE_RTM_OPT |
|
1879 |
// Restart collecting RTM locking abort statistic if the method |
|
1880 |
// is recompiled for a reason other than RTM state change. |
|
1881 |
// Assume that in new recompiled code the statistic could be different, |
|
1882 |
// for example, due to different inlining. |
|
1883 |
if ((reason != Reason_rtm_state_change) && (trap_mdo != NULL) && |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35499
diff
changeset
|
1884 |
UseRTMDeopt && (nm->as_nmethod()->rtm_state() != ProfileRTM)) { |
23491 | 1885 |
trap_mdo->atomic_set_rtm_state(ProfileRTM); |
1886 |
} |
|
1887 |
#endif |
|
24442
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1888 |
// For code aging we count traps separately here, using make_not_entrant() |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1889 |
// as a guard against simultaneous deopts in multiple threads. |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1890 |
if (reason == Reason_tenured && trap_mdo != NULL) { |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1891 |
trap_mdo->inc_tenure_traps(); |
4d4ae31dea26
8032463: VirtualDispatch test timeout with DeoptimizeALot
iveresov
parents:
24424
diff
changeset
|
1892 |
} |
1 | 1893 |
} |
1894 |
||
4754
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1895 |
if (inc_recompile_count) { |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1896 |
trap_mdo->inc_overflow_recompile_count(); |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1897 |
if ((uint)trap_mdo->overflow_recompile_count() > |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1898 |
(uint)PerBytecodeRecompilationCutoff) { |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1899 |
// Give up on the method containing the bad BCI. |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1900 |
if (trap_method() == nm->method()) { |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1901 |
make_not_compilable = true; |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1902 |
} else { |
15479 | 1903 |
trap_method->set_not_compilable(CompLevel_full_optimization, true, "overflow_recompile_count > PerBytecodeRecompilationCutoff"); |
4754
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1904 |
// But give grace to the enclosing nm->method(). |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1905 |
} |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1906 |
} |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1907 |
} |
1 | 1908 |
|
6453 | 1909 |
// Reprofile |
1910 |
if (reprofile) { |
|
1911 |
CompilationPolicy::policy()->reprofile(trap_scope, nm->is_osr_method()); |
|
1 | 1912 |
} |
1913 |
||
1914 |
// Give up compiling |
|
6453 | 1915 |
if (make_not_compilable && !nm->method()->is_not_compilable(CompLevel_full_optimization)) { |
1 | 1916 |
assert(make_not_entrant, "consistent"); |
6453 | 1917 |
nm->method()->set_not_compilable(CompLevel_full_optimization); |
1 | 1918 |
} |
1919 |
||
1920 |
} // Free marked resources |
|
1921 |
||
1922 |
} |
|
1923 |
JRT_END |
|
1924 |
||
1925 |
ProfileData* |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
1926 |
Deoptimization::query_update_method_data(MethodData* trap_mdo, |
1 | 1927 |
int trap_bci, |
1928 |
Deoptimization::DeoptReason reason, |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1929 |
bool update_total_trap_count, |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1930 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1931 |
bool is_osr, |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1932 |
#endif |
22916
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1933 |
Method* compiled_method, |
1 | 1934 |
//outputs: |
1935 |
uint& ret_this_trap_count, |
|
1936 |
bool& ret_maybe_prior_trap, |
|
1937 |
bool& ret_maybe_prior_recompile) { |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1938 |
bool maybe_prior_trap = false; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1939 |
bool maybe_prior_recompile = false; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1940 |
uint this_trap_count = 0; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1941 |
if (update_total_trap_count) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1942 |
uint idx = reason; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1943 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1944 |
if (is_osr) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1945 |
idx += Reason_LIMIT; |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1946 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1947 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1948 |
uint prior_trap_count = trap_mdo->trap_count(idx); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1949 |
this_trap_count = trap_mdo->inc_trap_count(idx); |
1 | 1950 |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1951 |
// If the runtime cannot find a place to store trap history, |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1952 |
// it is estimated based on the general condition of the method. |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1953 |
// If the method has ever been recompiled, or has ever incurred |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1954 |
// a trap with the present reason , then this BCI is assumed |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1955 |
// (pessimistically) to be the culprit. |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1956 |
maybe_prior_trap = (prior_trap_count != 0); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1957 |
maybe_prior_recompile = (trap_mdo->decompile_count() != 0); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1958 |
} |
1 | 1959 |
ProfileData* pdata = NULL; |
1960 |
||
1961 |
||
1962 |
// For reasons which are recorded per bytecode, we check per-BCI data. |
|
1963 |
DeoptReason per_bc_reason = reason_recorded_per_bytecode_if_any(reason); |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
1964 |
assert(per_bc_reason != Reason_none || update_total_trap_count, "must be"); |
1 | 1965 |
if (per_bc_reason != Reason_none) { |
1966 |
// Find the profile data for this BCI. If there isn't one, |
|
1967 |
// try to allocate one from the MDO's set of spares. |
|
1968 |
// This will let us detect a repeated trap at this point. |
|
22916
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1969 |
pdata = trap_mdo->allocate_bci_to_data(trap_bci, reason_is_speculate(reason) ? compiled_method : NULL); |
1 | 1970 |
|
1971 |
if (pdata != NULL) { |
|
22916
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1972 |
if (reason_is_speculate(reason) && !pdata->is_SpeculativeTrapData()) { |
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1973 |
if (LogCompilation && xtty != NULL) { |
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1974 |
ttyLocker ttyl; |
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1975 |
// no more room for speculative traps in this MDO |
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1976 |
xtty->elem("speculative_traps_oom"); |
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1977 |
} |
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
1978 |
} |
1 | 1979 |
// Query the trap state of this profile datum. |
1980 |
int tstate0 = pdata->trap_state(); |
|
1981 |
if (!trap_state_has_reason(tstate0, per_bc_reason)) |
|
1982 |
maybe_prior_trap = false; |
|
1983 |
if (!trap_state_is_recompiled(tstate0)) |
|
1984 |
maybe_prior_recompile = false; |
|
1985 |
||
1986 |
// Update the trap state of this profile datum. |
|
1987 |
int tstate1 = tstate0; |
|
1988 |
// Record the reason. |
|
1989 |
tstate1 = trap_state_add_reason(tstate1, per_bc_reason); |
|
1990 |
// Store the updated state on the MDO, for next time. |
|
1991 |
if (tstate1 != tstate0) |
|
1992 |
pdata->set_trap_state(tstate1); |
|
1993 |
} else { |
|
4754
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1994 |
if (LogCompilation && xtty != NULL) { |
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1995 |
ttyLocker ttyl; |
1 | 1996 |
// Missing MDP? Leave a small complaint in the log. |
1997 |
xtty->elem("missing_mdp bci='%d'", trap_bci); |
|
4754
8aef16f24e16
6614597: Performance variability in jvm2008 xml.validation
kvn
parents:
4752
diff
changeset
|
1998 |
} |
1 | 1999 |
} |
2000 |
} |
|
2001 |
||
2002 |
// Return results: |
|
2003 |
ret_this_trap_count = this_trap_count; |
|
2004 |
ret_maybe_prior_trap = maybe_prior_trap; |
|
2005 |
ret_maybe_prior_recompile = maybe_prior_recompile; |
|
2006 |
return pdata; |
|
2007 |
} |
|
2008 |
||
2009 |
void |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
2010 |
Deoptimization::update_method_data_from_interpreter(MethodData* trap_mdo, int trap_bci, int reason) { |
1 | 2011 |
ResourceMark rm; |
2012 |
// Ignored outputs: |
|
2013 |
uint ignore_this_trap_count; |
|
2014 |
bool ignore_maybe_prior_trap; |
|
2015 |
bool ignore_maybe_prior_recompile; |
|
22916
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
2016 |
assert(!reason_is_speculate(reason), "reason speculate only used by compiler"); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2017 |
// JVMCI uses the total counts to determine if deoptimizations are happening too frequently -> do not adjust total counts |
51073
34696f3aa22b
8206953: compiler/profiling/TestTypeProfiling.java fails when JVMCI build disabled
kvn
parents:
50858
diff
changeset
|
2018 |
bool update_total_counts = true JVMCI_ONLY( && !UseJVMCICompiler); |
1 | 2019 |
query_update_method_data(trap_mdo, trap_bci, |
2020 |
(DeoptReason)reason, |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2021 |
update_total_counts, |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2022 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2023 |
false, |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2024 |
#endif |
22916
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
2025 |
NULL, |
1 | 2026 |
ignore_this_trap_count, |
2027 |
ignore_maybe_prior_trap, |
|
2028 |
ignore_maybe_prior_recompile); |
|
2029 |
} |
|
2030 |
||
34173
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
2031 |
Deoptimization::UnrollBlock* Deoptimization::uncommon_trap(JavaThread* thread, jint trap_request, jint exec_mode) { |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2032 |
if (TraceDeoptimization) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2033 |
tty->print("Uncommon trap "); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2034 |
} |
1 | 2035 |
// Still in Java no safepoints |
2036 |
{ |
|
2037 |
// This enters VM and may safepoint |
|
2038 |
uncommon_trap_inner(thread, trap_request); |
|
2039 |
} |
|
34173
01bb07d23a5b
8141133: [JVMCI] crash during safepoint deopt if rethrow_exception is set
twisti
parents:
33632
diff
changeset
|
2040 |
return fetch_unroll_info_helper(thread, exec_mode); |
1 | 2041 |
} |
2042 |
||
2043 |
// Local derived constants. |
|
2044 |
// Further breakdown of DataLayout::trap_state, as promised by DataLayout. |
|
50577
bf7e2684cd0a
8204240: Extend MDO to allow more reasons to be recorded per bci
roland
parents:
49972
diff
changeset
|
2045 |
const int DS_REASON_MASK = ((uint)DataLayout::trap_mask) >> 1; |
1 | 2046 |
const int DS_RECOMPILE_BIT = DataLayout::trap_mask - DS_REASON_MASK; |
2047 |
||
2048 |
//---------------------------trap_state_reason--------------------------------- |
|
2049 |
Deoptimization::DeoptReason |
|
2050 |
Deoptimization::trap_state_reason(int trap_state) { |
|
2051 |
// This assert provides the link between the width of DataLayout::trap_bits |
|
2052 |
// and the encoding of "recorded" reasons. It ensures there are enough |
|
2053 |
// bits to store all needed reasons in the per-BCI MDO profile. |
|
2054 |
assert(DS_REASON_MASK >= Reason_RECORDED_LIMIT, "enough bits"); |
|
2055 |
int recompile_bit = (trap_state & DS_RECOMPILE_BIT); |
|
2056 |
trap_state -= recompile_bit; |
|
2057 |
if (trap_state == DS_REASON_MASK) { |
|
2058 |
return Reason_many; |
|
2059 |
} else { |
|
2060 |
assert((int)Reason_none == 0, "state=0 => Reason_none"); |
|
2061 |
return (DeoptReason)trap_state; |
|
2062 |
} |
|
2063 |
} |
|
2064 |
//-------------------------trap_state_has_reason------------------------------- |
|
2065 |
int Deoptimization::trap_state_has_reason(int trap_state, int reason) { |
|
2066 |
assert(reason_is_recorded_per_bytecode((DeoptReason)reason), "valid reason"); |
|
2067 |
assert(DS_REASON_MASK >= Reason_RECORDED_LIMIT, "enough bits"); |
|
2068 |
int recompile_bit = (trap_state & DS_RECOMPILE_BIT); |
|
2069 |
trap_state -= recompile_bit; |
|
2070 |
if (trap_state == DS_REASON_MASK) { |
|
2071 |
return -1; // true, unspecifically (bottom of state lattice) |
|
2072 |
} else if (trap_state == reason) { |
|
2073 |
return 1; // true, definitely |
|
2074 |
} else if (trap_state == 0) { |
|
2075 |
return 0; // false, definitely (top of state lattice) |
|
2076 |
} else { |
|
2077 |
return 0; // false, definitely |
|
2078 |
} |
|
2079 |
} |
|
2080 |
//-------------------------trap_state_add_reason------------------------------- |
|
2081 |
int Deoptimization::trap_state_add_reason(int trap_state, int reason) { |
|
2082 |
assert(reason_is_recorded_per_bytecode((DeoptReason)reason) || reason == Reason_many, "valid reason"); |
|
2083 |
int recompile_bit = (trap_state & DS_RECOMPILE_BIT); |
|
2084 |
trap_state -= recompile_bit; |
|
2085 |
if (trap_state == DS_REASON_MASK) { |
|
2086 |
return trap_state + recompile_bit; // already at state lattice bottom |
|
2087 |
} else if (trap_state == reason) { |
|
2088 |
return trap_state + recompile_bit; // the condition is already true |
|
2089 |
} else if (trap_state == 0) { |
|
2090 |
return reason + recompile_bit; // no condition has yet been true |
|
2091 |
} else { |
|
2092 |
return DS_REASON_MASK + recompile_bit; // fall to state lattice bottom |
|
2093 |
} |
|
2094 |
} |
|
2095 |
//-----------------------trap_state_is_recompiled------------------------------ |
|
2096 |
bool Deoptimization::trap_state_is_recompiled(int trap_state) { |
|
2097 |
return (trap_state & DS_RECOMPILE_BIT) != 0; |
|
2098 |
} |
|
2099 |
//-----------------------trap_state_set_recompiled----------------------------- |
|
2100 |
int Deoptimization::trap_state_set_recompiled(int trap_state, bool z) { |
|
2101 |
if (z) return trap_state | DS_RECOMPILE_BIT; |
|
2102 |
else return trap_state & ~DS_RECOMPILE_BIT; |
|
2103 |
} |
|
2104 |
//---------------------------format_trap_state--------------------------------- |
|
20010
c66a7254680c
8023134: Rename VM LogFile to hotspot_pid{pid}.log (was hotspot.log)
vlivanov
parents:
17872
diff
changeset
|
2105 |
// This is used for debugging and diagnostics, including LogFile output. |
1 | 2106 |
const char* Deoptimization::format_trap_state(char* buf, size_t buflen, |
2107 |
int trap_state) { |
|
33794 | 2108 |
assert(buflen > 0, "sanity"); |
1 | 2109 |
DeoptReason reason = trap_state_reason(trap_state); |
2110 |
bool recomp_flag = trap_state_is_recompiled(trap_state); |
|
2111 |
// Re-encode the state from its decoded components. |
|
2112 |
int decoded_state = 0; |
|
2113 |
if (reason_is_recorded_per_bytecode(reason) || reason == Reason_many) |
|
2114 |
decoded_state = trap_state_add_reason(decoded_state, reason); |
|
2115 |
if (recomp_flag) |
|
2116 |
decoded_state = trap_state_set_recompiled(decoded_state, recomp_flag); |
|
2117 |
// If the state re-encodes properly, format it symbolically. |
|
2118 |
// Because this routine is used for debugging and diagnostics, |
|
2119 |
// be robust even if the state is a strange value. |
|
2120 |
size_t len; |
|
2121 |
if (decoded_state != trap_state) { |
|
2122 |
// Random buggy state that doesn't decode?? |
|
2123 |
len = jio_snprintf(buf, buflen, "#%d", trap_state); |
|
2124 |
} else { |
|
2125 |
len = jio_snprintf(buf, buflen, "%s%s", |
|
2126 |
trap_reason_name(reason), |
|
2127 |
recomp_flag ? " recompiled" : ""); |
|
2128 |
} |
|
2129 |
return buf; |
|
2130 |
} |
|
2131 |
||
2132 |
||
2133 |
//--------------------------------statics-------------------------------------- |
|
24477 | 2134 |
const char* Deoptimization::_trap_reason_name[] = { |
1 | 2135 |
// Note: Keep this in sync. with enum DeoptReason. |
2136 |
"none", |
|
2137 |
"null_check", |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2138 |
"null_assert" JVMCI_ONLY("_or_unreached0"), |
1 | 2139 |
"range_check", |
2140 |
"class_check", |
|
2141 |
"array_check", |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2142 |
"intrinsic" JVMCI_ONLY("_or_type_checked_inlining"), |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2143 |
"bimorphic" JVMCI_ONLY("_or_optimized_type_check"), |
50623
5209d8a6303e
8203197: C2: consider all paths in loop body for loop predication
roland
parents:
50577
diff
changeset
|
2144 |
"profile_predicate", |
1 | 2145 |
"unloaded", |
2146 |
"uninitialized", |
|
2147 |
"unreached", |
|
2148 |
"unhandled", |
|
2149 |
"constraint", |
|
2150 |
"div0_check", |
|
4643 | 2151 |
"age", |
9446 | 2152 |
"predicate", |
22916
582da2ed4dfa
8031752: Failed speculative optimizations should be reattempted when root of compilation is different
roland
parents:
22876
diff
changeset
|
2153 |
"loop_limit_check", |
23491 | 2154 |
"speculate_class_check", |
23525
e3eb08ead679
8031755: Type speculation should be used to optimize explicit null checks
roland
parents:
23491
diff
changeset
|
2155 |
"speculate_null_check", |
46542
73dd19b96b5d
8181211: C2: Use profiling data to optimize on/off heap unsafe accesses
roland
parents:
46407
diff
changeset
|
2156 |
"speculate_null_assert", |
24477 | 2157 |
"rtm_state_change", |
24934
43a6fc13b7d4
8030976: Untaken paths should be more vigorously pruned at highest optimization level
rbackman
parents:
24477
diff
changeset
|
2158 |
"unstable_if", |
30183 | 2159 |
"unstable_fused_if", |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2160 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2161 |
"aliasing", |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2162 |
"transfer_to_interpreter", |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2163 |
"not_compiled_exception_handler", |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2164 |
"unresolved", |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2165 |
"jsr_mismatch", |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2166 |
#endif |
24477 | 2167 |
"tenured" |
1 | 2168 |
}; |
24477 | 2169 |
const char* Deoptimization::_trap_action_name[] = { |
1 | 2170 |
// Note: Keep this in sync. with enum DeoptAction. |
2171 |
"none", |
|
2172 |
"maybe_recompile", |
|
2173 |
"reinterpret", |
|
2174 |
"make_not_entrant", |
|
2175 |
"make_not_compilable" |
|
2176 |
}; |
|
2177 |
||
2178 |
const char* Deoptimization::trap_reason_name(int reason) { |
|
24477 | 2179 |
// Check that every reason has a name |
2180 |
STATIC_ASSERT(sizeof(_trap_reason_name)/sizeof(const char*) == Reason_LIMIT); |
|
2181 |
||
1 | 2182 |
if (reason == Reason_many) return "many"; |
2183 |
if ((uint)reason < Reason_LIMIT) |
|
2184 |
return _trap_reason_name[reason]; |
|
2185 |
static char buf[20]; |
|
2186 |
sprintf(buf, "reason%d", reason); |
|
2187 |
return buf; |
|
2188 |
} |
|
2189 |
const char* Deoptimization::trap_action_name(int action) { |
|
24477 | 2190 |
// Check that every action has a name |
2191 |
STATIC_ASSERT(sizeof(_trap_action_name)/sizeof(const char*) == Action_LIMIT); |
|
2192 |
||
1 | 2193 |
if ((uint)action < Action_LIMIT) |
2194 |
return _trap_action_name[action]; |
|
2195 |
static char buf[20]; |
|
2196 |
sprintf(buf, "action%d", action); |
|
2197 |
return buf; |
|
2198 |
} |
|
2199 |
||
20010
c66a7254680c
8023134: Rename VM LogFile to hotspot_pid{pid}.log (was hotspot.log)
vlivanov
parents:
17872
diff
changeset
|
2200 |
// This is used for debugging and diagnostics, including LogFile output. |
1 | 2201 |
const char* Deoptimization::format_trap_request(char* buf, size_t buflen, |
2202 |
int trap_request) { |
|
2203 |
jint unloaded_class_index = trap_request_index(trap_request); |
|
2204 |
const char* reason = trap_reason_name(trap_request_reason(trap_request)); |
|
2205 |
const char* action = trap_action_name(trap_request_action(trap_request)); |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2206 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2207 |
int debug_id = trap_request_debug_id(trap_request); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2208 |
#endif |
1 | 2209 |
size_t len; |
2210 |
if (unloaded_class_index < 0) { |
|
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2211 |
len = jio_snprintf(buf, buflen, "reason='%s' action='%s'" JVMCI_ONLY(" debug_id='%d'"), |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2212 |
reason, action |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2213 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2214 |
,debug_id |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2215 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2216 |
); |
1 | 2217 |
} else { |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2218 |
len = jio_snprintf(buf, buflen, "reason='%s' action='%s' index='%d'" JVMCI_ONLY(" debug_id='%d'"), |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2219 |
reason, action, unloaded_class_index |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2220 |
#if INCLUDE_JVMCI |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2221 |
,debug_id |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2222 |
#endif |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31593
diff
changeset
|
2223 |
); |
1 | 2224 |
} |
2225 |
return buf; |
|
2226 |
} |
|
2227 |
||
2228 |
juint Deoptimization::_deoptimization_hist |
|
2229 |
[Deoptimization::Reason_LIMIT] |
|
2230 |
[1 + Deoptimization::Action_LIMIT] |
|
2231 |
[Deoptimization::BC_CASE_LIMIT] |
|
2232 |
= {0}; |
|
2233 |
||
2234 |
enum { |
|
2235 |
LSB_BITS = 8, |
|
2236 |
LSB_MASK = right_n_bits(LSB_BITS) |
|
2237 |
}; |
|
2238 |
||
2239 |
void Deoptimization::gather_statistics(DeoptReason reason, DeoptAction action, |
|
2240 |
Bytecodes::Code bc) { |
|
2241 |
assert(reason >= 0 && reason < Reason_LIMIT, "oob"); |
|
2242 |
assert(action >= 0 && action < Action_LIMIT, "oob"); |
|
2243 |
_deoptimization_hist[Reason_none][0][0] += 1; // total |
|
2244 |
_deoptimization_hist[reason][0][0] += 1; // per-reason total |
|
2245 |
juint* cases = _deoptimization_hist[reason][1+action]; |
|
2246 |
juint* bc_counter_addr = NULL; |
|
2247 |
juint bc_counter = 0; |
|
2248 |
// Look for an unused counter, or an exact match to this BC. |
|
2249 |
if (bc != Bytecodes::_illegal) { |
|
2250 |
for (int bc_case = 0; bc_case < BC_CASE_LIMIT; bc_case++) { |
|
2251 |
juint* counter_addr = &cases[bc_case]; |
|
2252 |
juint counter = *counter_addr; |
|
2253 |
if ((counter == 0 && bc_counter_addr == NULL) |
|
2254 |
|| (Bytecodes::Code)(counter & LSB_MASK) == bc) { |
|
2255 |
// this counter is either free or is already devoted to this BC |
|
2256 |
bc_counter_addr = counter_addr; |
|
2257 |
bc_counter = counter | bc; |
|
2258 |
} |
|
2259 |
} |
|
2260 |
} |
|
2261 |
if (bc_counter_addr == NULL) { |
|
2262 |
// Overflow, or no given bytecode. |
|
2263 |
bc_counter_addr = &cases[BC_CASE_LIMIT-1]; |
|
2264 |
bc_counter = (*bc_counter_addr & ~LSB_MASK); // clear LSB |
|
2265 |
} |
|
2266 |
*bc_counter_addr = bc_counter + (1 << LSB_BITS); |
|
2267 |
} |
|
2268 |
||
2269 |
jint Deoptimization::total_deoptimization_count() { |
|
2270 |
return _deoptimization_hist[Reason_none][0][0]; |
|
2271 |
} |
|
2272 |
||
2273 |
void Deoptimization::print_statistics() { |
|
2274 |
juint total = total_deoptimization_count(); |
|
2275 |
juint account = total; |
|
2276 |
if (total != 0) { |
|
2277 |
ttyLocker ttyl; |
|
2278 |
if (xtty != NULL) xtty->head("statistics type='deoptimization'"); |
|
2279 |
tty->print_cr("Deoptimization traps recorded:"); |
|
2280 |
#define PRINT_STAT_LINE(name, r) \ |
|
2281 |
tty->print_cr(" %4d (%4.1f%%) %s", (int)(r), ((r) * 100.0) / total, name); |
|
2282 |
PRINT_STAT_LINE("total", total); |
|
2283 |
// For each non-zero entry in the histogram, print the reason, |
|
2284 |
// the action, and (if specifically known) the type of bytecode. |
|
2285 |
for (int reason = 0; reason < Reason_LIMIT; reason++) { |
|
2286 |
for (int action = 0; action < Action_LIMIT; action++) { |
|
2287 |
juint* cases = _deoptimization_hist[reason][1+action]; |
|
2288 |
for (int bc_case = 0; bc_case < BC_CASE_LIMIT; bc_case++) { |
|
2289 |
juint counter = cases[bc_case]; |
|
2290 |
if (counter != 0) { |
|
2291 |
char name[1*K]; |
|
2292 |
Bytecodes::Code bc = (Bytecodes::Code)(counter & LSB_MASK); |
|
2293 |
if (bc_case == BC_CASE_LIMIT && (int)bc == 0) |
|
2294 |
bc = Bytecodes::_illegal; |
|
2295 |
sprintf(name, "%s/%s/%s", |
|
2296 |
trap_reason_name(reason), |
|
2297 |
trap_action_name(action), |
|
2298 |
Bytecodes::is_defined(bc)? Bytecodes::name(bc): "other"); |
|
2299 |
juint r = counter >> LSB_BITS; |
|
2300 |
tty->print_cr(" %40s: " UINT32_FORMAT " (%.1f%%)", name, r, (r * 100.0) / total); |
|
2301 |
account -= r; |
|
2302 |
} |
|
2303 |
} |
|
2304 |
} |
|
2305 |
} |
|
2306 |
if (account != 0) { |
|
2307 |
PRINT_STAT_LINE("unaccounted", account); |
|
2308 |
} |
|
2309 |
#undef PRINT_STAT_LINE |
|
2310 |
if (xtty != NULL) xtty->tail("statistics"); |
|
2311 |
} |
|
2312 |
} |
|
47799 | 2313 |
#else // COMPILER2_OR_JVMCI |
1 | 2314 |
|
2315 |
||
2316 |
// Stubs for C1 only system. |
|
2317 |
bool Deoptimization::trap_state_is_recompiled(int trap_state) { |
|
2318 |
return false; |
|
2319 |
} |
|
2320 |
||
2321 |
const char* Deoptimization::trap_reason_name(int reason) { |
|
2322 |
return "unknown"; |
|
2323 |
} |
|
2324 |
||
2325 |
void Deoptimization::print_statistics() { |
|
2326 |
// no output |
|
2327 |
} |
|
2328 |
||
2329 |
void |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
2330 |
Deoptimization::update_method_data_from_interpreter(MethodData* trap_mdo, int trap_bci, int reason) { |
1 | 2331 |
// no udpate |
2332 |
} |
|
2333 |
||
2334 |
int Deoptimization::trap_state_has_reason(int trap_state, int reason) { |
|
2335 |
return 0; |
|
2336 |
} |
|
2337 |
||
2338 |
void Deoptimization::gather_statistics(DeoptReason reason, DeoptAction action, |
|
2339 |
Bytecodes::Code bc) { |
|
2340 |
// no update |
|
2341 |
} |
|
2342 |
||
2343 |
const char* Deoptimization::format_trap_state(char* buf, size_t buflen, |
|
2344 |
int trap_state) { |
|
2345 |
jio_snprintf(buf, buflen, "#%d", trap_state); |
|
2346 |
return buf; |
|
2347 |
} |
|
2348 |
||
47799 | 2349 |
#endif // COMPILER2_OR_JVMCI |