Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
target/riscv: Suppress pte update with is_debug
The debugger should not modify PTE_A or PTE_D.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Reviewed-by: Weiwei Li <liweiwei@iscas.ac.cn>
Tested-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
Message-Id: <20230325105429.1142530-22-richard.henderson@linaro.org>
Message-Id: <20230412114333.118895-22-richard.henderson@linaro.org>
Signed-off-by: Alistair Francis <alistair.francis@wdc.com>
  • Loading branch information
rth7680 authored and alistair23 committed May 5, 2023
1 parent 59688aa commit 0a19bf5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion target/riscv/cpu_helper.c
Expand Up @@ -1001,7 +1001,7 @@ static int get_physical_address(CPURISCVState *env, hwaddr *physical,
(access_type == MMU_DATA_STORE ? PTE_D : 0);

/* Page table updates need to be atomic with MTTCG enabled */
if (updated_pte != pte) {
if (updated_pte != pte && !is_debug) {
if (!hade) {
return TRANSLATE_FAIL;
}
Expand Down

0 comments on commit 0a19bf5

Please sign in to comment.