On 11/09/2010 04:06 PM, Gleb Natapov wrote: > On Thu, Nov 04, 2010 at 06:36:36PM +0800, Xiao Guangrong wrote: >> Retry #PF for softmmu only when the current vcpu has the same >> root shadow page as the time when #PF occurs. it means they >> have same paging environment. >> > Avi had an idea to allocate spte at the fault time, get reference > to it and populate it on completion instead of prefaulting. How hard > will it be? > Um. if we do it like this, we can't populate it directly, we should: - hold mmu_lock and walk vcpu's page table to see whether this spte is mapping in vcpu's page table (since the middle mapping can be zapped during apf is requesting and shadow page table can be switched) - the guest page mapping can be modified by other vcpu or other process, we need re-walk guest page table. I prefer to the way in this patch since it's very light. -- 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