KVM: arm/arm64: Fix bug in advertising KVM_CAP_MSI_DEVID capability
authorShanker Donthineni <shankerd@codeaurora.org>
Sat, 8 Jul 2017 13:48:30 +0000 (08:48 -0500)
committerMarc Zyngier <marc.zyngier@arm.com>
Tue, 25 Jul 2017 13:17:34 +0000 (14:17 +0100)
Commit 0e4e82f154e3 ("KVM: arm64: vgic-its: Enable ITS emulation as
a virtual MSI controller") tried to advertise KVM_CAP_MSI_DEVID, but
the code logic was not updating the dist->msis_require_devid field
correctly. If hypervisor tool creates the ITS device after VGIC
initialization then we don't advertise KVM_CAP_MSI_DEVID capability.

Update the field msis_require_devid to true inside vgic_its_create()
to fix the issue.

Fixes: 0e4e82f154e3 ("vgic-its: Enable ITS emulation as a virtual MSI controller")
Signed-off-by: Shanker Donthineni <shankerd@codeaurora.org>
Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
virt/kvm/arm/vgic/vgic-init.c
virt/kvm/arm/vgic/vgic-its.c

index 3a0b8999f011c6e3e1ff6ea7699577879a651fd3..5801261f3adddeaab819f4ffd23ecd5839d03c8a 100644 (file)
@@ -285,9 +285,6 @@ int vgic_init(struct kvm *kvm)
        if (ret)
                goto out;
 
-       if (vgic_has_its(kvm))
-               dist->msis_require_devid = true;
-
        kvm_for_each_vcpu(i, vcpu, kvm)
                kvm_vgic_vcpu_enable(vcpu);
 
index 2dff288b3a668e401d924e52cee10b9e85ecd8df..aa6b68db80b47c6089ef27c64dbf1e2654acc458 100644 (file)
@@ -1598,6 +1598,7 @@ static int vgic_its_create(struct kvm_device *dev, u32 type)
        INIT_LIST_HEAD(&its->device_list);
        INIT_LIST_HEAD(&its->collection_list);
 
+       dev->kvm->arch.vgic.msis_require_devid = true;
        dev->kvm->arch.vgic.has_its = true;
        its->enabled = false;
        its->dev = dev;