[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 5/6] target/microblaze: Replace magic values by proper definition
From: |
Philippe Mathieu-Daudé |
Subject: |
[PATCH 5/6] target/microblaze: Replace magic values by proper definitions |
Date: |
Thu, 3 Jun 2021 11:03:09 +0200 |
Use the Exception Status Register definitions from "cpu.h".
Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
target/microblaze/helper.c | 16 ++++++++++++----
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/target/microblaze/helper.c b/target/microblaze/helper.c
index 20dbd673136..d537f300ca6 100644
--- a/target/microblaze/helper.c
+++ b/target/microblaze/helper.c
@@ -99,14 +99,22 @@ bool mb_cpu_tlb_fill(CPUState *cs, vaddr address, int size,
mmu_idx, address);
env->ear = address;
+
+ env->esr = (access_type == MMU_DATA_STORE) ? ESR_S : 0;
switch (lu.err) {
case ERR_PROT:
- env->esr = access_type == MMU_INST_FETCH ? 17 : 16;
- env->esr |= (access_type == MMU_DATA_STORE) << 10;
+ if (access_type == MMU_INST_FETCH) {
+ env->esr |= ESR_EC_INSN_STORAGE;
+ } else {
+ env->esr |= ESR_EC_DATA_STORAGE;
+ }
break;
case ERR_MISS:
- env->esr = access_type == MMU_INST_FETCH ? 19 : 18;
- env->esr |= (access_type == MMU_DATA_STORE) << 10;
+ if (access_type == MMU_INST_FETCH) {
+ env->esr |= ESR_EC_INSN_TLB;
+ } else {
+ env->esr |= ESR_EC_DATA_TLB;
+ }
break;
default:
abort();
--
2.26.3
- Re: [PATCH 1/6] target/microblaze: Use the IEC binary prefix definitions, (continued)
- [PATCH 2/6] target/microblaze: Extract FPU helpers to fpu_helper.c, Philippe Mathieu-Daudé, 2021/06/03
- [PATCH 3/6] target/microblaze: Assert transaction failures have exception enabled, Philippe Mathieu-Daudé, 2021/06/03
- [PATCH 4/6] target/microblaze: Fix Exception Status Register 'Cause' definitions, Philippe Mathieu-Daudé, 2021/06/03
- [PATCH 5/6] target/microblaze: Replace magic values by proper definitions,
Philippe Mathieu-Daudé <=
- [PATCH 6/6] target/microblaze: Set OPB bits in tlb_fill, not in transaction_failed, Philippe Mathieu-Daudé, 2021/06/03