Re: [PATCH v5 4/7] pid: export pidfd_get_pid

From: Minchan Kim
Date: Tue Feb 18 2020 - 20:02:56 EST


On Sun, Feb 16, 2020 at 11:59:40PM -0800, Christoph Hellwig wrote:
> On Fri, Feb 14, 2020 at 09:05:17AM -0800, Minchan Kim wrote:
> > process_madvise syscall needs pidfd_get_pid function to translate
> > pidfd to pid so this patch exports the function.
>
> For that it should not need to exported, but then again the actual
> patch doesn't export it anyway, so this is just a commit log issue.
>
> > extern struct pid *pidfd_pid(const struct file *file);
> > +extern struct pid *pidfd_get_pid(unsigned int fd);
>
> ... and there is no need for the extern keyword on prototypes in
> headers.

Will fix new version. Thanks for the review!