On 5/9/19 8:02 AM, Richard Henderson wrote: > At present we give ret = 0, or TLBRET_MATCH. This gets matched > by the default case, which falls through to TLBRET_BADADDR. > However, it makes more sense to use a proper value. All of the > tlb-related exceptions are handled identically in cpu_loop.c, > so TLBRET_BADADDR is as good as any other. Retain it. > > Signed-off-by: Richard Henderson <richard.hender...@linaro.org> > --- > target/mips/helper.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/target/mips/helper.c b/target/mips/helper.c > index c44cdca3b5..cc7be7703a 100644 > --- a/target/mips/helper.c > +++ b/target/mips/helper.c > @@ -884,7 +884,7 @@ int mips_cpu_handle_mmu_fault(CPUState *cs, vaddr > address, int size, int rw, > int prot; > int access_type; > #endif > - int ret = 0; > + int ret = TLBRET_BADADDR; > > #if 0 > log_cpu_state(cs, 0); >
Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>