On Mon, Apr 23, 2018 at 7:21 PM, Song Liu <[email protected]> wrote: > Caller of uprobe_register is required to keep the inode and containing > mount point referenced. > > There was misuse of igrab() in uprobes.c and trace_uprobe.c. This is > because igrab() will not prevent umount of the containing mount point. > To fix this, we added path to struct trace_uprobe, which keeps the inode > and containing mount reference. > > For uprobes.c, it is not necessary to call igrab() in uprobe_register(), > as the caller is required to keep the inode reference. The igrab() is > removed and comments on this requirement is added to uprobe_register(). > > Link: > http://lkml.kernel.org/r/CAELBmZB2XX=qeoladvgg4cpx4gentcsnwqqujluk1ongrj3...@mail.gmail.com > Cc: Steven Rostedt <[email protected]> > Cc: Ingo Molnar <[email protected]> > Cc: Howard McLauchlan <[email protected]> > Cc: Josef Bacik <[email protected]> > Cc: Srikar Dronamraju <[email protected]> > Cc: Miklos Szeredi <[email protected]> > Signed-off-by: Song Liu <[email protected]>
Acked-by: Miklos Szeredi <[email protected]>
