diff options
author | Ben Cheng <bccheng@android.com> | 2011-10-27 14:04:12 -0700 |
---|---|---|
committer | Ben Cheng <bccheng@android.com> | 2011-10-27 17:35:28 -0700 |
commit | 0c0624bfad83503547b0226bd2dfac055782582a (patch) | |
tree | 84de5670bb42baa9f7190b9416c2f8f1d956eaad /vm/interp | |
parent | b4593f197532419746a0df6d93fb33f8b815affc (diff) | |
download | android_dalvik-0c0624bfad83503547b0226bd2dfac055782582a.tar.gz android_dalvik-0c0624bfad83503547b0226bd2dfac055782582a.tar.bz2 android_dalvik-0c0624bfad83503547b0226bd2dfac055782582a.zip |
Fix single-stepping resume mode in the interpreter.
When a resume attempt is cancelled due to other pending request make sure
the old native resume PC is cleared. Otherwise the JIT code cache may be
re-entered with mismatching Dalvik PC.
Also fix a code bloat problem where single-step count is not set properly
after executing a return instruction.
Change-Id: Ia926bc3a87cae1296719a1069838060b181ce54d
Diffstat (limited to 'vm/interp')
-rw-r--r-- | vm/interp/Interp.cpp | 3 | ||||
-rw-r--r-- | vm/interp/Jit.cpp | 7 |
2 files changed, 8 insertions, 2 deletions
diff --git a/vm/interp/Interp.cpp b/vm/interp/Interp.cpp index 85d06db13..f78b7a405 100644 --- a/vm/interp/Interp.cpp +++ b/vm/interp/Interp.cpp @@ -1839,6 +1839,9 @@ void dvmCheckBefore(const u2 *pc, u4 *fp, Thread* self) // Doesn't return dvmAbort(); } + // In case resume is blocked by non-zero breakFlags, clear + // jitResumeNPC here. + self->jitResumeNPC = NULL; self->jitResumeDPC = NULL; self->inJitCodeCache = NULL; #endif diff --git a/vm/interp/Jit.cpp b/vm/interp/Jit.cpp index d4c5a796e..494aae1a9 100644 --- a/vm/interp/Jit.cpp +++ b/vm/interp/Jit.cpp @@ -976,9 +976,12 @@ void dvmCheckJit(const u2* pc, Thread* self) if (allDone) { dvmDisableSubMode(self, kSubModeJitTraceBuild); if (stayOneMoreInst) { + // Clear jitResumeNPC explicitly since we know we don't need it + // here. + self->jitResumeNPC = NULL; // Keep going in single-step mode for at least one more inst - assert(self->jitResumeNPC == NULL); - self->singleStepCount = MIN(1, self->singleStepCount); + if (self->singleStepCount == 0) + self->singleStepCount = 1; dvmEnableSubMode(self, kSubModeCountedStep); } } |