target/arm: Use new CBAR encoding for all v8 CPUs, not all aarch64 CPUs
authorPeter Maydell <peter.maydell@linaro.org>
Tue, 6 Feb 2024 13:29:19 +0000 (13:29 +0000)
committerPeter Maydell <peter.maydell@linaro.org>
Thu, 15 Feb 2024 14:32:38 +0000 (14:32 +0000)
commitb2f24983db11dd1345db95f54a6d1dce184224a3
treed0da80f406faec87f32c46a3a5dade77b479782c
parent243b8602a6de74fbb56ede18e02ffb8b9de3b386
target/arm: Use new CBAR encoding for all v8 CPUs, not all aarch64 CPUs

We support two different encodings for the AArch32 IMPDEF
CBAR register -- older cores like the Cortex A9, A7, A15
have this at 4, c15, c0, 0; newer cores like the
Cortex A35, A53, A57 and A72 have it at 1 c15 c0 0.

When we implemented this we picked which encoding to
use based on whether the CPU set ARM_FEATURE_AARCH64.
However this isn't right for three cases:
 * the qemu-system-arm 'max' CPU, which is supposed to be
   a variant on a Cortex-A57; it ought to use the same
   encoding the A57 does and which the AArch64 'max'
   exposes to AArch32 guest code
 * the Cortex-R52, which is AArch32-only but has the CBAR
   at the newer encoding (and where we incorrectly are
   not yet setting ARM_FEATURE_CBAR_RO anyway)
 * any possible future support for other v8 AArch32
   only CPUs, or for supporting "boot the CPU into
   AArch32 mode" on our existing cores like the A57 etc

Make the decision of the encoding be based on whether
the CPU implements the ARM_FEATURE_V8 flag instead.

This changes the behaviour only for the qemu-system-arm
'-cpu max'. We don't expect anybody to be relying on the
old behaviour because:
 * it's not what the real hardware Cortex-A57 does
   (and that's what our ID register claims we are)
 * we don't implement the memory-mapped GICv3 support
   which is the only thing that exists at the peripheral
   base address pointed to by the register

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20240206132931.38376-2-peter.maydell@linaro.org
target/arm/helper.c