[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] target/mips: Fix ll/sc after 7dd547e5ab6b31e7a0cfc182d3ad131
From: |
Richard Henderson |
Subject: |
Re: [PATCH] target/mips: Fix ll/sc after 7dd547e5ab6b31e7a0cfc182d3ad131dd55a948f |
Date: |
Mon, 3 Feb 2020 07:52:07 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 |
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>
r~