mmu: don't pass *kvm to spte_write_protect and spte_*_dirty
authorBandan Das <bsd@redhat.com>
Tue, 2 Aug 2016 20:32:37 +0000 (16:32 -0400)
committerPaolo Bonzini <pbonzini@redhat.com>
Fri, 19 Aug 2016 15:51:40 +0000 (17:51 +0200)
That parameter isn't used in these functions,
it's probably a historical artifact.

Signed-off-by: Bandan Das <bsd@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/mmu.c

index 3d4cc8cc56a38b05752ef38133b74af9d0e7ae27..d9c7e986b4e4e7e0e7bd9bc28223485e42d7c3bb 100644 (file)
@@ -1207,7 +1207,7 @@ static void drop_large_spte(struct kvm_vcpu *vcpu, u64 *sptep)
  *
  * Return true if tlb need be flushed.
  */
-static bool spte_write_protect(struct kvm *kvm, u64 *sptep, bool pt_protect)
+static bool spte_write_protect(u64 *sptep, bool pt_protect)
 {
        u64 spte = *sptep;
 
@@ -1233,12 +1233,12 @@ static bool __rmap_write_protect(struct kvm *kvm,
        bool flush = false;
 
        for_each_rmap_spte(rmap_head, &iter, sptep)
-               flush |= spte_write_protect(kvm, sptep, pt_protect);
+               flush |= spte_write_protect(sptep, pt_protect);
 
        return flush;
 }
 
-static bool spte_clear_dirty(struct kvm *kvm, u64 *sptep)
+static bool spte_clear_dirty(u64 *sptep)
 {
        u64 spte = *sptep;
 
@@ -1256,12 +1256,12 @@ static bool __rmap_clear_dirty(struct kvm *kvm, struct kvm_rmap_head *rmap_head)
        bool flush = false;
 
        for_each_rmap_spte(rmap_head, &iter, sptep)
-               flush |= spte_clear_dirty(kvm, sptep);
+               flush |= spte_clear_dirty(sptep);
 
        return flush;
 }
 
-static bool spte_set_dirty(struct kvm *kvm, u64 *sptep)
+static bool spte_set_dirty(u64 *sptep)
 {
        u64 spte = *sptep;
 
@@ -1279,7 +1279,7 @@ static bool __rmap_set_dirty(struct kvm *kvm, struct kvm_rmap_head *rmap_head)
        bool flush = false;
 
        for_each_rmap_spte(rmap_head, &iter, sptep)
-               flush |= spte_set_dirty(kvm, sptep);
+               flush |= spte_set_dirty(sptep);
 
        return flush;
 }