KVM: MMU: fix forgot flush tlbs on sync_page path
authorXiao Guangrong <xiaoguangrong@cn.fujitsu.com>
Fri, 19 Nov 2010 09:01:40 +0000 (17:01 +0800)
committerAvi Kivity <avi@redhat.com>
Wed, 12 Jan 2011 09:29:43 +0000 (11:29 +0200)
We should flush all tlbs after drop spte on sync_page path since

Quote from Avi:
| sync_page
| drop_spte
| kvm_mmu_notifier_invalidate_page
| kvm_unmap_rmapp
| spte doesn't exist -> no flush
| page is freed
| guest can write into freed page?

KVM-Stable-Tag.
Signed-off-by: Xiao Guangrong <xiaoguangrong@cn.fujitsu.com>
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
arch/x86/kvm/paging_tmpl.h

index 590bf1223cb422024c29be2f4b3fb0ce5bd57106..ca0e5e834724afc4c737b2669736af84437a2c51 100644 (file)
@@ -786,6 +786,7 @@ static int FNAME(sync_page)(struct kvm_vcpu *vcpu, struct kvm_mmu_page *sp,
                        else
                                nonpresent = shadow_notrap_nonpresent_pte;
                        drop_spte(vcpu->kvm, &sp->spt[i], nonpresent);
+                       kvm_flush_remote_tlbs(vcpu->kvm);
                        continue;
                }