annotate src/cpu/x86/vm/frame_x86.cpp @ 4668:3dbcd1013cc8

added flag PrintDeoptimizationDetails changes so that TraceDeoptimization only prints basic information added nicer formatting for printing interpreter stack frames
author Christian Haeubl <christian.haeubl@oracle.com>
date Tue, 21 Feb 2012 16:05:46 -0800
parents 04b9a2566eec
children 33df1aeaebbf
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
0
a61af66fc99e Initial load
duke
parents:
diff changeset
1 /*
3336
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
2 * Copyright (c) 1997, 2011, Oracle and/or its affiliates. All rights reserved.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
a61af66fc99e Initial load
duke
parents:
diff changeset
4 *
a61af66fc99e Initial load
duke
parents:
diff changeset
5 * This code is free software; you can redistribute it and/or modify it
a61af66fc99e Initial load
duke
parents:
diff changeset
6 * under the terms of the GNU General Public License version 2 only, as
a61af66fc99e Initial load
duke
parents:
diff changeset
7 * published by the Free Software Foundation.
a61af66fc99e Initial load
duke
parents:
diff changeset
8 *
a61af66fc99e Initial load
duke
parents:
diff changeset
9 * This code is distributed in the hope that it will be useful, but WITHOUT
a61af66fc99e Initial load
duke
parents:
diff changeset
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
a61af66fc99e Initial load
duke
parents:
diff changeset
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
a61af66fc99e Initial load
duke
parents:
diff changeset
12 * version 2 for more details (a copy is included in the LICENSE file that
a61af66fc99e Initial load
duke
parents:
diff changeset
13 * accompanied this code).
a61af66fc99e Initial load
duke
parents:
diff changeset
14 *
a61af66fc99e Initial load
duke
parents:
diff changeset
15 * You should have received a copy of the GNU General Public License version
a61af66fc99e Initial load
duke
parents:
diff changeset
16 * 2 along with this work; if not, write to the Free Software Foundation,
a61af66fc99e Initial load
duke
parents:
diff changeset
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
a61af66fc99e Initial load
duke
parents:
diff changeset
18 *
1552
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1513
diff changeset
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
c18cbe5936b8 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1513
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: 1513
diff changeset
21 * questions.
0
a61af66fc99e Initial load
duke
parents:
diff changeset
22 *
a61af66fc99e Initial load
duke
parents:
diff changeset
23 */
a61af66fc99e Initial load
duke
parents:
diff changeset
24
1972
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
25 #include "precompiled.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
26 #include "interpreter/interpreter.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
27 #include "memory/resourceArea.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
28 #include "oops/markOop.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
29 #include "oops/methodOop.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
30 #include "oops/oop.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
31 #include "runtime/frame.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
32 #include "runtime/handles.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
33 #include "runtime/javaCalls.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
34 #include "runtime/monitorChunk.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
35 #include "runtime/signature.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
36 #include "runtime/stubCodeGenerator.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
37 #include "runtime/stubRoutines.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
38 #include "vmreg_x86.inline.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
39 #ifdef COMPILER1
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
40 #include "c1/c1_Runtime1.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
41 #include "runtime/vframeArray.hpp"
f95d63e2154a 6989984: Use standard include model for Hospot
stefank
parents: 1748
diff changeset
42 #endif
0
a61af66fc99e Initial load
duke
parents:
diff changeset
43
a61af66fc99e Initial load
duke
parents:
diff changeset
44 #ifdef ASSERT
a61af66fc99e Initial load
duke
parents:
diff changeset
45 void RegisterMap::check_location_valid() {
a61af66fc99e Initial load
duke
parents:
diff changeset
46 }
a61af66fc99e Initial load
duke
parents:
diff changeset
47 #endif
a61af66fc99e Initial load
duke
parents:
diff changeset
48
a61af66fc99e Initial load
duke
parents:
diff changeset
49
a61af66fc99e Initial load
duke
parents:
diff changeset
50 // Profiling/safepoint support
a61af66fc99e Initial load
duke
parents:
diff changeset
51
a61af66fc99e Initial load
duke
parents:
diff changeset
52 bool frame::safe_for_sender(JavaThread *thread) {
a61af66fc99e Initial load
duke
parents:
diff changeset
53 address sp = (address)_sp;
a61af66fc99e Initial load
duke
parents:
diff changeset
54 address fp = (address)_fp;
a61af66fc99e Initial load
duke
parents:
diff changeset
55 address unextended_sp = (address)_unextended_sp;
107
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
56 // sp must be within the stack
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
57 bool sp_safe = (sp <= thread->stack_base()) &&
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
58 (sp >= thread->stack_base() - thread->stack_size());
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
59
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
60 if (!sp_safe) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
61 return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
62 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
63
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
64 // unextended sp must be within the stack and above or equal sp
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
65 bool unextended_sp_safe = (unextended_sp <= thread->stack_base()) &&
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
66 (unextended_sp >= sp);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
67
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
68 if (!unextended_sp_safe) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
69 return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
70 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
71
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
72 // an fp must be within the stack and above (but not equal) sp
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
73 bool fp_safe = (fp <= thread->stack_base()) && (fp > sp);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
74
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
75 // We know sp/unextended_sp are safe only fp is questionable here
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
76
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
77 // If the current frame is known to the code cache then we can attempt to
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
78 // to construct the sender and do some validation of it. This goes a long way
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
79 // toward eliminating issues when we get in frame construction code
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
80
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
81 if (_cb != NULL ) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
82
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
83 // First check if frame is complete and tester is reliable
0
a61af66fc99e Initial load
duke
parents:
diff changeset
84 // Unfortunately we can only check frame complete for runtime stubs and nmethod
a61af66fc99e Initial load
duke
parents:
diff changeset
85 // other generic buffer blobs are more problematic so we just assume they are
a61af66fc99e Initial load
duke
parents:
diff changeset
86 // ok. adapter blobs never have a frame complete and are never ok.
107
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
87
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
88 if (!_cb->is_frame_complete_at(_pc)) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
89 if (_cb->is_nmethod() || _cb->is_adapter_blob() || _cb->is_runtime_stub()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
90 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
91 }
a61af66fc99e Initial load
duke
parents:
diff changeset
92 }
107
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
93 // Entry frame checks
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
94 if (is_entry_frame()) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
95 // an entry frame must have a valid fp.
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
96
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
97 if (!fp_safe) return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
98
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
99 // Validate the JavaCallWrapper an entry frame must have
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
100
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
101 address jcw = (address)entry_frame_call_wrapper();
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
102
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
103 bool jcw_safe = (jcw <= thread->stack_base()) && ( jcw > fp);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
104
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
105 return jcw_safe;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
106
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
107 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
108
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
109 intptr_t* sender_sp = NULL;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
110 address sender_pc = NULL;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
111
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
112 if (is_interpreted_frame()) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
113 // fp must be safe
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
114 if (!fp_safe) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
115 return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
116 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
117
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
118 sender_pc = (address) this->fp()[return_addr_offset];
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
119 sender_sp = (intptr_t*) addr_at(sender_sp_offset);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
120
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
121 } else {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
122 // must be some sort of compiled/runtime frame
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
123 // fp does not have to be safe (although it could be check for c1?)
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
124
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
125 sender_sp = _unextended_sp + _cb->frame_size();
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
126 // On Intel the return_address is always the word on the stack
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
127 sender_pc = (address) *(sender_sp-1);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
128 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
129
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
130 // We must always be able to find a recognizable pc
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
131 CodeBlob* sender_blob = CodeCache::find_blob_unsafe(sender_pc);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
132 if (sender_pc == NULL || sender_blob == NULL) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
133 return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
134 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
135
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
136
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
137 // If the potential sender is the interpreter then we can do some more checking
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
138 if (Interpreter::contains(sender_pc)) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
139
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
140 // ebp is always saved in a recognizable place in any code we generate. However
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
141 // only if the sender is interpreted/call_stub (c1 too?) are we certain that the saved ebp
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
142 // is really a frame pointer.
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
143
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
144 intptr_t *saved_fp = (intptr_t*)*(sender_sp - frame::sender_sp_offset);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
145 bool saved_fp_safe = ((address)saved_fp <= thread->stack_base()) && (saved_fp > sender_sp);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
146
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
147 if (!saved_fp_safe) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
148 return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
149 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
150
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
151 // construct the potential sender
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
152
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
153 frame sender(sender_sp, saved_fp, sender_pc);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
154
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
155 return sender.is_interpreted_frame_valid(thread);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
156
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
157 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
158
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
159 // Could just be some random pointer within the codeBlob
1748
3e8fbc61cee8 6978355: renaming for 6961697
twisti
parents: 1681
diff changeset
160 if (!sender_blob->code_contains(sender_pc)) {
3e8fbc61cee8 6978355: renaming for 6961697
twisti
parents: 1681
diff changeset
161 return false;
3e8fbc61cee8 6978355: renaming for 6961697
twisti
parents: 1681
diff changeset
162 }
107
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
163
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
164 // We should never be able to see an adapter if the current frame is something from code cache
1748
3e8fbc61cee8 6978355: renaming for 6961697
twisti
parents: 1681
diff changeset
165 if (sender_blob->is_adapter_blob()) {
107
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
166 return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
167 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
168
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
169 // Could be the call_stub
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
170
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
171 if (StubRoutines::returns_to_call_stub(sender_pc)) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
172 intptr_t *saved_fp = (intptr_t*)*(sender_sp - frame::sender_sp_offset);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
173 bool saved_fp_safe = ((address)saved_fp <= thread->stack_base()) && (saved_fp > sender_sp);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
174
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
175 if (!saved_fp_safe) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
176 return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
177 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
178
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
179 // construct the potential sender
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
180
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
181 frame sender(sender_sp, saved_fp, sender_pc);
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
182
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
183 // Validate the JavaCallWrapper an entry frame must have
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
184 address jcw = (address)sender.entry_frame_call_wrapper();
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
185
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
186 bool jcw_safe = (jcw <= thread->stack_base()) && ( jcw > (address)sender.fp());
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
187
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
188 return jcw_safe;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
189 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
190
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
191 // If the frame size is 0 something is bad because every nmethod has a non-zero frame size
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
192 // because the return address counts against the callee's frame.
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
193
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
194 if (sender_blob->frame_size() == 0) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
195 assert(!sender_blob->is_nmethod(), "should count return address at least");
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
196 return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
197 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
198
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
199 // We should never be able to see anything here except an nmethod. If something in the
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
200 // code cache (current frame) is called by an entity within the code cache that entity
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
201 // should not be anything but the call stub (already covered), the interpreter (already covered)
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
202 // or an nmethod.
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
203
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
204 assert(sender_blob->is_nmethod(), "Impossible call chain");
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
205
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
206 // Could put some more validation for the potential non-interpreted sender
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
207 // frame we'd create by calling sender if I could think of any. Wait for next crash in forte...
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
208
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
209 // One idea is seeing if the sender_pc we have is one that we'd expect to call to current cb
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
210
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
211 // We've validated the potential sender that would be created
0
a61af66fc99e Initial load
duke
parents:
diff changeset
212 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
213 }
107
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
214
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
215 // Must be native-compiled frame. Since sender will try and use fp to find
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
216 // linkages it must be safe
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
217
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
218 if (!fp_safe) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
219 return false;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
220 }
107
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
221
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
222 // Will the pc we fetch be non-zero (which we'll find at the oldest frame)
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
223
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
224 if ( (address) this->fp()[return_addr_offset] == NULL) return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
225
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
226
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
227 // could try and do some more potential verification of native frame if we could think of some...
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
228
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
229 return true;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
230
0
a61af66fc99e Initial load
duke
parents:
diff changeset
231 }
a61af66fc99e Initial load
duke
parents:
diff changeset
232
a61af66fc99e Initial load
duke
parents:
diff changeset
233
a61af66fc99e Initial load
duke
parents:
diff changeset
234 void frame::patch_pc(Thread* thread, address pc) {
4000
0abefdb54d21 7081938: JSR292: assert(magic_number_2() == MAGIC_NUMBER_2) failed
twisti
parents: 3931
diff changeset
235 address* pc_addr = &(((address*) sp())[-1]);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
236 if (TracePcPatching) {
4056
448691f285a5 7106944: assert(_pc == *pc_addr) failed may be too strong
twisti
parents: 4000
diff changeset
237 tty->print_cr("patch_pc at address " INTPTR_FORMAT " [" INTPTR_FORMAT " -> " INTPTR_FORMAT "]",
4000
0abefdb54d21 7081938: JSR292: assert(magic_number_2() == MAGIC_NUMBER_2) failed
twisti
parents: 3931
diff changeset
238 pc_addr, *pc_addr, pc);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
239 }
4056
448691f285a5 7106944: assert(_pc == *pc_addr) failed may be too strong
twisti
parents: 4000
diff changeset
240 // Either the return address is the original one or we are going to
448691f285a5 7106944: assert(_pc == *pc_addr) failed may be too strong
twisti
parents: 4000
diff changeset
241 // patch in the same address that's already there.
448691f285a5 7106944: assert(_pc == *pc_addr) failed may be too strong
twisti
parents: 4000
diff changeset
242 assert(_pc == *pc_addr || pc == *pc_addr, "must be");
4000
0abefdb54d21 7081938: JSR292: assert(magic_number_2() == MAGIC_NUMBER_2) failed
twisti
parents: 3931
diff changeset
243 *pc_addr = pc;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
244 _cb = CodeCache::find_blob(pc);
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
245 address original_pc = nmethod::get_deopt_original_pc(this);
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
246 if (original_pc != NULL) {
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
247 assert(original_pc == _pc, "expected original PC to be stored before patching");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
248 _deopt_state = is_deoptimized;
a61af66fc99e Initial load
duke
parents:
diff changeset
249 // leave _pc as is
a61af66fc99e Initial load
duke
parents:
diff changeset
250 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
251 _deopt_state = not_deoptimized;
a61af66fc99e Initial load
duke
parents:
diff changeset
252 _pc = pc;
a61af66fc99e Initial load
duke
parents:
diff changeset
253 }
a61af66fc99e Initial load
duke
parents:
diff changeset
254 }
a61af66fc99e Initial load
duke
parents:
diff changeset
255
a61af66fc99e Initial load
duke
parents:
diff changeset
256 bool frame::is_interpreted_frame() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
257 return Interpreter::contains(pc());
a61af66fc99e Initial load
duke
parents:
diff changeset
258 }
a61af66fc99e Initial load
duke
parents:
diff changeset
259
793
eacd97c88873 6848466: frame::frame_size() assertion failure with -XX:+DebugDeoptimization
cfang
parents: 304
diff changeset
260 int frame::frame_size(RegisterMap* map) const {
eacd97c88873 6848466: frame::frame_size() assertion failure with -XX:+DebugDeoptimization
cfang
parents: 304
diff changeset
261 frame sender = this->sender(map);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
262 return sender.sp() - sp();
a61af66fc99e Initial load
duke
parents:
diff changeset
263 }
a61af66fc99e Initial load
duke
parents:
diff changeset
264
a61af66fc99e Initial load
duke
parents:
diff changeset
265 intptr_t* frame::entry_frame_argument_at(int offset) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
266 // convert offset to index to deal with tsi
a61af66fc99e Initial load
duke
parents:
diff changeset
267 int index = (Interpreter::expr_offset_in_bytes(offset)/wordSize);
a61af66fc99e Initial load
duke
parents:
diff changeset
268 // Entry frame's arguments are always in relation to unextended_sp()
a61af66fc99e Initial load
duke
parents:
diff changeset
269 return &unextended_sp()[index];
a61af66fc99e Initial load
duke
parents:
diff changeset
270 }
a61af66fc99e Initial load
duke
parents:
diff changeset
271
a61af66fc99e Initial load
duke
parents:
diff changeset
272 // sender_sp
a61af66fc99e Initial load
duke
parents:
diff changeset
273 #ifdef CC_INTERP
a61af66fc99e Initial load
duke
parents:
diff changeset
274 intptr_t* frame::interpreter_frame_sender_sp() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
275 assert(is_interpreted_frame(), "interpreted frame expected");
a61af66fc99e Initial load
duke
parents:
diff changeset
276 // QQQ why does this specialize method exist if frame::sender_sp() does same thing?
a61af66fc99e Initial load
duke
parents:
diff changeset
277 // seems odd and if we always know interpreted vs. non then sender_sp() is really
a61af66fc99e Initial load
duke
parents:
diff changeset
278 // doing too much work.
a61af66fc99e Initial load
duke
parents:
diff changeset
279 return get_interpreterState()->sender_sp();
a61af66fc99e Initial load
duke
parents:
diff changeset
280 }
a61af66fc99e Initial load
duke
parents:
diff changeset
281
a61af66fc99e Initial load
duke
parents:
diff changeset
282 // monitor elements
a61af66fc99e Initial load
duke
parents:
diff changeset
283
a61af66fc99e Initial load
duke
parents:
diff changeset
284 BasicObjectLock* frame::interpreter_frame_monitor_begin() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
285 return get_interpreterState()->monitor_base();
a61af66fc99e Initial load
duke
parents:
diff changeset
286 }
a61af66fc99e Initial load
duke
parents:
diff changeset
287
a61af66fc99e Initial load
duke
parents:
diff changeset
288 BasicObjectLock* frame::interpreter_frame_monitor_end() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
289 return (BasicObjectLock*) get_interpreterState()->stack_base();
a61af66fc99e Initial load
duke
parents:
diff changeset
290 }
a61af66fc99e Initial load
duke
parents:
diff changeset
291
a61af66fc99e Initial load
duke
parents:
diff changeset
292 #else // CC_INTERP
a61af66fc99e Initial load
duke
parents:
diff changeset
293
a61af66fc99e Initial load
duke
parents:
diff changeset
294 intptr_t* frame::interpreter_frame_sender_sp() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
295 assert(is_interpreted_frame(), "interpreted frame expected");
a61af66fc99e Initial load
duke
parents:
diff changeset
296 return (intptr_t*) at(interpreter_frame_sender_sp_offset);
a61af66fc99e Initial load
duke
parents:
diff changeset
297 }
a61af66fc99e Initial load
duke
parents:
diff changeset
298
a61af66fc99e Initial load
duke
parents:
diff changeset
299 void frame::set_interpreter_frame_sender_sp(intptr_t* sender_sp) {
a61af66fc99e Initial load
duke
parents:
diff changeset
300 assert(is_interpreted_frame(), "interpreted frame expected");
a61af66fc99e Initial load
duke
parents:
diff changeset
301 ptr_at_put(interpreter_frame_sender_sp_offset, (intptr_t) sender_sp);
a61af66fc99e Initial load
duke
parents:
diff changeset
302 }
a61af66fc99e Initial load
duke
parents:
diff changeset
303
4668
3dbcd1013cc8 added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents: 4137
diff changeset
304 intptr_t** frame::interpreter_frame_sender_sp_addr() const {
3dbcd1013cc8 added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents: 4137
diff changeset
305 assert(is_interpreted_frame(), "interpreted frame expected");
3dbcd1013cc8 added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents: 4137
diff changeset
306 return (intptr_t**) addr_at(interpreter_frame_sender_sp_offset);
3dbcd1013cc8 added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents: 4137
diff changeset
307 }
3dbcd1013cc8 added flag PrintDeoptimizationDetails
Christian Haeubl <christian.haeubl@oracle.com>
parents: 4137
diff changeset
308
0
a61af66fc99e Initial load
duke
parents:
diff changeset
309
a61af66fc99e Initial load
duke
parents:
diff changeset
310 // monitor elements
a61af66fc99e Initial load
duke
parents:
diff changeset
311
a61af66fc99e Initial load
duke
parents:
diff changeset
312 BasicObjectLock* frame::interpreter_frame_monitor_begin() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
313 return (BasicObjectLock*) addr_at(interpreter_frame_monitor_block_bottom_offset);
a61af66fc99e Initial load
duke
parents:
diff changeset
314 }
a61af66fc99e Initial load
duke
parents:
diff changeset
315
a61af66fc99e Initial load
duke
parents:
diff changeset
316 BasicObjectLock* frame::interpreter_frame_monitor_end() const {
a61af66fc99e Initial load
duke
parents:
diff changeset
317 BasicObjectLock* result = (BasicObjectLock*) *addr_at(interpreter_frame_monitor_block_top_offset);
a61af66fc99e Initial load
duke
parents:
diff changeset
318 // make sure the pointer points inside the frame
1488
615a9d95d265 6946056: assert((intptr_t) sp()<=(intptr_t) result,"result must>=than stack pointer"), frame_x86.cpp:295
johnc
parents: 1204
diff changeset
319 assert(sp() <= (intptr_t*) result, "monitor end should be above the stack pointer");
615a9d95d265 6946056: assert((intptr_t) sp()<=(intptr_t) result,"result must>=than stack pointer"), frame_x86.cpp:295
johnc
parents: 1204
diff changeset
320 assert((intptr_t*) result < fp(), "monitor end should be strictly below the frame pointer");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
321 return result;
a61af66fc99e Initial load
duke
parents:
diff changeset
322 }
a61af66fc99e Initial load
duke
parents:
diff changeset
323
a61af66fc99e Initial load
duke
parents:
diff changeset
324 void frame::interpreter_frame_set_monitor_end(BasicObjectLock* value) {
a61af66fc99e Initial load
duke
parents:
diff changeset
325 *((BasicObjectLock**)addr_at(interpreter_frame_monitor_block_top_offset)) = value;
a61af66fc99e Initial load
duke
parents:
diff changeset
326 }
a61af66fc99e Initial load
duke
parents:
diff changeset
327
a61af66fc99e Initial load
duke
parents:
diff changeset
328 // Used by template based interpreter deoptimization
a61af66fc99e Initial load
duke
parents:
diff changeset
329 void frame::interpreter_frame_set_last_sp(intptr_t* sp) {
a61af66fc99e Initial load
duke
parents:
diff changeset
330 *((intptr_t**)addr_at(interpreter_frame_last_sp_offset)) = sp;
a61af66fc99e Initial load
duke
parents:
diff changeset
331 }
a61af66fc99e Initial load
duke
parents:
diff changeset
332 #endif // CC_INTERP
a61af66fc99e Initial load
duke
parents:
diff changeset
333
a61af66fc99e Initial load
duke
parents:
diff changeset
334 frame frame::sender_for_entry_frame(RegisterMap* map) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
335 assert(map != NULL, "map must be set");
a61af66fc99e Initial load
duke
parents:
diff changeset
336 // Java frame called from C; skip all C frames and return top C
a61af66fc99e Initial load
duke
parents:
diff changeset
337 // frame of that chunk as the sender
a61af66fc99e Initial load
duke
parents:
diff changeset
338 JavaFrameAnchor* jfa = entry_frame_call_wrapper()->anchor();
a61af66fc99e Initial load
duke
parents:
diff changeset
339 assert(!entry_frame_is_first(), "next Java fp must be non zero");
a61af66fc99e Initial load
duke
parents:
diff changeset
340 assert(jfa->last_Java_sp() > sp(), "must be above this frame on stack");
a61af66fc99e Initial load
duke
parents:
diff changeset
341 map->clear();
a61af66fc99e Initial load
duke
parents:
diff changeset
342 assert(map->include_argument_oops(), "should be set by clear");
a61af66fc99e Initial load
duke
parents:
diff changeset
343 if (jfa->last_Java_pc() != NULL ) {
a61af66fc99e Initial load
duke
parents:
diff changeset
344 frame fr(jfa->last_Java_sp(), jfa->last_Java_fp(), jfa->last_Java_pc());
a61af66fc99e Initial load
duke
parents:
diff changeset
345 return fr;
a61af66fc99e Initial load
duke
parents:
diff changeset
346 }
a61af66fc99e Initial load
duke
parents:
diff changeset
347 frame fr(jfa->last_Java_sp(), jfa->last_Java_fp());
a61af66fc99e Initial load
duke
parents:
diff changeset
348 return fr;
a61af66fc99e Initial load
duke
parents:
diff changeset
349 }
a61af66fc99e Initial load
duke
parents:
diff changeset
350
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
351 //------------------------------------------------------------------------------
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
352 // frame::verify_deopt_original_pc
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
353 //
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
354 // Verifies the calculated original PC of a deoptimization PC for the
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
355 // given unextended SP. The unextended SP might also be the saved SP
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
356 // for MethodHandle call sites.
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
357 #if ASSERT
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
358 void frame::verify_deopt_original_pc(nmethod* nm, intptr_t* unextended_sp, bool is_method_handle_return) {
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
359 frame fr;
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
360
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
361 // This is ugly but it's better than to change {get,set}_original_pc
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
362 // to take an SP value as argument. And it's only a debugging
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
363 // method anyway.
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
364 fr._unextended_sp = unextended_sp;
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
365
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
366 address original_pc = nm->get_original_pc(&fr);
1748
3e8fbc61cee8 6978355: renaming for 6961697
twisti
parents: 1681
diff changeset
367 assert(nm->insts_contains(original_pc), "original PC must be in nmethod");
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
368 assert(nm->is_method_handle_return(original_pc) == is_method_handle_return, "must be");
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
369 }
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
370 #endif
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
371
3363
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
372 //------------------------------------------------------------------------------
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
373 // frame::adjust_unextended_sp
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
374 void frame::adjust_unextended_sp() {
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
375 // If we are returning to a compiled MethodHandle call site, the
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
376 // saved_fp will in fact be a saved value of the unextended SP. The
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
377 // simplest way to tell whether we are returning to such a call site
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
378 // is as follows:
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
379
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
380 nmethod* sender_nm = (_cb == NULL) ? NULL : _cb->as_nmethod_or_null();
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
381 if (sender_nm != NULL) {
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
382 // If the sender PC is a deoptimization point, get the original
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
383 // PC. For MethodHandle call site the unextended_sp is stored in
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
384 // saved_fp.
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
385 if (sender_nm->is_deopt_mh_entry(_pc)) {
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
386 DEBUG_ONLY(verify_deopt_mh_original_pc(sender_nm, _fp));
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
387 _unextended_sp = _fp;
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
388 }
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
389 else if (sender_nm->is_deopt_entry(_pc)) {
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
390 DEBUG_ONLY(verify_deopt_original_pc(sender_nm, _unextended_sp));
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
391 }
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
392 else if (sender_nm->is_method_handle_return(_pc)) {
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
393 _unextended_sp = _fp;
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
394 }
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
395 }
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
396 }
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
397
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
398 //------------------------------------------------------------------------------
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
399 // frame::update_map_with_saved_link
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
400 void frame::update_map_with_saved_link(RegisterMap* map, intptr_t** link_addr) {
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
401 // The interpreter and compiler(s) always save EBP/RBP in a known
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
402 // location on entry. We must record where that location is
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
403 // so this if EBP/RBP was live on callout from c2 we can find
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
404 // the saved copy no matter what it called.
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
405
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
406 // Since the interpreter always saves EBP/RBP if we record where it is then
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
407 // we don't have to always save EBP/RBP on entry and exit to c2 compiled
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
408 // code, on entry will be enough.
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
409 map->set_location(rbp->as_VMReg(), (address) link_addr);
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
410 #ifdef AMD64
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
411 // this is weird "H" ought to be at a higher address however the
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
412 // oopMaps seems to have the "H" regs at the same address and the
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
413 // vanilla register.
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
414 // XXXX make this go away
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
415 if (true) {
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
416 map->set_location(rbp->as_VMReg()->next(), (address) link_addr);
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
417 }
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
418 #endif // AMD64
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
419 }
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
420
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
421
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
422 //------------------------------------------------------------------------------
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
423 // frame::sender_for_interpreter_frame
0
a61af66fc99e Initial load
duke
parents:
diff changeset
424 frame frame::sender_for_interpreter_frame(RegisterMap* map) const {
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
425 // SP is the raw SP from the sender after adapter or interpreter
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
426 // extension.
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
427 intptr_t* sender_sp = this->sender_sp();
0
a61af66fc99e Initial load
duke
parents:
diff changeset
428
a61af66fc99e Initial load
duke
parents:
diff changeset
429 // This is the sp before any possible extension (adapter/locals).
a61af66fc99e Initial load
duke
parents:
diff changeset
430 intptr_t* unextended_sp = interpreter_frame_sender_sp();
a61af66fc99e Initial load
duke
parents:
diff changeset
431
a61af66fc99e Initial load
duke
parents:
diff changeset
432 #ifdef COMPILER2
a61af66fc99e Initial load
duke
parents:
diff changeset
433 if (map->update_map()) {
3363
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
434 update_map_with_saved_link(map, (intptr_t**) addr_at(link_offset));
0
a61af66fc99e Initial load
duke
parents:
diff changeset
435 }
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
436 #endif // COMPILER2
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
437
3363
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
438 return frame(sender_sp, unextended_sp, link(), sender_pc());
0
a61af66fc99e Initial load
duke
parents:
diff changeset
439 }
a61af66fc99e Initial load
duke
parents:
diff changeset
440
a61af66fc99e Initial load
duke
parents:
diff changeset
441
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
442 //------------------------------------------------------------------------------
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
443 // frame::sender_for_compiled_frame
0
a61af66fc99e Initial load
duke
parents:
diff changeset
444 frame frame::sender_for_compiled_frame(RegisterMap* map) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
445 assert(map != NULL, "map must be set");
3363
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
446 assert(!is_ricochet_frame(), "caller must handle this");
0
a61af66fc99e Initial load
duke
parents:
diff changeset
447
a61af66fc99e Initial load
duke
parents:
diff changeset
448 // frame owned by optimizing compiler
a61af66fc99e Initial load
duke
parents:
diff changeset
449 assert(_cb->frame_size() >= 0, "must have non-zero frame size");
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
450 intptr_t* sender_sp = unextended_sp() + _cb->frame_size();
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
451 intptr_t* unextended_sp = sender_sp;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
452
a61af66fc99e Initial load
duke
parents:
diff changeset
453 // On Intel the return_address is always the word on the stack
a61af66fc99e Initial load
duke
parents:
diff changeset
454 address sender_pc = (address) *(sender_sp-1);
a61af66fc99e Initial load
duke
parents:
diff changeset
455
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
456 // This is the saved value of EBP which may or may not really be an FP.
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
457 // It is only an FP if the sender is an interpreter frame (or C1?).
3363
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
458 intptr_t** saved_fp_addr = (intptr_t**) (sender_sp - frame::sender_sp_offset);
1135
e66fd840cb6b 6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents: 844
diff changeset
459
0
a61af66fc99e Initial load
duke
parents:
diff changeset
460 if (map->update_map()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
461 // Tell GC to use argument oopmaps for some runtime stubs that need it.
a61af66fc99e Initial load
duke
parents:
diff changeset
462 // For C1, the runtime stub might not have oop maps, so set this flag
a61af66fc99e Initial load
duke
parents:
diff changeset
463 // outside of update_register_map.
a61af66fc99e Initial load
duke
parents:
diff changeset
464 map->set_include_argument_oops(_cb->caller_must_gc_arguments(map->thread()));
a61af66fc99e Initial load
duke
parents:
diff changeset
465 if (_cb->oop_maps() != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
466 OopMapSet::update_register_map(this, map);
a61af66fc99e Initial load
duke
parents:
diff changeset
467 }
3363
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
468
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
469 // Since the prolog does the save and restore of EBP there is no oopmap
0
a61af66fc99e Initial load
duke
parents:
diff changeset
470 // for it so we must fill in its location as if there was an oopmap entry
a61af66fc99e Initial load
duke
parents:
diff changeset
471 // since if our caller was compiled code there could be live jvm state in it.
3363
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
472 update_map_with_saved_link(map, saved_fp_addr);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
473 }
a61af66fc99e Initial load
duke
parents:
diff changeset
474
a61af66fc99e Initial load
duke
parents:
diff changeset
475 assert(sender_sp != sp(), "must have changed");
3363
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
476 return frame(sender_sp, unextended_sp, *saved_fp_addr, sender_pc);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
477 }
a61af66fc99e Initial load
duke
parents:
diff changeset
478
1204
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
479
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
480 //------------------------------------------------------------------------------
18a389214829 6921352: JSR 292 needs its own deopt handler
twisti
parents: 1201
diff changeset
481 // frame::sender
0
a61af66fc99e Initial load
duke
parents:
diff changeset
482 frame frame::sender(RegisterMap* map) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
483 // Default is we done have to follow them. The sender_for_xxx will
a61af66fc99e Initial load
duke
parents:
diff changeset
484 // update it accordingly
a61af66fc99e Initial load
duke
parents:
diff changeset
485 map->set_include_argument_oops(false);
a61af66fc99e Initial load
duke
parents:
diff changeset
486
a61af66fc99e Initial load
duke
parents:
diff changeset
487 if (is_entry_frame()) return sender_for_entry_frame(map);
a61af66fc99e Initial load
duke
parents:
diff changeset
488 if (is_interpreted_frame()) return sender_for_interpreter_frame(map);
a61af66fc99e Initial load
duke
parents:
diff changeset
489 assert(_cb == CodeCache::find_blob(pc()),"Must be the same");
3363
167b70ff3abc 6939861: JVM should handle more conversion operations
never
parents: 3336
diff changeset
490 if (is_ricochet_frame()) return sender_for_ricochet_frame(map);
0
a61af66fc99e Initial load
duke
parents:
diff changeset
491
a61af66fc99e Initial load
duke
parents:
diff changeset
492 if (_cb != NULL) {
a61af66fc99e Initial load
duke
parents:
diff changeset
493 return sender_for_compiled_frame(map);
a61af66fc99e Initial load
duke
parents:
diff changeset
494 }
a61af66fc99e Initial load
duke
parents:
diff changeset
495 // Must be native-compiled frame, i.e. the marshaling code for native
a61af66fc99e Initial load
duke
parents:
diff changeset
496 // methods that exists in the core system.
a61af66fc99e Initial load
duke
parents:
diff changeset
497 return frame(sender_sp(), link(), sender_pc());
a61af66fc99e Initial load
duke
parents:
diff changeset
498 }
a61af66fc99e Initial load
duke
parents:
diff changeset
499
a61af66fc99e Initial load
duke
parents:
diff changeset
500
a61af66fc99e Initial load
duke
parents:
diff changeset
501 bool frame::interpreter_frame_equals_unpacked_fp(intptr_t* fp) {
a61af66fc99e Initial load
duke
parents:
diff changeset
502 assert(is_interpreted_frame(), "must be interpreter frame");
a61af66fc99e Initial load
duke
parents:
diff changeset
503 methodOop method = interpreter_frame_method();
a61af66fc99e Initial load
duke
parents:
diff changeset
504 // When unpacking an optimized frame the frame pointer is
a61af66fc99e Initial load
duke
parents:
diff changeset
505 // adjusted with:
a61af66fc99e Initial load
duke
parents:
diff changeset
506 int diff = (method->max_locals() - method->size_of_parameters()) *
1506
2338d41fbd81 6943304: remove tagged stack interpreter
twisti
parents: 1204
diff changeset
507 Interpreter::stackElementWords;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
508 return _fp == (fp - diff);
a61af66fc99e Initial load
duke
parents:
diff changeset
509 }
a61af66fc99e Initial load
duke
parents:
diff changeset
510
a61af66fc99e Initial load
duke
parents:
diff changeset
511 void frame::pd_gc_epilog() {
a61af66fc99e Initial load
duke
parents:
diff changeset
512 // nothing done here now
a61af66fc99e Initial load
duke
parents:
diff changeset
513 }
a61af66fc99e Initial load
duke
parents:
diff changeset
514
107
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
515 bool frame::is_interpreted_frame_valid(JavaThread* thread) const {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
516 // QQQ
a61af66fc99e Initial load
duke
parents:
diff changeset
517 #ifdef CC_INTERP
a61af66fc99e Initial load
duke
parents:
diff changeset
518 #else
a61af66fc99e Initial load
duke
parents:
diff changeset
519 assert(is_interpreted_frame(), "Not an interpreted frame");
a61af66fc99e Initial load
duke
parents:
diff changeset
520 // These are reasonable sanity checks
a61af66fc99e Initial load
duke
parents:
diff changeset
521 if (fp() == 0 || (intptr_t(fp()) & (wordSize-1)) != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
522 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
523 }
a61af66fc99e Initial load
duke
parents:
diff changeset
524 if (sp() == 0 || (intptr_t(sp()) & (wordSize-1)) != 0) {
a61af66fc99e Initial load
duke
parents:
diff changeset
525 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
526 }
a61af66fc99e Initial load
duke
parents:
diff changeset
527 if (fp() + interpreter_frame_initial_sp_offset < sp()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
528 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
529 }
a61af66fc99e Initial load
duke
parents:
diff changeset
530 // These are hacks to keep us out of trouble.
a61af66fc99e Initial load
duke
parents:
diff changeset
531 // The problem with these is that they mask other problems
a61af66fc99e Initial load
duke
parents:
diff changeset
532 if (fp() <= sp()) { // this attempts to deal with unsigned comparison above
a61af66fc99e Initial load
duke
parents:
diff changeset
533 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
534 }
107
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
535
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
536 // do some validation of frame elements
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
537
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
538 // first the method
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
539
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
540 methodOop m = *interpreter_frame_method_addr();
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
541
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
542 // validate the method we'd find in this potential sender
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
543 if (!Universe::heap()->is_valid_method(m)) return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
544
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
545 // stack frames shouldn't be much larger than max_stack elements
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
546
1506
2338d41fbd81 6943304: remove tagged stack interpreter
twisti
parents: 1204
diff changeset
547 if (fp() - sp() > 1024 + m->max_stack()*Interpreter::stackElementSize) {
0
a61af66fc99e Initial load
duke
parents:
diff changeset
548 return false;
a61af66fc99e Initial load
duke
parents:
diff changeset
549 }
107
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
550
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
551 // validate bci/bcx
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
552
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
553 intptr_t bcx = interpreter_frame_bcx();
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
554 if (m->validate_bci_from_bcx(bcx) < 0) {
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
555 return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
556 }
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
557
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
558 // validate constantPoolCacheOop
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
559
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
560 constantPoolCacheOop cp = *interpreter_frame_cache_addr();
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
561
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
562 if (cp == NULL ||
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
563 !Space::is_aligned(cp) ||
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
564 !Universe::heap()->is_permanent((void*)cp)) return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
565
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
566 // validate locals
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
567
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
568 address locals = (address) *interpreter_frame_locals_addr();
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
569
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
570 if (locals > thread->stack_base() || locals < (address) fp()) return false;
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
571
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
572 // We'd have to be pretty unlucky to be mislead at this point
93b6525e3b82 6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents: 0
diff changeset
573
0
a61af66fc99e Initial load
duke
parents:
diff changeset
574 #endif // CC_INTERP
a61af66fc99e Initial load
duke
parents:
diff changeset
575 return true;
a61af66fc99e Initial load
duke
parents:
diff changeset
576 }
a61af66fc99e Initial load
duke
parents:
diff changeset
577
a61af66fc99e Initial load
duke
parents:
diff changeset
578 BasicType frame::interpreter_frame_result(oop* oop_result, jvalue* value_result) {
a61af66fc99e Initial load
duke
parents:
diff changeset
579 #ifdef CC_INTERP
1681
126ea7725993 6953477: Increase portability and flexibility of building Hotspot
bobv
parents: 1552
diff changeset
580 // Needed for JVMTI. The result should always be in the
126ea7725993 6953477: Increase portability and flexibility of building Hotspot
bobv
parents: 1552
diff changeset
581 // interpreterState object
0
a61af66fc99e Initial load
duke
parents:
diff changeset
582 interpreterState istate = get_interpreterState();
a61af66fc99e Initial load
duke
parents:
diff changeset
583 #endif // CC_INTERP
a61af66fc99e Initial load
duke
parents:
diff changeset
584 assert(is_interpreted_frame(), "interpreted frame expected");
a61af66fc99e Initial load
duke
parents:
diff changeset
585 methodOop method = interpreter_frame_method();
a61af66fc99e Initial load
duke
parents:
diff changeset
586 BasicType type = method->result_type();
a61af66fc99e Initial load
duke
parents:
diff changeset
587
a61af66fc99e Initial load
duke
parents:
diff changeset
588 intptr_t* tos_addr;
a61af66fc99e Initial load
duke
parents:
diff changeset
589 if (method->is_native()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
590 // Prior to calling into the runtime to report the method_exit the possible
a61af66fc99e Initial load
duke
parents:
diff changeset
591 // return value is pushed to the native stack. If the result is a jfloat/jdouble
a61af66fc99e Initial load
duke
parents:
diff changeset
592 // then ST0 is saved before EAX/EDX. See the note in generate_native_result
a61af66fc99e Initial load
duke
parents:
diff changeset
593 tos_addr = (intptr_t*)sp();
a61af66fc99e Initial load
duke
parents:
diff changeset
594 if (type == T_FLOAT || type == T_DOUBLE) {
a61af66fc99e Initial load
duke
parents:
diff changeset
595 // QQQ seems like this code is equivalent on the two platforms
a61af66fc99e Initial load
duke
parents:
diff changeset
596 #ifdef AMD64
a61af66fc99e Initial load
duke
parents:
diff changeset
597 // This is times two because we do a push(ltos) after pushing XMM0
a61af66fc99e Initial load
duke
parents:
diff changeset
598 // and that takes two interpreter stack slots.
1506
2338d41fbd81 6943304: remove tagged stack interpreter
twisti
parents: 1204
diff changeset
599 tos_addr += 2 * Interpreter::stackElementWords;
0
a61af66fc99e Initial load
duke
parents:
diff changeset
600 #else
a61af66fc99e Initial load
duke
parents:
diff changeset
601 tos_addr += 2;
a61af66fc99e Initial load
duke
parents:
diff changeset
602 #endif // AMD64
a61af66fc99e Initial load
duke
parents:
diff changeset
603 }
a61af66fc99e Initial load
duke
parents:
diff changeset
604 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
605 tos_addr = (intptr_t*)interpreter_frame_tos_address();
a61af66fc99e Initial load
duke
parents:
diff changeset
606 }
a61af66fc99e Initial load
duke
parents:
diff changeset
607
a61af66fc99e Initial load
duke
parents:
diff changeset
608 switch (type) {
a61af66fc99e Initial load
duke
parents:
diff changeset
609 case T_OBJECT :
a61af66fc99e Initial load
duke
parents:
diff changeset
610 case T_ARRAY : {
a61af66fc99e Initial load
duke
parents:
diff changeset
611 oop obj;
a61af66fc99e Initial load
duke
parents:
diff changeset
612 if (method->is_native()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
613 #ifdef CC_INTERP
a61af66fc99e Initial load
duke
parents:
diff changeset
614 obj = istate->_oop_temp;
a61af66fc99e Initial load
duke
parents:
diff changeset
615 #else
a61af66fc99e Initial load
duke
parents:
diff changeset
616 obj = (oop) at(interpreter_frame_oop_temp_offset);
a61af66fc99e Initial load
duke
parents:
diff changeset
617 #endif // CC_INTERP
a61af66fc99e Initial load
duke
parents:
diff changeset
618 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
619 oop* obj_p = (oop*)tos_addr;
a61af66fc99e Initial load
duke
parents:
diff changeset
620 obj = (obj_p == NULL) ? (oop)NULL : *obj_p;
a61af66fc99e Initial load
duke
parents:
diff changeset
621 }
a61af66fc99e Initial load
duke
parents:
diff changeset
622 assert(obj == NULL || Universe::heap()->is_in(obj), "sanity check");
a61af66fc99e Initial load
duke
parents:
diff changeset
623 *oop_result = obj;
a61af66fc99e Initial load
duke
parents:
diff changeset
624 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
625 }
a61af66fc99e Initial load
duke
parents:
diff changeset
626 case T_BOOLEAN : value_result->z = *(jboolean*)tos_addr; break;
a61af66fc99e Initial load
duke
parents:
diff changeset
627 case T_BYTE : value_result->b = *(jbyte*)tos_addr; break;
a61af66fc99e Initial load
duke
parents:
diff changeset
628 case T_CHAR : value_result->c = *(jchar*)tos_addr; break;
a61af66fc99e Initial load
duke
parents:
diff changeset
629 case T_SHORT : value_result->s = *(jshort*)tos_addr; break;
a61af66fc99e Initial load
duke
parents:
diff changeset
630 case T_INT : value_result->i = *(jint*)tos_addr; break;
a61af66fc99e Initial load
duke
parents:
diff changeset
631 case T_LONG : value_result->j = *(jlong*)tos_addr; break;
a61af66fc99e Initial load
duke
parents:
diff changeset
632 case T_FLOAT : {
a61af66fc99e Initial load
duke
parents:
diff changeset
633 #ifdef AMD64
a61af66fc99e Initial load
duke
parents:
diff changeset
634 value_result->f = *(jfloat*)tos_addr;
a61af66fc99e Initial load
duke
parents:
diff changeset
635 #else
a61af66fc99e Initial load
duke
parents:
diff changeset
636 if (method->is_native()) {
a61af66fc99e Initial load
duke
parents:
diff changeset
637 jdouble d = *(jdouble*)tos_addr; // Result was in ST0 so need to convert to jfloat
a61af66fc99e Initial load
duke
parents:
diff changeset
638 value_result->f = (jfloat)d;
a61af66fc99e Initial load
duke
parents:
diff changeset
639 } else {
a61af66fc99e Initial load
duke
parents:
diff changeset
640 value_result->f = *(jfloat*)tos_addr;
a61af66fc99e Initial load
duke
parents:
diff changeset
641 }
a61af66fc99e Initial load
duke
parents:
diff changeset
642 #endif // AMD64
a61af66fc99e Initial load
duke
parents:
diff changeset
643 break;
a61af66fc99e Initial load
duke
parents:
diff changeset
644 }
a61af66fc99e Initial load
duke
parents:
diff changeset
645 case T_DOUBLE : value_result->d = *(jdouble*)tos_addr; break;
a61af66fc99e Initial load
duke
parents:
diff changeset
646 case T_VOID : /* Nothing to do */ break;
a61af66fc99e Initial load
duke
parents:
diff changeset
647 default : ShouldNotReachHere();
a61af66fc99e Initial load
duke
parents:
diff changeset
648 }
a61af66fc99e Initial load
duke
parents:
diff changeset
649
a61af66fc99e Initial load
duke
parents:
diff changeset
650 return type;
a61af66fc99e Initial load
duke
parents:
diff changeset
651 }
a61af66fc99e Initial load
duke
parents:
diff changeset
652
a61af66fc99e Initial load
duke
parents:
diff changeset
653
a61af66fc99e Initial load
duke
parents:
diff changeset
654 intptr_t* frame::interpreter_frame_tos_at(jint offset) const {
a61af66fc99e Initial load
duke
parents:
diff changeset
655 int index = (Interpreter::expr_offset_in_bytes(offset)/wordSize);
a61af66fc99e Initial load
duke
parents:
diff changeset
656 return &interpreter_frame_tos_address()[index];
a61af66fc99e Initial load
duke
parents:
diff changeset
657 }
3336
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
658
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
659 #ifdef ASSERT
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
660
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
661 #define DESCRIBE_FP_OFFSET(name) \
3365
3cfb240033d1 7043301: assert(locals < caller->fp() || locals > (caller->fp() + 16)) failed: locals in save area
never
parents: 3363
diff changeset
662 values.describe(frame_no, fp() + frame::name##_offset, #name)
3336
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
663
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
664 void frame::describe_pd(FrameValues& values, int frame_no) {
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
665 if (is_interpreted_frame()) {
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
666 DESCRIBE_FP_OFFSET(interpreter_frame_sender_sp);
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
667 DESCRIBE_FP_OFFSET(interpreter_frame_last_sp);
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
668 DESCRIBE_FP_OFFSET(interpreter_frame_method);
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
669 DESCRIBE_FP_OFFSET(interpreter_frame_mdx);
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
670 DESCRIBE_FP_OFFSET(interpreter_frame_cache);
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
671 DESCRIBE_FP_OFFSET(interpreter_frame_locals);
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
672 DESCRIBE_FP_OFFSET(interpreter_frame_bcx);
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
673 DESCRIBE_FP_OFFSET(interpreter_frame_initial_sp);
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
674 }
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
675
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
676 }
2e038ad0c1d0 7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents: 1972
diff changeset
677 #endif
3931
5432047c7db7 7087445: Improve platform independence of JSR292 shared code
bdelsart
parents: 3365
diff changeset
678
5432047c7db7 7087445: Improve platform independence of JSR292 shared code
bdelsart
parents: 3365
diff changeset
679 intptr_t *frame::initial_deoptimization_info() {
5432047c7db7 7087445: Improve platform independence of JSR292 shared code
bdelsart
parents: 3365
diff changeset
680 // used to reset the saved FP
5432047c7db7 7087445: Improve platform independence of JSR292 shared code
bdelsart
parents: 3365
diff changeset
681 return fp();
5432047c7db7 7087445: Improve platform independence of JSR292 shared code
bdelsart
parents: 3365
diff changeset
682 }