[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/3] target/riscv: add log of PMP permission checking
From: |
Alistair Francis |
Subject: |
Re: [PATCH 2/3] target/riscv: add log of PMP permission checking |
Date: |
Tue, 16 Mar 2021 16:02:25 -0400 |
On Sun, Feb 21, 2021 at 10:31 AM Jim Shu <cwshu@andestech.com> wrote:
>
> Like MMU translation, add qemu log of PMP permission checking for
> debugging.
>
> Signed-off-by: Jim Shu <cwshu@andestech.com>
Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Alistair
> ---
> target/riscv/cpu_helper.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/target/riscv/cpu_helper.c b/target/riscv/cpu_helper.c
> index f6ac63bf0e..c1ecb8a710 100644
> --- a/target/riscv/cpu_helper.c
> +++ b/target/riscv/cpu_helper.c
> @@ -794,6 +794,12 @@ bool riscv_cpu_tlb_fill(CPUState *cs, vaddr address, int
> size,
> if (ret == TRANSLATE_SUCCESS) {
> ret = get_physical_address_pmp(env, &prot_pmp, &tlb_size, pa,
> size, access_type, mode);
> +
> + qemu_log_mask(CPU_LOG_MMU,
> + "%s PMP address=" TARGET_FMT_plx " ret %d prot"
> + " %d tlb_size " TARGET_FMT_lu "\n",
> + __func__, pa, ret, prot_pmp, tlb_size);
> +
> prot &= prot_pmp;
> }
>
> @@ -821,6 +827,12 @@ bool riscv_cpu_tlb_fill(CPUState *cs, vaddr address, int
> size,
> if (ret == TRANSLATE_SUCCESS) {
> ret = get_physical_address_pmp(env, &prot_pmp, &tlb_size, pa,
> size, access_type, mode);
> +
> + qemu_log_mask(CPU_LOG_MMU,
> + "%s PMP address=" TARGET_FMT_plx " ret %d prot"
> + " %d tlb_size " TARGET_FMT_lu "\n",
> + __func__, pa, ret, prot_pmp, tlb_size);
> +
> prot &= prot_pmp;
> }
> }
> --
> 2.30.1
>
>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH 2/3] target/riscv: add log of PMP permission checking,
Alistair Francis <=