KVM: x86: MMU: Use for_each_rmap_spte macro instead of pte_list_walk()
authorTakuya Yoshikawa <yoshikawa_takuya_b1@lab.ntt.co.jp>
Thu, 26 Nov 2015 12:15:38 +0000 (21:15 +0900)
committerPaolo Bonzini <pbonzini@redhat.com>
Thu, 26 Nov 2015 14:31:31 +0000 (15:31 +0100)
As kvm_mmu_get_page() was changed so that every parent pointer would not
get into the sp->parent_ptes chain before the entry pointed to by it was
set properly, we can use the for_each_rmap_spte macro instead of
pte_list_walk().

Signed-off-by: Takuya Yoshikawa <yoshikawa_takuya_b1@lab.ntt.co.jp>
Cc: Xiao Guangrong <guangrong.xiao@linux.intel.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
arch/x86/kvm/mmu.c

index ec61b223da0ad82ae24381effb18ece1ce1cea7c..204c7d4763d4c6b98c36fa1b31434ae0227791c7 100644 (file)
@@ -1007,26 +1007,6 @@ static void pte_list_remove(u64 *spte, struct kvm_rmap_head *rmap_head)
        }
 }
 
-typedef void (*pte_list_walk_fn) (u64 *spte);
-static void pte_list_walk(struct kvm_rmap_head *rmap_head, pte_list_walk_fn fn)
-{
-       struct pte_list_desc *desc;
-       int i;
-
-       if (!rmap_head->val)
-               return;
-
-       if (!(rmap_head->val & 1))
-               return fn((u64 *)rmap_head->val);
-
-       desc = (struct pte_list_desc *)(rmap_head->val & ~1ul);
-       while (desc) {
-               for (i = 0; i < PTE_LIST_EXT && desc->sptes[i]; ++i)
-                       fn(desc->sptes[i]);
-               desc = desc->more;
-       }
-}
-
 static struct kvm_rmap_head *__gfn_to_rmap(gfn_t gfn, int level,
                                           struct kvm_memory_slot *slot)
 {
@@ -1749,7 +1729,12 @@ static struct kvm_mmu_page *kvm_mmu_alloc_page(struct kvm_vcpu *vcpu, int direct
 static void mark_unsync(u64 *spte);
 static void kvm_mmu_mark_parents_unsync(struct kvm_mmu_page *sp)
 {
-       pte_list_walk(&sp->parent_ptes, mark_unsync);
+       u64 *sptep;
+       struct rmap_iterator iter;
+
+       for_each_rmap_spte(&sp->parent_ptes, &iter, sptep) {
+               mark_unsync(sptep);
+       }
 }
 
 static void mark_unsync(u64 *spte)