Harmony
  1. Harmony
  2. HARMONY-1668

[drlvm] tests.api.java.lang.reflect.InvocationTargetExceptionTest fails on interpreter

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: DRLVM
    • Labels:
      None
    • Environment:
      interpreter gcc/debug DRLVM

      Description

      The tests.api.java.lang.reflect.InvocationTargetExceptionTest fails on interpreter:
      java: /nfs/ins/proj/drl/coreapi/avarlamo/harmony/linux.ia32/svn-repo/drlvm/vm/interpreter/src/interpreter.cpp:2531: void interpreter(StackFrame&): Assertion `!check_current_thread_exception()' failed.
      SIGABRT in VM code.

                • STACK DUMP: ************
                  java/lang/Integer.<init>(I)V (Integer.java:94)
                  java/lang/reflect/VMReflection.invokeMethod(JLjava/lang/Object;[Ljava/lang/Object;)Ljava/lang/Object; (NULL:-2)
                  java/lang/reflect/Method.invoke(Ljava/lang/Object;[Ljava/lang/Object;)Ljava/lang/Object; (NULL:-1)
                  tests/api/java/lang/reflect/InvocationTargetExceptionTest.test_ConstructorLjava_lang_Throwable()V (InvocationTargetExceptionTest.java:128)
                  java/lang/reflect/VMReflection.invokeMethod(JLjava/lang/Object;[Ljava/lang/Object;)Ljava/lang/Object; (NULL:-2)
                  java/lang/reflect/Method.invoke(Ljava/lang/Object;[Ljava/lang/Object;)Ljava/lang/Object; (NULL:-1)
                  junit/framework/TestCase.runTest()V (TestCase.java:164)
                  junit/framework/TestCase.runBare()V (TestCase.java:132)
                  junit/framework/TestResult$1.protect()V (TestResult.java:107)
                  junit/framework/TestResult.runProtected(Ljunit/framework/Test;Ljunit/framework/Protectable;)V (TestResult.java:135)
                  junit/framework/TestResult.run(Ljunit/framework/TestCase;)V (TestResult.java:112)
                  junit/framework/TestCase.run(Ljunit/framework/TestResult;)V (TestCase.java:119)
                  junit/framework/TestSuite.runTest(Ljunit/framework/Test;Ljunit/framework/TestResult;)V (TestSuite.java:209)
                  junit/framework/TestSuite.run(Ljunit/framework/TestResult;)V (TestSuite.java:205)
                  junit/textui/TestRunner.doRun(Ljunit/framework/Test;Z)Ljunit/framework/TestResult; (TestRunner.java:122)
                  junit/textui/TestRunner.start([Ljava/lang/String;)Ljunit/framework/TestResult; (TestRunner.java:175)
                  junit/textui/TestRunner.main([Ljava/lang/String;)V (TestRunner.java:146)

        Issue Links

          Activity

          Alexey Varlamov created issue -
          Hide
          Alexey Varlamov added a comment -

          Fixed.

          Show
          Alexey Varlamov added a comment - Fixed.
          Alexey Varlamov made changes -
          Field Original Value New Value
          Attachment H-1668.patch [ 12342176 ]
          Hide
          Alexei Fedotov added a comment -
          Show
          Alexei Fedotov added a comment - [drlvm] [unit] Blocks http://wiki.apache.org/harmony/Unit_Tests_Pass_on_DRLVM
          made changes -
          Hide
          Pavel Afremov added a comment -

          Interpreter and JIT mismatch in exception status assertion.

          Java Methods must not be launched in the "exception raised" state. But DRL VM asserts it now in the interpreter mode only. To fix it assert(!exn_raised()) was added into vm_execute_java_method_array.

          Show
          Pavel Afremov added a comment - Interpreter and JIT mismatch in exception status assertion. Java Methods must not be launched in the "exception raised" state. But DRL VM asserts it now in the interpreter mode only. To fix it assert(!exn_raised()) was added into vm_execute_java_method_array.
          Pavel Afremov made changes -
          Serguei Zapreyev made changes -
          Link This issue blocks HARMONY-1834 [ HARMONY-1834 ]
          Hide
          Alexey Varlamov added a comment -

          Both patches should be applied

          Show
          Alexey Varlamov added a comment - Both patches should be applied
          Geir Magnusson Jr made changes -
          Assignee Geir Magnusson Jr [ geir ]
          Geir Magnusson Jr made changes -
          Status Open [ 1 ] In Progress [ 3 ]
          Hide
          Geir Magnusson Jr added a comment -

          r464670

          Both patches applied.

          Passes on Ubuntu 6 - testcase, smoke, c-unit, ~kernel

          Show
          Geir Magnusson Jr added a comment - r464670 Both patches applied. Passes on Ubuntu 6 - testcase, smoke, c-unit, ~kernel
          Geir Magnusson Jr made changes -
          Resolution Fixed [ 1 ]
          Status In Progress [ 3 ] Resolved [ 5 ]
          Geir Magnusson Jr made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          Gavin made changes -
          Link This issue blocks HARMONY-1834 [ HARMONY-1834 ]
          Gavin made changes -
          Link This issue is depended upon by HARMONY-1834 [ HARMONY-1834 ]

            People

            • Assignee:
              Geir Magnusson Jr
              Reporter:
              Alexey Varlamov
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development