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