KVM: s390: Removing untriggerable BUG_ONs
authorDominik Dingel <dingel@linux.vnet.ibm.com>
Mon, 10 Mar 2014 14:23:34 +0000 (15:23 +0100)
committerChristian Borntraeger <borntraeger@de.ibm.com>
Mon, 17 Mar 2014 10:06:45 +0000 (11:06 +0100)
The BUG_ON in kvm-s390.c is unreachable, as we get the vcpu per common code,
which itself does this from the private_data field of the file descriptor,
and there is no KVM_UNCREATE_VCPU.

The __{set,unset}_cpu_idle BUG_ONs are not triggerable because the vcpu
creation code already checks against KVM_MAX_VCPUS.

Signed-off-by: Dominik Dingel <dingel@linux.vnet.ibm.com>
Acked-by: Cornelia Huck <cornelia.huck@de.ibm.com>
Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
arch/s390/kvm/interrupt.c
arch/s390/kvm/kvm-s390.c

index 1d0f9d532c0b85ba1caf84400d80fe7d1609dda9..79d2e4fa9f9c2619c4b6c6d6e89e99c459b2f2fd 100644 (file)
@@ -114,14 +114,12 @@ static int __interrupt_is_deliverable(struct kvm_vcpu *vcpu,
 
 static void __set_cpu_idle(struct kvm_vcpu *vcpu)
 {
-       BUG_ON(vcpu->vcpu_id > KVM_MAX_VCPUS - 1);
        atomic_set_mask(CPUSTAT_WAIT, &vcpu->arch.sie_block->cpuflags);
        set_bit(vcpu->vcpu_id, vcpu->arch.local_int.float_int->idle_mask);
 }
 
 static void __unset_cpu_idle(struct kvm_vcpu *vcpu)
 {
-       BUG_ON(vcpu->vcpu_id > KVM_MAX_VCPUS - 1);
        atomic_clear_mask(CPUSTAT_WAIT, &vcpu->arch.sie_block->cpuflags);
        clear_bit(vcpu->vcpu_id, vcpu->arch.local_int.float_int->idle_mask);
 }
index 9136f8d408503d1a2c23e8d9315267dde7ef35fc..62683576f8e2cc87a9fa646f130df36484c0d148 100644 (file)
@@ -949,8 +949,6 @@ int kvm_arch_vcpu_ioctl_run(struct kvm_vcpu *vcpu, struct kvm_run *kvm_run)
 
        atomic_clear_mask(CPUSTAT_STOPPED, &vcpu->arch.sie_block->cpuflags);
 
-       BUG_ON(kvm_get_vcpu(vcpu->kvm, vcpu->vcpu_id) == NULL);
-
        switch (kvm_run->exit_reason) {
        case KVM_EXIT_S390_SIEIC:
        case KVM_EXIT_UNKNOWN: