Merge tag 'kvm-arm-for-4.10' of git://git.kernel.org/pub/scm/linux/kernel/git/kvmarm/kvmarm into HEAD

KVM/ARM updates for 4.10:

- Support for the GICv3 ITS on 32bit platforms
- A handful of timer and GIC emulation fixes
- A PMU architecture fix

6 files changed, 14 insertions, 47 deletions

diff --git a/virt/kvm/arm/arch_timer.c b/virt/kvm/arm/arch_timer.c
index 27a1f6341d41..ae95fc0e3214 100644
--- a/virt/kvm/arm/arch_timer.c
+++ b/virt/kvm/arm/arch_timer.c
@@ -425,6 +425,11 @@ int kvm_timer_hyp_init(void)
         info = arch_timer_get_kvm_info();
         timecounter = &info->timecounter;
 
+        if (!timecounter->cc) {
+                kvm_err("kvm_arch_timer: uninitialized timecounter\n");
+                return -ENODEV;
+        }
+
         if (info->virtual_irq <= 0) {
                 kvm_err("kvm_arch_timer: invalid virtual timer IRQ: %d\n",
                         info->virtual_irq);
@@ -498,17 +503,7 @@ int kvm_timer_enable(struct kvm_vcpu *vcpu)
         if (ret)
                 return ret;
 
-
-        /*
-         * There is a potential race here between VCPUs starting for the first
-         * time, which may be enabling the timer multiple times.  That doesn't
-         * hurt though, because we're just setting a variable to the same
-         * variable that it already was.  The important thing is that all
-         * VCPUs have the enabled variable set, before entering the guest, if
-         * the arch timers are enabled.
-         */
-        if (timecounter)
-                timer->enabled = 1;
+        timer->enabled = 1;
 
         return 0;
 }
diff --git a/virt/kvm/arm/vgic/vgic-its.c b/virt/kvm/arm/vgic/vgic-its.c
index 4660a7d04eea..8c2b3cdcb2c5 100644
--- a/virt/kvm/arm/vgic/vgic-its.c
+++ b/virt/kvm/arm/vgic/vgic-its.c
@@ -632,21 +632,22 @@ static bool vgic_its_check_id(struct vgic_its *its, u64 baser, int id)
         int index;
         u64 indirect_ptr;
         gfn_t gfn;
+        int esz = GITS_BASER_ENTRY_SIZE(baser);
 
         if (!(baser & GITS_BASER_INDIRECT)) {
                 phys_addr_t addr;
 
-                if (id >= (l1_tbl_size / GITS_BASER_ENTRY_SIZE(baser)))
+                if (id >= (l1_tbl_size / esz))
                         return false;
 
-                addr = BASER_ADDRESS(baser) + id * GITS_BASER_ENTRY_SIZE(baser);
+                addr = BASER_ADDRESS(baser) + id * esz;
                 gfn = addr >> PAGE_SHIFT;
 
                 return kvm_is_visible_gfn(its->dev->kvm, gfn);
         }
 
         /* calculate and check the index into the 1st level */
-        index = id / (SZ_64K / GITS_BASER_ENTRY_SIZE(baser));
+        index = id / (SZ_64K / esz);
         if (index >= (l1_tbl_size / sizeof(u64)))
                 return false;
 
@@ -670,8 +671,8 @@ static bool vgic_its_check_id(struct vgic_its *its, u64 baser, int id)
         indirect_ptr &= GENMASK_ULL(51, 16);
 
         /* Find the address of the actual entry */
-        index = id % (SZ_64K / GITS_BASER_ENTRY_SIZE(baser));
-        indirect_ptr += index * GITS_BASER_ENTRY_SIZE(baser);
+        index = id % (SZ_64K / esz);
+        indirect_ptr += index * esz;
         gfn = indirect_ptr >> PAGE_SHIFT;
 
         return kvm_is_visible_gfn(its->dev->kvm, gfn);
diff --git a/virt/kvm/arm/vgic/vgic-kvm-device.c b/virt/kvm/arm/vgic/vgic-kvm-device.c
index ce1f4ed9daf4..fbe87a63d250 100644
--- a/virt/kvm/arm/vgic/vgic-kvm-device.c
+++ b/virt/kvm/arm/vgic/vgic-kvm-device.c
@@ -221,11 +221,9 @@ int kvm_register_vgic_device(unsigned long type)
                 ret = kvm_register_device_ops(&kvm_arm_vgic_v3_ops,
                                               KVM_DEV_TYPE_ARM_VGIC_V3);
 
-#ifdef CONFIG_KVM_ARM_VGIC_V3_ITS
                 if (ret)
                         break;
                 ret = kvm_vgic_register_its_device();
-#endif
                 break;
         }
 
diff --git a/virt/kvm/arm/vgic/vgic-mmio-v2.c b/virt/kvm/arm/vgic/vgic-mmio-v2.c
index b44b359cbbad..78e34bc4d89b 100644
--- a/virt/kvm/arm/vgic/vgic-mmio-v2.c
+++ b/virt/kvm/arm/vgic/vgic-mmio-v2.c
@@ -129,6 +129,7 @@ static void vgic_mmio_write_target(struct kvm_vcpu *vcpu,
                                    unsigned long val)
 {
         u32 intid = VGIC_ADDR_TO_INTID(addr, 8);
+        u8 cpu_mask = GENMASK(atomic_read(&vcpu->kvm->online_vcpus) - 1, 0);
         int i;
 
         /* GICD_ITARGETSR[0-7] are read-only */
@@ -141,7 +142,7 @@ static void vgic_mmio_write_target(struct kvm_vcpu *vcpu,
 
                 spin_lock(&irq->irq_lock);
 
-                irq->targets = (val >> (i * 8)) & 0xff;
+                irq->targets = (val >> (i * 8)) & cpu_mask;
                 target = irq->targets ? __ffs(irq->targets) : 0;
                 irq->target_vcpu = kvm_get_vcpu(vcpu->kvm, target);
 
diff --git a/virt/kvm/arm/vgic/vgic-mmio-v3.c b/virt/kvm/arm/vgic/vgic-mmio-v3.c
index 0d3c76a4208b..50f42f0f8c4f 100644
--- a/virt/kvm/arm/vgic/vgic-mmio-v3.c
+++ b/virt/kvm/arm/vgic/vgic-mmio-v3.c
@@ -42,7 +42,6 @@ u64 update_64bit_reg(u64 reg, unsigned int offset, unsigned int len,
         return reg | ((u64)val << lower);
 }
 
-#ifdef CONFIG_KVM_ARM_VGIC_V3_ITS
 bool vgic_has_its(struct kvm *kvm)
 {
         struct vgic_dist *dist = &kvm->arch.vgic;
@@ -52,7 +51,6 @@ bool vgic_has_its(struct kvm *kvm)
 
         return dist->has_its;
 }
-#endif
 
 static unsigned long vgic_mmio_read_v3_misc(struct kvm_vcpu *vcpu,
                                             gpa_t addr, unsigned int len)
diff --git a/virt/kvm/arm/vgic/vgic.h b/virt/kvm/arm/vgic/vgic.h
index 9d9e014765a2..859f65c6e056 100644
--- a/virt/kvm/arm/vgic/vgic.h
+++ b/virt/kvm/arm/vgic/vgic.h
@@ -84,37 +84,11 @@ int vgic_v3_probe(const struct gic_kvm_info *info);
 int vgic_v3_map_resources(struct kvm *kvm);
 int vgic_register_redist_iodevs(struct kvm *kvm, gpa_t dist_base_address);
 
-#ifdef CONFIG_KVM_ARM_VGIC_V3_ITS
 int vgic_register_its_iodevs(struct kvm *kvm);
 bool vgic_has_its(struct kvm *kvm);
 int kvm_vgic_register_its_device(void);
 void vgic_enable_lpis(struct kvm_vcpu *vcpu);
 int vgic_its_inject_msi(struct kvm *kvm, struct kvm_msi *msi);
-#else
-static inline int vgic_register_its_iodevs(struct kvm *kvm)
-{
-        return -ENODEV;
-}
-
-static inline bool vgic_has_its(struct kvm *kvm)
-{
-        return false;
-}
-
-static inline int kvm_vgic_register_its_device(void)
-{
-        return -ENODEV;
-}
-
-static inline void vgic_enable_lpis(struct kvm_vcpu *vcpu)
-{
-}
-
-static inline int vgic_its_inject_msi(struct kvm *kvm, struct kvm_msi *msi)
-{
-        return -ENODEV;
-}
-#endif
 
 int kvm_register_vgic_device(unsigned long type);
 int vgic_lazy_init(struct kvm *kvm);