]> git.proxmox.com Git - mirror_qemu.git/commit - target/arm/helper.c
target-arm: Handle "extended small page" descriptors correctly
authorPeter Maydell <peter.maydell@linaro.org>
Mon, 15 Jun 2015 17:06:07 +0000 (18:06 +0100)
committerPeter Maydell <peter.maydell@linaro.org>
Mon, 15 Jun 2015 17:06:07 +0000 (18:06 +0100)
commitfc1891c74ae122a9dc7854f38bae7db03cd911e6
treeb039e7dbedae54c639545f5a92ed6d956aa13fd6
parentb500e4db8e3e0b5f41a2dd14e2001200e5fc7d6b
target-arm: Handle "extended small page" descriptors correctly

The old ARMv5-style page table format includes a kind of second level
descriptor named the "extended small page" format, whose primary purpose
is to allow specification of the TEX memory attribute bits on a 4K page.
This exists on ARMv6 and also (as an implementation extension) on XScale
CPUs; it's UNPREDICTABLE on v5.

We were mishandling this in two ways:
 (1) we weren't implementing it for v6 (probably never noticed because
Linux will use the new-style v6 page table format there)
 (2) we were not correctly setting the page_size, which is 4K, not 1K

The latter bug went unnoticed for years because the only thing which
the page_size affects is which TLB entries get flushed when the guest
does a TLB invalidate on an address in the page, and prior to commit
2f0d8631b7 we were doing a full TLB flush very frequently due to Linux's
habit of writing the SCTLR pointlessly a lot.

(We can assume that after commit 2f0d8631b7 the bug went unnoticed
for a year because nobody's actually using the Zaurus/XScale emulation...)

Report the correct page size for these descriptors, and permit them
on ARMv6 CPUs. This fixes a problem where a kernel image for Zaurus
can boot the kernel OK but gets random segfaults when it tries to
run userspace programs.

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-id: 1432844085-16441-1-git-send-email-peter.maydell@linaro.org
target-arm/helper.c