Richard Henderson <richard.hender...@linaro.org> 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 <peter.mayd...@linaro.org> > Reviewed-by: Philippe Mathieu-Daudé <f4...@amsat.org> > Signed-off-by: Richard Henderson <richard.hender...@linaro.org> > --- > 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 <alex.ben...@linaro.org> -- Alex Bennée