> > > > rcu_read_lock(); > > tsk = mm->owner; > > if (tsk) > > get_task_struct(tsk); > > rcu_read_unlock(); > > if (!tsk) > > return ret; > > > > Probably looks cleaner. > > Yes, plus we should do "tsk = rcu_dereference(mm->owner);" and wrap the > whole thing in a static uprobes_get_mm_owner() or similar. > Okay, will do that. -- 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 internet charges in Canada: sign http://stopthemeter.ca/ Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>