Re: [PATCH] Add pidfs filesystem

2017-02-27 Thread Michael Kerrisk
[CC += linux-...@vger.kernel.org] Hi Alexey This is a change to the kernel-user-space API. Please CC linux-...@vger.kernel.org on any future iterations of this patch. Thanks, Michael On Sat, Feb 18, 2017 at 11:53 PM, Alexey Gladkov wrote: > The pidfs filesystem contains a subset of the /proc

Re: [PATCH] Add pidfs filesystem

2017-02-22 Thread Richard Weinberger
On Mon, Feb 20, 2017 at 5:05 AM, Eric W. Biederman wrote: > Alexey Gladkov writes: > >> The pidfs filesystem contains a subset of the /proc file system which >> contains only information about the processes. > > My summary of your motivation. > > It hurts when I create a container with a processe

Re: [PATCH] Add pidfs filesystem

2017-02-22 Thread Alexey Gladkov
On Wed, Feb 22, 2017 at 06:48:03PM +0100, Oleg Nesterov wrote: > On 02/22, Dmitry V. Levin wrote: > > > > On Tue, Feb 21, 2017 at 03:57:47PM +0100, Oleg Nesterov wrote: > > > > > > But perhaps we can use mnt_flags? For example, lets abuse MNT_NODEV, see > > > the simple patch below. Not sure it is

Re: [PATCH] Add pidfs filesystem

2017-02-22 Thread Oleg Nesterov
On 02/22, Dmitry V. Levin wrote: > > On Tue, Feb 21, 2017 at 03:57:47PM +0100, Oleg Nesterov wrote: > > > > But perhaps we can use mnt_flags? For example, lets abuse MNT_NODEV, see > > the simple patch below. Not sure it is correct/complete, just to illustrate > > the idea. ... > I like the idea

Re: [PATCH] Add pidfs filesystem

2017-02-22 Thread Dmitry V. Levin
On Tue, Feb 21, 2017 at 03:57:47PM +0100, Oleg Nesterov wrote: > On 02/18, Alexey Gladkov wrote: > > > > This patch allows to mount only the part of /proc related to pids > > without rest objects. Since this is an addon to /proc, flags applied to > > /proc have an effect on this pidfs filesystem. >

Re: [PATCH] Add pidfs filesystem

2017-02-22 Thread Pavel Emelyanov
On 02/22/2017 03:04 PM, Alexey Gladkov wrote: > On Wed, Feb 22, 2017 at 10:40:49AM +0300, Pavel Emelyanov wrote: >> On 02/21/2017 05:57 PM, Oleg Nesterov wrote: >>> On 02/18, Alexey Gladkov wrote: This patch allows to mount only the part of /proc related to pids without rest objects.

Re: [PATCH] Add pidfs filesystem

2017-02-22 Thread Alexey Gladkov
On Wed, Feb 22, 2017 at 10:40:49AM +0300, Pavel Emelyanov wrote: > On 02/21/2017 05:57 PM, Oleg Nesterov wrote: > > On 02/18, Alexey Gladkov wrote: > >> > >> This patch allows to mount only the part of /proc related to pids > >> without rest objects. Since this is an addon to /proc, flags applied t

Re: [PATCH] Add pidfs filesystem

2017-02-22 Thread Alexey Gladkov
On Tue, Feb 21, 2017 at 03:57:47PM +0100, Oleg Nesterov wrote: > On 02/18, Alexey Gladkov wrote: > > > > This patch allows to mount only the part of /proc related to pids > > without rest objects. Since this is an addon to /proc, flags applied to > > /proc have an effect on this pidfs filesystem. >

Re: [PATCH] Add pidfs filesystem

2017-02-21 Thread Pavel Emelyanov
On 02/21/2017 05:57 PM, Oleg Nesterov wrote: > On 02/18, Alexey Gladkov wrote: >> >> This patch allows to mount only the part of /proc related to pids >> without rest objects. Since this is an addon to /proc, flags applied to >> /proc have an effect on this pidfs filesystem. > > I leave this to yo

Re: [PATCH] Add pidfs filesystem

2017-02-21 Thread Oleg Nesterov
On 02/18, Alexey Gladkov wrote: > > This patch allows to mount only the part of /proc related to pids > without rest objects. Since this is an addon to /proc, flags applied to > /proc have an effect on this pidfs filesystem. I leave this to you and Eric, but imo it would be nice to avoid another f

Re: [PATCH] Add pidfs filesystem

2017-02-20 Thread Alexey Gladkov
On Mon, Feb 20, 2017 at 05:05:18PM +1300, Eric W. Biederman wrote: > Alexey Gladkov writes: > > > The pidfs filesystem contains a subset of the /proc file system which > > contains only information about the processes. > > My summary of your motivation. > > It hurts when I create a container wi

Re: [PATCH] Add pidfs filesystem

2017-02-19 Thread Eric W. Biederman
Alexey Gladkov writes: > The pidfs filesystem contains a subset of the /proc file system which > contains only information about the processes. My summary of your motivation. It hurts when I create a container with a processes with uid 0 inside of it. This generates lots of hacks to attempt to

Re: [PATCH] Add pidfs filesystem

2017-02-18 Thread kbuild test robot
Hi Alexey, [auto build test WARNING on linus/master] [also build test WARNING on v4.10-rc8 next-20170217] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Alexey-Gladkov/Add-pidfs-filesystem/20170

Re: [PATCH] Add pidfs filesystem

2017-02-18 Thread kbuild test robot
Hi Alexey, [auto build test ERROR on linus/master] [also build test ERROR on v4.10-rc8 next-20170217] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Alexey-Gladkov/Add-pidfs-filesystem/20170219-

[PATCH] Add pidfs filesystem

2017-02-18 Thread Alexey Gladkov
The pidfs filesystem contains a subset of the /proc file system which contains only information about the processes. Some of the container virtualization systems are mounted /proc inside the container. This is done in most cases to operate with information about the processes. Knowing that /proc f