author | coleenp |
Tue, 14 Mar 2017 14:07:24 -0400 | |
changeset 46327 | 91576389a517 |
parent 46289 | 1904e7ec236e |
child 46329 | 53ccc37bda19 |
permissions | -rw-r--r-- |
34253 | 1 |
/* |
46271
979ebd346ecf
8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents:
41664
diff
changeset
|
2 |
* Copyright (c) 2015, 2017, Oracle and/or its affiliates. All rights reserved. |
34253 | 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 |
* |
|
19 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
20 |
* or visit www.oracle.com if you need additional information or have any |
|
21 |
* questions. |
|
22 |
* |
|
23 |
*/ |
|
24 |
||
25 |
#include "precompiled.hpp" |
|
26 |
#include "classfile/javaClasses.hpp" |
|
27 |
#include "classfile/javaClasses.inline.hpp" |
|
28 |
#include "classfile/vmSymbols.hpp" |
|
41664
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
29 |
#include "logging/log.hpp" |
34253 | 30 |
#include "memory/oopFactory.hpp" |
31 |
#include "oops/oop.inline.hpp" |
|
32 |
#include "oops/objArrayOop.inline.hpp" |
|
33 |
#include "prims/stackwalk.hpp" |
|
34 |
#include "runtime/globals.hpp" |
|
35 |
#include "runtime/handles.inline.hpp" |
|
36 |
#include "runtime/javaCalls.hpp" |
|
37 |
#include "runtime/vframe.hpp" |
|
38 |
#include "utilities/globalDefinitions.hpp" |
|
39 |
||
40 |
// setup and cleanup actions |
|
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
41 |
void BaseFrameStream::setup_magic_on_entry(objArrayHandle frames_array) { |
37438 | 42 |
frames_array->obj_at_put(magic_pos, _thread->threadObj()); |
34253 | 43 |
_anchor = address_value(); |
37438 | 44 |
assert(check_magic(frames_array), "invalid magic"); |
34253 | 45 |
} |
46 |
||
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
47 |
bool BaseFrameStream::check_magic(objArrayHandle frames_array) { |
37438 | 48 |
oop m1 = frames_array->obj_at(magic_pos); |
34253 | 49 |
jlong m2 = _anchor; |
50 |
if (m1 == _thread->threadObj() && m2 == address_value()) return true; |
|
51 |
return false; |
|
52 |
} |
|
53 |
||
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
54 |
bool BaseFrameStream::cleanup_magic_on_exit(objArrayHandle frames_array) { |
37438 | 55 |
bool ok = check_magic(frames_array); |
56 |
frames_array->obj_at_put(magic_pos, NULL); |
|
34253 | 57 |
_anchor = 0L; |
58 |
return ok; |
|
59 |
} |
|
60 |
||
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
61 |
JavaFrameStream::JavaFrameStream(JavaThread* thread, int mode) |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
62 |
: BaseFrameStream(thread), _vfst(thread) { |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
63 |
_need_method_info = StackWalk::need_method_info(mode); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
64 |
} |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
65 |
|
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
66 |
// Returns the BaseFrameStream for the current stack being traversed. |
34253 | 67 |
// |
68 |
// Parameters: |
|
69 |
// thread Current Java thread. |
|
70 |
// magic Magic value used for each stack walking |
|
37438 | 71 |
// frames_array User-supplied buffers. The 0th element is reserved |
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
72 |
// for this BaseFrameStream to use |
34253 | 73 |
// |
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
74 |
BaseFrameStream* BaseFrameStream::from_current(JavaThread* thread, jlong magic, |
37438 | 75 |
objArrayHandle frames_array) |
34253 | 76 |
{ |
77 |
assert(thread != NULL && thread->is_Java_thread(), ""); |
|
37438 | 78 |
oop m1 = frames_array->obj_at(magic_pos); |
34253 | 79 |
if (m1 != thread->threadObj()) return NULL; |
80 |
if (magic == 0L) return NULL; |
|
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
81 |
BaseFrameStream* stream = (BaseFrameStream*) (intptr_t) magic; |
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
82 |
if (!stream->is_valid_in(thread, frames_array)) return NULL; |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
83 |
return stream; |
34253 | 84 |
} |
85 |
||
86 |
// Unpacks one or more frames into user-supplied buffers. |
|
87 |
// Updates the end index, and returns the number of unpacked frames. |
|
88 |
// Always start with the existing vfst.method and bci. |
|
89 |
// Do not call vfst.next to advance over the last returned value. |
|
90 |
// In other words, do not leave any stale data in the vfst. |
|
91 |
// |
|
92 |
// Parameters: |
|
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
93 |
// mode Restrict which frames to be decoded. |
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
94 |
// BaseFrameStream stream of frames |
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
95 |
// max_nframes Maximum number of frames to be filled. |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
96 |
// start_index Start index to the user-supplied buffers. |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
97 |
// frames_array Buffer to store Class or StackFrame in, starting at start_index. |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
98 |
// frames array is a Class<?>[] array when only getting caller |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
99 |
// reference, and a StackFrameInfo[] array (or derivative) |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
100 |
// otherwise. It should never be null. |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
101 |
// end_index End index to the user-supplied buffers with unpacked frames. |
34253 | 102 |
// |
103 |
// Returns the number of frames whose information was transferred into the buffers. |
|
104 |
// |
|
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
105 |
int StackWalk::fill_in_frames(jlong mode, BaseFrameStream& stream, |
34253 | 106 |
int max_nframes, int start_index, |
107 |
objArrayHandle frames_array, |
|
108 |
int& end_index, TRAPS) { |
|
41664
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
109 |
log_debug(stackwalk)("fill_in_frames limit=%d start=%d frames length=%d", |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
110 |
max_nframes, start_index, frames_array->length()); |
34253 | 111 |
assert(max_nframes > 0, "invalid max_nframes"); |
37438 | 112 |
assert(start_index + max_nframes <= frames_array->length(), "oob"); |
34253 | 113 |
|
114 |
int frames_decoded = 0; |
|
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
115 |
for (; !stream.at_end(); stream.next()) { |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
116 |
Method* method = stream.method(); |
34253 | 117 |
|
118 |
if (method == NULL) continue; |
|
41095
97cca074c2be
8157464: Disallow StackWalker.getCallerClass() be called by caller-sensitive method
mchung
parents:
40001
diff
changeset
|
119 |
|
97cca074c2be
8157464: Disallow StackWalker.getCallerClass() be called by caller-sensitive method
mchung
parents:
40001
diff
changeset
|
120 |
// skip hidden frames for default StackWalker option (i.e. SHOW_HIDDEN_FRAMES |
97cca074c2be
8157464: Disallow StackWalker.getCallerClass() be called by caller-sensitive method
mchung
parents:
40001
diff
changeset
|
121 |
// not set) and when StackWalker::getCallerClass is called |
97cca074c2be
8157464: Disallow StackWalker.getCallerClass() be called by caller-sensitive method
mchung
parents:
40001
diff
changeset
|
122 |
if (!ShowHiddenFrames && (skip_hidden_frames(mode) || get_caller_class(mode))) { |
34253 | 123 |
if (method->is_hidden()) { |
41664
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
124 |
if (log_is_enabled(Debug, stackwalk)) { |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
125 |
ResourceMark rm(THREAD); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
126 |
outputStream* st = Log(stackwalk)::debug_stream(); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
127 |
st->print(" hidden method: "); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
128 |
method->print_short_name(st); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
129 |
st->cr(); |
34253 | 130 |
} |
131 |
continue; |
|
132 |
} |
|
133 |
} |
|
134 |
||
135 |
int index = end_index++; |
|
41664
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
136 |
if (log_is_enabled(Debug, stackwalk)) { |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
137 |
ResourceMark rm(THREAD); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
138 |
outputStream* st = Log(stackwalk)::debug_stream(); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
139 |
st->print(" %d: frame method: ", index); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
140 |
method->print_short_name(st); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
141 |
st->print_cr(" bci=%d", stream.bci()); |
34253 | 142 |
} |
143 |
||
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
144 |
if (!need_method_info(mode) && get_caller_class(mode) && |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
145 |
index == start_index && method->caller_sensitive()) { |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
146 |
ResourceMark rm(THREAD); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
147 |
THROW_MSG_0(vmSymbols::java_lang_UnsupportedOperationException(), |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
148 |
err_msg("StackWalker::getCallerClass called from @CallerSensitive %s method", |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
149 |
method->name_and_sig_as_C_string())); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
150 |
} |
34253 | 151 |
// fill in StackFrameInfo and initialize MemberName |
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
152 |
stream.fill_frame(index, frames_array, method, CHECK_0); |
34253 | 153 |
if (++frames_decoded >= max_nframes) break; |
154 |
} |
|
155 |
return frames_decoded; |
|
156 |
} |
|
157 |
||
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
158 |
// Fill in the LiveStackFrameInfo at the given index in frames_array |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
159 |
void LiveFrameStream::fill_frame(int index, objArrayHandle frames_array, |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
160 |
const methodHandle& method, TRAPS) { |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
161 |
Handle stackFrame(THREAD, frames_array->obj_at(index)); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
162 |
fill_live_stackframe(stackFrame, method, CHECK); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
163 |
} |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
164 |
|
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
165 |
// Fill in the StackFrameInfo at the given index in frames_array |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
166 |
void JavaFrameStream::fill_frame(int index, objArrayHandle frames_array, |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
167 |
const methodHandle& method, TRAPS) { |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
168 |
if (_need_method_info) { |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
169 |
Handle stackFrame(THREAD, frames_array->obj_at(index)); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
170 |
fill_stackframe(stackFrame, method); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
171 |
} else { |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
172 |
frames_array->obj_at_put(index, method->method_holder()->java_mirror()); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
173 |
} |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
174 |
} |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
175 |
|
43677
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
176 |
// Create and return a LiveStackFrame.PrimitiveSlot (if needed) for the |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
177 |
// StackValue at the given index. 'type' is expected to be T_INT, T_LONG, |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
178 |
// T_OBJECT, or T_CONFLICT. |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
179 |
oop LiveFrameStream::create_primitive_slot_instance(StackValueCollection* values, |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
180 |
int i, BasicType type, TRAPS) { |
34253 | 181 |
Klass* k = SystemDictionary::resolve_or_null(vmSymbols::java_lang_LiveStackFrameInfo(), CHECK_NULL); |
182 |
instanceKlassHandle ik (THREAD, k); |
|
183 |
||
184 |
JavaValue result(T_OBJECT); |
|
185 |
JavaCallArguments args; |
|
186 |
Symbol* signature = NULL; |
|
187 |
||
188 |
// ## TODO: type is only available in LocalVariable table, if present. |
|
43677
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
189 |
// ## StackValue type is T_INT or T_OBJECT (or converted to T_LONG on 64-bit) |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
190 |
switch (type) { |
34253 | 191 |
case T_INT: |
192 |
args.push_int(values->int_at(i)); |
|
193 |
signature = vmSymbols::asPrimitive_int_signature(); |
|
194 |
break; |
|
195 |
||
196 |
case T_LONG: |
|
197 |
args.push_long(values->long_at(i)); |
|
198 |
signature = vmSymbols::asPrimitive_long_signature(); |
|
199 |
break; |
|
200 |
||
201 |
case T_FLOAT: |
|
202 |
case T_DOUBLE: |
|
203 |
case T_BYTE: |
|
204 |
case T_SHORT: |
|
205 |
case T_CHAR: |
|
206 |
case T_BOOLEAN: |
|
43677
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
207 |
THROW_MSG_(vmSymbols::java_lang_InternalError(), "Unexpected StackValue type", NULL); |
34253 | 208 |
|
209 |
case T_OBJECT: |
|
210 |
return values->obj_at(i)(); |
|
211 |
||
212 |
case T_CONFLICT: |
|
213 |
// put a non-null slot |
|
43677
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
214 |
#ifdef _LP64 |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
215 |
args.push_long(0); |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
216 |
signature = vmSymbols::asPrimitive_long_signature(); |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
217 |
#else |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
218 |
args.push_int(0); |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
219 |
signature = vmSymbols::asPrimitive_int_signature(); |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
220 |
#endif |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
221 |
|
34253 | 222 |
break; |
223 |
||
224 |
default: ShouldNotReachHere(); |
|
225 |
} |
|
226 |
JavaCalls::call_static(&result, |
|
227 |
ik, |
|
228 |
vmSymbols::asPrimitive_name(), |
|
229 |
signature, |
|
230 |
&args, |
|
231 |
CHECK_NULL); |
|
232 |
return (instanceOop) result.get_jobject(); |
|
233 |
} |
|
234 |
||
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
235 |
objArrayHandle LiveFrameStream::values_to_object_array(StackValueCollection* values, TRAPS) { |
34253 | 236 |
objArrayHandle empty; |
237 |
int length = values->size(); |
|
238 |
objArrayOop array_oop = oopFactory::new_objArray(SystemDictionary::Object_klass(), |
|
239 |
length, CHECK_(empty)); |
|
240 |
objArrayHandle array_h(THREAD, array_oop); |
|
241 |
for (int i = 0; i < values->size(); i++) { |
|
242 |
StackValue* st = values->at(i); |
|
43677
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
243 |
BasicType type = st->type(); |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
244 |
int index = i; |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
245 |
#ifdef _LP64 |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
246 |
if (type != T_OBJECT && type != T_CONFLICT) { |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
247 |
intptr_t ret = st->get_int(); // read full 64-bit slot |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
248 |
type = T_LONG; // treat as long |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
249 |
index--; // undo +1 in StackValueCollection::long_at |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
250 |
} |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
251 |
#endif |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
252 |
oop obj = create_primitive_slot_instance(values, index, type, CHECK_(empty)); |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
253 |
if (obj != NULL) { |
34253 | 254 |
array_h->obj_at_put(i, obj); |
43677
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
255 |
} |
34253 | 256 |
} |
257 |
return array_h; |
|
258 |
} |
|
259 |
||
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
260 |
objArrayHandle LiveFrameStream::monitors_to_object_array(GrowableArray<MonitorInfo*>* monitors, TRAPS) { |
34253 | 261 |
int length = monitors->length(); |
262 |
objArrayOop array_oop = oopFactory::new_objArray(SystemDictionary::Object_klass(), |
|
263 |
length, CHECK_(objArrayHandle())); |
|
264 |
objArrayHandle array_h(THREAD, array_oop); |
|
265 |
for (int i = 0; i < length; i++) { |
|
266 |
MonitorInfo* monitor = monitors->at(i); |
|
267 |
array_h->obj_at_put(i, monitor->owner()); |
|
268 |
} |
|
269 |
return array_h; |
|
270 |
} |
|
271 |
||
272 |
// Fill StackFrameInfo with declaringClass and bci and initialize memberName |
|
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
273 |
void BaseFrameStream::fill_stackframe(Handle stackFrame, const methodHandle& method) { |
34253 | 274 |
java_lang_StackFrameInfo::set_declaringClass(stackFrame(), method->method_holder()->java_mirror()); |
46271
979ebd346ecf
8169881: Remove implicit Handle conversions oop->Handle
coleenp
parents:
41664
diff
changeset
|
275 |
java_lang_StackFrameInfo::set_method_and_bci(stackFrame, method, bci()); |
34253 | 276 |
} |
277 |
||
278 |
// Fill LiveStackFrameInfo with locals, monitors, and expressions |
|
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
279 |
void LiveFrameStream::fill_live_stackframe(Handle stackFrame, |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
280 |
const methodHandle& method, TRAPS) { |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
281 |
fill_stackframe(stackFrame, method); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
282 |
if (_jvf != NULL) { |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
283 |
StackValueCollection* locals = _jvf->locals(); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
284 |
StackValueCollection* expressions = _jvf->expressions(); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
285 |
GrowableArray<MonitorInfo*>* monitors = _jvf->monitors(); |
34253 | 286 |
|
43677
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
287 |
int mode = 0; |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
288 |
if (_jvf->is_interpreted_frame()) { |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
289 |
mode = MODE_INTERPRETED; |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
290 |
} else if (_jvf->is_compiled_frame()) { |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
291 |
mode = MODE_COMPILED; |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
292 |
} |
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
293 |
|
34253 | 294 |
if (!locals->is_empty()) { |
295 |
objArrayHandle locals_h = values_to_object_array(locals, CHECK); |
|
296 |
java_lang_LiveStackFrameInfo::set_locals(stackFrame(), locals_h()); |
|
297 |
} |
|
298 |
if (!expressions->is_empty()) { |
|
299 |
objArrayHandle expressions_h = values_to_object_array(expressions, CHECK); |
|
300 |
java_lang_LiveStackFrameInfo::set_operands(stackFrame(), expressions_h()); |
|
301 |
} |
|
302 |
if (monitors->length() > 0) { |
|
303 |
objArrayHandle monitors_h = monitors_to_object_array(monitors, CHECK); |
|
304 |
java_lang_LiveStackFrameInfo::set_monitors(stackFrame(), monitors_h()); |
|
305 |
} |
|
43677
5228814c1da2
8156073: 2-slot LiveStackFrame locals (long and double) are incorrect
bchristi
parents:
41664
diff
changeset
|
306 |
java_lang_LiveStackFrameInfo::set_mode(stackFrame(), mode); |
34253 | 307 |
} |
308 |
} |
|
309 |
||
310 |
// Begins stack walking. |
|
311 |
// |
|
312 |
// Parameters: |
|
313 |
// stackStream StackStream object |
|
314 |
// mode Stack walking mode. |
|
315 |
// skip_frames Number of frames to be skipped. |
|
316 |
// frame_count Number of frames to be traversed. |
|
317 |
// start_index Start index to the user-supplied buffers. |
|
318 |
// frames_array Buffer to store StackFrame in, starting at start_index. |
|
37438 | 319 |
// frames array is a Class<?>[] array when only getting caller |
320 |
// reference, and a StackFrameInfo[] array (or derivative) |
|
321 |
// otherwise. It should never be null. |
|
34253 | 322 |
// |
323 |
// Returns Object returned from AbstractStackWalker::doStackWalk call. |
|
324 |
// |
|
325 |
oop StackWalk::walk(Handle stackStream, jlong mode, |
|
326 |
int skip_frames, int frame_count, int start_index, |
|
327 |
objArrayHandle frames_array, |
|
328 |
TRAPS) { |
|
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
329 |
ResourceMark rm(THREAD); |
34253 | 330 |
JavaThread* jt = (JavaThread*)THREAD; |
41664
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
331 |
log_debug(stackwalk)("Start walking: mode " JLONG_FORMAT " skip %d frames batch size %d", |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
332 |
mode, skip_frames, frame_count); |
34253 | 333 |
|
37438 | 334 |
if (frames_array.is_null()) { |
335 |
THROW_MSG_(vmSymbols::java_lang_NullPointerException(), "frames_array is NULL", NULL); |
|
34253 | 336 |
} |
337 |
||
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
338 |
// Setup traversal onto my stack. |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
339 |
if (live_frame_info(mode)) { |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
340 |
assert (use_frames_array(mode), "Bad mode for get live frame"); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
341 |
RegisterMap regMap(jt, true); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
342 |
LiveFrameStream stream(jt, ®Map); |
41307
a89905798992
8166501: compilation error in stackwalk.cpp on some gccs
bchristi
parents:
41280
diff
changeset
|
343 |
return fetchFirstBatch(stream, stackStream, mode, skip_frames, frame_count, |
a89905798992
8166501: compilation error in stackwalk.cpp on some gccs
bchristi
parents:
41280
diff
changeset
|
344 |
start_index, frames_array, THREAD); |
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
345 |
} else { |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
346 |
JavaFrameStream stream(jt, mode); |
41307
a89905798992
8166501: compilation error in stackwalk.cpp on some gccs
bchristi
parents:
41280
diff
changeset
|
347 |
return fetchFirstBatch(stream, stackStream, mode, skip_frames, frame_count, |
a89905798992
8166501: compilation error in stackwalk.cpp on some gccs
bchristi
parents:
41280
diff
changeset
|
348 |
start_index, frames_array, THREAD); |
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
349 |
} |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
350 |
} |
34253 | 351 |
|
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
352 |
oop StackWalk::fetchFirstBatch(BaseFrameStream& stream, Handle stackStream, |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
353 |
jlong mode, int skip_frames, int frame_count, |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
354 |
int start_index, objArrayHandle frames_array, TRAPS) { |
34253 | 355 |
methodHandle m_doStackWalk(THREAD, Universe::do_stack_walk_method()); |
356 |
||
357 |
{ |
|
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
358 |
Klass* stackWalker_klass = SystemDictionary::StackWalker_klass(); |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
359 |
Klass* abstractStackWalker_klass = SystemDictionary::AbstractStackWalker_klass(); |
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
360 |
while (!stream.at_end()) { |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
361 |
InstanceKlass* ik = stream.method()->method_holder(); |
37438 | 362 |
if (ik != stackWalker_klass && |
363 |
ik != abstractStackWalker_klass && ik->super() != abstractStackWalker_klass) { |
|
364 |
break; |
|
34253 | 365 |
} |
366 |
||
41664
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
367 |
if (log_is_enabled(Debug, stackwalk)) { |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
368 |
ResourceMark rm(THREAD); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
369 |
outputStream* st = Log(stackwalk)::debug_stream(); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
370 |
st->print(" skip "); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
371 |
stream.method()->print_short_name(st); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
372 |
st->cr(); |
34253 | 373 |
} |
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
374 |
stream.next(); |
34253 | 375 |
} |
376 |
||
377 |
// stack frame has been traversed individually and resume stack walk |
|
378 |
// from the stack frame at depth == skip_frames. |
|
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
379 |
for (int n=0; n < skip_frames && !stream.at_end(); stream.next(), n++) { |
41664
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
380 |
if (log_is_enabled(Debug, stackwalk)) { |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
381 |
ResourceMark rm(THREAD); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
382 |
outputStream* st = Log(stackwalk)::debug_stream(); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
383 |
st->print(" skip "); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
384 |
stream.method()->print_short_name(st); |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
385 |
st->cr(); |
34253 | 386 |
} |
387 |
} |
|
388 |
} |
|
389 |
||
390 |
int end_index = start_index; |
|
391 |
int numFrames = 0; |
|
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
392 |
if (!stream.at_end()) { |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
393 |
numFrames = fill_in_frames(mode, stream, frame_count, start_index, |
34253 | 394 |
frames_array, end_index, CHECK_NULL); |
395 |
if (numFrames < 1) { |
|
396 |
THROW_MSG_(vmSymbols::java_lang_InternalError(), "stack walk: decode failed", NULL); |
|
397 |
} |
|
398 |
} |
|
399 |
||
400 |
// JVM_CallStackWalk walks the stack and fills in stack frames, then calls to |
|
401 |
// Java method java.lang.StackStreamFactory.AbstractStackWalker::doStackWalk |
|
402 |
// which calls the implementation to consume the stack frames. |
|
403 |
// When JVM_CallStackWalk returns, it invalidates the stack stream. |
|
404 |
JavaValue result(T_OBJECT); |
|
405 |
JavaCallArguments args(stackStream); |
|
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
406 |
args.push_long(stream.address_value()); |
34253 | 407 |
args.push_int(skip_frames); |
408 |
args.push_int(frame_count); |
|
409 |
args.push_int(start_index); |
|
410 |
args.push_int(end_index); |
|
411 |
||
412 |
// Link the thread and vframe stream into the callee-visible object |
|
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
413 |
stream.setup_magic_on_entry(frames_array); |
34253 | 414 |
|
415 |
JavaCalls::call(&result, m_doStackWalk, &args, THREAD); |
|
416 |
||
417 |
// Do this before anything else happens, to disable any lingering stream objects |
|
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
418 |
bool ok = stream.cleanup_magic_on_exit(frames_array); |
34253 | 419 |
|
420 |
// Throw pending exception if we must |
|
421 |
(void) (CHECK_NULL); |
|
422 |
||
423 |
if (!ok) { |
|
424 |
THROW_MSG_(vmSymbols::java_lang_InternalError(), "doStackWalk: corrupted buffers on exit", NULL); |
|
425 |
} |
|
426 |
||
427 |
// Return normally |
|
428 |
return (oop)result.get_jobject(); |
|
429 |
} |
|
430 |
||
431 |
// Walk the next batch of stack frames |
|
432 |
// |
|
433 |
// Parameters: |
|
434 |
// stackStream StackStream object |
|
435 |
// mode Stack walking mode. |
|
436 |
// magic Must be valid value to continue the stack walk |
|
437 |
// frame_count Number of frames to be decoded. |
|
438 |
// start_index Start index to the user-supplied buffers. |
|
439 |
// frames_array Buffer to store StackFrame in, starting at start_index. |
|
440 |
// |
|
441 |
// Returns the end index of frame filled in the buffer. |
|
442 |
// |
|
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
443 |
jint StackWalk::fetchNextBatch(Handle stackStream, jlong mode, jlong magic, |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
444 |
int frame_count, int start_index, |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
445 |
objArrayHandle frames_array, |
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
446 |
TRAPS) |
34253 | 447 |
{ |
448 |
JavaThread* jt = (JavaThread*)THREAD; |
|
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
449 |
BaseFrameStream* existing_stream = BaseFrameStream::from_current(jt, magic, frames_array); |
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
450 |
if (existing_stream == NULL) { |
34253 | 451 |
THROW_MSG_(vmSymbols::java_lang_InternalError(), "doStackWalk: corrupted buffers", 0L); |
452 |
} |
|
453 |
||
37438 | 454 |
if (frames_array.is_null()) { |
34253 | 455 |
THROW_MSG_(vmSymbols::java_lang_NullPointerException(), "frames_array is NULL", 0L); |
456 |
} |
|
457 |
||
41664
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
458 |
log_debug(stackwalk)("StackWalk::fetchNextBatch frame_count %d existing_stream " |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
459 |
PTR_FORMAT " start %d frames %d", |
07e88a4b405f
8160064: StackWalker implementation added logging option without using UL
rprotacio
parents:
41307
diff
changeset
|
460 |
frame_count, p2i(existing_stream), start_index, frames_array->length()); |
34253 | 461 |
int end_index = start_index; |
462 |
if (frame_count <= 0) { |
|
463 |
return end_index; // No operation. |
|
464 |
} |
|
465 |
||
466 |
int count = frame_count + start_index; |
|
37438 | 467 |
assert (frames_array->length() >= count, "not enough space in buffers"); |
34253 | 468 |
|
41280
74b314ad1e9a
8165372: StackWalker performance regression following JDK-8147039
bchristi
parents:
41095
diff
changeset
|
469 |
BaseFrameStream& stream = (*existing_stream); |
38213
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
470 |
if (!stream.at_end()) { |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
471 |
stream.next(); // advance past the last frame decoded in previous batch |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
472 |
if (!stream.at_end()) { |
2f32787e31b1
8147039: Incorrect locals and operands in compiled frames
bchristi
parents:
37438
diff
changeset
|
473 |
int n = fill_in_frames(mode, stream, frame_count, start_index, |
34253 | 474 |
frames_array, end_index, CHECK_0); |
475 |
if (n < 1) { |
|
476 |
THROW_MSG_(vmSymbols::java_lang_InternalError(), "doStackWalk: later decode failed", 0L); |
|
477 |
} |
|
478 |
return end_index; |
|
479 |
} |
|
480 |
} |
|
481 |
return end_index; |
|
482 |
} |