[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH 3/3] target/arm: Merge ISS for data abort from tag check fail
From: |
Richard Henderson |
Subject: |
[PATCH 3/3] target/arm: Merge ISS for data abort from tag check fail |
Date: |
Wed, 12 Aug 2020 10:19:46 -0700 |
Merge the ISS data saved by the translator for the originating insn.
Use merge_syn_data_abort so that we properly take the target EL into
account. Set the WnR bit properly in all cases.
Reported-by: Andrey Konovalov <andreyknvl@google.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
---
target/arm/mte_helper.c | 14 +++++++++-----
1 file changed, 9 insertions(+), 5 deletions(-)
diff --git a/target/arm/mte_helper.c b/target/arm/mte_helper.c
index a40454588d..2dff4c548d 100644
--- a/target/arm/mte_helper.c
+++ b/target/arm/mte_helper.c
@@ -519,7 +519,7 @@ static void mte_check_fail(CPUARMState *env, uint32_t desc,
{
int mmu_idx = FIELD_EX32(desc, MTEDESC, MIDX);
ARMMMUIdx arm_mmu_idx = core_to_aa64_mmu_idx(mmu_idx);
- int el, reg_el, tcf, select;
+ int el, target_el, reg_el, tcf, select, is_write, syn;
uint64_t sctlr;
reg_el = regime_el(env, arm_mmu_idx);
@@ -543,13 +543,17 @@ static void mte_check_fail(CPUARMState *env, uint32_t
desc,
*
* In restore_state_to_opc, we set the exception syndrome
* for the load or store operation. Unwind first so we
- * may overwrite that with the syndrome for the tag check.
+ * may merge that with the syndrome for the tag check.
*/
cpu_restore_state(env_cpu(env), ra, true);
env->exception.vaddress = dirty_ptr;
- raise_exception(env, EXCP_DATA_ABORT,
- syn_data_abort_no_iss(el != 0, 0, 0, 0, 0, 0, 0x11),
- exception_target_el(env));
+
+ target_el = exception_target_el(env);
+ is_write = FIELD_EX32(desc, MTEDESC, WRITE);
+ syn = merge_syn_data_abort(env->exception.syndrome, target_el,
+ target_el == el, 0, 0, is_write, 0x11);
+
+ raise_exception(env, EXCP_DATA_ABORT, syn, target_el);
/* noreturn, but fall through to the assert anyway */
case 0:
--
2.25.1
- [PATCH 0/3] target/arm: Complete ISS for MTE tag check fail, Richard Henderson, 2020/08/12
- [PATCH 1/3] target/arm: Export merge_syn_data_abort from tlb_helper.c, Richard Henderson, 2020/08/12
- [PATCH 2/3] target/arm: Pass the entire mte descriptor to mte_check_fail, Richard Henderson, 2020/08/12
- [PATCH 3/3] target/arm: Merge ISS for data abort from tag check fail,
Richard Henderson <=
- Re: [PATCH 0/3] target/arm: Complete ISS for MTE tag check fail, Andrey Konovalov, 2020/08/12
- Re: [PATCH 0/3] target/arm: Complete ISS for MTE tag check fail, Richard Henderson, 2020/08/12
- Re: [PATCH 0/3] target/arm: Complete ISS for MTE tag check fail, Richard Henderson, 2020/08/12
- Re: [PATCH 0/3] target/arm: Complete ISS for MTE tag check fail, Andrey Konovalov, 2020/08/12
- Re: [PATCH 0/3] target/arm: Complete ISS for MTE tag check fail, Evgenii Stepanov, 2020/08/12
- Re: [PATCH 0/3] target/arm: Complete ISS for MTE tag check fail, Kevin Brodsky, 2020/08/13
- Re: [PATCH 0/3] target/arm: Complete ISS for MTE tag check fail, Andrey Konovalov, 2020/08/13