[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [Xen-devel] [PATCH v3 07/17] arm64: vgic-v3: Add ICV_EOIR1_EL1 handler
This patch is ported to xen from linux commit b6f49035b4bf6e2709f2a5fed3107f5438c1fd02 Add a handler for writing the guest's view of the ICC_EOIR1_EL1 register. This involves dropping the priority of the interrupt, and deactivating it if required (EOImode == 0). Signed-off-by : Manish Jaggi <manish.jaggi@xxxxxxxxxx> diff --git a/xen/arch/arm/arm64/vgic-v3-sr.c b/xen/arch/arm/arm64/vgic-v3-sr.c index 1b47351bbb..2192547cc4 100644 --- a/xen/arch/arm/arm64/vgic-v3-sr.c +++ b/xen/arch/arm/arm64/vgic-v3-sr.c @@ -21,6 +21,7 @@ */ #include <asm/current.h> +#include <asm-arm/gic.h> #include <asm/gic_v3_defs.h> #include <asm/regs.h> #include <asm/system.h> @@ -36,6 +37,7 @@ #define ICC_IAR1_EL1_SPURIOUS 0x3ff #define VGIC_MAX_SPI 1019 +#define VGIC_MIN_LPI 8192 /* Provide wrappers to read write VMCR similar to linux */ static uint64_t vgic_v3_read_vmcr(void) @@ -463,6 +465,134 @@ spurious: set_user_reg(regs, rt, ICC_IAR1_EL1_SPURIOUS); } +static int vgic_v3_find_active_lr(int intid, uint64_t *lr_val) +{ + int i; + unsigned int used_lr; + unsigned int nr_lrs = gic_get_nr_lrs(); + unsigned long *lr_mask = (unsigned long *) &this_cpu(lr_mask); + + for_each_set_bit(used_lr, lr_mask, nr_lrs) + { + uint64_t val = gicv3_ich_read_lr(used_lr); + + if ( (val & ICH_LR_VIRTUAL_ID_MASK) == intid && + (val & ICH_LR_ACTIVE_BIT) ) + { + *lr_val = val; + return i; + } + } + + *lr_val = ICC_IAR1_EL1_SPURIOUS; + return -1; +} + +static int vgic_v3_clear_highest_active_priority(void) +{ + int i; + uint32_t hap = 0; + uint8_t nr_apr_regs = vtr_to_nr_apr_regs(READ_SYSREG32(ICH_VTR_EL2)); + + for ( i = 0; i < nr_apr_regs; i++ ) + { + uint32_t ap0, ap1; + int c0, c1; + + ap0 = vgic_v3_read_ap0rn(i); + ap1 = vgic_v3_read_ap1rn(i); + if ( !ap0 && !ap1 ) + { + hap += 32; + continue; + } + + c0 = ap0 ? __ffs(ap0) : 32; + c1 = ap1 ? __ffs(ap1) : 32; + + /* Always clear the LSB, which is the highest priority */ + if ( c0 < c1 ) + { + ap0 &= ~BIT(c0); + vgic_v3_write_ap0rn(ap0, i); + hap += c0; + } + else + { + ap1 &= ~BIT(c1); + vgic_v3_write_ap1rn(ap1, i); + hap += c1; + } + + /* Rescale to 8 bits of priority */ + return hap << vgic_v3_bpr_min(); + } + + return GICV3_IDLE_PRIORITY; +} + +static void vgic_v3_clear_active_lr(int lr, uint64_t lr_val) +{ + lr_val &= ~ICH_LR_ACTIVE_BIT; + if ( lr_val & ICH_LR_HW ) + { + uint32_t pid; + + pid = (lr_val & ICH_LR_PHYS_ID_MASK) >> ICH_LR_PHYS_ID_SHIFT; + WRITE_SYSREG32(pid, ICC_DIR_EL1); + } + gicv3_ich_write_lr(lr, lr_val); +} + +static void vgic_v3_bump_eoicount(void) +{ + uint32_t hcr; + + hcr = READ_SYSREG32(ICH_HCR_EL2); + hcr += 1 << ICH_HCR_EOIcount_SHIFT; + WRITE_SYSREG32(hcr, ICH_HCR_EL2); +} + +static void vgic_v3_write_eoir(struct cpu_user_regs *regs, uint32_t vmcr, + int rt) +{ + uint64_t lr_val; + uint8_t lr_prio, act_prio; + int lr, grp; + const union hsr hsr = { .bits = regs->hsr }; + register_t vid = get_user_reg(regs, hsr.sysreg.reg); + + grp = vgic_v3_get_group(hsr); + + /* Drop priority in any case */ + act_prio = vgic_v3_clear_highest_active_priority(); + + /* If EOIing an LPI, no deactivate to be performed */ + if ( vid >= VGIC_MIN_LPI ) + return; + + /* EOImode == 1, nothing to be done here */ + if ( vmcr & ICH_VMCR_EOIM_MASK ) + return; + + lr = vgic_v3_find_active_lr(vid, &lr_val); + if ( lr == -1 ) + { + vgic_v3_bump_eoicount(); + return; + } + + lr_prio = (lr_val & ICH_LR_PRIORITY_MASK) >> ICH_LR_PRIORITY_SHIFT; + + /* If priorities or group do not match, the guest has fscked-up. */ + if ( grp != !!(lr_val & ICH_LR_GROUP) || + vgic_v3_pri_to_pre(lr_prio, vmcr, grp) != act_prio ) + return; + + /* Let's now perform the deactivation */ + vgic_v3_clear_active_lr(lr, lr_val); +} + /* vgic_v3_handle_cpuif_access * returns: true if the register is emulated * false if not a sysreg @@ -508,6 +638,10 @@ bool vgic_v3_handle_cpuif_access(struct cpu_user_regs *regs) fn = vgic_v3_read_iar; break; + case HSR_SYSREG_ICC_EOIR1_EL1: + fn = vgic_v3_write_eoir; + break; + default: ret = false; goto end; diff --git a/xen/include/asm-arm/arm64/sysregs.h b/xen/include/asm-arm/arm64/sysregs.h index dc8bcabe4c..f7422f1649 100644 --- a/xen/include/asm-arm/arm64/sysregs.h +++ b/xen/include/asm-arm/arm64/sysregs.h @@ -92,6 +92,7 @@ #define HSR_SYSREG_ICC_BPR1_EL1 HSR_SYSREG(3,0,c12,c12,3) #define HSR_SYSREG_ICC_IGRPEN1_EL1 HSR_SYSREG(3,0,c12,c12,7) #define HSR_SYSREG_ICC_IAR1_EL1 HSR_SYSREG(3,0,c12,c12,0) +#define HSR_SYSREG_ICC_EOIR1_EL1 HSR_SYSREG(3,0,c12,c12,1) #define HSR_SYSREG_CONTEXTIDR_EL1 HSR_SYSREG(3,0,c13,c0,1) #define HSR_SYSREG_PMCR_EL0 HSR_SYSREG(3,3,c9,c12,0) diff --git a/xen/include/asm-arm/gic_v3_defs.h b/xen/include/asm-arm/gic_v3_defs.h index 50316477bb..52af0466df 100644 --- a/xen/include/asm-arm/gic_v3_defs.h +++ b/xen/include/asm-arm/gic_v3_defs.h @@ -177,6 +177,10 @@ #define ICH_VMCR_ENG0_MASK (1 << ICH_VMCR_ENG0_SHIFT) #define ICH_VMCR_PMR_SHIFT 24 #define ICH_VMCR_PMR_MASK (0xffUL << ICH_VMCR_PMR_SHIFT) +#define ICH_VMCR_EOIM_SHIFT 9 +#define ICH_VMCR_EOIM_MASK (1 << ICH_VMCR_EOIM_SHIFT) +#define ICH_HCR_EOIcount_SHIFT 27 +#define ICH_HCR_EOIcount_MASK (0x1f << ICH_HCR_EOIcount_SHIFT) #define ICH_LR_VIRTUAL_MASK 0xffff #define ICH_LR_VIRTUAL_SHIFT 0 -- 2.14.1 _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/mailman/listinfo/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |