On Wed, 03 Jul 2013 10:53:51 +0200 Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote: > Il 03/07/2013 10:50, Xiao Guangrong ha scritto: > >> > Please wait a while. I can not understand it very clearly. > >> > > >> > This conditional check will cause caching a overflow value into mmio spte. > >> > The simple case is that kvm adds new slots for many times, the mmio-gen is easily > >> > more than MMIO_MAX_GEN. > >> > > > Actually, the double zapping can be avoided by moving kvm_mmu_invalidate_mmio_sptes to > > the end of install_new_memslots(). > > > > > > Yes, the actual operation would be the same as this patch. You can > rename kvm_mmu_invalidate_mmio_sptes to kvm_arch_memslots_installed, or > something like that. But it would have to touch all architectures. I tried to avoid introducing x86-centric code into the generic one. If another arch can gain something by such function, I'm willing to touch all arch code. Takuya -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html