Hello, David. On Tue, Apr 16, 2013 at 07:27:30PM +0100, David Howells wrote: > +struct pid *get_proc_pid(const struct inode *inode) > +{ > + return PROC_I(inode)->pid; > +} Maybe we can add /** comment explaining what this is about? Other than that, it looks fine to me. It probably would be best to route this through -mm given it touches /proc too, so cc Andrew on the next posting? Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html