On Mon 06-11-17 13:00:25, Peter Zijlstra wrote: > On Mon, Nov 06, 2017 at 11:43:54AM +0100, Michal Hocko wrote: > > > Yes the comment is very much accurate. > > > > Which suggests that print_vma_addr might be problematic, right? > > Shouldn't we do trylock on mmap_sem instead? > > Yes that's complete rubbish. trylock will get spurious failures to print > when the lock is contended.
Yes, but I guess that it is acceptable to to not print the state under that condition. -- Michal Hocko SUSE Labs