KVM: s390: Use kvm_get_vcpu() instead of open-coded access
authorMarc Zyngier <maz@kernel.org>
Tue, 16 Nov 2021 16:03:59 +0000 (16:03 +0000)
committerPaolo Bonzini <pbonzini@redhat.com>
Wed, 8 Dec 2021 09:24:14 +0000 (04:24 -0500)
As we are about to change the way vcpus are allocated, mandate
the use of kvm_get_vcpu() instead of open-coding the access.

Reviewed-by: Claudio Imbrenda <imbrenda@linux.ibm.com>
Signed-off-by: Marc Zyngier <maz@kernel.org>
Message-Id: <20211116160403.4074052-4-maz@kernel.org>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/s390/kvm/kvm-s390.c

index 6eeb59a..7a0f5ab 100644 (file)
@@ -4574,7 +4574,7 @@ int kvm_s390_vcpu_start(struct kvm_vcpu *vcpu)
        }
 
        for (i = 0; i < online_vcpus; i++) {
-               if (!is_vcpu_stopped(vcpu->kvm->vcpus[i]))
+               if (!is_vcpu_stopped(kvm_get_vcpu(vcpu->kvm, i)))
                        started_vcpus++;
        }
 
@@ -4636,9 +4636,11 @@ int kvm_s390_vcpu_stop(struct kvm_vcpu *vcpu)
        __disable_ibs_on_vcpu(vcpu);
 
        for (i = 0; i < online_vcpus; i++) {
-               if (!is_vcpu_stopped(vcpu->kvm->vcpus[i])) {
+               struct kvm_vcpu *tmp = kvm_get_vcpu(vcpu->kvm, i);
+
+               if (!is_vcpu_stopped(tmp)) {
                        started_vcpus++;
-                       started_vcpu = vcpu->kvm->vcpus[i];
+                       started_vcpu = tmp;
                }
        }