Hi!
On Wed, Aug 06, 2025 at 05:56:12PM +0100, Marc Zyngier wrote:
> FEAT_RASv1p1 system registeres are not handled at all so far.
*registers
> KVM will give an embarassed warning on the console and inject
*embarrassed
> an UNDEF, despite RASv1p1 being exposed to the guest on suitable HW.
>
> Handle these registers similarly to FEAT_RAS, with the added fun
> that there are *two* way to indicate the presence of FEAT_RASv1p1.
>
> Signed-off-by: Marc Zyngier <maz@xxxxxxxxxx>
Reviewed-by: Joey Gouly <joey.gouly@xxxxxxx>
> ---
> arch/arm64/kvm/sys_regs.c | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/arch/arm64/kvm/sys_regs.c b/arch/arm64/kvm/sys_regs.c
> index ad25484772574..1b4114790024e 100644
> --- a/arch/arm64/kvm/sys_regs.c
> +++ b/arch/arm64/kvm/sys_regs.c
> @@ -2695,6 +2695,18 @@ static bool access_ras(struct kvm_vcpu *vcpu,
> struct kvm *kvm = vcpu->kvm;
>
> switch(reg_to_encoding(r)) {
> + case SYS_ERXPFGCDN_EL1:
> + case SYS_ERXPFGCTL_EL1:
> + case SYS_ERXPFGF_EL1:
> + case SYS_ERXMISC2_EL1:
> + case SYS_ERXMISC3_EL1:
> + if (!(kvm_has_feat(kvm, ID_AA64PFR0_EL1, RAS, V1P1) ||
> + (kvm_has_feat_enum(kvm, ID_AA64PFR0_EL1, RAS, IMP) &&
> + kvm_has_feat(kvm, ID_AA64PFR1_EL1, RAS_frac, RASv1p1)))) {
> + kvm_inject_undefined(vcpu);
> + return false;
> + }
> + break;
> default:
> if (!kvm_has_feat(kvm, ID_AA64PFR0_EL1, RAS, IMP)) {
> kvm_inject_undefined(vcpu);
> @@ -3058,8 +3070,13 @@ static const struct sys_reg_desc sys_reg_descs[] = {
> { SYS_DESC(SYS_ERXCTLR_EL1), access_ras },
> { SYS_DESC(SYS_ERXSTATUS_EL1), access_ras },
> { SYS_DESC(SYS_ERXADDR_EL1), access_ras },
> + { SYS_DESC(SYS_ERXPFGF_EL1), access_ras },
> + { SYS_DESC(SYS_ERXPFGCTL_EL1), access_ras },
> + { SYS_DESC(SYS_ERXPFGCDN_EL1), access_ras },
> { SYS_DESC(SYS_ERXMISC0_EL1), access_ras },
> { SYS_DESC(SYS_ERXMISC1_EL1), access_ras },
> + { SYS_DESC(SYS_ERXMISC2_EL1), access_ras },
> + { SYS_DESC(SYS_ERXMISC3_EL1), access_ras },
>
> MTE_REG(TFSR_EL1),
> MTE_REG(TFSRE0_EL1),
> --
> 2.39.2
>
