Re: [PATCH for-9.0] target/arm: take HSTR traps of cp15 accesses to EL2, not EL1

[Richard Henderson]

On 3/25/24 03:31, Peter Maydell wrote:
> The HSTR_EL2 register allows the hypervisor to trap AArch32 EL1 and
> EL0 accesses to cp15 registers.  We incorrectly implemented this so
> they trap to EL1 when we detect the need for a HSTR trap at code
> generation time.  (The check in access_check_cp_reg() which we do at
> runtime to catch traps from EL0 is correctly routing them to EL2.)
>
> Use the correct target EL when generating the code to take the trap.
>
> Cc:qemu-stable@nongnu.org
> Resolves:https://gitlab.com/qemu-project/qemu/-/issues/2226
> Fixes: 049edada5e93df ("target/arm: Make HSTR_EL2 traps take priority over 
> UNDEF-at-EL1")
> Signed-off-by: Peter Maydell<peter.maydell@linaro.org>
> ---
>   target/arm/tcg/translate.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~