r = 0;
@@ -1195,11 +1232,16 @@ void mark_page_dirty(struct kvm *kvm, gfn_t gfn)
        gfn = unalias_gfn(kvm, gfn);
        memslot = gfn_to_memslot_unaliased(kvm, gfn);
        if (memslot&&  memslot->dirty_bitmap) {
-               unsigned long rel_gfn = gfn - memslot->base_gfn;
+               int nr = generic_le_bit_offset(gfn - memslot->base_gfn);

-               generic___set_le_bit(rel_gfn, memslot->dirty_bitmap);
+               if (kvm_set_bit_user(nr, memslot->dirty_bitmap))
+                       goto out_fault;

mark_page_dirty is called with the mmu_lock spinlock held in set_spte.
Must find a way to move it outside of the spinlock section.


Oh, it's a serious problem. I have to consider it.


Thanks,
  Takuya
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to