Re: [PATCH 25/28] proc: Supply an accessor to get the process IDassociated with some proc files [RFC]

From: Tejun Heo
Date: Thu Apr 18 2013 - 14:40:00 EST


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-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/