aboutsummaryrefslogtreecommitdiff
path: root/hw
diff options
context:
space:
mode:
authorPeter Maydell <peter.maydell@linaro.org>2017-11-07 13:03:51 +0000
committerPeter Maydell <peter.maydell@linaro.org>2017-11-07 13:03:51 +0000
commit3448d47b3172015006b79197eb5a69826c6a7b6d (patch)
tree4e6e40c1b2e15fec55130428379639970d1d09e9 /hw
parente4e05b7b3e28970bcb9c0032dc46e30950e75f18 (diff)
downloadqemu-3448d47b3172015006b79197eb5a69826c6a7b6d.zip
qemu-3448d47b3172015006b79197eb5a69826c6a7b6d.tar.gz
qemu-3448d47b3172015006b79197eb5a69826c6a7b6d.tar.bz2
translate.c: Fix usermode big-endian AArch32 LDREXD and STREXD
For AArch32 LDREXD and STREXD, architecturally the 32-bit word at the lowest address is always Rt and the one at addr+4 is Rt2, even if the CPU is big-endian. Our implementation does these with a single 64-bit store, so if we're big-endian then we need to put the two 32-bit halves together in the opposite order to little-endian, so that they end up in the right places. We were trying to do this with the gen_aa32_frob64() function, but that is not correct for the usermode emulator, because there there is a distinction between "load a 64 bit value" (which does a BE 64-bit access and doesn't need swapping) and "load two 32 bit values as one 64 bit access" (where we still need to do the swapping, like system mode BE32). Fixes: https://bugs.launchpad.net/qemu/+bug/1725267 Cc: qemu-stable@nongnu.org Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Message-id: 1509622400-13351-1-git-send-email-peter.maydell@linaro.org
Diffstat (limited to 'hw')
0 files changed, 0 insertions, 0 deletions