target-mips: fix offset calculation for Interrupts

Correct computation of vector offsets for EXCP_EXT_INTERRUPT.
For instance, if Cause.IV is 0 the vector offset should be 0x180.

Simplify the finding vector number logic for the Vectored Interrupts.

Signed-off-by: Yongbok Kim <yongbok.kim@imgtec.com>
Reviewed-by: Leon Alrae <leon.alrae@imgtec.com>
[leon.alrae@imgtec.com: cosmetic changes]
Signed-off-by: Leon Alrae <leon.alrae@imgtec.com>
This commit is contained in:
Yongbok Kim 2015-07-10 12:10:02 +01:00 committed by Leon Alrae
parent 8bcbb834a0
commit da52a4dfcc
2 changed files with 19 additions and 25 deletions

View file

@ -565,34 +565,30 @@ void mips_cpu_do_interrupt(CPUState *cs)
break; break;
case EXCP_EXT_INTERRUPT: case EXCP_EXT_INTERRUPT:
cause = 0; cause = 0;
if (env->CP0_Cause & (1 << CP0Ca_IV)) if (env->CP0_Cause & (1 << CP0Ca_IV)) {
offset = 0x200; uint32_t spacing = (env->CP0_IntCtl >> CP0IntCtl_VS) & 0x1f;
if (env->CP0_Config3 & ((1 << CP0C3_VInt) | (1 << CP0C3_VEIC))) { if ((env->CP0_Status & (1 << CP0St_BEV)) || spacing == 0) {
/* Vectored Interrupts. */ offset = 0x200;
unsigned int spacing; } else {
unsigned int vector; uint32_t vector = 0;
unsigned int pending = (env->CP0_Cause & CP0Ca_IP_mask) >> 8; uint32_t pending = (env->CP0_Cause & CP0Ca_IP_mask) >> CP0Ca_IP;
pending &= env->CP0_Status >> 8; if (env->CP0_Config3 & (1 << CP0C3_VEIC)) {
/* Compute the Vector Spacing. */ /* For VEIC mode, the external interrupt controller feeds
spacing = (env->CP0_IntCtl >> CP0IntCtl_VS) & ((1 << 6) - 1); * the vector through the CP0Cause IP lines. */
spacing <<= 5; vector = pending;
} else {
if (env->CP0_Config3 & (1 << CP0C3_VInt)) { /* Vectored Interrupts
/* For VInt mode, the MIPS computes the vector internally. */ * Mask with Status.IM7-IM0 to get enabled interrupts. */
for (vector = 7; vector > 0; vector--) { pending &= (env->CP0_Status >> CP0St_IM) & 0xff;
if (pending & (1 << vector)) { /* Find the highest-priority interrupt. */
/* Found it. */ while (pending >>= 1) {
break; vector++;
} }
} }
} else { offset = 0x200 + (vector * (spacing << 5));
/* For VEIC mode, the external interrupt controller feeds the
vector through the CP0Cause IP lines. */
vector = pending;
} }
offset = 0x200 + vector * spacing;
} }
goto set_EPC; goto set_EPC;
case EXCP_LTLBL: case EXCP_LTLBL:

View file

@ -1432,7 +1432,6 @@ void helper_mttc0_status(CPUMIPSState *env, target_ulong arg1)
void helper_mtc0_intctl(CPUMIPSState *env, target_ulong arg1) void helper_mtc0_intctl(CPUMIPSState *env, target_ulong arg1)
{ {
/* vectored interrupts not implemented, no performance counters. */
env->CP0_IntCtl = (env->CP0_IntCtl & ~0x000003e0) | (arg1 & 0x000003e0); env->CP0_IntCtl = (env->CP0_IntCtl & ~0x000003e0) | (arg1 & 0x000003e0);
} }
@ -1473,7 +1472,6 @@ target_ulong helper_mftc0_ebase(CPUMIPSState *env)
void helper_mtc0_ebase(CPUMIPSState *env, target_ulong arg1) void helper_mtc0_ebase(CPUMIPSState *env, target_ulong arg1)
{ {
/* vectored interrupts not implemented */
env->CP0_EBase = (env->CP0_EBase & ~0x3FFFF000) | (arg1 & 0x3FFFF000); env->CP0_EBase = (env->CP0_EBase & ~0x3FFFF000) | (arg1 & 0x3FFFF000);
} }