Re: [PATCH 2/5] uprobes: suppress uprobe_munmap() from mmput()

2012-07-11 Thread Srikar Dronamraju
* Oleg Nesterov [2012-07-08 22:30:03]: > uprobe_munmap() does get_user_pages() and it is also called from > the final mmput()->exit_mmap() path. This slows down exit/mmput() > for no reason, and I think it is simply dangerous/wrong to try to > fault-in a page into the dying mm. If nothing else,

Re: [PATCH 2/5] uprobes: suppress uprobe_munmap() from mmput()

2012-07-09 Thread Srikar Dronamraju
* Oleg Nesterov [2012-07-09 12:09:20]: > On 07/09, Peter Zijlstra wrote: > > > > On Sun, 2012-07-08 at 22:30 +0200, Oleg Nesterov wrote: > > > uprobe_munmap() does get_user_pages() and it is also called from > > > the final mmput()->exit_mmap() path. This slows down exit/mmput() > > > for no reas

Re: [PATCH 2/5] uprobes: suppress uprobe_munmap() from mmput()

2012-07-09 Thread Peter Zijlstra
On Mon, 2012-07-09 at 12:09 +0200, Oleg Nesterov wrote: > No, mmap/munmap do not participate in uprobe refcounting. This code > does put_uprobe() for each uprobe, yes, but only because the counter > was incremented in build_probe_list(). > > uprobe_munmap() is only needed to decrement mm->uprobes_

Re: [PATCH 2/5] uprobes: suppress uprobe_munmap() from mmput()

2012-07-09 Thread Oleg Nesterov
On 07/09, Peter Zijlstra wrote: > > On Sun, 2012-07-08 at 22:30 +0200, Oleg Nesterov wrote: > > uprobe_munmap() does get_user_pages() and it is also called from > > the final mmput()->exit_mmap() path. This slows down exit/mmput() > > for no reason, and I think it is simply dangerous/wrong to try

Re: [PATCH 2/5] uprobes: suppress uprobe_munmap() from mmput()

2012-07-09 Thread Peter Zijlstra
On Sun, 2012-07-08 at 22:30 +0200, Oleg Nesterov wrote: > uprobe_munmap() does get_user_pages() and it is also called from > the final mmput()->exit_mmap() path. This slows down exit/mmput() > for no reason, and I think it is simply dangerous/wrong to try to > fault-in a page into the dying mm. If

[PATCH 2/5] uprobes: suppress uprobe_munmap() from mmput()

2012-07-08 Thread Oleg Nesterov
uprobe_munmap() does get_user_pages() and it is also called from the final mmput()->exit_mmap() path. This slows down exit/mmput() for no reason, and I think it is simply dangerous/wrong to try to fault-in a page into the dying mm. If nothing else, this happens after the last sync_mm_rss(), afaics