diff options
author | Takuya Yoshikawa <yoshikawa_takuya_b1@lab.ntt.co.jp> | 2013-12-13 01:07:21 -0500 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2013-12-13 08:23:45 -0500 |
commit | c08ac06ab3f3cdb8d34376c3a8a5e46a31a62c8f (patch) | |
tree | 775a9dadbab5d86bd83a264ece99d42079bbdb50 /arch | |
parent | 6bb05ef78553491b2e2e0710dde364331dc3cdd2 (diff) |
KVM: Use cond_resched() directly and remove useless kvm_resched()
Since the commit 15ad7146 ("KVM: Use the scheduler preemption notifiers
to make kvm preemptible"), the remaining stuff in this function is a
simple cond_resched() call with an extra need_resched() check which was
there to avoid dropping VCPUs unnecessarily. Now it is meaningless.
Signed-off-by: Takuya Yoshikawa <yoshikawa_takuya_b1@lab.ntt.co.jp>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'arch')
-rw-r--r-- | arch/ia64/kvm/kvm-ia64.c | 2 | ||||
-rw-r--r-- | arch/powerpc/kvm/book3s_hv.c | 2 | ||||
-rw-r--r-- | arch/x86/kvm/x86.c | 2 |
3 files changed, 3 insertions, 3 deletions
diff --git a/arch/ia64/kvm/kvm-ia64.c b/arch/ia64/kvm/kvm-ia64.c index 985bf80c622e..53f44bee9ebb 100644 --- a/arch/ia64/kvm/kvm-ia64.c +++ b/arch/ia64/kvm/kvm-ia64.c | |||
@@ -702,7 +702,7 @@ again: | |||
702 | out: | 702 | out: |
703 | srcu_read_unlock(&vcpu->kvm->srcu, idx); | 703 | srcu_read_unlock(&vcpu->kvm->srcu, idx); |
704 | if (r > 0) { | 704 | if (r > 0) { |
705 | kvm_resched(vcpu); | 705 | cond_resched(); |
706 | idx = srcu_read_lock(&vcpu->kvm->srcu); | 706 | idx = srcu_read_lock(&vcpu->kvm->srcu); |
707 | goto again; | 707 | goto again; |
708 | } | 708 | } |
diff --git a/arch/powerpc/kvm/book3s_hv.c b/arch/powerpc/kvm/book3s_hv.c index 072287f1c3bc..3fa99b20894f 100644 --- a/arch/powerpc/kvm/book3s_hv.c +++ b/arch/powerpc/kvm/book3s_hv.c | |||
@@ -1348,7 +1348,7 @@ static void kvmppc_run_core(struct kvmppc_vcore *vc) | |||
1348 | kvm_guest_exit(); | 1348 | kvm_guest_exit(); |
1349 | 1349 | ||
1350 | preempt_enable(); | 1350 | preempt_enable(); |
1351 | kvm_resched(vcpu); | 1351 | cond_resched(); |
1352 | 1352 | ||
1353 | spin_lock(&vc->lock); | 1353 | spin_lock(&vc->lock); |
1354 | now = get_tb(); | 1354 | now = get_tb(); |
diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c index 21ef1ba184ae..4fb1ee619c1c 100644 --- a/arch/x86/kvm/x86.c +++ b/arch/x86/kvm/x86.c | |||
@@ -6125,7 +6125,7 @@ static int __vcpu_run(struct kvm_vcpu *vcpu) | |||
6125 | } | 6125 | } |
6126 | if (need_resched()) { | 6126 | if (need_resched()) { |
6127 | srcu_read_unlock(&kvm->srcu, vcpu->srcu_idx); | 6127 | srcu_read_unlock(&kvm->srcu, vcpu->srcu_idx); |
6128 | kvm_resched(vcpu); | 6128 | cond_resched(); |
6129 | vcpu->srcu_idx = srcu_read_lock(&kvm->srcu); | 6129 | vcpu->srcu_idx = srcu_read_lock(&kvm->srcu); |
6130 | } | 6130 | } |
6131 | } | 6131 | } |