Peter Maydell <[email protected]> writes: > In the definition of ID_PFR1 we have an ifdef block; we specify the > accessfn once in the common part of the ifdef and once in the > not-user-only part, which is redundant but harmless. > > The accessfn will always return success in user-only mode (because > we won't trap to EL2), so specify it only in the not-user-only > half of the ifdef, as was probably the intention. > > This is only cc'd to stable to avoid a textual conflict with > the following patch, which is a bug fix. > > Cc: [email protected] > Fixes: 0f150c8499e970bd ("target/arm: Constify ID_PFR1 on user emulation") > Signed-off-by: Peter Maydell <[email protected]>
Reviewed-by: Alex Bennée <[email protected]> -- Alex Bennée Virtualisation Tech Lead @ Linaro
