Re: [Qemu-devel] [PATCH 2/7] target/sparc: Check for transaction failures in MMU passthrough ASIs

[Richard Henderson]

On 8/1/19 11:30 AM, Peter Maydell wrote:
> Currently the ld/st_asi helper functions make calls to the
> ld*_phys() and st*_phys() functions for those ASIs which
> imply direct accesses to physical addresses. These implicitly
> rely on the unassigned_access hook to cause them to generate
> an MMU fault if the access fails.
> 
> Switch to using the address_space_* functions instead, which
> return a MemTxResult that we can check. This means that when
> we switch SPARC over to using the do_transaction_failed hook
> we'll still get the same MMU faults we did before.
> 
> This commit converts the ASIs which do "MMU passthrough".
> 
> Signed-off-by: Peter Maydell <address@hidden>
> ---
>  target/sparc/ldst_helper.c | 49 +++++++++++++++++++++++++-------------
>  1 file changed, 33 insertions(+), 16 deletions(-)

Reviewed-by: Richard Henderson <address@hidden>


r~