[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v5 03/13] target/riscv: Make the short cut really work in pmp
From: |
Alistair Francis |
Subject: |
Re: [PATCH v5 03/13] target/riscv: Make the short cut really work in pmp_hart_has_privs |
Date: |
Wed, 17 May 2023 12:07:29 +1000 |
On Sat, Apr 29, 2023 at 12:38 AM Weiwei Li <liweiwei@iscas.ac.cn> wrote:
>
> Return the result directly for short cut, since We needn't do the
> following check on the PMP entries if there is no PMP rules.
>
> Signed-off-by: Weiwei Li <liweiwei@iscas.ac.cn>
> Signed-off-by: Junqiang Wang <wangjunqiang@iscas.ac.cn>
Reviewed-by: Alistair Francis <alistair.francis@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 ad20a319c1..86abe1e7cd 100644
> --- a/target/riscv/pmp.c
> +++ b/target/riscv/pmp.c
> @@ -316,6 +316,7 @@ int pmp_hart_has_privs(CPURISCVState *env, target_ulong
> addr,
> allowed_privs, mode)) {
> ret = MAX_RISCV_PMPS;
> }
> + return ret;
> }
>
> if (size == 0) {
> --
> 2.25.1
>
>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH v5 03/13] target/riscv: Make the short cut really work in pmp_hart_has_privs,
Alistair Francis <=