On Thu 16-11-17 09:44:57, Minchan Kim wrote: > On Wed, Nov 15, 2017 at 09:14:52AM +0100, Michal Hocko wrote: > > On Mon 13-11-17 09:28:33, Minchan Kim wrote: > > [...] > > > void arch_tlb_gather_mmu(...) > > > > > > tlb->fullmm = !(start | (end + 1)) && atomic_read(&mm->mm_users) == 0; > > > > Sorry, I should have realized sooner but this will not work for the oom > > reaper. It _can_ race with the final exit_mmap and run with mm_users == 0 > > If someone see mm_users is zero, it means there is no user to access > address space by stale TLB. Am I missing something? You are probably right but changing the flushing policy in the middle of the address space tear down makes me nervous. While this might work right now, it is kind of tricky and it has some potential to kick us back in future. Just note how the current arm64 optimization went unnoticed because the the oom reaper is such a rare event that nobody has actually noticed this. And I suspect that the likelyhood of failure is very low even when applied for anybody to notice in the real life. So I would very much like to make the behavior really explicit for everybody to see what is going on there. -- Michal Hocko SUSE Labs -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>
