Re: [Qemu-devel] [PATCH] tcg/tci: enable bswap16_i64

[Richard Henderson]

On 06/13/2017 12:11 PM, Alex Bennée wrote:
> Philippe Mathieu-Daudé <address@hidden> writes:
>
> > remove some copy/paste leftover, code seems sane.
> >
> > while running Alex Bennée's image aarch64-linux-3.15rc2-buildroot.img:
> >
> > Trace 0x7fa1904b0890 [0: ffffffc00036cd04]
> > ----------------
> > IN:
> > 0xffffffc00036cd24:  5ac00694      rev16 w20, w20
> >
> > OP:
> >   ---- ffffffc00036cd24 0000000000000000 0000000000000000
> >   ext32u_i64 tmp3,x20
> >   ext16u_i64 tmp2,tmp3
> >   bswap16_i64 x20,tmp2
> >   movi_i64 tmp4,$0x10
> >   shr_i64 tmp2,tmp3,tmp4
> >   ext16u_i64 tmp2,tmp2
> >   bswap16_i64 tmp2,tmp2
> >   deposit_i64 x20,x20,tmp2,$0x10,$0x10
> >
> > Linking TBs 0x7fa1904b0890 [ffffffc00036cd04] index 0 -> 0x7fa1904b0aa0 
> > [ffffffc00036cd24]
> > Trace 0x7fa1904b0aa0 [0: ffffffc00036cd24]
> > TODO qemu/tci.c:1049: tcg_qemu_tb_exec()
> > qemu/tci.c:1049: tcg fatal error
> > Aborted
> >
> > Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
>
> Possibly the only person who has run tci recently ;-)

Is it time to remove it?  I'm pretty sure the only hosts for which it will work 
have proper backends...


r~