Re: [PATCH 5/5] uprobes: kill insert_vm_struct()->uprobe_mmap()

2012-07-13 Thread Srikar Dronamraju
* Oleg Nesterov [2012-07-13 15:29:16]: > On 07/13, Srikar Dronamraju wrote: > > > > * Oleg Nesterov [2012-07-08 22:30:11]: > > > > > Kill insert_vm_struct()->uprobe_mmap(). It is not needed, nobody > > > except arch/ia64/kernel/perfmon.c uses insert_vm_struct(vma) with > > > vma->vm_file !=

Re: [PATCH 5/5] uprobes: kill insert_vm_struct()->uprobe_mmap()

2012-07-13 Thread Srikar Dronamraju
* Oleg Nesterov [2012-07-08 22:30:11]: > Kill insert_vm_struct()->uprobe_mmap(). It is not needed, nobody > except arch/ia64/kernel/perfmon.c uses insert_vm_struct(vma) with > vma->vm_file != NULL. > > And it is wrong. Again, get_user_pages() can not succeed before > vma_link(vma) makes is

Re: [PATCH 5/5] uprobes: kill insert_vm_struct()->uprobe_mmap()

2012-07-13 Thread Oleg Nesterov
On 07/13, Srikar Dronamraju wrote: > > * Oleg Nesterov [2012-07-08 22:30:11]: > > > Kill insert_vm_struct()->uprobe_mmap(). It is not needed, nobody > > except arch/ia64/kernel/perfmon.c uses insert_vm_struct(vma) with > > vma->vm_file != NULL. > > > > Right, but somebody else might start using

Re: [PATCH 5/5] uprobes: kill insert_vm_struct()->uprobe_mmap()

2012-07-13 Thread Srikar Dronamraju
* Oleg Nesterov [2012-07-08 22:30:11]: > Kill insert_vm_struct()->uprobe_mmap(). It is not needed, nobody > except arch/ia64/kernel/perfmon.c uses insert_vm_struct(vma) with > vma->vm_file != NULL. > Right, but somebody else might start using this later. I cant think of a use case though. >

Re: [PATCH 5/5] uprobes: kill insert_vm_struct()-uprobe_mmap()

2012-07-13 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-07-08 22:30:11]: Kill insert_vm_struct()-uprobe_mmap(). It is not needed, nobody except arch/ia64/kernel/perfmon.c uses insert_vm_struct(vma) with vma-vm_file != NULL. Right, but somebody else might start using this later. I cant think of a use case

Re: [PATCH 5/5] uprobes: kill insert_vm_struct()-uprobe_mmap()

2012-07-13 Thread Oleg Nesterov
On 07/13, Srikar Dronamraju wrote: * Oleg Nesterov o...@redhat.com [2012-07-08 22:30:11]: Kill insert_vm_struct()-uprobe_mmap(). It is not needed, nobody except arch/ia64/kernel/perfmon.c uses insert_vm_struct(vma) with vma-vm_file != NULL. Right, but somebody else might start using

Re: [PATCH 5/5] uprobes: kill insert_vm_struct()-uprobe_mmap()

2012-07-13 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-07-08 22:30:11]: Kill insert_vm_struct()-uprobe_mmap(). It is not needed, nobody except arch/ia64/kernel/perfmon.c uses insert_vm_struct(vma) with vma-vm_file != NULL. And it is wrong. Again, get_user_pages() can not succeed before vma_link(vma) makes

Re: [PATCH 5/5] uprobes: kill insert_vm_struct()-uprobe_mmap()

2012-07-13 Thread Srikar Dronamraju
* Oleg Nesterov o...@redhat.com [2012-07-13 15:29:16]: On 07/13, Srikar Dronamraju wrote: * Oleg Nesterov o...@redhat.com [2012-07-08 22:30:11]: Kill insert_vm_struct()-uprobe_mmap(). It is not needed, nobody except arch/ia64/kernel/perfmon.c uses insert_vm_struct(vma) with

[PATCH 5/5] uprobes: kill insert_vm_struct()->uprobe_mmap()

2012-07-08 Thread Oleg Nesterov
Kill insert_vm_struct()->uprobe_mmap(). It is not needed, nobody except arch/ia64/kernel/perfmon.c uses insert_vm_struct(vma) with vma->vm_file != NULL. And it is wrong. Again, get_user_pages() can not succeed before vma_link(vma) makes is visible to find_vma(). And even if this worked, we must

[PATCH 5/5] uprobes: kill insert_vm_struct()-uprobe_mmap()

2012-07-08 Thread Oleg Nesterov
Kill insert_vm_struct()-uprobe_mmap(). It is not needed, nobody except arch/ia64/kernel/perfmon.c uses insert_vm_struct(vma) with vma-vm_file != NULL. And it is wrong. Again, get_user_pages() can not succeed before vma_link(vma) makes is visible to find_vma(). And even if this worked, we must not