Re: [PATCH 3/7] cgroup: introduce cgroup namespaces

2015-12-08 Thread Serge E. Hallyn
On Tue, Dec 08, 2015 at 02:46:00PM -0500, Tejun Heo wrote: > Hello, Serge. > > On Tue, Dec 08, 2015 at 01:34:31PM -0600, Serge E. Hallyn wrote: > > > I'd prefer collecting all ns related declarations in a single place. > > > > I can group some of them, but free_cgroup_ns needs the > > cgroup_name

Re: [PATCH 3/7] cgroup: introduce cgroup namespaces

2015-12-08 Thread Tejun Heo
Hello, Serge. On Tue, Dec 08, 2015 at 01:34:31PM -0600, Serge E. Hallyn wrote: > > I'd prefer collecting all ns related declarations in a single place. > > I can group some of them, but free_cgroup_ns needs the > cgroup_namespace definition, put_cgroup_ns() needs free_cgroup_ns(), > and free_cgro

Re: [PATCH 3/7] cgroup: introduce cgroup namespaces

2015-12-08 Thread Serge E. Hallyn
On Tue, Dec 08, 2015 at 11:04:53AM -0500, Tejun Heo wrote: > On Mon, Dec 07, 2015 at 05:06:18PM -0600, serge.hal...@ubuntu.com wrote: > > static const char *proc_ns_follow_link(struct dentry *dentry, void > > **cookie) > > diff --git a/include/linux/cgroup.h b/include/linux/cgroup.h > > index 2b3

Re: [PATCH 3/7] cgroup: introduce cgroup namespaces

2015-12-08 Thread Tejun Heo
On Mon, Dec 07, 2015 at 05:06:18PM -0600, serge.hal...@ubuntu.com wrote: > static const char *proc_ns_follow_link(struct dentry *dentry, void **cookie) > diff --git a/include/linux/cgroup.h b/include/linux/cgroup.h > index 2b3e2314..906f240 100644 > --- a/include/linux/cgroup.h > +++ b/include/lin