Hi,

On 03/05/2017 12:45, Marc Zyngier wrote:
> A number of Group-0 registers can be handled by the same accessors
> as that of Group-1, so let's add the required system register encodings
> and catch them in the dispatching function.
> 
> Signed-off-by: Marc Zyngier <[email protected]>
Reviewed-by: Eric Auger <[email protected]>

Eric
> ---
>  arch/arm64/include/asm/sysreg.h | 4 ++++
>  virt/kvm/arm/hyp/vgic-v3-sr.c   | 7 +++++++
>  2 files changed, 11 insertions(+)
> 
> diff --git a/arch/arm64/include/asm/sysreg.h b/arch/arm64/include/asm/sysreg.h
> index ba93bc7ac8e4..9971c5c435a7 100644
> --- a/arch/arm64/include/asm/sysreg.h
> +++ b/arch/arm64/include/asm/sysreg.h
> @@ -180,7 +180,11 @@
>  
>  #define SYS_VBAR_EL1                 sys_reg(3, 0, 12, 0, 0)
>  
> +#define SYS_ICC_IAR0_EL1             sys_reg(3, 0, 12, 8, 0)
> +#define SYS_ICC_EOIR0_EL1            sys_reg(3, 0, 12, 8, 1)
> +#define SYS_ICC_HPPIR0_EL1           sys_reg(3, 0, 12, 8, 2)
>  #define SYS_ICC_BPR0_EL1             sys_reg(3, 0, 12, 8, 3)
> +#define SYS_ICC_AP0Rn_EL1(n)         sys_reg(3, 0, 12, 8, 4 | n)
>  #define SYS_ICC_AP1Rn_EL1(n)         sys_reg(3, 0, 12, 9, n)
>  #define SYS_ICC_DIR_EL1                      sys_reg(3, 0, 12, 11, 1)
>  #define SYS_ICC_SGI1R_EL1            sys_reg(3, 0, 12, 11, 5)
> diff --git a/virt/kvm/arm/hyp/vgic-v3-sr.c b/virt/kvm/arm/hyp/vgic-v3-sr.c
> index 77d5d12389ec..9c639f57268b 100644
> --- a/virt/kvm/arm/hyp/vgic-v3-sr.c
> +++ b/virt/kvm/arm/hyp/vgic-v3-sr.c
> @@ -848,9 +848,11 @@ int __hyp_text __vgic_v3_perform_cpuif_access(struct 
> kvm_vcpu *vcpu)
>       is_read = (esr & ESR_ELx_SYS64_ISS_DIR_MASK) == 
> ESR_ELx_SYS64_ISS_DIR_READ;
>  
>       switch (sysreg) {
> +     case SYS_ICC_IAR0_EL1:
>       case SYS_ICC_IAR1_EL1:
>               fn = __vgic_v3_read_iar;
>               break;
> +     case SYS_ICC_EOIR0_EL1:
>       case SYS_ICC_EOIR1_EL1:
>               fn = __vgic_v3_write_eoir;
>               break;
> @@ -866,30 +868,35 @@ int __hyp_text __vgic_v3_perform_cpuif_access(struct 
> kvm_vcpu *vcpu)
>               else
>                       fn = __vgic_v3_write_bpr1;
>               break;
> +     case SYS_ICC_AP0Rn_EL1(0):
>       case SYS_ICC_AP1Rn_EL1(0):
>               if (is_read)
>                       fn = __vgic_v3_read_apxr0;
>               else
>                       fn = __vgic_v3_write_apxr0;
>               break;
> +     case SYS_ICC_AP0Rn_EL1(1):
>       case SYS_ICC_AP1Rn_EL1(1):
>               if (is_read)
>                       fn = __vgic_v3_read_apxr1;
>               else
>                       fn = __vgic_v3_write_apxr1;
>               break;
> +     case SYS_ICC_AP0Rn_EL1(2):
>       case SYS_ICC_AP1Rn_EL1(2):
>               if (is_read)
>                       fn = __vgic_v3_read_apxr2;
>               else
>                       fn = __vgic_v3_write_apxr2;
>               break;
> +     case SYS_ICC_AP0Rn_EL1(3):
>       case SYS_ICC_AP1Rn_EL1(3):
>               if (is_read)
>                       fn = __vgic_v3_read_apxr3;
>               else
>                       fn = __vgic_v3_write_apxr3;
>               break;
> +     case SYS_ICC_HPPIR0_EL1:
>       case SYS_ICC_HPPIR1_EL1:
>               fn = __vgic_v3_read_hppir;
>               break;
> 
_______________________________________________
kvmarm mailing list
[email protected]
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

Reply via email to