diff options
| author | BALATON Zoltan <balaton@eik.bme.hu> | 2024-05-13 01:27:49 +0200 |
|---|---|---|
| committer | Nicholas Piggin <npiggin@gmail.com> | 2024-05-24 09:43:06 +1000 |
| commit | f178e4f8949ec75d0e1e34f9b1ace646d1e6a031 (patch) | |
| tree | 633ba8e497b4019066d0a9b1e1d36d2618bcae0f /target/ppc | |
| parent | 9e9ca54cdb493721f8444030e6dcf680400c8d0b (diff) | |
| download | qemu-f178e4f8949ec75d0e1e34f9b1ace646d1e6a031.zip qemu-f178e4f8949ec75d0e1e34f9b1ace646d1e6a031.tar.gz qemu-f178e4f8949ec75d0e1e34f9b1ace646d1e6a031.tar.bz2 | |
target/ppc: Replace hard coded constants in ppc_jumbo_xlate()
The "2" in booke206_update_mas_tlb_miss() call corresponds to
MMU_INST_FETCH which is the value of access_type in this branch;
mmubooke206_esr() only checks for MMU_DATA_STORE and it's called from
code access so using MMU_DATA_LOAD here seems wrong so replace it with
access_type here as well that yields the same result. This also makes
these calls the same as the data access branch further down.
Reviewed-by: Nicholas Piggin <npiggin@gmail.com>
Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu>
Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
Diffstat (limited to 'target/ppc')
| -rw-r--r-- | target/ppc/mmu_common.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/target/ppc/mmu_common.c b/target/ppc/mmu_common.c index f40481b..6570b28 100644 --- a/target/ppc/mmu_common.c +++ b/target/ppc/mmu_common.c @@ -1287,13 +1287,13 @@ static bool ppc_jumbo_xlate(PowerPCCPU *cpu, vaddr eaddr, env->spr[SPR_40x_ESR] = 0x00000000; break; case POWERPC_MMU_BOOKE206: - booke206_update_mas_tlb_miss(env, eaddr, 2, mmu_idx); + booke206_update_mas_tlb_miss(env, eaddr, access_type, mmu_idx); /* fall through */ case POWERPC_MMU_BOOKE: cs->exception_index = POWERPC_EXCP_ITLB; env->error_code = 0; env->spr[SPR_BOOKE_DEAR] = eaddr; - env->spr[SPR_BOOKE_ESR] = mmubooke206_esr(mmu_idx, MMU_DATA_LOAD); + env->spr[SPR_BOOKE_ESR] = mmubooke206_esr(mmu_idx, access_type); break; case POWERPC_MMU_REAL: cpu_abort(cs, "PowerPC in real mode should never raise " |