Re: [EXTERNAL]Re: [PATCH] target/mips: Fix ll/sc after 7dd547e5ab6b31e7a0cfc182d3ad131dd55a948f

[Aleksandar Markovic]

> From: Richard Henderson <address@hidden>
> Sent: Monday, February 3, 2020 8:52 AM
> To: James Clarke; address@hidden
> Cc: Alex Richardson; Aurelien Jarno; Aleksandar Markovic; Aleksandar Rikalo
> Subject: [EXTERNAL]Re: [PATCH] target/mips: Fix ll/sc after > 
> 7dd547e5ab6b31e7a0cfc182d3ad131dd55a948f
> 
> On 2/2/20 3:34 PM, James Clarke wrote:
> > From: Alex Richardson <address@hidden>
> >
> > After 7dd547e5ab6b31e7a0cfc182d3ad131dd55a948f the env->llval value is
> > loaded as an unsigned value (instead of sign-extended as before).
> > Therefore, the CMPXCHG in gen_st_cond() in translate.c fails if the sign
> > bit is set in the loaded value.
> > Fix this by sign-extending the llval value for the 32-bit case.
> >
> > I discovered this issue because FreeBSD MIPS64 was looping forever in an
> > atomic helper function when trying to start /sbin/init.
> >
> > Signed-off-by: Alex Richardson <address@hidden>
> > Fixes: 7dd547e5ab6b ("target/mips: Use cpu_*_mmuidx_ra instead of 
> > MMU_MODE*_SUFFIX")
> > Buglink: https://bugs.launchpad.net/qemu/+bug/1861605
> > Cc: Aurelien Jarno <address@hidden>
> > Cc: Aleksandar Markovic <address@hidden>
> > Cc: Aleksandar Rikalo <address@hidden>
> > Cc: Richard Henderson <address@hidden>
> > Signed-off-by: James Clarke <address@hidden>
> > ---
> >  target/mips/op_helper.c | 8 ++++----
> >  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> My fault.  Sorry about that.
> 
> Reviewed-by: Richard Henderson <address@hidden>
> 

Applied to the next mips queue.

Thanks to everyone involved!!

Aleksandar

> 
r~