Aurelien Jarno <aurel...@aurel32.net> writes: That said this does implement neither the specification nor the silicon behaviour. This is fine for 1.5 as we are in freeze period, but this should be fixed for the 1.6 release.
I talked to IBM now. Reserved fields should be ignored by hardware. The architecture owner is IBM, not Freescale. That Freescale deviates from the architecture, is something that you may decide to ignore, unless it is vital for qemu's behaviour in practice. I very much doubt that L = 1 often, for code targeting a 32-bit processor. Trying to mimic decoding flaws on a per-processor basis, is going to take a lot of research, and will be prone to errors. So as far as I can tell, the patch is correct as per the architecture specification. One caveat though: Does 32-bit implementations define the SF bit, or else, does qemu define it and make sure it is 0 for 32-bit emulation? If not, the patch might cause trouble. Congrats, you read a "user message" until the last line. :-) -- Torbjörn
