Re: [PATCH v2 0/5] pid: add pidfd_open()

From: Christian Brauner
Date: Sat Mar 30 2019 - 12:19:42 EST


On March 30, 2019 5:09:10 PM GMT+01:00, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> wrote:
>On Fri, Mar 29, 2019 at 8:54 AM Christian Brauner
><christian@xxxxxxxxxx> wrote:
>>
>> /* Introduction */
>> This adds the pidfd_open() syscall.
>> pidfd_open() allows to retrieve file descriptors for a given pid.
>This
>> includes both file descriptors for processes and file descriptors for
>> threads.
>
>I'm ok with the pidfd_open() call to just get a pidfd, but that

From pure API perspective that's all I care about: independence of procfs.
Once we have pidfd_open() we can cleanly signal threads etc.

>"pidfd_to_profs()" needs to go away.
>
>If you want to open the /proc/<pid>/status file, then just do that.
>This whole "do one and convert to the other" needs to die. No, no, no.
>
> Linus