From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Tosatti Subject: [patch 3/3] KVM: move postcommit flush to x86, as mmio sptes are x86 specific Date: Fri, 24 Aug 2012 15:54:59 -0300 Message-ID: <20120824185634.625676530@amt.cnet> References: <20120824185456.658814016@amt.cnet> Cc: Xiao Guangrong , kvm@vger.kernel.org, Marcelo Tosatti To: Paul Mackerras , Avi Kivity Return-path: Received: from mx1.redhat.com ([209.132.183.28]:28563 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759645Ab2HXS6M (ORCPT ); Fri, 24 Aug 2012 14:58:12 -0400 Content-Disposition: inline; filename=03-move-postcommit-flush-to-x86 Sender: kvm-owner@vger.kernel.org List-ID: Other arches do not need this. Signed-off-by: Marcelo Tosatti Index: kvm/arch/x86/kvm/x86.c =================================================================== --- kvm.orig/arch/x86/kvm/x86.c +++ kvm/arch/x86/kvm/x86.c @@ -6455,6 +6455,14 @@ void kvm_arch_commit_memory_region(struc kvm_mmu_change_mmu_pages(kvm, nr_mmu_pages); kvm_mmu_slot_remove_write_access(kvm, mem->slot); spin_unlock(&kvm->mmu_lock); + /* + * If the new memory slot is created, we need to clear all + * mmio sptes. + */ + if (old.npages == 0 && npages) { + kvm_mmu_zap_all(kvm); + kvm_reload_remote_mmus(kvm); + } } void kvm_arch_flush_shadow_all(struct kvm *kvm) Index: kvm/virt/kvm/kvm_main.c =================================================================== --- kvm.orig/virt/kvm/kvm_main.c +++ kvm/virt/kvm/kvm_main.c @@ -849,13 +849,6 @@ int __kvm_set_memory_region(struct kvm * kvm_arch_commit_memory_region(kvm, mem, old, user_alloc); - /* - * If the new memory slot is created, we need to clear all - * mmio sptes. - */ - if (npages && old.base_gfn != mem->guest_phys_addr >> PAGE_SHIFT) - kvm_arch_flush_shadow_all(kvm); - kvm_free_physmem_slot(&old, &new); kfree(old_memslots);