Sounds like a obvious bug to me. Other places doing tlb flush do the right 
thing.

Signed-off-by: Shaohua Li <[EMAIL PROTECTED]>

Index: kvm-28/kernel/mmu.c
===================================================================
--- kvm-28.orig/kernel/mmu.c    2007-06-20 16:52:46.000000000 +0800
+++ kvm-28/kernel/mmu.c 2007-06-20 17:10:02.000000000 +0800
@@ -441,8 +441,8 @@
                BUG_ON(!(*spte & PT_WRITABLE_MASK));
                rmap_printk("rmap_write_protect: spte %p %llx\n", spte, *spte);
                rmap_remove(vcpu, spte);
-               kvm_flush_remote_tlbs(vcpu->kvm);
                set_shadow_pte(spte, *spte & ~PT_WRITABLE_MASK);
+               kvm_flush_remote_tlbs(vcpu->kvm);
        }
 }
 

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel

Reply via email to