author | coleenp |
Fri, 16 Mar 2018 09:12:13 -0400 | |
changeset 49449 | ef5d5d343e2a |
parent 47799 | 1772ebf07d1f |
child 51263 | b5aac518b097 |
permissions | -rw-r--r-- |
1 | 1 |
/* |
46589 | 2 |
* Copyright (c) 1997, 2017, Oracle and/or its affiliates. All rights reserved. |
1 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. |
|
8 |
* |
|
9 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
10 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
11 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
12 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
13 |
* accompanied this code). |
|
14 |
* |
|
15 |
* You should have received a copy of the GNU General Public License version |
|
16 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
17 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
18 |
* |
|
5547
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5426
diff
changeset
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5426
diff
changeset
|
20 |
* or visit www.oracle.com if you need additional information or have any |
f4b087cbb361
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
5426
diff
changeset
|
21 |
* questions. |
1 | 22 |
* |
23 |
*/ |
|
24 |
||
7397 | 25 |
#include "precompiled.hpp" |
26 |
#include "interpreter/interpreter.hpp" |
|
27 |
#include "memory/resourceArea.hpp" |
|
28 |
#include "oops/markOop.hpp" |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
29 |
#include "oops/method.hpp" |
7397 | 30 |
#include "oops/oop.inline.hpp" |
11632 | 31 |
#include "prims/methodHandles.hpp" |
7397 | 32 |
#include "runtime/frame.inline.hpp" |
33 |
#include "runtime/handles.inline.hpp" |
|
34 |
#include "runtime/javaCalls.hpp" |
|
35 |
#include "runtime/monitorChunk.hpp" |
|
25468
5331df506290
8048241: Introduce umbrella header os.inline.hpp and clean up includes
goetz
parents:
24424
diff
changeset
|
36 |
#include "runtime/os.inline.hpp" |
7397 | 37 |
#include "runtime/signature.hpp" |
38 |
#include "runtime/stubCodeGenerator.hpp" |
|
39 |
#include "runtime/stubRoutines.hpp" |
|
40 |
#include "vmreg_x86.inline.hpp" |
|
41 |
#ifdef COMPILER1 |
|
42 |
#include "c1/c1_Runtime1.hpp" |
|
43 |
#include "runtime/vframeArray.hpp" |
|
44 |
#endif |
|
1 | 45 |
|
46 |
#ifdef ASSERT |
|
47 |
void RegisterMap::check_location_valid() { |
|
48 |
} |
|
49 |
#endif |
|
50 |
||
51 |
// Profiling/safepoint support |
|
52 |
||
53 |
bool frame::safe_for_sender(JavaThread *thread) { |
|
54 |
address sp = (address)_sp; |
|
55 |
address fp = (address)_fp; |
|
56 |
address unextended_sp = (address)_unextended_sp; |
|
18025 | 57 |
|
58 |
// consider stack guards when trying to determine "safe" stack pointers |
|
35201
996db89f378e
8139864: Improve handling of stack protection zones.
goetz
parents:
33198
diff
changeset
|
59 |
static size_t stack_guard_size = os::uses_stack_guard_pages() ? |
996db89f378e
8139864: Improve handling of stack protection zones.
goetz
parents:
33198
diff
changeset
|
60 |
JavaThread::stack_red_zone_size() + JavaThread::stack_yellow_zone_size() : 0; |
18025 | 61 |
size_t usable_stack_size = thread->stack_size() - stack_guard_size; |
62 |
||
63 |
// sp must be within the usable part of the stack (not in guards) |
|
64 |
bool sp_safe = (sp < thread->stack_base()) && |
|
65 |
(sp >= thread->stack_base() - usable_stack_size); |
|
66 |
||
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
67 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
68 |
if (!sp_safe) { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
69 |
return false; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
70 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
71 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
72 |
// unextended sp must be within the stack and above or equal sp |
18025 | 73 |
bool unextended_sp_safe = (unextended_sp < thread->stack_base()) && |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
74 |
(unextended_sp >= sp); |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
75 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
76 |
if (!unextended_sp_safe) { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
77 |
return false; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
78 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
79 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
80 |
// an fp must be within the stack and above (but not equal) sp |
18025 | 81 |
// second evaluation on fp+ is added to handle situation where fp is -1 |
82 |
bool fp_safe = (fp < thread->stack_base() && (fp > sp) && (((fp + (return_addr_offset * sizeof(void*))) < thread->stack_base()))); |
|
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
83 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
84 |
// We know sp/unextended_sp are safe only fp is questionable here |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
85 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
86 |
// If the current frame is known to the code cache then we can attempt to |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
87 |
// to construct the sender and do some validation of it. This goes a long way |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
88 |
// toward eliminating issues when we get in frame construction code |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
89 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
90 |
if (_cb != NULL ) { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
91 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
92 |
// First check if frame is complete and tester is reliable |
1 | 93 |
// Unfortunately we can only check frame complete for runtime stubs and nmethod |
94 |
// other generic buffer blobs are more problematic so we just assume they are |
|
95 |
// ok. adapter blobs never have a frame complete and are never ok. |
|
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
96 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
97 |
if (!_cb->is_frame_complete_at(_pc)) { |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
98 |
if (_cb->is_compiled() || _cb->is_adapter_blob() || _cb->is_runtime_stub()) { |
1 | 99 |
return false; |
100 |
} |
|
101 |
} |
|
15801
fefae82e1067
8008340: [sampling] assert(upper->pc_offset() >= pc_offset) failed: sanity
rbackman
parents:
14579
diff
changeset
|
102 |
|
fefae82e1067
8008340: [sampling] assert(upper->pc_offset() >= pc_offset) failed: sanity
rbackman
parents:
14579
diff
changeset
|
103 |
// Could just be some random pointer within the codeBlob |
fefae82e1067
8008340: [sampling] assert(upper->pc_offset() >= pc_offset) failed: sanity
rbackman
parents:
14579
diff
changeset
|
104 |
if (!_cb->code_contains(_pc)) { |
fefae82e1067
8008340: [sampling] assert(upper->pc_offset() >= pc_offset) failed: sanity
rbackman
parents:
14579
diff
changeset
|
105 |
return false; |
fefae82e1067
8008340: [sampling] assert(upper->pc_offset() >= pc_offset) failed: sanity
rbackman
parents:
14579
diff
changeset
|
106 |
} |
fefae82e1067
8008340: [sampling] assert(upper->pc_offset() >= pc_offset) failed: sanity
rbackman
parents:
14579
diff
changeset
|
107 |
|
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
108 |
// Entry frame checks |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
109 |
if (is_entry_frame()) { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
110 |
// an entry frame must have a valid fp. |
40332
a52d1e719c4d
8159284: bigapps/Jetty - assert(jfa->last_Java_sp() > sp()) failed with JFR in use
coleenp
parents:
38144
diff
changeset
|
111 |
return fp_safe && is_entry_frame_valid(thread); |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
112 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
113 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
114 |
intptr_t* sender_sp = NULL; |
28824
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
115 |
intptr_t* sender_unextended_sp = NULL; |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
116 |
address sender_pc = NULL; |
28824
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
117 |
intptr_t* saved_fp = NULL; |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
118 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
119 |
if (is_interpreted_frame()) { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
120 |
// fp must be safe |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
121 |
if (!fp_safe) { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
122 |
return false; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
123 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
124 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
125 |
sender_pc = (address) this->fp()[return_addr_offset]; |
28824
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
126 |
// for interpreted frames, the value below is the sender "raw" sp, |
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
127 |
// which can be different from the sender unextended sp (the sp seen |
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
128 |
// by the sender) because of current frame local variables |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
129 |
sender_sp = (intptr_t*) addr_at(sender_sp_offset); |
28824
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
130 |
sender_unextended_sp = (intptr_t*) this->fp()[interpreter_frame_sender_sp_offset]; |
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
131 |
saved_fp = (intptr_t*) this->fp()[link_offset]; |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
132 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
133 |
} else { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
134 |
// must be some sort of compiled/runtime frame |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
135 |
// fp does not have to be safe (although it could be check for c1?) |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
136 |
|
22217
163b4d58a6f2
8028412: AsyncGetCallTrace() is broken on x86 in JDK 7u40
mgronlun
parents:
20282
diff
changeset
|
137 |
// check for a valid frame_size, otherwise we are unlikely to get a valid sender_pc |
163b4d58a6f2
8028412: AsyncGetCallTrace() is broken on x86 in JDK 7u40
mgronlun
parents:
20282
diff
changeset
|
138 |
if (_cb->frame_size() <= 0) { |
163b4d58a6f2
8028412: AsyncGetCallTrace() is broken on x86 in JDK 7u40
mgronlun
parents:
20282
diff
changeset
|
139 |
return false; |
163b4d58a6f2
8028412: AsyncGetCallTrace() is broken on x86 in JDK 7u40
mgronlun
parents:
20282
diff
changeset
|
140 |
} |
163b4d58a6f2
8028412: AsyncGetCallTrace() is broken on x86 in JDK 7u40
mgronlun
parents:
20282
diff
changeset
|
141 |
|
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
142 |
sender_sp = _unextended_sp + _cb->frame_size(); |
42042
681bd315c66e
8160411: SIGSEGV in frame::safe_for_sender on incomplete DeoptimizationBlob frame
dlong
parents:
40644
diff
changeset
|
143 |
// Is sender_sp safe? |
681bd315c66e
8160411: SIGSEGV in frame::safe_for_sender on incomplete DeoptimizationBlob frame
dlong
parents:
40644
diff
changeset
|
144 |
if ((address)sender_sp >= thread->stack_base()) { |
681bd315c66e
8160411: SIGSEGV in frame::safe_for_sender on incomplete DeoptimizationBlob frame
dlong
parents:
40644
diff
changeset
|
145 |
return false; |
681bd315c66e
8160411: SIGSEGV in frame::safe_for_sender on incomplete DeoptimizationBlob frame
dlong
parents:
40644
diff
changeset
|
146 |
} |
28824
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
147 |
sender_unextended_sp = sender_sp; |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
148 |
// On Intel the return_address is always the word on the stack |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
149 |
sender_pc = (address) *(sender_sp-1); |
28824
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
150 |
// Note: frame::sender_sp_offset is only valid for compiled frame |
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
151 |
saved_fp = (intptr_t*) *(sender_sp - frame::sender_sp_offset); |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
152 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
153 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
154 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
155 |
// If the potential sender is the interpreter then we can do some more checking |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
156 |
if (Interpreter::contains(sender_pc)) { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
157 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
158 |
// ebp is always saved in a recognizable place in any code we generate. However |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
159 |
// only if the sender is interpreted/call_stub (c1 too?) are we certain that the saved ebp |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
160 |
// is really a frame pointer. |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
161 |
|
18025 | 162 |
bool saved_fp_safe = ((address)saved_fp < thread->stack_base()) && (saved_fp > sender_sp); |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
163 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
164 |
if (!saved_fp_safe) { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
165 |
return false; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
166 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
167 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
168 |
// construct the potential sender |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
169 |
|
28824
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
170 |
frame sender(sender_sp, sender_unextended_sp, saved_fp, sender_pc); |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
171 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
172 |
return sender.is_interpreted_frame_valid(thread); |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
173 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
174 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
175 |
|
18025 | 176 |
// We must always be able to find a recognizable pc |
177 |
CodeBlob* sender_blob = CodeCache::find_blob_unsafe(sender_pc); |
|
178 |
if (sender_pc == NULL || sender_blob == NULL) { |
|
179 |
return false; |
|
180 |
} |
|
181 |
||
182 |
// Could be a zombie method |
|
183 |
if (sender_blob->is_zombie() || sender_blob->is_unloaded()) { |
|
184 |
return false; |
|
185 |
} |
|
186 |
||
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
187 |
// Could just be some random pointer within the codeBlob |
6418 | 188 |
if (!sender_blob->code_contains(sender_pc)) { |
189 |
return false; |
|
190 |
} |
|
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
191 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
192 |
// We should never be able to see an adapter if the current frame is something from code cache |
6418 | 193 |
if (sender_blob->is_adapter_blob()) { |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
194 |
return false; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
195 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
196 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
197 |
// Could be the call_stub |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
198 |
if (StubRoutines::returns_to_call_stub(sender_pc)) { |
18025 | 199 |
bool saved_fp_safe = ((address)saved_fp < thread->stack_base()) && (saved_fp > sender_sp); |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
200 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
201 |
if (!saved_fp_safe) { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
202 |
return false; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
203 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
204 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
205 |
// construct the potential sender |
42051
0264f170da65
8166679: JNI AsyncGetCallTrace replaces topmost frame name with <no Java callstack recorded> starting with Java 9 b133
cjplummer
parents:
42042
diff
changeset
|
206 |
|
28824
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
207 |
frame sender(sender_sp, sender_unextended_sp, saved_fp, sender_pc); |
42051
0264f170da65
8166679: JNI AsyncGetCallTrace replaces topmost frame name with <no Java callstack recorded> starting with Java 9 b133
cjplummer
parents:
42042
diff
changeset
|
208 |
|
0264f170da65
8166679: JNI AsyncGetCallTrace replaces topmost frame name with <no Java callstack recorded> starting with Java 9 b133
cjplummer
parents:
42042
diff
changeset
|
209 |
// Validate the JavaCallWrapper an entry frame must have |
0264f170da65
8166679: JNI AsyncGetCallTrace replaces topmost frame name with <no Java callstack recorded> starting with Java 9 b133
cjplummer
parents:
42042
diff
changeset
|
210 |
address jcw = (address)sender.entry_frame_call_wrapper(); |
0264f170da65
8166679: JNI AsyncGetCallTrace replaces topmost frame name with <no Java callstack recorded> starting with Java 9 b133
cjplummer
parents:
42042
diff
changeset
|
211 |
|
0264f170da65
8166679: JNI AsyncGetCallTrace replaces topmost frame name with <no Java callstack recorded> starting with Java 9 b133
cjplummer
parents:
42042
diff
changeset
|
212 |
bool jcw_safe = (jcw < thread->stack_base()) && (jcw > (address)sender.fp()); |
0264f170da65
8166679: JNI AsyncGetCallTrace replaces topmost frame name with <no Java callstack recorded> starting with Java 9 b133
cjplummer
parents:
42042
diff
changeset
|
213 |
|
0264f170da65
8166679: JNI AsyncGetCallTrace replaces topmost frame name with <no Java callstack recorded> starting with Java 9 b133
cjplummer
parents:
42042
diff
changeset
|
214 |
return jcw_safe; |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
215 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
216 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
217 |
CompiledMethod* nm = sender_blob->as_compiled_method_or_null(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
218 |
if (nm != NULL) { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
219 |
if (nm->is_deopt_mh_entry(sender_pc) || nm->is_deopt_entry(sender_pc) || |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
220 |
nm->method()->is_method_handle_intrinsic()) { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
221 |
return false; |
18025 | 222 |
} |
223 |
} |
|
224 |
||
225 |
// If the frame size is 0 something (or less) is bad because every nmethod has a non-zero frame size |
|
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
226 |
// because the return address counts against the callee's frame. |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
227 |
|
18025 | 228 |
if (sender_blob->frame_size() <= 0) { |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
229 |
assert(!sender_blob->is_compiled(), "should count return address at least"); |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
230 |
return false; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
231 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
232 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
233 |
// We should never be able to see anything here except an nmethod. If something in the |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
234 |
// code cache (current frame) is called by an entity within the code cache that entity |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
235 |
// should not be anything but the call stub (already covered), the interpreter (already covered) |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
236 |
// or an nmethod. |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
237 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
238 |
if (!sender_blob->is_compiled()) { |
18025 | 239 |
return false; |
240 |
} |
|
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
241 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
242 |
// Could put some more validation for the potential non-interpreted sender |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
243 |
// frame we'd create by calling sender if I could think of any. Wait for next crash in forte... |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
244 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
245 |
// One idea is seeing if the sender_pc we have is one that we'd expect to call to current cb |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
246 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
247 |
// We've validated the potential sender that would be created |
1 | 248 |
return true; |
249 |
} |
|
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
250 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
251 |
// Must be native-compiled frame. Since sender will try and use fp to find |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
252 |
// linkages it must be safe |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
253 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
254 |
if (!fp_safe) { |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
255 |
return false; |
1 | 256 |
} |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
257 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
258 |
// Will the pc we fetch be non-zero (which we'll find at the oldest frame) |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
259 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
260 |
if ( (address) this->fp()[return_addr_offset] == NULL) return false; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
261 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
262 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
263 |
// could try and do some more potential verification of native frame if we could think of some... |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
264 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
265 |
return true; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
266 |
|
1 | 267 |
} |
268 |
||
269 |
||
270 |
void frame::patch_pc(Thread* thread, address pc) { |
|
10733
49cc61508e86
7081938: JSR292: assert(magic_number_2() == MAGIC_NUMBER_2) failed
twisti
parents:
10539
diff
changeset
|
271 |
address* pc_addr = &(((address*) sp())[-1]); |
1 | 272 |
if (TracePcPatching) { |
10980
8b23df12c7f6
7106944: assert(_pc == *pc_addr) failed may be too strong
twisti
parents:
10733
diff
changeset
|
273 |
tty->print_cr("patch_pc at address " INTPTR_FORMAT " [" INTPTR_FORMAT " -> " INTPTR_FORMAT "]", |
33148
68fa8b6c4340
8042893: compiler: PRAGMA_FORMAT_MUTE_WARNINGS_FOR_GCC needs to be removed from source files
david
parents:
31389
diff
changeset
|
274 |
p2i(pc_addr), p2i(*pc_addr), p2i(pc)); |
1 | 275 |
} |
10980
8b23df12c7f6
7106944: assert(_pc == *pc_addr) failed may be too strong
twisti
parents:
10733
diff
changeset
|
276 |
// Either the return address is the original one or we are going to |
8b23df12c7f6
7106944: assert(_pc == *pc_addr) failed may be too strong
twisti
parents:
10733
diff
changeset
|
277 |
// patch in the same address that's already there. |
8b23df12c7f6
7106944: assert(_pc == *pc_addr) failed may be too strong
twisti
parents:
10733
diff
changeset
|
278 |
assert(_pc == *pc_addr || pc == *pc_addr, "must be"); |
10733
49cc61508e86
7081938: JSR292: assert(magic_number_2() == MAGIC_NUMBER_2) failed
twisti
parents:
10539
diff
changeset
|
279 |
*pc_addr = pc; |
1 | 280 |
_cb = CodeCache::find_blob(pc); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
281 |
address original_pc = CompiledMethod::get_deopt_original_pc(this); |
4752 | 282 |
if (original_pc != NULL) { |
283 |
assert(original_pc == _pc, "expected original PC to be stored before patching"); |
|
1 | 284 |
_deopt_state = is_deoptimized; |
285 |
// leave _pc as is |
|
286 |
} else { |
|
287 |
_deopt_state = not_deoptimized; |
|
288 |
_pc = pc; |
|
289 |
} |
|
290 |
} |
|
291 |
||
292 |
bool frame::is_interpreted_frame() const { |
|
293 |
return Interpreter::contains(pc()); |
|
294 |
} |
|
295 |
||
2880
c2974244a496
6848466: frame::frame_size() assertion failure with -XX:+DebugDeoptimization
cfang
parents:
1066
diff
changeset
|
296 |
int frame::frame_size(RegisterMap* map) const { |
c2974244a496
6848466: frame::frame_size() assertion failure with -XX:+DebugDeoptimization
cfang
parents:
1066
diff
changeset
|
297 |
frame sender = this->sender(map); |
1 | 298 |
return sender.sp() - sp(); |
299 |
} |
|
300 |
||
301 |
intptr_t* frame::entry_frame_argument_at(int offset) const { |
|
302 |
// convert offset to index to deal with tsi |
|
303 |
int index = (Interpreter::expr_offset_in_bytes(offset)/wordSize); |
|
304 |
// Entry frame's arguments are always in relation to unextended_sp() |
|
305 |
return &unextended_sp()[index]; |
|
306 |
} |
|
307 |
||
308 |
// sender_sp |
|
309 |
||
310 |
intptr_t* frame::interpreter_frame_sender_sp() const { |
|
311 |
assert(is_interpreted_frame(), "interpreted frame expected"); |
|
312 |
return (intptr_t*) at(interpreter_frame_sender_sp_offset); |
|
313 |
} |
|
314 |
||
315 |
void frame::set_interpreter_frame_sender_sp(intptr_t* sender_sp) { |
|
316 |
assert(is_interpreted_frame(), "interpreted frame expected"); |
|
317 |
ptr_at_put(interpreter_frame_sender_sp_offset, (intptr_t) sender_sp); |
|
318 |
} |
|
319 |
||
320 |
||
321 |
// monitor elements |
|
322 |
||
323 |
BasicObjectLock* frame::interpreter_frame_monitor_begin() const { |
|
324 |
return (BasicObjectLock*) addr_at(interpreter_frame_monitor_block_bottom_offset); |
|
325 |
} |
|
326 |
||
327 |
BasicObjectLock* frame::interpreter_frame_monitor_end() const { |
|
328 |
BasicObjectLock* result = (BasicObjectLock*) *addr_at(interpreter_frame_monitor_block_top_offset); |
|
329 |
// make sure the pointer points inside the frame |
|
5401
30bda607cb67
6946056: assert((intptr_t) sp()<=(intptr_t) result,"result must>=than stack pointer"), frame_x86.cpp:295
johnc
parents:
4752
diff
changeset
|
330 |
assert(sp() <= (intptr_t*) result, "monitor end should be above the stack pointer"); |
30bda607cb67
6946056: assert((intptr_t) sp()<=(intptr_t) result,"result must>=than stack pointer"), frame_x86.cpp:295
johnc
parents:
4752
diff
changeset
|
331 |
assert((intptr_t*) result < fp(), "monitor end should be strictly below the frame pointer"); |
1 | 332 |
return result; |
333 |
} |
|
334 |
||
335 |
void frame::interpreter_frame_set_monitor_end(BasicObjectLock* value) { |
|
336 |
*((BasicObjectLock**)addr_at(interpreter_frame_monitor_block_top_offset)) = value; |
|
337 |
} |
|
338 |
||
339 |
// Used by template based interpreter deoptimization |
|
340 |
void frame::interpreter_frame_set_last_sp(intptr_t* sp) { |
|
341 |
*((intptr_t**)addr_at(interpreter_frame_last_sp_offset)) = sp; |
|
342 |
} |
|
343 |
||
344 |
frame frame::sender_for_entry_frame(RegisterMap* map) const { |
|
345 |
assert(map != NULL, "map must be set"); |
|
346 |
// Java frame called from C; skip all C frames and return top C |
|
347 |
// frame of that chunk as the sender |
|
348 |
JavaFrameAnchor* jfa = entry_frame_call_wrapper()->anchor(); |
|
349 |
assert(!entry_frame_is_first(), "next Java fp must be non zero"); |
|
350 |
assert(jfa->last_Java_sp() > sp(), "must be above this frame on stack"); |
|
40644
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
351 |
// Since we are walking the stack now this nested anchor is obviously walkable |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
352 |
// even if it wasn't when it was stacked. |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
353 |
if (!jfa->walkable()) { |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
354 |
// Capture _last_Java_pc (if needed) and mark anchor walkable. |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
355 |
jfa->capture_last_Java_pc(); |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
356 |
} |
1 | 357 |
map->clear(); |
358 |
assert(map->include_argument_oops(), "should be set by clear"); |
|
40644
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
359 |
vmassert(jfa->last_Java_pc() != NULL, "not walkable"); |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
360 |
frame fr(jfa->last_Java_sp(), jfa->last_Java_fp(), jfa->last_Java_pc()); |
1 | 361 |
return fr; |
362 |
} |
|
363 |
||
4752 | 364 |
//------------------------------------------------------------------------------ |
365 |
// frame::verify_deopt_original_pc |
|
366 |
// |
|
367 |
// Verifies the calculated original PC of a deoptimization PC for the |
|
30305
b92a97e1e9cb
8068945: Use RBP register as proper frame pointer in JIT compiled code on x86
zmajo
parents:
28824
diff
changeset
|
368 |
// given unextended SP. |
15937 | 369 |
#ifdef ASSERT |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
370 |
void frame::verify_deopt_original_pc(CompiledMethod* nm, intptr_t* unextended_sp) { |
4752 | 371 |
frame fr; |
372 |
||
373 |
// This is ugly but it's better than to change {get,set}_original_pc |
|
374 |
// to take an SP value as argument. And it's only a debugging |
|
375 |
// method anyway. |
|
376 |
fr._unextended_sp = unextended_sp; |
|
377 |
||
378 |
address original_pc = nm->get_original_pc(&fr); |
|
43474
8fbf946045f6
8172844: Assert fails in deoptimization due to original PC at the end of code section
zmajo
parents:
42051
diff
changeset
|
379 |
assert(nm->insts_contains_inclusive(original_pc), |
8fbf946045f6
8172844: Assert fails in deoptimization due to original PC at the end of code section
zmajo
parents:
42051
diff
changeset
|
380 |
"original PC must be in the main code section of the the compiled method (or must be immediately following it)"); |
4752 | 381 |
} |
382 |
#endif |
|
383 |
||
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
384 |
//------------------------------------------------------------------------------ |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
385 |
// frame::adjust_unextended_sp |
47625
60dae13727b8
8189244: x86: eliminate frame::adjust_unextended_sp() overhead
dlong
parents:
47216
diff
changeset
|
386 |
#ifdef ASSERT |
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
387 |
void frame::adjust_unextended_sp() { |
30305
b92a97e1e9cb
8068945: Use RBP register as proper frame pointer in JIT compiled code on x86
zmajo
parents:
28824
diff
changeset
|
388 |
// On x86, sites calling method handle intrinsics and lambda forms are treated |
b92a97e1e9cb
8068945: Use RBP register as proper frame pointer in JIT compiled code on x86
zmajo
parents:
28824
diff
changeset
|
389 |
// as any other call site. Therefore, no special action is needed when we are |
b92a97e1e9cb
8068945: Use RBP register as proper frame pointer in JIT compiled code on x86
zmajo
parents:
28824
diff
changeset
|
390 |
// returning to any of these call sites. |
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
391 |
|
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
392 |
if (_cb != NULL) { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
393 |
CompiledMethod* sender_cm = _cb->as_compiled_method_or_null(); |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
394 |
if (sender_cm != NULL) { |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
395 |
// If the sender PC is a deoptimization point, get the original PC. |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
396 |
if (sender_cm->is_deopt_entry(_pc) || |
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
397 |
sender_cm->is_deopt_mh_entry(_pc)) { |
47625
60dae13727b8
8189244: x86: eliminate frame::adjust_unextended_sp() overhead
dlong
parents:
47216
diff
changeset
|
398 |
verify_deopt_original_pc(sender_cm, _unextended_sp); |
38133
78b95467b9f1
8151956: Support non-continuous CodeBlobs in HotSpot
rbackman
parents:
35214
diff
changeset
|
399 |
} |
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
400 |
} |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
401 |
} |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
402 |
} |
47625
60dae13727b8
8189244: x86: eliminate frame::adjust_unextended_sp() overhead
dlong
parents:
47216
diff
changeset
|
403 |
#endif |
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
404 |
|
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
405 |
//------------------------------------------------------------------------------ |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
406 |
// frame::update_map_with_saved_link |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
407 |
void frame::update_map_with_saved_link(RegisterMap* map, intptr_t** link_addr) { |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
408 |
// The interpreter and compiler(s) always save EBP/RBP in a known |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
409 |
// location on entry. We must record where that location is |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
410 |
// so this if EBP/RBP was live on callout from c2 we can find |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
411 |
// the saved copy no matter what it called. |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
412 |
|
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
413 |
// Since the interpreter always saves EBP/RBP if we record where it is then |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
414 |
// we don't have to always save EBP/RBP on entry and exit to c2 compiled |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
415 |
// code, on entry will be enough. |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
416 |
map->set_location(rbp->as_VMReg(), (address) link_addr); |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
417 |
#ifdef AMD64 |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
418 |
// this is weird "H" ought to be at a higher address however the |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
419 |
// oopMaps seems to have the "H" regs at the same address and the |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
420 |
// vanilla register. |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
421 |
// XXXX make this go away |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
422 |
if (true) { |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
423 |
map->set_location(rbp->as_VMReg()->next(), (address) link_addr); |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
424 |
} |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
425 |
#endif // AMD64 |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
426 |
} |
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
427 |
|
4752 | 428 |
|
429 |
//------------------------------------------------------------------------------ |
|
430 |
// frame::sender_for_interpreter_frame |
|
1 | 431 |
frame frame::sender_for_interpreter_frame(RegisterMap* map) const { |
4752 | 432 |
// SP is the raw SP from the sender after adapter or interpreter |
433 |
// extension. |
|
434 |
intptr_t* sender_sp = this->sender_sp(); |
|
1 | 435 |
|
436 |
// This is the sp before any possible extension (adapter/locals). |
|
437 |
intptr_t* unextended_sp = interpreter_frame_sender_sp(); |
|
438 |
||
47799 | 439 |
#if COMPILER2_OR_JVMCI |
1 | 440 |
if (map->update_map()) { |
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
441 |
update_map_with_saved_link(map, (intptr_t**) addr_at(link_offset)); |
1 | 442 |
} |
47799 | 443 |
#endif // COMPILER2_OR_JVMCI |
4752 | 444 |
|
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
445 |
return frame(sender_sp, unextended_sp, link(), sender_pc()); |
1 | 446 |
} |
447 |
||
448 |
||
4752 | 449 |
//------------------------------------------------------------------------------ |
450 |
// frame::sender_for_compiled_frame |
|
1 | 451 |
frame frame::sender_for_compiled_frame(RegisterMap* map) const { |
452 |
assert(map != NULL, "map must be set"); |
|
453 |
||
454 |
// frame owned by optimizing compiler |
|
455 |
assert(_cb->frame_size() >= 0, "must have non-zero frame size"); |
|
4752 | 456 |
intptr_t* sender_sp = unextended_sp() + _cb->frame_size(); |
457 |
intptr_t* unextended_sp = sender_sp; |
|
1 | 458 |
|
459 |
// On Intel the return_address is always the word on the stack |
|
460 |
address sender_pc = (address) *(sender_sp-1); |
|
461 |
||
4752 | 462 |
// This is the saved value of EBP which may or may not really be an FP. |
463 |
// It is only an FP if the sender is an interpreter frame (or C1?). |
|
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
464 |
intptr_t** saved_fp_addr = (intptr_t**) (sender_sp - frame::sender_sp_offset); |
4564
55dfb20908d0
6893081: method handle & invokedynamic code needs additional cleanup (post 6815692, 6858164)
twisti
parents:
3261
diff
changeset
|
465 |
|
1 | 466 |
if (map->update_map()) { |
467 |
// Tell GC to use argument oopmaps for some runtime stubs that need it. |
|
468 |
// For C1, the runtime stub might not have oop maps, so set this flag |
|
469 |
// outside of update_register_map. |
|
470 |
map->set_include_argument_oops(_cb->caller_must_gc_arguments(map->thread())); |
|
471 |
if (_cb->oop_maps() != NULL) { |
|
472 |
OopMapSet::update_register_map(this, map); |
|
473 |
} |
|
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
474 |
|
4752 | 475 |
// Since the prolog does the save and restore of EBP there is no oopmap |
1 | 476 |
// for it so we must fill in its location as if there was an oopmap entry |
477 |
// since if our caller was compiled code there could be live jvm state in it. |
|
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
478 |
update_map_with_saved_link(map, saved_fp_addr); |
1 | 479 |
} |
480 |
||
481 |
assert(sender_sp != sp(), "must have changed"); |
|
9630
d6419e4395e3
6939861: JVM should handle more conversion operations
never
parents:
9437
diff
changeset
|
482 |
return frame(sender_sp, unextended_sp, *saved_fp_addr, sender_pc); |
1 | 483 |
} |
484 |
||
4752 | 485 |
|
486 |
//------------------------------------------------------------------------------ |
|
487 |
// frame::sender |
|
1 | 488 |
frame frame::sender(RegisterMap* map) const { |
489 |
// Default is we done have to follow them. The sender_for_xxx will |
|
490 |
// update it accordingly |
|
491 |
map->set_include_argument_oops(false); |
|
492 |
||
493 |
if (is_entry_frame()) return sender_for_entry_frame(map); |
|
494 |
if (is_interpreted_frame()) return sender_for_interpreter_frame(map); |
|
495 |
assert(_cb == CodeCache::find_blob(pc()),"Must be the same"); |
|
496 |
||
497 |
if (_cb != NULL) { |
|
498 |
return sender_for_compiled_frame(map); |
|
499 |
} |
|
500 |
// Must be native-compiled frame, i.e. the marshaling code for native |
|
501 |
// methods that exists in the core system. |
|
502 |
return frame(sender_sp(), link(), sender_pc()); |
|
503 |
} |
|
504 |
||
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
505 |
bool frame::is_interpreted_frame_valid(JavaThread* thread) const { |
1 | 506 |
assert(is_interpreted_frame(), "Not an interpreted frame"); |
507 |
// These are reasonable sanity checks |
|
508 |
if (fp() == 0 || (intptr_t(fp()) & (wordSize-1)) != 0) { |
|
509 |
return false; |
|
510 |
} |
|
511 |
if (sp() == 0 || (intptr_t(sp()) & (wordSize-1)) != 0) { |
|
512 |
return false; |
|
513 |
} |
|
514 |
if (fp() + interpreter_frame_initial_sp_offset < sp()) { |
|
515 |
return false; |
|
516 |
} |
|
517 |
// These are hacks to keep us out of trouble. |
|
518 |
// The problem with these is that they mask other problems |
|
519 |
if (fp() <= sp()) { // this attempts to deal with unsigned comparison above |
|
520 |
return false; |
|
521 |
} |
|
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
522 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
523 |
// do some validation of frame elements |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
524 |
// first the method |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
525 |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
526 |
Method* m = *interpreter_frame_method_addr(); |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
527 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
528 |
// validate the method we'd find in this potential sender |
14579
7f6ce6e3dd80
8003635: NPG: AsynchGetCallTrace broken by Method* virtual call
coleenp
parents:
13728
diff
changeset
|
529 |
if (!m->is_valid_method()) return false; |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
530 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
531 |
// stack frames shouldn't be much larger than max_stack elements |
28824
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
532 |
// this test requires the use the unextended_sp which is the sp as seen by |
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
533 |
// the current frame, and not sp which is the "raw" pc which could point |
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
534 |
// further because of local variables of the callee method inserted after |
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
535 |
// method arguments |
ad5e6cd229c0
8068655: frame::safe_for_sender() computes incorrect sender_sp value for interpreted frames
fparain
parents:
26821
diff
changeset
|
536 |
if (fp() - unextended_sp() > 1024 + m->max_stack()*Interpreter::stackElementSize) { |
1 | 537 |
return false; |
538 |
} |
|
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
539 |
|
25714
87fa6860b5ae
8004128: NPG: remove stackwalking in Threads::gc_prologue and gc_epilogue code
coleenp
parents:
25468
diff
changeset
|
540 |
// validate bci/bcp |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
541 |
|
25714
87fa6860b5ae
8004128: NPG: remove stackwalking in Threads::gc_prologue and gc_epilogue code
coleenp
parents:
25468
diff
changeset
|
542 |
address bcp = interpreter_frame_bcp(); |
87fa6860b5ae
8004128: NPG: remove stackwalking in Threads::gc_prologue and gc_epilogue code
coleenp
parents:
25468
diff
changeset
|
543 |
if (m->validate_bci_from_bcp(bcp) < 0) { |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
544 |
return false; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
545 |
} |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
546 |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
547 |
// validate ConstantPoolCache* |
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
548 |
ConstantPoolCache* cp = *interpreter_frame_cache_addr(); |
18439 | 549 |
if (cp == NULL || !cp->is_metaspace_object()) return false; |
354
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
550 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
551 |
// validate locals |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
552 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
553 |
address locals = (address) *interpreter_frame_locals_addr(); |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
554 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
555 |
if (locals > thread->stack_base() || locals < (address) fp()) return false; |
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
556 |
|
3b42d6fdcb82
6603919: Stackwalking crash on x86 -server with Sun Studio's collect -j on
sgoldman
parents:
1
diff
changeset
|
557 |
// We'd have to be pretty unlucky to be mislead at this point |
1 | 558 |
return true; |
559 |
} |
|
560 |
||
561 |
BasicType frame::interpreter_frame_result(oop* oop_result, jvalue* value_result) { |
|
562 |
assert(is_interpreted_frame(), "interpreted frame expected"); |
|
13728
882756847a04
6964458: Reimplement class meta-data storage to use native memory
coleenp
parents:
13391
diff
changeset
|
563 |
Method* method = interpreter_frame_method(); |
1 | 564 |
BasicType type = method->result_type(); |
565 |
||
566 |
intptr_t* tos_addr; |
|
567 |
if (method->is_native()) { |
|
568 |
// Prior to calling into the runtime to report the method_exit the possible |
|
569 |
// return value is pushed to the native stack. If the result is a jfloat/jdouble |
|
570 |
// then ST0 is saved before EAX/EDX. See the note in generate_native_result |
|
571 |
tos_addr = (intptr_t*)sp(); |
|
572 |
if (type == T_FLOAT || type == T_DOUBLE) { |
|
573 |
// QQQ seems like this code is equivalent on the two platforms |
|
574 |
#ifdef AMD64 |
|
575 |
// This is times two because we do a push(ltos) after pushing XMM0 |
|
576 |
// and that takes two interpreter stack slots. |
|
5419 | 577 |
tos_addr += 2 * Interpreter::stackElementWords; |
1 | 578 |
#else |
579 |
tos_addr += 2; |
|
580 |
#endif // AMD64 |
|
581 |
} |
|
582 |
} else { |
|
583 |
tos_addr = (intptr_t*)interpreter_frame_tos_address(); |
|
584 |
} |
|
585 |
||
586 |
switch (type) { |
|
587 |
case T_OBJECT : |
|
588 |
case T_ARRAY : { |
|
589 |
oop obj; |
|
590 |
if (method->is_native()) { |
|
20282
7f9cbdf89af2
7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents:
18439
diff
changeset
|
591 |
obj = cast_to_oop(at(interpreter_frame_oop_temp_offset)); |
1 | 592 |
} else { |
593 |
oop* obj_p = (oop*)tos_addr; |
|
594 |
obj = (obj_p == NULL) ? (oop)NULL : *obj_p; |
|
595 |
} |
|
596 |
assert(obj == NULL || Universe::heap()->is_in(obj), "sanity check"); |
|
597 |
*oop_result = obj; |
|
598 |
break; |
|
599 |
} |
|
600 |
case T_BOOLEAN : value_result->z = *(jboolean*)tos_addr; break; |
|
601 |
case T_BYTE : value_result->b = *(jbyte*)tos_addr; break; |
|
602 |
case T_CHAR : value_result->c = *(jchar*)tos_addr; break; |
|
603 |
case T_SHORT : value_result->s = *(jshort*)tos_addr; break; |
|
604 |
case T_INT : value_result->i = *(jint*)tos_addr; break; |
|
605 |
case T_LONG : value_result->j = *(jlong*)tos_addr; break; |
|
606 |
case T_FLOAT : { |
|
607 |
#ifdef AMD64 |
|
608 |
value_result->f = *(jfloat*)tos_addr; |
|
609 |
#else |
|
610 |
if (method->is_native()) { |
|
611 |
jdouble d = *(jdouble*)tos_addr; // Result was in ST0 so need to convert to jfloat |
|
612 |
value_result->f = (jfloat)d; |
|
613 |
} else { |
|
614 |
value_result->f = *(jfloat*)tos_addr; |
|
615 |
} |
|
616 |
#endif // AMD64 |
|
617 |
break; |
|
618 |
} |
|
619 |
case T_DOUBLE : value_result->d = *(jdouble*)tos_addr; break; |
|
620 |
case T_VOID : /* Nothing to do */ break; |
|
621 |
default : ShouldNotReachHere(); |
|
622 |
} |
|
623 |
||
624 |
return type; |
|
625 |
} |
|
626 |
||
627 |
||
628 |
intptr_t* frame::interpreter_frame_tos_at(jint offset) const { |
|
629 |
int index = (Interpreter::expr_offset_in_bytes(offset)/wordSize); |
|
630 |
return &interpreter_frame_tos_address()[index]; |
|
631 |
} |
|
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
632 |
|
11571
23f825a42a85
7120468: SPARC/x86: use frame::describe to enhance trace_method_handle
bdelsart
parents:
11565
diff
changeset
|
633 |
#ifndef PRODUCT |
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
634 |
|
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
635 |
#define DESCRIBE_FP_OFFSET(name) \ |
9632
cd86c748c12b
7043301: assert(locals < caller->fp() || locals > (caller->fp() + 16)) failed: locals in save area
never
parents:
9630
diff
changeset
|
636 |
values.describe(frame_no, fp() + frame::name##_offset, #name) |
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
637 |
|
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
638 |
void frame::describe_pd(FrameValues& values, int frame_no) { |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11632
diff
changeset
|
639 |
if (is_interpreted_frame()) { |
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
640 |
DESCRIBE_FP_OFFSET(interpreter_frame_sender_sp); |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
641 |
DESCRIBE_FP_OFFSET(interpreter_frame_last_sp); |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
642 |
DESCRIBE_FP_OFFSET(interpreter_frame_method); |
38074
8475fdc6dcc3
8154580: Save mirror in interpreter frame to enable cleanups of CLDClosure
coleenp
parents:
35214
diff
changeset
|
643 |
DESCRIBE_FP_OFFSET(interpreter_frame_mirror); |
25714
87fa6860b5ae
8004128: NPG: remove stackwalking in Threads::gc_prologue and gc_epilogue code
coleenp
parents:
25468
diff
changeset
|
644 |
DESCRIBE_FP_OFFSET(interpreter_frame_mdp); |
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
645 |
DESCRIBE_FP_OFFSET(interpreter_frame_cache); |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
646 |
DESCRIBE_FP_OFFSET(interpreter_frame_locals); |
25714
87fa6860b5ae
8004128: NPG: remove stackwalking in Threads::gc_prologue and gc_epilogue code
coleenp
parents:
25468
diff
changeset
|
647 |
DESCRIBE_FP_OFFSET(interpreter_frame_bcp); |
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
648 |
DESCRIBE_FP_OFFSET(interpreter_frame_initial_sp); |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31389
diff
changeset
|
649 |
#ifdef AMD64 |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31389
diff
changeset
|
650 |
} else if (is_entry_frame()) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31389
diff
changeset
|
651 |
// This could be more descriptive if we use the enum in |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31389
diff
changeset
|
652 |
// stubGenerator to map to real names but it's most important to |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31389
diff
changeset
|
653 |
// claim these frame slots so the error checking works. |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31389
diff
changeset
|
654 |
for (int i = 0; i < entry_frame_after_call_words; i++) { |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31389
diff
changeset
|
655 |
values.describe(frame_no, fp() - i, err_msg("call_stub word fp - %d", i)); |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31389
diff
changeset
|
656 |
} |
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31389
diff
changeset
|
657 |
#endif // AMD64 |
9437
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
658 |
} |
9981851b4b8c
7009361: JSR 292 Invalid value on stack on solaris-sparc with -Xcomp
never
parents:
7397
diff
changeset
|
659 |
} |
33160
c59f1676d27e
8136421: JEP 243: Java-Level JVM Compiler Interface
twisti
parents:
31389
diff
changeset
|
660 |
#endif // !PRODUCT |
10539
f87cedf7983c
7087445: Improve platform independence of JSR292 shared code
bdelsart
parents:
9632
diff
changeset
|
661 |
|
f87cedf7983c
7087445: Improve platform independence of JSR292 shared code
bdelsart
parents:
9632
diff
changeset
|
662 |
intptr_t *frame::initial_deoptimization_info() { |
f87cedf7983c
7087445: Improve platform independence of JSR292 shared code
bdelsart
parents:
9632
diff
changeset
|
663 |
// used to reset the saved FP |
f87cedf7983c
7087445: Improve platform independence of JSR292 shared code
bdelsart
parents:
9632
diff
changeset
|
664 |
return fp(); |
f87cedf7983c
7087445: Improve platform independence of JSR292 shared code
bdelsart
parents:
9632
diff
changeset
|
665 |
} |
11486
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
666 |
|
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
667 |
intptr_t* frame::real_fp() const { |
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
668 |
if (_cb != NULL) { |
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
669 |
// use the frame size if valid |
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
670 |
int size = _cb->frame_size(); |
13391
30245956af37
7023639: JSR 292 method handle invocation needs a fast path for compiled code
twisti
parents:
11632
diff
changeset
|
671 |
if (size > 0) { |
11486
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
672 |
return unextended_sp() + size; |
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
673 |
} |
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
674 |
} |
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
675 |
// else rely on fp() |
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
676 |
assert(! is_compiled_frame(), "unknown compiled frame size"); |
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
677 |
return fp(); |
cdc636532368
7120448: Fix FP values for compiled frames in frame::describe
bdelsart
parents:
10980
diff
changeset
|
678 |
} |
26821
ce9f82507dc2
8058345: Refactor native stack printing from vmError.cpp to debug.cpp to make it available in gdb as well
simonis
parents:
25714
diff
changeset
|
679 |
|
ce9f82507dc2
8058345: Refactor native stack printing from vmError.cpp to debug.cpp to make it available in gdb as well
simonis
parents:
25714
diff
changeset
|
680 |
#ifndef PRODUCT |
ce9f82507dc2
8058345: Refactor native stack printing from vmError.cpp to debug.cpp to make it available in gdb as well
simonis
parents:
25714
diff
changeset
|
681 |
// This is a generic constructor which is only used by pns() in debug.cpp. |
ce9f82507dc2
8058345: Refactor native stack printing from vmError.cpp to debug.cpp to make it available in gdb as well
simonis
parents:
25714
diff
changeset
|
682 |
frame::frame(void* sp, void* fp, void* pc) { |
ce9f82507dc2
8058345: Refactor native stack printing from vmError.cpp to debug.cpp to make it available in gdb as well
simonis
parents:
25714
diff
changeset
|
683 |
init((intptr_t*)sp, (intptr_t*)fp, (address)pc); |
ce9f82507dc2
8058345: Refactor native stack printing from vmError.cpp to debug.cpp to make it available in gdb as well
simonis
parents:
25714
diff
changeset
|
684 |
} |
46589 | 685 |
|
686 |
void frame::pd_ps() {} |
|
26821
ce9f82507dc2
8058345: Refactor native stack printing from vmError.cpp to debug.cpp to make it available in gdb as well
simonis
parents:
25714
diff
changeset
|
687 |
#endif |
40644
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
688 |
|
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
689 |
void JavaFrameAnchor::make_walkable(JavaThread* thread) { |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
690 |
// last frame set? |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
691 |
if (last_Java_sp() == NULL) return; |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
692 |
// already walkable? |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
693 |
if (walkable()) return; |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
694 |
vmassert(Thread::current() == (Thread*)thread, "not current thread"); |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
695 |
vmassert(last_Java_sp() != NULL, "not called from Java code?"); |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
696 |
vmassert(last_Java_pc() == NULL, "already walkable"); |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
697 |
capture_last_Java_pc(); |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
698 |
vmassert(walkable(), "something went wrong"); |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
699 |
} |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
700 |
|
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
701 |
void JavaFrameAnchor::capture_last_Java_pc() { |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
702 |
vmassert(_last_Java_sp != NULL, "no last frame set"); |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
703 |
vmassert(_last_Java_pc == NULL, "already walkable"); |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
704 |
_last_Java_pc = (address)_last_Java_sp[-1]; |
39e631ed7145
8161598: Kitchensink fails: assert(nm->insts_contains(original_pc)) failed: original PC must be in nmethod/CompiledMethod
dlong
parents:
40332
diff
changeset
|
705 |
} |