summaryrefslogtreecommitdiffstats
path: root/hw/i386/acpi-dsdt.hex.generated
diff options
context:
space:
mode:
authorPeter Maydell <peter.maydell@linaro.org>2014-04-08 16:51:11 +0100
committerPeter Maydell <peter.maydell@linaro.org>2014-04-08 18:37:45 +0100
commit7dc176bce49c61551b513508def136d5bb632b72 (patch)
tree465316199814f3411c52ed844a70d98da5f93c46 /hw/i386/acpi-dsdt.hex.generated
parent9bc1a1d817670702f642633a325da346047f7508 (diff)
downloadhqemu-7dc176bce49c61551b513508def136d5bb632b72.zip
hqemu-7dc176bce49c61551b513508def136d5bb632b72.tar.gz
hw/pci-host/prep: Don't reverse IO accesses on bigendian hosts
The raven_io_read() and raven_io_write() functions pass and return values in little-endian format (since the IO op struct is marked DEVICE_LITTLE_ENDIAN); however they were storing the values in the buffer to pass to address_space_read/write() in host-endian order, which meant that on big-endian hosts the values were inadvertently reversed. Use the *_le_p() accessors instead so that we are consistent regardless of host endianness. Strictly speaking the byte order of the buffer for address_space_rw() is target byte order (which for PPC will be BE) but it doesn't actually matter as long as we are consistent about the marking on the IO op struct and which stl_*_p(). This bug was probably introduced due to confusion caused by the two different versions of ldl_p() and friends: bswap.h defines versions meaning "host endianness access" cpu-all.h defines versions meaning "target endianness access" As a target-independent source file prep.c gets the bswap.h versions; the very similar looking code in ioport.c is compiled per-target and gets the cpu-all.h versions. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Message-id: 1396972271-22660-1-git-send-email-peter.maydell@linaro.org Reviewed-by: Richard Henderson <rth@twiddle.net>
Diffstat (limited to 'hw/i386/acpi-dsdt.hex.generated')
0 files changed, 0 insertions, 0 deletions
OpenPOWER on IntegriCloud