On Thu, Apr 13, 2023 at 7:03 PM Weiwei Li <liwei...@iscas.ac.cn> wrote: > > TLB should be flushed not only for pmpcfg csr changes, but also for > pmpaddr csr changes. > > Signed-off-by: Weiwei Li <liwei...@iscas.ac.cn> > Signed-off-by: Junqiang Wang <wangjunqi...@iscas.ac.cn>
Reviewed-by: Alistair Francis <alistair.fran...@wdc.com> Alistair > --- > target/riscv/pmp.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/target/riscv/pmp.c b/target/riscv/pmp.c > index 4f9389e73c..6d4813806b 100644 > --- a/target/riscv/pmp.c > +++ b/target/riscv/pmp.c > @@ -537,6 +537,7 @@ void pmpaddr_csr_write(CPURISCVState *env, uint32_t > addr_index, > if (!pmp_is_locked(env, addr_index)) { > env->pmp_state.pmp[addr_index].addr_reg = val; > pmp_update_rule(env, addr_index); > + tlb_flush(env_cpu(env)); > } else { > qemu_log_mask(LOG_GUEST_ERROR, > "ignoring pmpaddr write - locked\n"); > -- > 2.25.1 > >