* Peter Zijlstra <peterz@xxxxxxxxxxxxx> [2011-01-26 16:16:49]: > On Wed, 2011-01-26 at 20:29 +0530, Srikar Dronamraju wrote: > > list_for_each_entry_safe(mm, tmpmm, &tmp_list, uprobes_list) { > > down_read(&mm->map_sem); > > if (!install_uprobe(mm, uprobe)) > > ret = 0; > > up_read(&mm->map_sem); > > list_del(&mm->uprobes_list); > > mmput(mm); > > } > > and the tmp_list thing works because new mm's will hit the mmap callback > and you cannot loose mm's due to the refcount, right? > Right, In other words, the tmp_list has all mm's that have already running and have this inode mapped as executable text. Those process that are yet to start or yet to map the inode as executable text will hit mmap and then we look at inserting the probes thro uprobes_mmap. -- Thanks and Regards Srikar > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxxx For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom policy in Canada: sign http://dissolvethecrtc.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>