KVM: Move vcpu_load to arch-specific kvm_arch_vcpu_ioctl_get_mpstate
Move vcpu_load() and vcpu_put() into the architecture specific implementations of kvm_arch_vcpu_ioctl_get_mpstate(). Reviewed-by: David Hildenbrand <david@redhat.com> Signed-off-by: Christoffer Dall <christoffer.dall@linaro.org> Reviewed-by: Cornelia Huck <cohuck@redhat.com> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
parent
b4ef9d4e8c
commit
fd2325612c
@ -2833,9 +2833,16 @@ int kvm_arch_vcpu_ioctl_set_guest_debug(struct kvm_vcpu *vcpu,
|
|||||||
int kvm_arch_vcpu_ioctl_get_mpstate(struct kvm_vcpu *vcpu,
|
int kvm_arch_vcpu_ioctl_get_mpstate(struct kvm_vcpu *vcpu,
|
||||||
struct kvm_mp_state *mp_state)
|
struct kvm_mp_state *mp_state)
|
||||||
{
|
{
|
||||||
|
int ret;
|
||||||
|
|
||||||
|
vcpu_load(vcpu);
|
||||||
|
|
||||||
/* CHECK_STOP and LOAD are not supported yet */
|
/* CHECK_STOP and LOAD are not supported yet */
|
||||||
return is_vcpu_stopped(vcpu) ? KVM_MP_STATE_STOPPED :
|
ret = is_vcpu_stopped(vcpu) ? KVM_MP_STATE_STOPPED :
|
||||||
KVM_MP_STATE_OPERATING;
|
KVM_MP_STATE_OPERATING;
|
||||||
|
|
||||||
|
vcpu_put(vcpu);
|
||||||
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
int kvm_arch_vcpu_ioctl_set_mpstate(struct kvm_vcpu *vcpu,
|
int kvm_arch_vcpu_ioctl_set_mpstate(struct kvm_vcpu *vcpu,
|
||||||
|
@ -7465,6 +7465,8 @@ int kvm_arch_vcpu_ioctl_get_sregs(struct kvm_vcpu *vcpu,
|
|||||||
int kvm_arch_vcpu_ioctl_get_mpstate(struct kvm_vcpu *vcpu,
|
int kvm_arch_vcpu_ioctl_get_mpstate(struct kvm_vcpu *vcpu,
|
||||||
struct kvm_mp_state *mp_state)
|
struct kvm_mp_state *mp_state)
|
||||||
{
|
{
|
||||||
|
vcpu_load(vcpu);
|
||||||
|
|
||||||
kvm_apic_accept_events(vcpu);
|
kvm_apic_accept_events(vcpu);
|
||||||
if (vcpu->arch.mp_state == KVM_MP_STATE_HALTED &&
|
if (vcpu->arch.mp_state == KVM_MP_STATE_HALTED &&
|
||||||
vcpu->arch.pv.pv_unhalted)
|
vcpu->arch.pv.pv_unhalted)
|
||||||
@ -7472,6 +7474,7 @@ int kvm_arch_vcpu_ioctl_get_mpstate(struct kvm_vcpu *vcpu,
|
|||||||
else
|
else
|
||||||
mp_state->mp_state = vcpu->arch.mp_state;
|
mp_state->mp_state = vcpu->arch.mp_state;
|
||||||
|
|
||||||
|
vcpu_put(vcpu);
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -381,11 +381,14 @@ static void vcpu_power_off(struct kvm_vcpu *vcpu)
|
|||||||
int kvm_arch_vcpu_ioctl_get_mpstate(struct kvm_vcpu *vcpu,
|
int kvm_arch_vcpu_ioctl_get_mpstate(struct kvm_vcpu *vcpu,
|
||||||
struct kvm_mp_state *mp_state)
|
struct kvm_mp_state *mp_state)
|
||||||
{
|
{
|
||||||
|
vcpu_load(vcpu);
|
||||||
|
|
||||||
if (vcpu->arch.power_off)
|
if (vcpu->arch.power_off)
|
||||||
mp_state->mp_state = KVM_MP_STATE_STOPPED;
|
mp_state->mp_state = KVM_MP_STATE_STOPPED;
|
||||||
else
|
else
|
||||||
mp_state->mp_state = KVM_MP_STATE_RUNNABLE;
|
mp_state->mp_state = KVM_MP_STATE_RUNNABLE;
|
||||||
|
|
||||||
|
vcpu_put(vcpu);
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -2634,9 +2634,7 @@ out_free1:
|
|||||||
case KVM_GET_MP_STATE: {
|
case KVM_GET_MP_STATE: {
|
||||||
struct kvm_mp_state mp_state;
|
struct kvm_mp_state mp_state;
|
||||||
|
|
||||||
vcpu_load(vcpu);
|
|
||||||
r = kvm_arch_vcpu_ioctl_get_mpstate(vcpu, &mp_state);
|
r = kvm_arch_vcpu_ioctl_get_mpstate(vcpu, &mp_state);
|
||||||
vcpu_put(vcpu);
|
|
||||||
if (r)
|
if (r)
|
||||||
goto out;
|
goto out;
|
||||||
r = -EFAULT;
|
r = -EFAULT;
|
||||||
|
Loading…
Reference in New Issue
Block a user