From f356a436f04c20db31d307e2e31e4dab21d0543b Mon Sep 17 00:00:00 2001 From: Keith Seitz Date: Sat, 28 Oct 2006 02:15:12 +0000 Subject: [PATCH] java-stack.h (ncodeMap): Declare. * include/java-stack.h (ncodeMap): Declare. (_Jv_StackTrace): Make _Jv_GetMethodDeclaringClass friend. * java/lang/Class.h (_Jv_GetMethodDeclaringClass): Declare. * java/lang/natClass.cc (_Jv_GetMethodDeclaringClass): New function. * stacktrace.cc (ncodeMap): Redefine from file global to global for class _Jv_StackTrace. (_Jv_StackTrace::UpdateNCodeMap): Add interpreted classes, too, so that _Jv_GetMethodDeclaringClass can find them all. (_Jv_StackTrace::ClassForFrame): Exclude interpreted classes. * jvmti.cc (_Jv_JVMTI_GetMethodDeclaringClass): New function. (_Jv_JVMTI_Interface): Define GetMethodDeclaringClass function. From-SVN: r118100 --- libjava/ChangeLog | 15 +++++++++++++ libjava/include/java-stack.h | 5 ++++- libjava/java/lang/Class.h | 4 ++++ libjava/java/lang/natClass.cc | 9 ++++++++ libjava/jvmti.cc | 20 +++++++++++++++++- libjava/stacktrace.cc | 40 +++++++++++++++++++---------------- 6 files changed, 73 insertions(+), 20 deletions(-) diff --git a/libjava/ChangeLog b/libjava/ChangeLog index e18d7caf2a0..34c044bdfcb 100644 --- a/libjava/ChangeLog +++ b/libjava/ChangeLog @@ -1,3 +1,18 @@ +2006-10-27 Keith Seitz + + * include/java-stack.h (ncodeMap): Declare. + (_Jv_StackTrace): Make _Jv_GetMethodDeclaringClass friend. + * java/lang/Class.h (_Jv_GetMethodDeclaringClass): Declare. + * java/lang/natClass.cc (_Jv_GetMethodDeclaringClass): New + function. + * stacktrace.cc (ncodeMap): Redefine from file global to global + for class _Jv_StackTrace. + (_Jv_StackTrace::UpdateNCodeMap): Add interpreted classes, too, + so that _Jv_GetMethodDeclaringClass can find them all. + (_Jv_StackTrace::ClassForFrame): Exclude interpreted classes. + * jvmti.cc (_Jv_JVMTI_GetMethodDeclaringClass): New function. + (_Jv_JVMTI_Interface): Define GetMethodDeclaringClass function. + 2006-10-25 Andreas Tobler * testsuite/libjava.jvmti/natevents.cc (do_callback_arg_tests): Replace diff --git a/libjava/include/java-stack.h b/libjava/include/java-stack.h index f7209512b2c..b9cf09a551c 100644 --- a/libjava/include/java-stack.h +++ b/libjava/include/java-stack.h @@ -23,6 +23,7 @@ details. */ #include #include #include +#include #include @@ -102,6 +103,7 @@ private: int length; _Jv_StackFrame frames[]; + static java::util::IdentityHashMap *ncodeMap; static void UpdateNCodeMap (); static jclass ClassForFrame (_Jv_StackFrame *frame); static void FillInFrameInfo (_Jv_StackFrame *frame); @@ -126,7 +128,8 @@ public: static JArray *GetClassContext (jclass checkClass); static ClassLoader *GetFirstNonSystemClassLoader (void); static jobjectArray GetAccessControlStack (); - + + friend jclass _Jv_GetMethodDeclaringClass (jmethodID); }; // Information about a given address. diff --git a/libjava/java/lang/Class.h b/libjava/java/lang/Class.h index a884cd6c59b..82a66fb814b 100644 --- a/libjava/java/lang/Class.h +++ b/libjava/java/lang/Class.h @@ -290,6 +290,10 @@ class java::io::VMObjectStreamClass; void _Jv_sharedlib_register_hook (jclass klass); +/* Find the class that defines the given method. Returns NULL + if it cannot be found. Searches both interpreted and native + classes. */ +jclass _Jv_GetMethodDeclaringClass (jmethodID method); class java::lang::Class : public java::lang::Object { diff --git a/libjava/java/lang/natClass.cc b/libjava/java/lang/natClass.cc index 12984291ee1..491faa7989a 100644 --- a/libjava/java/lang/natClass.cc +++ b/libjava/java/lang/natClass.cc @@ -1267,3 +1267,12 @@ _Jv_GetClassNameUtf8 (jclass klass) { return klass->name; } + +jclass +_Jv_GetMethodDeclaringClass (jmethodID method) +{ + _Jv_StackTrace::UpdateNCodeMap (); + jobject obj = reinterpret_cast (method->ncode); + return reinterpret_cast (_Jv_StackTrace::ncodeMap->get (obj)); +} + diff --git a/libjava/jvmti.cc b/libjava/jvmti.cc index d1ad6e3fcb0..c1bdc788411 100644 --- a/libjava/jvmti.cc +++ b/libjava/jvmti.cc @@ -465,6 +465,24 @@ _Jv_JVMTI_IsMethodSynthetic (MAYBE_UNUSED jvmtiEnv *env, jmethodID method, return JVMTI_ERROR_NONE; } +static jvmtiError JNICALL +_Jv_JVMTI_GetMethodDeclaringClass (MAYBE_UNUSED jvmtiEnv *env, + jmethodID method, + jclass *declaring_class_ptr) +{ + REQUIRE_PHASE (env, JVMTI_PHASE_LIVE); + NULL_CHECK (declaring_class_ptr); + + jclass klass = _Jv_GetMethodDeclaringClass (method); + if (klass != NULL) + { + *declaring_class_ptr = klass; + return JVMTI_ERROR_NONE; + } + + return JVMTI_ERROR_INVALID_METHODID; +} + static jvmtiError JNICALL _Jv_JVMTI_GetClassLoaderClasses (MAYBE_UNUSED jvmtiEnv *env, jobject init_loader, @@ -1287,7 +1305,7 @@ struct _Jv_jvmtiEnv _Jv_JVMTI_Interface = _Jv_JVMTI_GetFieldModifiers, // GetFieldModifiers _Jv_JVMTI_IsFieldSynthetic, // IsFieldSynthetic UNIMPLEMENTED, // GetMethodName - UNIMPLEMENTED, // GetMethodDeclaringClass + _Jv_JVMTI_GetMethodDeclaringClass, // GetMethodDeclaringClass _Jv_JVMTI_GetMethodModifiers, // GetMethodModifers RESERVED, // reserved67 UNIMPLEMENTED, // GetMaxLocals diff --git a/libjava/stacktrace.cc b/libjava/stacktrace.cc index 7f967baccf0..77a28644641 100644 --- a/libjava/stacktrace.cc +++ b/libjava/stacktrace.cc @@ -23,7 +23,6 @@ details. */ #include #include #include -#include #include #include #include @@ -41,7 +40,7 @@ using namespace gnu::gcj::runtime; // NOTE: Currently this Map contradicts class GC for native classes. This map // (and the "new class stack") will need to use WeakReferences in order to // enable native class GC. -static java::util::IdentityHashMap *ncodeMap; +java::util::IdentityHashMap *_Jv_StackTrace::ncodeMap; // Check the "class stack" for any classes initialized since we were last // called, and add them to ncodeMap. @@ -56,21 +55,20 @@ _Jv_StackTrace::UpdateNCodeMap () jclass klass; while ((klass = _Jv_PopClass ())) - if (!_Jv_IsInterpretedClass (klass)) - { - //printf ("got %s\n", klass->name->data); - for (int i = 0; i < klass->method_count; i++) - { - _Jv_Method *method = &klass->methods[i]; - void *ncode = method->ncode; - // Add non-abstract methods to ncodeMap. - if (ncode) - { - ncode = UNWRAP_FUNCTION_DESCRIPTOR (ncode); - ncodeMap->put ((java::lang::Object *) ncode, klass); - } - } - } + { + //printf ("got %s\n", klass->name->data); + for (int i = 0; i < klass->method_count; i++) + { + _Jv_Method *method = &klass->methods[i]; + void *ncode = method->ncode; + // Add non-abstract methods to ncodeMap. + if (ncode) + { + ncode = UNWRAP_FUNCTION_DESCRIPTOR (ncode); + ncodeMap->put ((java::lang::Object *) ncode, klass); + } + } + } } // Given a native frame, return the class which this code belongs @@ -85,7 +83,13 @@ _Jv_StackTrace::ClassForFrame (_Jv_StackFrame *frame) // look it up in ncodeMap if (frame->start_ip) - klass = (jclass) ncodeMap->get ((jobject) frame->start_ip); + { + klass = (jclass) ncodeMap->get ((jobject) frame->start_ip); + + // Exclude interpreted classes + if (klass != NULL && _Jv_IsInterpretedClass (klass)) + klass = NULL; + } return klass; }