On 1/8/21 9:51 AM, Peter Maydell wrote: > In commit cd8be50e58f63413c0 we converted the A32 coprocessor > insns to decodetree. This accidentally broke XScale/iWMMXt insns, > because it moved the handling of "cp insns which are handled > by looking up the cp register in the hashtable" from after the > call to the legacy disas_xscale_insn() decode to before it, > with the result that all XScale/iWMMXt insns now UNDEF. > > Update valid_cp() so that it knows that on XScale cp 0 and 1 > are not standard coprocessor instructions; this will cause > the decodetree trans_ functions to ignore them, so that > execution will correctly get through to the legacy decode again. > > Cc: qemu-sta...@nongnu.org > Reported-by: Guenter Roeck <li...@roeck-us.net> > Signed-off-by: Peter Maydell <peter.mayd...@linaro.org> > --- > With this Guenter's test image now successfully boots > and shuts down again. > ---
Reviewed-by: Richard Henderson <richard.hender...@linaro.org> r~
