On 06/11/2018 02:56 AM, Peter Maydell wrote: > The API for cpu_transaction_failed() says that it takes the physical > address for the failed transaction. However we were actually passing > it the offset within the target MemoryRegion. We don't currently > have any target CPU implementations of this hook that require the > physical address; fix this bug so we don't get confused if we ever > do add one. > > Suggested-by: Paolo Bonzini <pbonz...@redhat.com> > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > include/exec/exec-all.h | 13 ++++++++++-- > accel/tcg/cputlb.c | 44 +++++++++++++++++++++++++++++------------ > exec.c | 5 +++-- > 3 files changed, 45 insertions(+), 17 deletions(-)
Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~
