Richard Henderson <[email protected]> writes:

> Add an interface to return the CPUTLBEntryFull struct
> that goes with the lookup.  The result is not intended
> to be valid across multiple lookups, so the user must
> use the results immediately.
>
> Reviewed-by: Peter Maydell <[email protected]>
> Reviewed-by: Philippe Mathieu-Daudé <[email protected]>
> Signed-off-by: Richard Henderson <[email protected]>
> ---
>  include/exec/exec-all.h | 11 ++++++++++
>  accel/tcg/cputlb.c      | 47 +++++++++++++++++++++++++----------------
>  2 files changed, 40 insertions(+), 18 deletions(-)
>
> diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h
> index bcad607c4e..758cf6bcc7 100644
> --- a/include/exec/exec-all.h
> +++ b/include/exec/exec-all.h
> @@ -434,6 +434,17 @@ int probe_access_flags(CPUArchState *env, target_ulong 
> addr,
>                         MMUAccessType access_type, int mmu_idx,
>                         bool nonfault, void **phost, uintptr_t retaddr);
>  
> +#ifndef CONFIG_USER_ONLY
> +/**
> + * probe_access_full:
> + * Like probe_access_flags, except also return into @pfull.
> + */

That lifetime requirement on @pfull really should be documented here as
well as the commit message.

Otherwise:

Reviewed-by: Alex Bennée <[email protected]>

-- 
Alex Bennée

Reply via email to