Revert "Move Trace to use trampolines"
This reverts commit 19af3b9b7fda03417a724fb7e465a3457d528eab.
Caused the following tests to fail.
test-art-host-run-test-debug-prebuild-interpreter-no-relocate-trace-cms-checkjni-picimage-pictest-ndebuggable-no-jvmti-cdex-fast-916-obsolete-jit32
test-art-host-run-test-debug-prebuild-interpreter-no-relocate-trace-cms-checkjni-picimage-pictest-ndebuggable-no-jvmti-cdex-fast-916-obsolete-jit64
It seems we did not setup the tracing fully correctly.
Reason for revert: Caused failure of 2 tests on buildbots
Test: None
Change-Id: Ied420b95c24ce530a19cdd39dc2403e5cc99fde5
diff --git a/runtime/trace.cc b/runtime/trace.cc
index 1986eec..292cac6 100644
--- a/runtime/trace.cc
+++ b/runtime/trace.cc
@@ -433,12 +433,7 @@
instrumentation::Instrumentation::kMethodExited |
instrumentation::Instrumentation::kMethodUnwind);
// TODO: In full-PIC mode, we don't need to fully deopt.
- // TODO: We can only use trampoline entrypoints if we are java-debuggable since in that case
- // we know that inlining and other problematic optimizations are disabled. We might just
- // want to use the trampolines anyway since it is faster. It makes the story with disabling
- // jit-gc more complex though.
- runtime->GetInstrumentation()->EnableMethodTracing(
- kTracerInstrumentationKey, /*needs_interpreter*/!runtime->IsJavaDebuggable());
+ runtime->GetInstrumentation()->EnableMethodTracing(kTracerInstrumentationKey);
}
}
}
diff --git a/test/knownfailures.json b/test/knownfailures.json
index 2b7a9b0..9ba2b50 100644
--- a/test/knownfailures.json
+++ b/test/knownfailures.json
@@ -193,22 +193,6 @@
"bug": "http://b/34369284"
},
{
- "tests": ["018-stack-overflow",
- "107-int-math2",
- "667-jit-jni-stub"],
- "description": ["We run for long enough for jit to compile some of the methods ",
- "which means it takes so long to finish the test that it will ",
- "timeout. The timeout is due to having to walk the stack ",
- "when entering and exiting methods due to the way the instrumentation ",
- "trampoline is implemented."],
- "variant": "debuggable & jit & trace | debuggable & jit & stream"
- },
- {
- "tests": "1935-get-set-current-frame-jit",
- "description": ["Test expects that OSR works but tracing turns this feature off."],
- "variant": "trace | stream"
- },
- {
"tests": "1940-ddms-ext",
"description": ["Test expects to be able to start tracing but we cannot",
"do that if tracing is already ongoing."],
@@ -217,7 +201,7 @@
{
"tests": "137-cfi",
"description": ["This test unrolls and expects managed frames, but",
- "tracing means we run the interpreter or trampolines."],
+ "tracing means we run the interpreter."],
"variant": "trace | stream"
},
{