On Wed, Mar 29, 2023 at 8:20 PM Weiwei Li <liwei...@iscas.ac.cn> wrote: > > Function get_physical_address() translates both virtual address and > guest physical address, and the latter is 34-bits for Sv32x4. So we > should use vaddr type for 'addr' parameter. > > 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/cpu_helper.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/target/riscv/cpu_helper.c b/target/riscv/cpu_helper.c > index f88c503cf4..cd6fbaeddc 100644 > --- a/target/riscv/cpu_helper.c > +++ b/target/riscv/cpu_helper.c > @@ -739,7 +739,7 @@ static int get_physical_address_pmp(CPURISCVState *env, > int *prot, > * @env: CPURISCVState > * @physical: This will be set to the calculated physical address > * @prot: The returned protection attributes > - * @addr: The virtual address to be translated > + * @addr: The virtual address or guest physical address to be translated > * @fault_pte_addr: If not NULL, this will be set to fault pte address > * when a error occurs on pte address translation. > * This will already be shifted to match htval. > @@ -751,7 +751,7 @@ static int get_physical_address_pmp(CPURISCVState *env, > int *prot, > * @is_debug: Is this access from a debugger or the monitor? > */ > static int get_physical_address(CPURISCVState *env, hwaddr *physical, > - int *prot, target_ulong addr, > + int *prot, vaddr addr, > target_ulong *fault_pte_addr, > int access_type, int mmu_idx, > bool first_stage, bool two_stage, > -- > 2.25.1 > >
