Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-22 Thread Tejun Heo
Hello, On Wed, Oct 22, 2014 at 11:37:55AM -0700, Aditya Kali wrote: ... > Actually, there is no right answer here. Our options are: > * show relative path > -- this will break userspace as /proc//cgroup does not show > relative paths today. This is also very ambiguous (is it relative to >

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-22 Thread Andy Lutomirski
On Wed, Oct 22, 2014 at 11:37 AM, Aditya Kali wrote: > On Tue, Oct 21, 2014 at 5:58 PM, Andy Lutomirski wrote: >> On Tue, Oct 21, 2014 at 5:46 PM, Aditya Kali wrote: >>> On Tue, Oct 21, 2014 at 3:42 PM, Andy Lutomirski >>> wrote: On Tue, Oct 21, 2014 at 3:33 PM, Aditya Kali wrote:

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-22 Thread Aditya Kali
On Tue, Oct 21, 2014 at 5:58 PM, Andy Lutomirski wrote: > On Tue, Oct 21, 2014 at 5:46 PM, Aditya Kali wrote: >> On Tue, Oct 21, 2014 at 3:42 PM, Andy Lutomirski wrote: >>> On Tue, Oct 21, 2014 at 3:33 PM, Aditya Kali wrote: On Tue, Oct 21, 2014 at 12:02 PM, Andy Lutomirski wrote:

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-22 Thread Aditya Kali
On Tue, Oct 21, 2014 at 5:58 PM, Andy Lutomirski l...@amacapital.net wrote: On Tue, Oct 21, 2014 at 5:46 PM, Aditya Kali adityak...@google.com wrote: On Tue, Oct 21, 2014 at 3:42 PM, Andy Lutomirski l...@amacapital.net wrote: On Tue, Oct 21, 2014 at 3:33 PM, Aditya Kali adityak...@google.com

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-22 Thread Andy Lutomirski
On Wed, Oct 22, 2014 at 11:37 AM, Aditya Kali adityak...@google.com wrote: On Tue, Oct 21, 2014 at 5:58 PM, Andy Lutomirski l...@amacapital.net wrote: On Tue, Oct 21, 2014 at 5:46 PM, Aditya Kali adityak...@google.com wrote: On Tue, Oct 21, 2014 at 3:42 PM, Andy Lutomirski l...@amacapital.net

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-22 Thread Tejun Heo
Hello, On Wed, Oct 22, 2014 at 11:37:55AM -0700, Aditya Kali wrote: ... Actually, there is no right answer here. Our options are: * show relative path -- this will break userspace as /proc/pid/cgroup does not show relative paths today. This is also very ambiguous (is it relative to

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Andy Lutomirski
On Tue, Oct 21, 2014 at 5:46 PM, Aditya Kali wrote: > On Tue, Oct 21, 2014 at 3:42 PM, Andy Lutomirski wrote: >> On Tue, Oct 21, 2014 at 3:33 PM, Aditya Kali wrote: >>> On Tue, Oct 21, 2014 at 12:02 PM, Andy Lutomirski >>> wrote: On Tue, Oct 21, 2014 at 11:49 AM, Aditya Kali

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Aditya Kali
On Tue, Oct 21, 2014 at 3:42 PM, Andy Lutomirski wrote: > On Tue, Oct 21, 2014 at 3:33 PM, Aditya Kali wrote: >> On Tue, Oct 21, 2014 at 12:02 PM, Andy Lutomirski >> wrote: >>> On Tue, Oct 21, 2014 at 11:49 AM, Aditya Kali wrote: On Mon, Oct 20, 2014 at 10:49 PM, Andy Lutomirski

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Andy Lutomirski
On Tue, Oct 21, 2014 at 3:33 PM, Aditya Kali wrote: > On Tue, Oct 21, 2014 at 12:02 PM, Andy Lutomirski wrote: >> On Tue, Oct 21, 2014 at 11:49 AM, Aditya Kali wrote: >>> On Mon, Oct 20, 2014 at 10:49 PM, Andy Lutomirski >>> wrote: On Mon, Oct 20, 2014 at 10:42 PM, Eric W. Biederman

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Aditya Kali
On Tue, Oct 21, 2014 at 12:02 PM, Andy Lutomirski wrote: > On Tue, Oct 21, 2014 at 11:49 AM, Aditya Kali wrote: >> On Mon, Oct 20, 2014 at 10:49 PM, Andy Lutomirski >> wrote: >>> On Mon, Oct 20, 2014 at 10:42 PM, Eric W. Biederman >>> wrote: I do wonder if we think of this as

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Andy Lutomirski
On Tue, Oct 21, 2014 at 11:49 AM, Aditya Kali wrote: > On Mon, Oct 20, 2014 at 10:49 PM, Andy Lutomirski wrote: >> On Mon, Oct 20, 2014 at 10:42 PM, Eric W. Biederman >> wrote: >>> >>> I do wonder if we think of this as chcgrouproot if there is a simpler >>> implementation. >> >> Could be.

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Aditya Kali
On Mon, Oct 20, 2014 at 10:49 PM, Andy Lutomirski wrote: > On Mon, Oct 20, 2014 at 10:42 PM, Eric W. Biederman > wrote: >> Andy Lutomirski writes: >> >>> On Mon, Oct 20, 2014 at 9:49 PM, Eric W. Biederman >>> wrote: Andy Lutomirski writes: > Possible solution: > > Ditch the

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Aditya Kali
On Mon, Oct 20, 2014 at 10:49 PM, Andy Lutomirski l...@amacapital.net wrote: On Mon, Oct 20, 2014 at 10:42 PM, Eric W. Biederman ebied...@xmission.com wrote: Andy Lutomirski l...@amacapital.net writes: On Mon, Oct 20, 2014 at 9:49 PM, Eric W. Biederman ebied...@xmission.com wrote: Andy

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Andy Lutomirski
On Tue, Oct 21, 2014 at 11:49 AM, Aditya Kali adityak...@google.com wrote: On Mon, Oct 20, 2014 at 10:49 PM, Andy Lutomirski l...@amacapital.net wrote: On Mon, Oct 20, 2014 at 10:42 PM, Eric W. Biederman ebied...@xmission.com wrote: I do wonder if we think of this as chcgrouproot if there is

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Aditya Kali
On Tue, Oct 21, 2014 at 12:02 PM, Andy Lutomirski l...@amacapital.net wrote: On Tue, Oct 21, 2014 at 11:49 AM, Aditya Kali adityak...@google.com wrote: On Mon, Oct 20, 2014 at 10:49 PM, Andy Lutomirski l...@amacapital.net wrote: On Mon, Oct 20, 2014 at 10:42 PM, Eric W. Biederman

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Andy Lutomirski
On Tue, Oct 21, 2014 at 3:33 PM, Aditya Kali adityak...@google.com wrote: On Tue, Oct 21, 2014 at 12:02 PM, Andy Lutomirski l...@amacapital.net wrote: On Tue, Oct 21, 2014 at 11:49 AM, Aditya Kali adityak...@google.com wrote: On Mon, Oct 20, 2014 at 10:49 PM, Andy Lutomirski l...@amacapital.net

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Aditya Kali
On Tue, Oct 21, 2014 at 3:42 PM, Andy Lutomirski l...@amacapital.net wrote: On Tue, Oct 21, 2014 at 3:33 PM, Aditya Kali adityak...@google.com wrote: On Tue, Oct 21, 2014 at 12:02 PM, Andy Lutomirski l...@amacapital.net wrote: On Tue, Oct 21, 2014 at 11:49 AM, Aditya Kali

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-21 Thread Andy Lutomirski
On Tue, Oct 21, 2014 at 5:46 PM, Aditya Kali adityak...@google.com wrote: On Tue, Oct 21, 2014 at 3:42 PM, Andy Lutomirski l...@amacapital.net wrote: On Tue, Oct 21, 2014 at 3:33 PM, Aditya Kali adityak...@google.com wrote: On Tue, Oct 21, 2014 at 12:02 PM, Andy Lutomirski l...@amacapital.net

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-20 Thread Andy Lutomirski
On Mon, Oct 20, 2014 at 10:42 PM, Eric W. Biederman wrote: > Andy Lutomirski writes: > >> On Mon, Oct 20, 2014 at 9:49 PM, Eric W. Biederman >> wrote: >>> Andy Lutomirski writes: Possible solution: Ditch the pinning. That is, if you're outside a cgroupns (or you have a

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-20 Thread Eric W. Biederman
Andy Lutomirski writes: > On Mon, Oct 20, 2014 at 9:49 PM, Eric W. Biederman > wrote: >> Andy Lutomirski writes: >> >>> On Sun, Oct 19, 2014 at 9:55 PM, Eric W.Biederman >>> wrote: On October 19, 2014 1:26:29 PM CDT, Andy Lutomirski wrote: >> > Is the idea >that

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-20 Thread Andy Lutomirski
On Mon, Oct 20, 2014 at 9:49 PM, Eric W. Biederman wrote: > Andy Lutomirski writes: > >> On Sun, Oct 19, 2014 at 9:55 PM, Eric W.Biederman >> wrote: >>> >>> >>> On October 19, 2014 1:26:29 PM CDT, Andy Lutomirski >>> wrote: > Is the idea that you want a privileged user wrt a

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-20 Thread Eric W. Biederman
Andy Lutomirski writes: > On Sun, Oct 19, 2014 at 9:55 PM, Eric W.Biederman > wrote: >> >> >> On October 19, 2014 1:26:29 PM CDT, Andy Lutomirski >> wrote: >>> Is the idea >>>that you want a privileged user wrt a cgroupns's userns to be able to >>>use this? If so: >>> >>>Yes, that

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-20 Thread Andy Lutomirski
On Sun, Oct 19, 2014 at 9:55 PM, Eric W.Biederman wrote: > > > On October 19, 2014 1:26:29 PM CDT, Andy Lutomirski > wrote: >>On Sat, Oct 18, 2014 at 10:23 PM, Eric W. Biederman >> wrote: >>> "Serge E. Hallyn" writes: >>> Quoting Aditya Kali (adityak...@google.com): > On Thu, Oct 16,

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-20 Thread Andy Lutomirski
On Sun, Oct 19, 2014 at 9:55 PM, Eric W.Biederman ebied...@xmission.com wrote: On October 19, 2014 1:26:29 PM CDT, Andy Lutomirski l...@amacapital.net wrote: On Sat, Oct 18, 2014 at 10:23 PM, Eric W. Biederman ebied...@xmission.com wrote: Serge E. Hallyn se...@hallyn.com writes: Quoting

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-20 Thread Eric W. Biederman
Andy Lutomirski l...@amacapital.net writes: On Sun, Oct 19, 2014 at 9:55 PM, Eric W.Biederman ebied...@xmission.com wrote: On October 19, 2014 1:26:29 PM CDT, Andy Lutomirski l...@amacapital.net wrote: Is the idea that you want a privileged user wrt a cgroupns's userns to be able to use

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-20 Thread Andy Lutomirski
On Mon, Oct 20, 2014 at 9:49 PM, Eric W. Biederman ebied...@xmission.com wrote: Andy Lutomirski l...@amacapital.net writes: On Sun, Oct 19, 2014 at 9:55 PM, Eric W.Biederman ebied...@xmission.com wrote: On October 19, 2014 1:26:29 PM CDT, Andy Lutomirski l...@amacapital.net wrote: Is

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-20 Thread Eric W. Biederman
Andy Lutomirski l...@amacapital.net writes: On Mon, Oct 20, 2014 at 9:49 PM, Eric W. Biederman ebied...@xmission.com wrote: Andy Lutomirski l...@amacapital.net writes: On Sun, Oct 19, 2014 at 9:55 PM, Eric W.Biederman ebied...@xmission.com wrote: On October 19, 2014 1:26:29 PM CDT, Andy

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-20 Thread Andy Lutomirski
On Mon, Oct 20, 2014 at 10:42 PM, Eric W. Biederman ebied...@xmission.com wrote: Andy Lutomirski l...@amacapital.net writes: On Mon, Oct 20, 2014 at 9:49 PM, Eric W. Biederman ebied...@xmission.com wrote: Andy Lutomirski l...@amacapital.net writes: Possible solution: Ditch the pinning.

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-19 Thread Eric W.Biederman
On October 19, 2014 1:26:29 PM CDT, Andy Lutomirski wrote: >On Sat, Oct 18, 2014 at 10:23 PM, Eric W. Biederman > wrote: >> "Serge E. Hallyn" writes: >> >>> Quoting Aditya Kali (adityak...@google.com): On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn >wrote: > Quoting Aditya Kali

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-19 Thread Andy Lutomirski
On Sat, Oct 18, 2014 at 10:23 PM, Eric W. Biederman wrote: > "Serge E. Hallyn" writes: > >> Quoting Aditya Kali (adityak...@google.com): >>> On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn wrote: >>> > Quoting Aditya Kali (adityak...@google.com): >>> >> setns on a cgroup namespace is allowed

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-19 Thread Andy Lutomirski
On Sat, Oct 18, 2014 at 10:23 PM, Eric W. Biederman ebied...@xmission.com wrote: Serge E. Hallyn se...@hallyn.com writes: Quoting Aditya Kali (adityak...@google.com): On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn se...@hallyn.com wrote: Quoting Aditya Kali (adityak...@google.com): setns

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-19 Thread Eric W.Biederman
On October 19, 2014 1:26:29 PM CDT, Andy Lutomirski l...@amacapital.net wrote: On Sat, Oct 18, 2014 at 10:23 PM, Eric W. Biederman ebied...@xmission.com wrote: Serge E. Hallyn se...@hallyn.com writes: Quoting Aditya Kali (adityak...@google.com): On Thu, Oct 16, 2014 at 2:12 PM, Serge E.

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-18 Thread Eric W. Biederman
"Serge E. Hallyn" writes: > Quoting Aditya Kali (adityak...@google.com): >> On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn wrote: >> > Quoting Aditya Kali (adityak...@google.com): >> >> setns on a cgroup namespace is allowed only if >> >> * task has CAP_SYS_ADMIN in its current user-namespace

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-18 Thread Eric W. Biederman
Serge E. Hallyn se...@hallyn.com writes: Quoting Aditya Kali (adityak...@google.com): On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn se...@hallyn.com wrote: Quoting Aditya Kali (adityak...@google.com): setns on a cgroup namespace is allowed only if * task has CAP_SYS_ADMIN in its

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-17 Thread Serge E. Hallyn
Quoting Aditya Kali (adityak...@google.com): > setns on a cgroup namespace is allowed only if > * task has CAP_SYS_ADMIN in its current user-namespace and > over the user-namespace associated with target cgroupns. > * task's current cgroup is descendent of the target cgroupns-root > cgroup. >

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-17 Thread Serge E. Hallyn
Quoting Aditya Kali (adityak...@google.com): setns on a cgroup namespace is allowed only if * task has CAP_SYS_ADMIN in its current user-namespace and over the user-namespace associated with target cgroupns. * task's current cgroup is descendent of the target cgroupns-root cgroup. *

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-16 Thread Serge E. Hallyn
Quoting Aditya Kali (adityak...@google.com): > On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn wrote: > > Quoting Aditya Kali (adityak...@google.com): > >> setns on a cgroup namespace is allowed only if > >> * task has CAP_SYS_ADMIN in its current user-namespace and > >> over the

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-16 Thread Aditya Kali
On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn wrote: > Quoting Aditya Kali (adityak...@google.com): >> setns on a cgroup namespace is allowed only if >> * task has CAP_SYS_ADMIN in its current user-namespace and >> over the user-namespace associated with target cgroupns. >> * task's current

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-16 Thread Andy Lutomirski
On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn wrote: > Quoting Aditya Kali (adityak...@google.com): >> setns on a cgroup namespace is allowed only if >> * task has CAP_SYS_ADMIN in its current user-namespace and >> over the user-namespace associated with target cgroupns. >> * task's current

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-16 Thread Serge E. Hallyn
Quoting Aditya Kali (adityak...@google.com): > setns on a cgroup namespace is allowed only if > * task has CAP_SYS_ADMIN in its current user-namespace and > over the user-namespace associated with target cgroupns. > * task's current cgroup is descendent of the target cgroupns-root > cgroup.

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-16 Thread Serge E. Hallyn
Quoting Aditya Kali (adityak...@google.com): setns on a cgroup namespace is allowed only if * task has CAP_SYS_ADMIN in its current user-namespace and over the user-namespace associated with target cgroupns. * task's current cgroup is descendent of the target cgroupns-root cgroup. What

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-16 Thread Andy Lutomirski
On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn se...@hallyn.com wrote: Quoting Aditya Kali (adityak...@google.com): setns on a cgroup namespace is allowed only if * task has CAP_SYS_ADMIN in its current user-namespace and over the user-namespace associated with target cgroupns. * task's

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-16 Thread Aditya Kali
On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn se...@hallyn.com wrote: Quoting Aditya Kali (adityak...@google.com): setns on a cgroup namespace is allowed only if * task has CAP_SYS_ADMIN in its current user-namespace and over the user-namespace associated with target cgroupns. * task's

Re: [PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-16 Thread Serge E. Hallyn
Quoting Aditya Kali (adityak...@google.com): On Thu, Oct 16, 2014 at 2:12 PM, Serge E. Hallyn se...@hallyn.com wrote: Quoting Aditya Kali (adityak...@google.com): setns on a cgroup namespace is allowed only if * task has CAP_SYS_ADMIN in its current user-namespace and over the

[PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-13 Thread Aditya Kali
setns on a cgroup namespace is allowed only if * task has CAP_SYS_ADMIN in its current user-namespace and over the user-namespace associated with target cgroupns. * task's current cgroup is descendent of the target cgroupns-root cgroup. * target cgroupns-root is same as or deeper than task's

[PATCHv1 7/8] cgroup: cgroup namespace setns support

2014-10-13 Thread Aditya Kali
setns on a cgroup namespace is allowed only if * task has CAP_SYS_ADMIN in its current user-namespace and over the user-namespace associated with target cgroupns. * task's current cgroup is descendent of the target cgroupns-root cgroup. * target cgroupns-root is same as or deeper than task's