diff options
author | Maciej W. Rozycki <macro@codesourcery.com> | 2014-11-07 20:05:35 +0000 |
---|---|---|
committer | Leon Alrae <leon.alrae@imgtec.com> | 2014-12-16 12:45:19 +0000 |
commit | c3577479815f5bcf9d38993967bca2115af245d8 (patch) | |
tree | 0aac6c1eae93ffde21d7a83d4db52cf4e727905e /translate-all.c | |
parent | 8fc605b8aa257feb3e69d44794a765bd492b573b (diff) | |
download | qemu-c3577479815f5bcf9d38993967bca2115af245d8.tar.gz qemu-c3577479815f5bcf9d38993967bca2115af245d8.tar.bz2 qemu-c3577479815f5bcf9d38993967bca2115af245d8.zip |
target-mips: Correct MIPS16/microMIPS branch size calculation
Correct MIPS16/microMIPS branch size calculation in PC adjustment
needed:
- to set the value of CP0.ErrorEPC at the entry to the reset exception,
- for the purpose of branch reexecution in the context of device I/O.
Follow the approach taken in `exception_resume_pc' for ordinary, Debug
and NMI exceptions.
MIPS16 and microMIPS branches can be 2 or 4 bytes in size and that has
to be reflected in calculation. Original MIPS ISA branches, which is
where this code originates from, are always 4 bytes long, just as all
original MIPS ISA instructions.
Signed-off-by: Nathan Froyd <froydnj@codesourcery.com>
Signed-off-by: Maciej W. Rozycki <macro@codesourcery.com>
Reviewed-by: Leon Alrae <leon.alrae@imgtec.com>
Signed-off-by: Leon Alrae <leon.alrae@imgtec.com>
Diffstat (limited to 'translate-all.c')
-rw-r--r-- | translate-all.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/translate-all.c b/translate-all.c index cf05472008..d930a5ce67 100644 --- a/translate-all.c +++ b/translate-all.c @@ -1540,7 +1540,7 @@ void cpu_io_recompile(CPUState *cpu, uintptr_t retaddr) branch. */ #if defined(TARGET_MIPS) if ((env->hflags & MIPS_HFLAG_BMASK) != 0 && n > 1) { - env->active_tc.PC -= 4; + env->active_tc.PC -= (env->hflags & MIPS_HFLAG_B16 ? 2 : 4); cpu->icount_decr.u16.low++; env->hflags &= ~MIPS_HFLAG_BMASK; } |