On 04/21, Jiri Olsa wrote:
>
> @@ -1483,7 +1483,7 @@ static int unapply_uprobe(struct uprobe *uprobe, struct
> mm_struct *mm)
> struct vm_area_struct *vma;
> int err = 0;
>
> - mmap_read_lock(mm);
> + mmap_write_lock(mm);
So uprobe_write_opcode() is always called under down_write(), right?
Then this
* Called with mm->mmap_lock held for read or write.
comment should be probably updated.
And perhaps the comment above mmap_write_lock() in register_for_each_vma()
should be updated too... or even removed.
Oleg.
