diff options
author | Jan Kiszka <jan.kiszka@siemens.com> | 2011-12-02 12:35:24 -0500 |
---|---|---|
committer | Avi Kivity <avi@redhat.com> | 2011-12-27 04:22:15 -0500 |
commit | 3d56cbdf359c953f8bfcab68aa5cf766e4480799 (patch) | |
tree | 64c81ba2658afe8b91b2e8609f5c9eadc701cc27 /arch/x86/kvm | |
parent | 086c9855019935854311b477b33498a6ea357ef6 (diff) |
KVM: MMU: Drop unused return value of kvm_mmu_remove_some_alloc_mmu_pages
freed_pages is never evaluated, so remove it as well as the return code
kvm_mmu_remove_some_alloc_mmu_pages so far delivered to its only user.
Signed-off-by: Jan Kiszka <jan.kiszka@siemens.com>
Signed-off-by: Avi Kivity <avi@redhat.com>
Diffstat (limited to 'arch/x86/kvm')
-rw-r--r-- | arch/x86/kvm/mmu.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c index 7a8e99c6dc81..2a2a9b40db19 100644 --- a/arch/x86/kvm/mmu.c +++ b/arch/x86/kvm/mmu.c | |||
@@ -3890,14 +3890,14 @@ restart: | |||
3890 | spin_unlock(&kvm->mmu_lock); | 3890 | spin_unlock(&kvm->mmu_lock); |
3891 | } | 3891 | } |
3892 | 3892 | ||
3893 | static int kvm_mmu_remove_some_alloc_mmu_pages(struct kvm *kvm, | 3893 | static void kvm_mmu_remove_some_alloc_mmu_pages(struct kvm *kvm, |
3894 | struct list_head *invalid_list) | 3894 | struct list_head *invalid_list) |
3895 | { | 3895 | { |
3896 | struct kvm_mmu_page *page; | 3896 | struct kvm_mmu_page *page; |
3897 | 3897 | ||
3898 | page = container_of(kvm->arch.active_mmu_pages.prev, | 3898 | page = container_of(kvm->arch.active_mmu_pages.prev, |
3899 | struct kvm_mmu_page, link); | 3899 | struct kvm_mmu_page, link); |
3900 | return kvm_mmu_prepare_zap_page(kvm, page, invalid_list); | 3900 | kvm_mmu_prepare_zap_page(kvm, page, invalid_list); |
3901 | } | 3901 | } |
3902 | 3902 | ||
3903 | static int mmu_shrink(struct shrinker *shrink, struct shrink_control *sc) | 3903 | static int mmu_shrink(struct shrinker *shrink, struct shrink_control *sc) |
@@ -3912,15 +3912,15 @@ static int mmu_shrink(struct shrinker *shrink, struct shrink_control *sc) | |||
3912 | raw_spin_lock(&kvm_lock); | 3912 | raw_spin_lock(&kvm_lock); |
3913 | 3913 | ||
3914 | list_for_each_entry(kvm, &vm_list, vm_list) { | 3914 | list_for_each_entry(kvm, &vm_list, vm_list) { |
3915 | int idx, freed_pages; | 3915 | int idx; |
3916 | LIST_HEAD(invalid_list); | 3916 | LIST_HEAD(invalid_list); |
3917 | 3917 | ||
3918 | idx = srcu_read_lock(&kvm->srcu); | 3918 | idx = srcu_read_lock(&kvm->srcu); |
3919 | spin_lock(&kvm->mmu_lock); | 3919 | spin_lock(&kvm->mmu_lock); |
3920 | if (!kvm_freed && nr_to_scan > 0 && | 3920 | if (!kvm_freed && nr_to_scan > 0 && |
3921 | kvm->arch.n_used_mmu_pages > 0) { | 3921 | kvm->arch.n_used_mmu_pages > 0) { |
3922 | freed_pages = kvm_mmu_remove_some_alloc_mmu_pages(kvm, | 3922 | kvm_mmu_remove_some_alloc_mmu_pages(kvm, |
3923 | &invalid_list); | 3923 | &invalid_list); |
3924 | kvm_freed = kvm; | 3924 | kvm_freed = kvm; |
3925 | } | 3925 | } |
3926 | nr_to_scan--; | 3926 | nr_to_scan--; |