Re: [PATCH 00/85] target/sparc: Convert to decodetree

[Richard Henderson]

On 10/15/23 13:12, Mark Cave-Ayland wrote:
> 1 ./qemu-system-sparc -cdrom debian-40r4a-sparc-netinst.iso -boot d -bios ss5.bin (Boot 
> with real SS-5 PROM instead of OpenBIOS)
>
> -> Hangs during PROM memory test
>
> -> Bisected to:
> 91b579b5293c4c5c3cfaf0214a5523b655dea4fe is the first bad commit
> commit 91b579b5293c4c5c3cfaf0214a5523b655dea4fe
> Author: Richard Henderson <richard.henderson@linaro.org>
> Date:   Fri Oct 13 14:27:57 2023 -0700
>
>      target/sparc: Move JMPL, RETT, RETURN to decodetree
>
>      Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
>      Message-Id: <20231013212846.165724-37-richard.henderson@linaro.org>

Incorrectly translated "call %o7".

Old decoder always computed the source into a temp; new decoder tries to use cpu_gpr[] 
directly.  Clobbered %o7 with return address before setting npc.  Fixed by moving the 
return address store later.


r~