Re: [PATCH 08/12] KVM: MMU: move zap/flush to kvm_mmu_get_page

2016-02-25 Thread Paolo Bonzini
On 25/02/2016 08:32, Xiao Guangrong wrote: > > As you have already merged this patchset to the kvm tree, i will post > a patch to comment these cases to make the code be more understandable. I've only merged it to kvm/queue so that it gets into all my testing (and the buildbot's). I won't

Re: [PATCH 08/12] KVM: MMU: move zap/flush to kvm_mmu_get_page

2016-02-25 Thread Paolo Bonzini
On 25/02/2016 08:32, Xiao Guangrong wrote: > > As you have already merged this patchset to the kvm tree, i will post > a patch to comment these cases to make the code be more understandable. I've only merged it to kvm/queue so that it gets into all my testing (and the buildbot's). I won't

Re: [PATCH 08/12] KVM: MMU: move zap/flush to kvm_mmu_get_page

2016-02-24 Thread Xiao Guangrong
On 02/24/2016 09:17 PM, Paolo Bonzini wrote: kvm_mmu_get_page is the only caller of kvm_sync_page_transient and kvm_sync_pages. Moving the handling of the invalid_list there removes the need for the underdocumented kvm_sync_page_transient function. Signed-off-by: Paolo Bonzini

Re: [PATCH 08/12] KVM: MMU: move zap/flush to kvm_mmu_get_page

2016-02-24 Thread Xiao Guangrong
On 02/24/2016 09:17 PM, Paolo Bonzini wrote: kvm_mmu_get_page is the only caller of kvm_sync_page_transient and kvm_sync_pages. Moving the handling of the invalid_list there removes the need for the underdocumented kvm_sync_page_transient function. Signed-off-by: Paolo Bonzini ---

[PATCH 08/12] KVM: MMU: move zap/flush to kvm_mmu_get_page

2016-02-24 Thread Paolo Bonzini
kvm_mmu_get_page is the only caller of kvm_sync_page_transient and kvm_sync_pages. Moving the handling of the invalid_list there removes the need for the underdocumented kvm_sync_page_transient function. Signed-off-by: Paolo Bonzini --- Guangrong, at this point I am

[PATCH 08/12] KVM: MMU: move zap/flush to kvm_mmu_get_page

2016-02-24 Thread Paolo Bonzini
kvm_mmu_get_page is the only caller of kvm_sync_page_transient and kvm_sync_pages. Moving the handling of the invalid_list there removes the need for the underdocumented kvm_sync_page_transient function. Signed-off-by: Paolo Bonzini --- Guangrong, at this point I am confused about why