Re: [PATCH] KVM/MMU: Combine flushing remote tlb in mmu_set_spte()

2018-07-25 Thread Xiao Guangrong
On 07/24/2018 10:35 PM, Paolo Bonzini wrote: On 24/07/2018 10:17, Tianyu Lan wrote: mmu_set_spte() flushes remote tlbs for drop_parent_pte/drop_spte() and set_spte() separately. This may introduce redundant flush. This patch is to combine these flushes and check flush request after calling

Re: [PATCH] KVM/MMU: Combine flushing remote tlb in mmu_set_spte()

2018-07-25 Thread Xiao Guangrong
On 07/24/2018 10:35 PM, Paolo Bonzini wrote: On 24/07/2018 10:17, Tianyu Lan wrote: mmu_set_spte() flushes remote tlbs for drop_parent_pte/drop_spte() and set_spte() separately. This may introduce redundant flush. This patch is to combine these flushes and check flush request after calling

Re: [PATCH] KVM/MMU: Combine flushing remote tlb in mmu_set_spte()

2018-07-24 Thread Tianyu Lan
Hi Junaid: Thanks for your review. On 7/25/2018 10:26 AM, Junaid Shahid wrote: > On 07/24/2018 07:35 AM, Paolo Bonzini wrote: >> On 24/07/2018 10:17, Tianyu Lan wrote: >>> mmu_set_spte() flushes remote tlbs for drop_parent_pte/drop_spte() >>> and set_spte() separately. This may introduce

Re: [PATCH] KVM/MMU: Combine flushing remote tlb in mmu_set_spte()

2018-07-24 Thread Tianyu Lan
Hi Junaid: Thanks for your review. On 7/25/2018 10:26 AM, Junaid Shahid wrote: > On 07/24/2018 07:35 AM, Paolo Bonzini wrote: >> On 24/07/2018 10:17, Tianyu Lan wrote: >>> mmu_set_spte() flushes remote tlbs for drop_parent_pte/drop_spte() >>> and set_spte() separately. This may introduce

Re: [PATCH] KVM/MMU: Combine flushing remote tlb in mmu_set_spte()

2018-07-24 Thread Junaid Shahid
On 07/24/2018 07:35 AM, Paolo Bonzini wrote: > On 24/07/2018 10:17, Tianyu Lan wrote: >> mmu_set_spte() flushes remote tlbs for drop_parent_pte/drop_spte() >> and set_spte() separately. This may introduce redundant flush. This >> patch is to combine these flushes and check flush request after >>

Re: [PATCH] KVM/MMU: Combine flushing remote tlb in mmu_set_spte()

2018-07-24 Thread Junaid Shahid
On 07/24/2018 07:35 AM, Paolo Bonzini wrote: > On 24/07/2018 10:17, Tianyu Lan wrote: >> mmu_set_spte() flushes remote tlbs for drop_parent_pte/drop_spte() >> and set_spte() separately. This may introduce redundant flush. This >> patch is to combine these flushes and check flush request after >>

Re: [PATCH] KVM/MMU: Combine flushing remote tlb in mmu_set_spte()

2018-07-24 Thread Paolo Bonzini
On 24/07/2018 10:17, Tianyu Lan wrote: > mmu_set_spte() flushes remote tlbs for drop_parent_pte/drop_spte() > and set_spte() separately. This may introduce redundant flush. This > patch is to combine these flushes and check flush request after > calling set_spte(). > > Signed-off-by: Lan Tianyu

Re: [PATCH] KVM/MMU: Combine flushing remote tlb in mmu_set_spte()

2018-07-24 Thread Paolo Bonzini
On 24/07/2018 10:17, Tianyu Lan wrote: > mmu_set_spte() flushes remote tlbs for drop_parent_pte/drop_spte() > and set_spte() separately. This may introduce redundant flush. This > patch is to combine these flushes and check flush request after > calling set_spte(). > > Signed-off-by: Lan Tianyu

[PATCH] KVM/MMU: Combine flushing remote tlb in mmu_set_spte()

2018-07-24 Thread Tianyu Lan
mmu_set_spte() flushes remote tlbs for drop_parent_pte/drop_spte() and set_spte() separately. This may introduce redundant flush. This patch is to combine these flushes and check flush request after calling set_spte(). Signed-off-by: Lan Tianyu --- arch/x86/kvm/mmu.c | 7 --- 1 file

[PATCH] KVM/MMU: Combine flushing remote tlb in mmu_set_spte()

2018-07-24 Thread Tianyu Lan
mmu_set_spte() flushes remote tlbs for drop_parent_pte/drop_spte() and set_spte() separately. This may introduce redundant flush. This patch is to combine these flushes and check flush request after calling set_spte(). Signed-off-by: Lan Tianyu --- arch/x86/kvm/mmu.c | 7 --- 1 file