Mercurial > hg > truffle
annotate src/share/vm/runtime/vframeArray.cpp @ 4678:a03f3fd16b22
Fix reexecute boolean in HotSpot debug information. Introduce "duringCall" flag in FrameState that indicates that the bci of the frame state denotes an invoke that should *not* be reexecuted.
| author | Thomas Wuerthinger <thomas.wuerthinger@oracle.com> |
|---|---|
| date | Thu, 23 Feb 2012 21:43:59 +0100 |
| parents | 3dbcd1013cc8 |
| children | 07bcee8b70a4 |
| rev | line source |
|---|---|
| 0 | 1 /* |
| 2142 | 2 * Copyright (c) 1997, 2011, 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:
1506
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
1506
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:
1506
diff
changeset
|
21 * questions. |
| 0 | 22 * |
| 23 */ | |
| 24 | |
| 1972 | 25 #include "precompiled.hpp" |
| 26 #include "classfile/vmSymbols.hpp" | |
|
3153
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
27 #include "code/scopeDesc.hpp" |
| 1972 | 28 #include "interpreter/interpreter.hpp" |
| 29 #include "memory/allocation.inline.hpp" | |
| 30 #include "memory/resourceArea.hpp" | |
| 31 #include "memory/universe.inline.hpp" | |
| 32 #include "oops/methodDataOop.hpp" | |
| 33 #include "oops/oop.inline.hpp" | |
| 34 #include "prims/jvmtiThreadState.hpp" | |
| 35 #include "runtime/handles.inline.hpp" | |
| 36 #include "runtime/monitorChunk.hpp" | |
| 37 #include "runtime/sharedRuntime.hpp" | |
| 38 #include "runtime/vframe.hpp" | |
| 39 #include "runtime/vframeArray.hpp" | |
| 40 #include "runtime/vframe_hp.hpp" | |
| 41 #include "utilities/events.hpp" | |
| 42 #ifdef COMPILER2 | |
| 43 #include "opto/runtime.hpp" | |
| 44 #endif | |
| 0 | 45 |
| 46 | |
| 47 int vframeArrayElement:: bci(void) const { return (_bci == SynchronizationEntryBCI ? 0 : _bci); } | |
| 48 | |
| 49 void vframeArrayElement::free_monitors(JavaThread* jt) { | |
| 50 if (_monitors != NULL) { | |
| 51 MonitorChunk* chunk = _monitors; | |
| 52 _monitors = NULL; | |
| 53 jt->remove_monitor_chunk(chunk); | |
| 54 delete chunk; | |
| 55 } | |
| 56 } | |
| 57 | |
| 58 void vframeArrayElement::fill_in(compiledVFrame* vf) { | |
| 59 | |
| 60 // Copy the information from the compiled vframe to the | |
| 61 // interpreter frame we will be creating to replace vf | |
| 62 | |
| 63 _method = vf->method(); | |
| 64 _bci = vf->raw_bci(); | |
|
900
9987d9d5eb0e
6833129: specjvm98 fails with NullPointerException in the compiler with -XX:DeoptimizeALot
cfang
parents:
818
diff
changeset
|
65 _reexecute = vf->should_reexecute(); |
|
3153
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
66 |
| 0 | 67 int index; |
| 68 | |
| 69 // Get the monitors off-stack | |
| 70 | |
| 71 GrowableArray<MonitorInfo*>* list = vf->monitors(); | |
| 72 if (list->is_empty()) { | |
| 73 _monitors = NULL; | |
| 74 } else { | |
| 75 | |
| 76 // Allocate monitor chunk | |
| 77 _monitors = new MonitorChunk(list->length()); | |
| 78 vf->thread()->add_monitor_chunk(_monitors); | |
| 79 | |
| 80 // Migrate the BasicLocks from the stack to the monitor chunk | |
| 81 for (index = 0; index < list->length(); index++) { | |
| 82 MonitorInfo* monitor = list->at(index); | |
|
818
b109e761e927
6837472: com/sun/jdi/MonitorFrameInfo.java fails with AggressiveOpts in 6u14
kvn
parents:
0
diff
changeset
|
83 assert(!monitor->owner_is_scalar_replaced(), "object should be reallocated already"); |
| 0 | 84 assert(monitor->owner() == NULL || (!monitor->owner()->is_unlocked() && !monitor->owner()->has_bias_pattern()), "object must be null or locked, and unbiased"); |
| 85 BasicObjectLock* dest = _monitors->at(index); | |
| 86 dest->set_obj(monitor->owner()); | |
| 87 monitor->lock()->move_to(monitor->owner(), dest->lock()); | |
| 88 } | |
| 89 } | |
| 90 | |
| 91 // Convert the vframe locals and expressions to off stack | |
| 92 // values. Because we will not gc all oops can be converted to | |
| 93 // intptr_t (i.e. a stack slot) and we are fine. This is | |
| 94 // good since we are inside a HandleMark and the oops in our | |
| 95 // collection would go away between packing them here and | |
| 96 // unpacking them in unpack_on_stack. | |
| 97 | |
| 98 // First the locals go off-stack | |
| 99 | |
| 100 // FIXME this seems silly it creates a StackValueCollection | |
| 101 // in order to get the size to then copy them and | |
| 102 // convert the types to intptr_t size slots. Seems like it | |
| 103 // could do it in place... Still uses less memory than the | |
| 104 // old way though | |
| 105 | |
| 106 StackValueCollection *locs = vf->locals(); | |
| 107 _locals = new StackValueCollection(locs->size()); | |
| 108 for(index = 0; index < locs->size(); index++) { | |
| 109 StackValue* value = locs->at(index); | |
| 110 switch(value->type()) { | |
| 111 case T_OBJECT: | |
|
818
b109e761e927
6837472: com/sun/jdi/MonitorFrameInfo.java fails with AggressiveOpts in 6u14
kvn
parents:
0
diff
changeset
|
112 assert(!value->obj_is_scalar_replaced(), "object should be reallocated already"); |
| 0 | 113 // preserve object type |
| 114 _locals->add( new StackValue((intptr_t) (value->get_obj()()), T_OBJECT )); | |
| 115 break; | |
| 116 case T_CONFLICT: | |
| 117 // A dead local. Will be initialized to null/zero. | |
| 118 _locals->add( new StackValue()); | |
| 119 break; | |
| 120 case T_INT: | |
| 121 _locals->add( new StackValue(value->get_int())); | |
| 122 break; | |
| 123 default: | |
| 124 ShouldNotReachHere(); | |
| 125 } | |
| 126 } | |
| 127 | |
| 128 // Now the expressions off-stack | |
| 129 // Same silliness as above | |
|
3153
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
130 bool rethrow_exception = vf->scope()->rethrow_exception(); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
131 if (rethrow_exception) { |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
132 // (tw) Make sure there are only null pointers on the stack, because the stack values do not correspond to the GC map at the bytecode at which the exception is rethrown. |
|
4678
a03f3fd16b22
Fix reexecute boolean in HotSpot debug information. Introduce "duringCall" flag in FrameState that indicates that the bci of the frame state denotes an invoke that should *not* be reexecuted.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
4668
diff
changeset
|
133 // TODO: Fix this! Locals map might be wrong too. |
|
3153
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
134 _expressions = new StackValueCollection(vf->method()->max_stack()); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
135 assert(Thread::current()->has_pending_exception(), "just checking"); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
136 for (int i=0; i<vf->method()->max_stack(); ++i) { |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
137 _expressions->add( new StackValue()); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
138 } |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
139 } else { |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
140 StackValueCollection *exprs = vf->expressions(); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
141 _expressions = new StackValueCollection(exprs->size()); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
142 for(index = 0; index < exprs->size(); index++) { |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
143 StackValue* value = exprs->at(index); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
144 switch(value->type()) { |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
145 case T_OBJECT: |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
146 assert(!value->obj_is_scalar_replaced(), "object should be reallocated already"); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
147 // preserve object type |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
148 _expressions->add( new StackValue((intptr_t) (value->get_obj()()), T_OBJECT )); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
149 break; |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
150 case T_CONFLICT: |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
151 // A dead stack element. Will be initialized to null/zero. |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
152 // This can occur when the compiler emits a state in which stack |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
153 // elements are known to be dead (because of an imminent exception). |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
154 _expressions->add( new StackValue()); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
155 break; |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
156 case T_INT: |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
157 _expressions->add( new StackValue(value->get_int())); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
158 break; |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
159 default: |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
160 ShouldNotReachHere(); |
|
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
161 } |
| 0 | 162 } |
| 163 } | |
| 164 } | |
| 165 | |
| 166 int unpack_counter = 0; | |
| 167 | |
|
3369
3d2ab563047a
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
2142
diff
changeset
|
168 void vframeArrayElement::unpack_on_stack(int caller_actual_parameters, |
|
3d2ab563047a
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
2142
diff
changeset
|
169 int callee_parameters, |
| 0 | 170 int callee_locals, |
| 171 frame* caller, | |
| 172 bool is_top_frame, | |
| 173 int exec_mode) { | |
| 174 JavaThread* thread = (JavaThread*) Thread::current(); | |
| 175 | |
| 176 // Look at bci and decide on bcp and continuation pc | |
| 177 address bcp; | |
| 178 // C++ interpreter doesn't need a pc since it will figure out what to do when it | |
| 179 // begins execution | |
| 180 address pc; | |
|
900
9987d9d5eb0e
6833129: specjvm98 fails with NullPointerException in the compiler with -XX:DeoptimizeALot
cfang
parents:
818
diff
changeset
|
181 bool use_next_mdp = false; // true if we should use the mdp associated with the next bci |
|
9987d9d5eb0e
6833129: specjvm98 fails with NullPointerException in the compiler with -XX:DeoptimizeALot
cfang
parents:
818
diff
changeset
|
182 // rather than the one associated with bcp |
| 0 | 183 if (raw_bci() == SynchronizationEntryBCI) { |
| 184 // We are deoptimizing while hanging in prologue code for synchronized method | |
| 185 bcp = method()->bcp_from(0); // first byte code | |
| 186 pc = Interpreter::deopt_entry(vtos, 0); // step = 0 since we don't skip current bytecode | |
|
900
9987d9d5eb0e
6833129: specjvm98 fails with NullPointerException in the compiler with -XX:DeoptimizeALot
cfang
parents:
818
diff
changeset
|
187 } else if (should_reexecute()) { //reexecute this bytecode |
|
9987d9d5eb0e
6833129: specjvm98 fails with NullPointerException in the compiler with -XX:DeoptimizeALot
cfang
parents:
818
diff
changeset
|
188 assert(is_top_frame, "reexecute allowed only for the top frame"); |
|
9987d9d5eb0e
6833129: specjvm98 fails with NullPointerException in the compiler with -XX:DeoptimizeALot
cfang
parents:
818
diff
changeset
|
189 bcp = method()->bcp_from(bci()); |
|
9987d9d5eb0e
6833129: specjvm98 fails with NullPointerException in the compiler with -XX:DeoptimizeALot
cfang
parents:
818
diff
changeset
|
190 pc = Interpreter::deopt_reexecute_entry(method(), bcp); |
| 0 | 191 } else { |
| 192 bcp = method()->bcp_from(bci()); | |
|
900
9987d9d5eb0e
6833129: specjvm98 fails with NullPointerException in the compiler with -XX:DeoptimizeALot
cfang
parents:
818
diff
changeset
|
193 pc = Interpreter::deopt_continue_after_entry(method(), bcp, callee_parameters, is_top_frame); |
|
9987d9d5eb0e
6833129: specjvm98 fails with NullPointerException in the compiler with -XX:DeoptimizeALot
cfang
parents:
818
diff
changeset
|
194 use_next_mdp = true; |
| 0 | 195 } |
| 196 assert(Bytecodes::is_defined(*bcp), "must be a valid bytecode"); | |
| 197 | |
| 198 // Monitorenter and pending exceptions: | |
| 199 // | |
| 200 // For Compiler2, there should be no pending exception when deoptimizing at monitorenter | |
| 201 // because there is no safepoint at the null pointer check (it is either handled explicitly | |
| 202 // or prior to the monitorenter) and asynchronous exceptions are not made "pending" by the | |
| 203 // runtime interface for the slow case (see JRT_ENTRY_FOR_MONITORENTER). If an asynchronous | |
| 204 // exception was processed, the bytecode pointer would have to be extended one bytecode beyond | |
| 205 // the monitorenter to place it in the proper exception range. | |
| 206 // | |
| 207 // For Compiler1, deoptimization can occur while throwing a NullPointerException at monitorenter, | |
| 208 // in which case bcp should point to the monitorenter since it is within the exception's range. | |
| 209 | |
| 210 assert(*bcp != Bytecodes::_monitorenter || is_top_frame, "a _monitorenter must be a top frame"); | |
|
1814
fd5d4527cdf5
6986270: guarantee(*bcp != Bytecodes::_monitorenter || exec_mode != Deoptimization::Unpack_exception) fails
iveresov
parents:
1603
diff
changeset
|
211 assert(thread->deopt_nmethod() != NULL, "nmethod should be known"); |
|
fd5d4527cdf5
6986270: guarantee(*bcp != Bytecodes::_monitorenter || exec_mode != Deoptimization::Unpack_exception) fails
iveresov
parents:
1603
diff
changeset
|
212 guarantee(!(thread->deopt_nmethod()->is_compiled_by_c2() && |
|
fd5d4527cdf5
6986270: guarantee(*bcp != Bytecodes::_monitorenter || exec_mode != Deoptimization::Unpack_exception) fails
iveresov
parents:
1603
diff
changeset
|
213 *bcp == Bytecodes::_monitorenter && |
|
fd5d4527cdf5
6986270: guarantee(*bcp != Bytecodes::_monitorenter || exec_mode != Deoptimization::Unpack_exception) fails
iveresov
parents:
1603
diff
changeset
|
214 exec_mode == Deoptimization::Unpack_exception), |
|
fd5d4527cdf5
6986270: guarantee(*bcp != Bytecodes::_monitorenter || exec_mode != Deoptimization::Unpack_exception) fails
iveresov
parents:
1603
diff
changeset
|
215 "shouldn't get exception during monitorenter"); |
| 0 | 216 |
| 217 int popframe_preserved_args_size_in_bytes = 0; | |
| 218 int popframe_preserved_args_size_in_words = 0; | |
| 219 if (is_top_frame) { | |
|
1255
e3a4305c6bc3
6925249: assert(last_sp < (intptr_t*) interpreter_frame_monitor_begin(),"bad tos")
kvn
parents:
903
diff
changeset
|
220 JvmtiThreadState *state = thread->jvmti_thread_state(); |
| 0 | 221 if (JvmtiExport::can_pop_frame() && |
| 222 (thread->has_pending_popframe() || thread->popframe_forcing_deopt_reexecution())) { | |
| 223 if (thread->has_pending_popframe()) { | |
| 224 // Pop top frame after deoptimization | |
| 225 #ifndef CC_INTERP | |
| 226 pc = Interpreter::remove_activation_preserving_args_entry(); | |
| 227 #else | |
| 228 // Do an uncommon trap type entry. c++ interpreter will know | |
| 229 // to pop frame and preserve the args | |
| 230 pc = Interpreter::deopt_entry(vtos, 0); | |
| 231 use_next_mdp = false; | |
| 232 #endif | |
| 233 } else { | |
| 234 // Reexecute invoke in top frame | |
| 235 pc = Interpreter::deopt_entry(vtos, 0); | |
| 236 use_next_mdp = false; | |
| 237 popframe_preserved_args_size_in_bytes = in_bytes(thread->popframe_preserved_args_size()); | |
| 238 // Note: the PopFrame-related extension of the expression stack size is done in | |
| 239 // Deoptimization::fetch_unroll_info_helper | |
| 240 popframe_preserved_args_size_in_words = in_words(thread->popframe_preserved_args_size_in_words()); | |
| 241 } | |
| 242 } else if (JvmtiExport::can_force_early_return() && state != NULL && state->is_earlyret_pending()) { | |
| 243 // Force early return from top frame after deoptimization | |
| 244 #ifndef CC_INTERP | |
| 245 pc = Interpreter::remove_activation_early_entry(state->earlyret_tos()); | |
| 246 #else | |
| 247 // TBD: Need to implement ForceEarlyReturn for CC_INTERP (ia64) | |
| 248 #endif | |
| 249 } else { | |
| 250 // Possibly override the previous pc computation of the top (youngest) frame | |
| 251 switch (exec_mode) { | |
| 252 case Deoptimization::Unpack_deopt: | |
| 253 // use what we've got | |
| 254 break; | |
| 255 case Deoptimization::Unpack_exception: | |
| 256 // exception is pending | |
| 1295 | 257 pc = SharedRuntime::raw_exception_handler_for_return_address(thread, pc); |
| 0 | 258 // [phh] We're going to end up in some handler or other, so it doesn't |
| 259 // matter what mdp we point to. See exception_handler_for_exception() | |
| 260 // in interpreterRuntime.cpp. | |
| 261 break; | |
| 262 case Deoptimization::Unpack_uncommon_trap: | |
| 263 case Deoptimization::Unpack_reexecute: | |
| 264 // redo last byte code | |
|
4678
a03f3fd16b22
Fix reexecute boolean in HotSpot debug information. Introduce "duringCall" flag in FrameState that indicates that the bci of the frame state denotes an invoke that should *not* be reexecuted.
Thomas Wuerthinger <thomas.wuerthinger@oracle.com>
parents:
4668
diff
changeset
|
265 assert(should_reexecute(), ""); |
| 0 | 266 pc = Interpreter::deopt_entry(vtos, 0); |
| 267 use_next_mdp = false; | |
| 268 break; | |
| 269 default: | |
| 270 ShouldNotReachHere(); | |
| 271 } | |
| 272 } | |
| 273 } | |
| 274 | |
| 275 // Setup the interpreter frame | |
| 276 | |
| 277 assert(method() != NULL, "method must exist"); | |
| 278 int temps = expressions()->size(); | |
| 279 | |
| 280 int locks = monitors() == NULL ? 0 : monitors()->number_of_monitors(); | |
| 281 | |
| 282 Interpreter::layout_activation(method(), | |
| 283 temps + callee_parameters, | |
| 284 popframe_preserved_args_size_in_words, | |
| 285 locks, | |
|
3369
3d2ab563047a
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
2142
diff
changeset
|
286 caller_actual_parameters, |
| 0 | 287 callee_parameters, |
| 288 callee_locals, | |
| 289 caller, | |
| 290 iframe(), | |
| 291 is_top_frame); | |
| 292 | |
| 293 // Update the pc in the frame object and overwrite the temporary pc | |
| 294 // we placed in the skeletal frame now that we finally know the | |
| 295 // exact interpreter address we should use. | |
| 296 | |
| 297 _frame.patch_pc(thread, pc); | |
| 298 | |
| 299 assert (!method()->is_synchronized() || locks > 0, "synchronized methods must have monitors"); | |
| 300 | |
| 301 BasicObjectLock* top = iframe()->interpreter_frame_monitor_begin(); | |
| 302 for (int index = 0; index < locks; index++) { | |
| 303 top = iframe()->previous_monitor_in_interpreter_frame(top); | |
| 304 BasicObjectLock* src = _monitors->at(index); | |
| 305 top->set_obj(src->obj()); | |
| 306 src->lock()->move_to(src->obj(), top->lock()); | |
| 307 } | |
| 308 if (ProfileInterpreter) { | |
| 309 iframe()->interpreter_frame_set_mdx(0); // clear out the mdp. | |
| 310 } | |
| 311 iframe()->interpreter_frame_set_bcx((intptr_t)bcp); // cannot use bcp because frame is not initialized yet | |
| 312 if (ProfileInterpreter) { | |
| 313 methodDataOop mdo = method()->method_data(); | |
| 314 if (mdo != NULL) { | |
| 315 int bci = iframe()->interpreter_frame_bci(); | |
| 316 if (use_next_mdp) ++bci; | |
| 317 address mdp = mdo->bci_to_dp(bci); | |
| 318 iframe()->interpreter_frame_set_mdp(mdp); | |
| 319 } | |
| 320 } | |
|
2607
008adfd6d850
Fixed the stateBefore of invokes and monitorenter instructions to include the arguments of the instruction.
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2181
diff
changeset
|
321 |
|
4668
3dbcd1013cc8
added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents:
3464
diff
changeset
|
322 if (PrintDeoptimizationDetails) { |
|
2607
008adfd6d850
Fixed the stateBefore of invokes and monitorenter instructions to include the arguments of the instruction.
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2181
diff
changeset
|
323 tty->print_cr("Expressions size: %d", expressions()->size()); |
|
008adfd6d850
Fixed the stateBefore of invokes and monitorenter instructions to include the arguments of the instruction.
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2181
diff
changeset
|
324 } |
| 0 | 325 |
| 326 // Unpack expression stack | |
| 327 // If this is an intermediate frame (i.e. not top frame) then this | |
| 328 // only unpacks the part of the expression stack not used by callee | |
| 329 // as parameters. The callee parameters are unpacked as part of the | |
| 330 // callee locals. | |
|
3153
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
331 for(int i = 0; i < expressions()->size(); i++) { |
| 0 | 332 StackValue *value = expressions()->at(i); |
| 333 intptr_t* addr = iframe()->interpreter_frame_expression_stack_at(i); | |
| 334 switch(value->type()) { | |
| 335 case T_INT: | |
| 336 *addr = value->get_int(); | |
|
1936
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
337 #ifndef PRODUCT |
|
4668
3dbcd1013cc8
added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents:
3464
diff
changeset
|
338 if (PrintDeoptimizationDetails) { |
|
1936
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
339 tty->print_cr("Reconstructed expression %d (INT): %d", i, (int)(*addr)); |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
340 } |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
341 #endif |
| 0 | 342 break; |
| 343 case T_OBJECT: | |
| 344 *addr = value->get_int(T_OBJECT); | |
|
1936
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
345 #ifndef PRODUCT |
|
4668
3dbcd1013cc8
added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents:
3464
diff
changeset
|
346 if (PrintDeoptimizationDetails) { |
|
1936
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
347 tty->print("Reconstructed expression %d (OBJECT): ", i); |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
348 oop o = (oop)(*addr); |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
349 if (o == NULL) { |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
350 tty->print_cr("NULL"); |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
351 } else { |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
352 tty->print_cr(err_msg("%s", o->blueprint()->name()->as_C_string())); |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
353 } |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
354 } |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
355 #endif |
| 0 | 356 break; |
| 357 case T_CONFLICT: | |
| 358 // A dead stack slot. Initialize to null in case it is an oop. | |
| 359 *addr = NULL_WORD; | |
| 360 break; | |
| 361 default: | |
| 362 ShouldNotReachHere(); | |
| 363 } | |
| 364 } | |
| 365 | |
| 366 | |
| 367 // Unpack the locals | |
|
3153
5ca1332171c8
Fixed an issue in the HotSpot code for rethrowing an exception at deopt (wrong values on the expression stack caused a crash when GC and deopt happened at the same time).
Thomas Wuerthinger <thomas@wuerthinger.net>
parents:
2607
diff
changeset
|
368 for(int i = 0; i < locals()->size(); i++) { |
| 0 | 369 StackValue *value = locals()->at(i); |
| 370 intptr_t* addr = iframe()->interpreter_frame_local_at(i); | |
| 371 switch(value->type()) { | |
| 372 case T_INT: | |
| 373 *addr = value->get_int(); | |
|
1936
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
374 #ifndef PRODUCT |
|
4668
3dbcd1013cc8
added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents:
3464
diff
changeset
|
375 if (PrintDeoptimizationDetails) { |
|
1936
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
376 tty->print_cr("Reconstructed local %d (INT): %d", i, (int)(*addr)); |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
377 } |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
378 #endif |
| 0 | 379 break; |
| 380 case T_OBJECT: | |
| 381 *addr = value->get_int(T_OBJECT); | |
|
1936
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
382 #ifndef PRODUCT |
|
4668
3dbcd1013cc8
added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents:
3464
diff
changeset
|
383 if (PrintDeoptimizationDetails) { |
|
1936
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
384 tty->print("Reconstructed local %d (OBJECT): ", i); |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
385 oop o = (oop)(*addr); |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
386 if (o == NULL) { |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
387 tty->print_cr("NULL"); |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
388 } else { |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
389 tty->print_cr(err_msg("%s", o->blueprint()->name()->as_C_string())); |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
390 } |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
391 } |
|
8d88c9ac9247
Correct deopt handler entry. New flag -XX:+TraceSignals. More detailed deopt printing.
Thomas Wuerthinger <wuerthinger@ssw.jku.at>
parents:
1814
diff
changeset
|
392 #endif |
| 0 | 393 break; |
| 394 case T_CONFLICT: | |
| 395 // A dead location. If it is an oop then we need a NULL to prevent GC from following it | |
| 396 *addr = NULL_WORD; | |
| 397 break; | |
| 398 default: | |
| 399 ShouldNotReachHere(); | |
| 400 } | |
| 401 } | |
| 402 | |
| 403 if (is_top_frame && JvmtiExport::can_pop_frame() && thread->popframe_forcing_deopt_reexecution()) { | |
| 404 // An interpreted frame was popped but it returns to a deoptimized | |
| 405 // frame. The incoming arguments to the interpreted activation | |
| 406 // were preserved in thread-local storage by the | |
| 407 // remove_activation_preserving_args_entry in the interpreter; now | |
| 408 // we put them back into the just-unpacked interpreter frame. | |
| 409 // Note that this assumes that the locals arena grows toward lower | |
| 410 // addresses. | |
| 411 if (popframe_preserved_args_size_in_words != 0) { | |
| 412 void* saved_args = thread->popframe_preserved_args(); | |
| 413 assert(saved_args != NULL, "must have been saved by interpreter"); | |
| 414 #ifdef ASSERT | |
| 415 assert(popframe_preserved_args_size_in_words <= | |
| 1506 | 416 iframe()->interpreter_frame_expression_stack_size()*Interpreter::stackElementWords, |
| 0 | 417 "expression stack size should have been extended"); |
| 418 #endif // ASSERT | |
| 419 int top_element = iframe()->interpreter_frame_expression_stack_size()-1; | |
| 420 intptr_t* base; | |
| 421 if (frame::interpreter_frame_expression_stack_direction() < 0) { | |
| 422 base = iframe()->interpreter_frame_expression_stack_at(top_element); | |
| 423 } else { | |
| 424 base = iframe()->interpreter_frame_expression_stack(); | |
| 425 } | |
|
1603
d93949c5bdcc
6730276: JDI_REGRESSION tests fail with "Error: count must be non-zero" error on x86
kvn
parents:
1552
diff
changeset
|
426 Copy::conjoint_jbytes(saved_args, |
|
d93949c5bdcc
6730276: JDI_REGRESSION tests fail with "Error: count must be non-zero" error on x86
kvn
parents:
1552
diff
changeset
|
427 base, |
|
d93949c5bdcc
6730276: JDI_REGRESSION tests fail with "Error: count must be non-zero" error on x86
kvn
parents:
1552
diff
changeset
|
428 popframe_preserved_args_size_in_bytes); |
| 0 | 429 thread->popframe_free_preserved_args(); |
| 430 } | |
| 431 } | |
| 432 | |
| 433 #ifndef PRODUCT | |
|
4668
3dbcd1013cc8
added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents:
3464
diff
changeset
|
434 if (PrintDeoptimizationDetails) { |
| 0 | 435 ttyLocker ttyl; |
| 436 tty->print_cr("[%d Interpreted Frame]", ++unpack_counter); | |
| 437 iframe()->print_on(tty); | |
| 438 RegisterMap map(thread); | |
| 439 vframe* f = vframe::new_vframe(iframe(), &map, thread); | |
| 440 f->print(); | |
| 441 tty->cr(); | |
| 442 // method()->print_codes(); | |
| 443 } else if (TraceDeoptimization) { | |
| 444 tty->print(" "); | |
| 445 method()->print_value(); | |
| 2142 | 446 Bytecodes::Code code = Bytecodes::java_code_at(method(), bcp); |
| 0 | 447 int bci = method()->bci_from(bcp); |
| 448 tty->print(" - %s", Bytecodes::name(code)); | |
| 449 tty->print(" @ bci %d ", bci); | |
| 450 tty->print_cr("sp = " PTR_FORMAT, iframe()->sp()); | |
| 451 } | |
| 452 #endif // PRODUCT | |
| 453 | |
| 454 // The expression stack and locals are in the resource area don't leave | |
| 455 // a dangling pointer in the vframeArray we leave around for debug | |
| 456 // purposes | |
| 457 | |
| 458 _locals = _expressions = NULL; | |
| 459 | |
| 460 } | |
| 461 | |
|
3369
3d2ab563047a
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
2142
diff
changeset
|
462 int vframeArrayElement::on_stack_size(int caller_actual_parameters, |
|
3d2ab563047a
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
2142
diff
changeset
|
463 int callee_parameters, |
| 0 | 464 int callee_locals, |
| 465 bool is_top_frame, | |
| 466 int popframe_extra_stack_expression_els) const { | |
| 467 assert(method()->max_locals() == locals()->size(), "just checking"); | |
| 468 int locks = monitors() == NULL ? 0 : monitors()->number_of_monitors(); | |
| 469 int temps = expressions()->size(); | |
| 470 return Interpreter::size_activation(method(), | |
| 471 temps + callee_parameters, | |
| 472 popframe_extra_stack_expression_els, | |
| 473 locks, | |
|
3369
3d2ab563047a
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
2142
diff
changeset
|
474 caller_actual_parameters, |
| 0 | 475 callee_parameters, |
| 476 callee_locals, | |
| 477 is_top_frame); | |
| 478 } | |
| 479 | |
| 480 | |
| 481 | |
| 482 vframeArray* vframeArray::allocate(JavaThread* thread, int frame_size, GrowableArray<compiledVFrame*>* chunk, | |
| 483 RegisterMap *reg_map, frame sender, frame caller, frame self) { | |
| 484 | |
| 485 // Allocate the vframeArray | |
| 486 vframeArray * result = (vframeArray*) AllocateHeap(sizeof(vframeArray) + // fixed part | |
| 487 sizeof(vframeArrayElement) * (chunk->length() - 1), // variable part | |
| 488 "vframeArray::allocate"); | |
| 489 result->_frames = chunk->length(); | |
| 490 result->_owner_thread = thread; | |
| 491 result->_sender = sender; | |
| 492 result->_caller = caller; | |
| 493 result->_original = self; | |
| 494 result->set_unroll_block(NULL); // initialize it | |
| 495 result->fill_in(thread, frame_size, chunk, reg_map); | |
| 496 return result; | |
| 497 } | |
| 498 | |
| 499 void vframeArray::fill_in(JavaThread* thread, | |
| 500 int frame_size, | |
| 501 GrowableArray<compiledVFrame*>* chunk, | |
| 502 const RegisterMap *reg_map) { | |
| 503 // Set owner first, it is used when adding monitor chunks | |
| 504 | |
| 505 _frame_size = frame_size; | |
| 506 for(int i = 0; i < chunk->length(); i++) { | |
| 507 element(i)->fill_in(chunk->at(i)); | |
| 508 } | |
| 509 | |
| 510 // Copy registers for callee-saved registers | |
| 511 if (reg_map != NULL) { | |
| 512 for(int i = 0; i < RegisterMap::reg_count; i++) { | |
| 513 #ifdef AMD64 | |
| 514 // The register map has one entry for every int (32-bit value), so | |
| 515 // 64-bit physical registers have two entries in the map, one for | |
| 516 // each half. Ignore the high halves of 64-bit registers, just like | |
| 517 // frame::oopmapreg_to_location does. | |
| 518 // | |
| 519 // [phh] FIXME: this is a temporary hack! This code *should* work | |
| 520 // correctly w/o this hack, possibly by changing RegisterMap::pd_location | |
| 521 // in frame_amd64.cpp and the values of the phantom high half registers | |
| 522 // in amd64.ad. | |
| 523 // if (VMReg::Name(i) < SharedInfo::stack0 && is_even(i)) { | |
| 524 intptr_t* src = (intptr_t*) reg_map->location(VMRegImpl::as_VMReg(i)); | |
| 525 _callee_registers[i] = src != NULL ? *src : NULL_WORD; | |
| 526 // } else { | |
| 527 // jint* src = (jint*) reg_map->location(VMReg::Name(i)); | |
| 528 // _callee_registers[i] = src != NULL ? *src : NULL_WORD; | |
| 529 // } | |
| 530 #else | |
| 531 jint* src = (jint*) reg_map->location(VMRegImpl::as_VMReg(i)); | |
| 532 _callee_registers[i] = src != NULL ? *src : NULL_WORD; | |
| 533 #endif | |
| 534 if (src == NULL) { | |
| 535 set_location_valid(i, false); | |
| 536 } else { | |
| 537 set_location_valid(i, true); | |
| 538 jint* dst = (jint*) register_location(i); | |
| 539 *dst = *src; | |
| 540 } | |
| 541 } | |
| 542 } | |
| 543 } | |
| 544 | |
|
3369
3d2ab563047a
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
2142
diff
changeset
|
545 void vframeArray::unpack_to_stack(frame &unpack_frame, int exec_mode, int caller_actual_parameters) { |
| 0 | 546 // stack picture |
| 547 // unpack_frame | |
| 548 // [new interpreter frames ] (frames are skeletal but walkable) | |
| 549 // caller_frame | |
| 550 // | |
| 551 // This routine fills in the missing data for the skeletal interpreter frames | |
| 552 // in the above picture. | |
| 553 | |
| 554 // Find the skeletal interpreter frames to unpack into | |
| 555 RegisterMap map(JavaThread::current(), false); | |
| 556 // Get the youngest frame we will unpack (last to be unpacked) | |
| 557 frame me = unpack_frame.sender(&map); | |
| 558 int index; | |
| 559 for (index = 0; index < frames(); index++ ) { | |
| 560 *element(index)->iframe() = me; | |
| 561 // Get the caller frame (possibly skeletal) | |
| 562 me = me.sender(&map); | |
| 563 } | |
| 564 | |
| 565 frame caller_frame = me; | |
| 566 | |
| 567 // Do the unpacking of interpreter frames; the frame at index 0 represents the top activation, so it has no callee | |
| 568 | |
| 569 // Unpack the frames from the oldest (frames() -1) to the youngest (0) | |
| 570 | |
| 571 for (index = frames() - 1; index >= 0 ; index--) { | |
| 572 int callee_parameters = index == 0 ? 0 : element(index-1)->method()->size_of_parameters(); | |
| 573 int callee_locals = index == 0 ? 0 : element(index-1)->method()->max_locals(); | |
|
3369
3d2ab563047a
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
2142
diff
changeset
|
574 element(index)->unpack_on_stack(caller_actual_parameters, |
|
3d2ab563047a
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
2142
diff
changeset
|
575 callee_parameters, |
| 0 | 576 callee_locals, |
| 577 &caller_frame, | |
| 578 index == 0, | |
| 579 exec_mode); | |
| 580 if (index == frames() - 1) { | |
| 581 Deoptimization::unwind_callee_save_values(element(index)->iframe(), this); | |
| 582 } | |
| 583 caller_frame = *element(index)->iframe(); | |
|
3369
3d2ab563047a
7043461: VM crashes in void LinkResolver::runtime_resolve_virtual_method
never
parents:
2142
diff
changeset
|
584 caller_actual_parameters = callee_parameters; |
| 0 | 585 } |
| 586 | |
| 587 | |
| 588 deallocate_monitor_chunks(); | |
| 589 } | |
| 590 | |
| 591 void vframeArray::deallocate_monitor_chunks() { | |
| 592 JavaThread* jt = JavaThread::current(); | |
| 593 for (int index = 0; index < frames(); index++ ) { | |
| 594 element(index)->free_monitors(jt); | |
| 595 } | |
| 596 } | |
| 597 | |
| 598 #ifndef PRODUCT | |
| 599 | |
| 600 bool vframeArray::structural_compare(JavaThread* thread, GrowableArray<compiledVFrame*>* chunk) { | |
| 601 if (owner_thread() != thread) return false; | |
| 602 int index = 0; | |
| 603 #if 0 // FIXME can't do this comparison | |
| 604 | |
| 605 // Compare only within vframe array. | |
| 606 for (deoptimizedVFrame* vf = deoptimizedVFrame::cast(vframe_at(first_index())); vf; vf = vf->deoptimized_sender_or_null()) { | |
| 607 if (index >= chunk->length() || !vf->structural_compare(chunk->at(index))) return false; | |
| 608 index++; | |
| 609 } | |
| 610 if (index != chunk->length()) return false; | |
| 611 #endif | |
| 612 | |
| 613 return true; | |
| 614 } | |
| 615 | |
| 616 #endif | |
| 617 | |
| 618 address vframeArray::register_location(int i) const { | |
| 619 assert(0 <= i && i < RegisterMap::reg_count, "index out of bounds"); | |
| 620 return (address) & _callee_registers[i]; | |
| 621 } | |
| 622 | |
| 623 | |
| 624 #ifndef PRODUCT | |
| 625 | |
| 626 // Printing | |
| 627 | |
| 628 // Note: we cannot have print_on as const, as we allocate inside the method | |
| 629 void vframeArray::print_on_2(outputStream* st) { | |
| 630 st->print_cr(" - sp: " INTPTR_FORMAT, sp()); | |
| 631 st->print(" - thread: "); | |
| 632 Thread::current()->print(); | |
| 633 st->print_cr(" - frame size: %d", frame_size()); | |
| 634 for (int index = 0; index < frames() ; index++ ) { | |
| 635 element(index)->print(st); | |
| 636 } | |
| 637 } | |
| 638 | |
| 639 void vframeArrayElement::print(outputStream* st) { | |
|
1255
e3a4305c6bc3
6925249: assert(last_sp < (intptr_t*) interpreter_frame_monitor_begin(),"bad tos")
kvn
parents:
903
diff
changeset
|
640 st->print_cr(" - interpreter_frame -> sp: " INTPTR_FORMAT, iframe()->sp()); |
| 0 | 641 } |
| 642 | |
| 643 void vframeArray::print_value_on(outputStream* st) const { | |
| 644 st->print_cr("vframeArray [%d] ", frames()); | |
| 645 } | |
| 646 | |
| 647 | |
| 648 #endif |