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