[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 33/40] target/arm: Fix exception case in allocation_tag_mem_probe
From: |
Alex Bennée |
Subject: |
[PULL 33/40] target/arm: Fix exception case in allocation_tag_mem_probe |
Date: |
Fri, 5 Jul 2024 16:30:45 +0100 |
From: Gustavo Romero <gustavo.romero@linaro.org>
If page in 'ptr_access' is inaccessible and probe is 'true'
allocation_tag_mem_probe should not throw an exception, but currently it
does, so fix it.
Signed-off-by: Gustavo Romero <gustavo.romero@linaro.org>
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-Id: <20240628050850.536447-5-gustavo.romero@linaro.org>
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
Message-Id: <20240705084047.857176-34-alex.bennee@linaro.org>
diff --git a/target/arm/tcg/mte_helper.c b/target/arm/tcg/mte_helper.c
index 037ac6dd60..a50d576294 100644
--- a/target/arm/tcg/mte_helper.c
+++ b/target/arm/tcg/mte_helper.c
@@ -96,6 +96,9 @@ static uint8_t *allocation_tag_mem_probe(CPUARMState *env,
int ptr_mmu_idx,
assert(!(probe && ra));
if (!(flags & (ptr_access == MMU_DATA_STORE ? PAGE_WRITE_ORG :
PAGE_READ))) {
+ if (probe) {
+ return NULL;
+ }
cpu_loop_exit_sigsegv(env_cpu(env), ptr, ptr_access,
!(flags & PAGE_VALID), ra);
}
--
2.39.2
- [PULL 25/40] plugins/lockstep: mention the one-insn-per-tb option, (continued)
- [PULL 25/40] plugins/lockstep: mention the one-insn-per-tb option, Alex Bennée, 2024/07/05
- [PULL 24/40] plugins/lockstep: make mixed-mode safe, Alex Bennée, 2024/07/05
- [PULL 23/40] plugins/lockstep: preserve sock_path, Alex Bennée, 2024/07/05
- [PULL 36/40] gdbstub: Make hex conversion function non-internal, Alex Bennée, 2024/07/05
- [PULL 39/40] gdbstub: Add support for MTE in user mode, Alex Bennée, 2024/07/05
- [PULL 18/40] tests/tcg/arm: Use vmrs/vmsr instead of mcr/mrc, Alex Bennée, 2024/07/05
- [PULL 19/40] linux-user/main: Suppress out-of-range comparison warning for clang, Alex Bennée, 2024/07/05
- [PULL 28/40] plugins: Free CPUPluginState before destroying vCPU state, Alex Bennée, 2024/07/05
- [PULL 38/40] gdbstub: Use true to set cmd_startswith, Alex Bennée, 2024/07/05
- [PULL 33/40] target/arm: Fix exception case in allocation_tag_mem_probe,
Alex Bennée <=
- [PULL 30/40] gdbstub: Clean up process_string_cmd, Alex Bennée, 2024/07/05
- [PULL 26/40] plugins/lockstep: clean-up output, Alex Bennée, 2024/07/05
- [PULL 22/40] test/plugins: preserve the instruction record over translations, Alex Bennée, 2024/07/05
- [PULL 20/40] gitlab: don't bother with KVM for TCI builds, Alex Bennée, 2024/07/05
- [PULL 31/40] gdbstub: Move GdbCmdParseEntry into a new header file, Alex Bennée, 2024/07/05
- Re: [PULL 00/40] maintainer updates for testing, plugins and gdbstub, Richard Henderson, 2024/07/05