On Mon, Aug 14, 2017 at 12:09:14PM +0900, Minchan Kim wrote: > @@ -446,9 +450,7 @@ void tlb_finish_mmu(struct mmu_gather *tlb, > * > */ > bool force = mm_tlb_flush_nested(tlb->mm); > - > arch_tlb_finish_mmu(tlb, start, end, force); > - dec_tlb_flush_pending(tlb->mm); > }
No, I think this breaks all the mm_tlb_flush_pending() users. They need the decrement to not be visible until the TLB flush is complete.