7009923: JSR 292: VM crash in JavaThread::last_frame
Summary: Handle stack overflow before the first frame is called, by printing out the called method and not walking the stack.
Reviewed-by: dholmes, phh, dsamersoff
--- a/hotspot/src/share/vm/classfile/javaClasses.cpp Sat Apr 23 00:33:38 2011 -0400
+++ b/hotspot/src/share/vm/classfile/javaClasses.cpp Tue Apr 26 14:04:43 2011 -0400
@@ -1357,7 +1357,7 @@
};
-void java_lang_Throwable::fill_in_stack_trace(Handle throwable, TRAPS) {
+void java_lang_Throwable::fill_in_stack_trace(Handle throwable, methodHandle method, TRAPS) {
if (!StackTraceInThrowable) return;
ResourceMark rm(THREAD);
@@ -1374,6 +1374,16 @@
JavaThread* thread = (JavaThread*)THREAD;
BacktraceBuilder bt(CHECK);
+ // If there is no Java frame just return the method that was being called
+ // with bci 0
+ if (!thread->has_last_Java_frame()) {
+ if (max_depth >= 1 && method() != NULL) {
+ bt.push(method(), 0, CHECK);
+ set_backtrace(throwable(), bt.backtrace());
+ }
+ return;
+ }
+
// Instead of using vframe directly, this version of fill_in_stack_trace
// basically handles everything by hand. This significantly improved the
// speed of this method call up to 28.5% on Solaris sparc. 27.1% on Windows.
@@ -1477,7 +1487,7 @@
set_backtrace(throwable(), bt.backtrace());
}
-void java_lang_Throwable::fill_in_stack_trace(Handle throwable) {
+void java_lang_Throwable::fill_in_stack_trace(Handle throwable, methodHandle method) {
// No-op if stack trace is disabled
if (!StackTraceInThrowable) {
return;
@@ -1491,7 +1501,7 @@
PRESERVE_EXCEPTION_MARK;
JavaThread* thread = JavaThread::active();
- fill_in_stack_trace(throwable, thread);
+ fill_in_stack_trace(throwable, method, thread);
// ignore exceptions thrown during stack trace filling
CLEAR_PENDING_EXCEPTION;
}
--- a/hotspot/src/share/vm/classfile/javaClasses.hpp Sat Apr 23 00:33:38 2011 -0400
+++ b/hotspot/src/share/vm/classfile/javaClasses.hpp Tue Apr 26 14:04:43 2011 -0400
@@ -440,8 +440,8 @@
static void fill_in_stack_trace_of_preallocated_backtrace(Handle throwable);
// Fill in current stack trace, can cause GC
- static void fill_in_stack_trace(Handle throwable, TRAPS);
- static void fill_in_stack_trace(Handle throwable);
+ static void fill_in_stack_trace(Handle throwable, methodHandle method, TRAPS);
+ static void fill_in_stack_trace(Handle throwable, methodHandle method = methodHandle());
// Programmatic access to stack trace
static oop get_stack_trace_element(oop throwable, int index, TRAPS);
static int get_stack_trace_depth(oop throwable, TRAPS);
--- a/hotspot/src/share/vm/runtime/javaCalls.cpp Sat Apr 23 00:33:38 2011 -0400
+++ b/hotspot/src/share/vm/runtime/javaCalls.cpp Tue Apr 26 14:04:43 2011 -0400
@@ -389,7 +389,7 @@
// to Java
if (!os::stack_shadow_pages_available(THREAD, method)) {
// Throw stack overflow exception with preinitialized exception.
- Exceptions::throw_stack_overflow_exception(THREAD, __FILE__, __LINE__);
+ Exceptions::throw_stack_overflow_exception(THREAD, __FILE__, __LINE__, method);
return;
} else {
// Touch pages checked if the OS needs them to be touched to be mapped.
--- a/hotspot/src/share/vm/utilities/exceptions.cpp Sat Apr 23 00:33:38 2011 -0400
+++ b/hotspot/src/share/vm/utilities/exceptions.cpp Tue Apr 26 14:04:43 2011 -0400
@@ -207,7 +207,7 @@
}
-void Exceptions::throw_stack_overflow_exception(Thread* THREAD, const char* file, int line) {
+void Exceptions::throw_stack_overflow_exception(Thread* THREAD, const char* file, int line, methodHandle method) {
Handle exception;
if (!THREAD->has_pending_exception()) {
klassOop k = SystemDictionary::StackOverflowError_klass();
@@ -215,13 +215,13 @@
exception = Handle(THREAD, e); // fill_in_stack trace does gc
assert(instanceKlass::cast(k)->is_initialized(), "need to increase min_stack_allowed calculation");
if (StackTraceInThrowable) {
- java_lang_Throwable::fill_in_stack_trace(exception);
+ java_lang_Throwable::fill_in_stack_trace(exception, method());
}
} else {
// if prior exception, throw that one instead
exception = Handle(THREAD, THREAD->pending_exception());
}
- _throw_oop(THREAD, file, line, exception());
+ _throw(THREAD, file, line, exception);
}
void Exceptions::fthrow(Thread* thread, const char* file, int line, Symbol* h_name, const char* format, ...) {
--- a/hotspot/src/share/vm/utilities/exceptions.hpp Sat Apr 23 00:33:38 2011 -0400
+++ b/hotspot/src/share/vm/utilities/exceptions.hpp Tue Apr 26 14:04:43 2011 -0400
@@ -144,7 +144,7 @@
const char* message,
ExceptionMsgToUtf8Mode to_utf8_safe = safe_to_utf8);
- static void throw_stack_overflow_exception(Thread* thread, const char* file, int line);
+ static void throw_stack_overflow_exception(Thread* thread, const char* file, int line, methodHandle method);
// for AbortVMOnException flag
NOT_PRODUCT(static void debug_check_abort(Handle exception, const char* message = NULL);)