[Devel] Re: [RFC][PATCH] another swap controller for cgroup

2008-03-17 Thread Daisuke Nishimura
Hi, Yamamoto-san. I'm reviewing and testing your patch now. I think your implementation is better because: - the group to be charged is determined correctly at the point of swapout, without fixing the behavior of move_task of memcg. (I think the behavior of move_task of memcg should be

[Devel] [PATCH -mm] cgroup: fix boot option parsing

2008-03-17 Thread Li Zefan
When boot with 'cgroup_disable=cpuacct', it turns out subsystem 'cpu' is disabled. When Balbir posted the patch to add cgroup boot option support, Paul M noticed this problem, but the patch was accepted without fixing it. Signed-off-by: Li Zefan [EMAIL PROTECTED] --- kernel/cgroup.c | 15

[Devel] Re: [PATCH -mm] cgroup: fix boot option parsing

2008-03-17 Thread Paul Menage
On Mon, Mar 17, 2008 at 4:24 PM, Li Zefan [EMAIL PROTECTED] wrote: When boot with 'cgroup_disable=cpuacct', it turns out subsystem 'cpu' is disabled. When Balbir posted the patch to add cgroup boot option support, Paul M noticed this problem, but the patch was accepted without fixing it.

[Devel] Re: [RFC][PATCH] another swap controller for cgroup

2008-03-17 Thread YAMAMOTO Takashi
- anonymous objects (shmem) are not accounted. IMHO, shmem should be accounted. I agree it's difficult in your implementation, but are you going to support it? it should be trivial to track how much swap an anonymous object is using. i'm not sure how it should be associated with cgroups,

[Devel] Re: [PATCH] cgroups: implement device whitelist lsm (v3)

2008-03-17 Thread Serge E. Hallyn
Quoting Casey Schaufler ([EMAIL PROTECTED]): --- Stephen Smalley [EMAIL PROTECTED] wrote: ... I completely disagree. We have two separate frameworks in the kernel, one to enforce generic additional security stuff, and one to track tasks. When I need a feature which tracks

[Devel] Re: [RFC][PATCH 0/4] Object creation with a specified id

2008-03-17 Thread Serge E. Hallyn
Quoting Oren Laadan ([EMAIL PROTECTED]): Serge E. Hallyn wrote: Quoting Oren Laadan ([EMAIL PROTECTED]): Nadia Derbey wrote: Oren Laadan wrote: Nadia Derbey wrote: Oren Laadan wrote: [EMAIL PROTECTED] wrote: A couple of weeks ago, a discussion has started after Pierre's proposal

[Devel] Re: [PATCH] cgroups: implement device whitelist lsm (v3)

2008-03-17 Thread Stephen Smalley
On Mon, 2008-03-17 at 09:16 -0700, Casey Schaufler wrote: --- Serge E. Hallyn [EMAIL PROTECTED] wrote: Quoting Casey Schaufler ([EMAIL PROTECTED]): ... In particular, capabilities are not an access control mechanism, they are a privilege mechanism. A lot of discussion about LSM has

[Devel] Re: [PATCH] cgroups: implement device whitelist lsm (v3)

2008-03-17 Thread Stephen Smalley
On Fri, 2008-03-14 at 15:44 -0700, Casey Schaufler wrote: --- Stephen Smalley [EMAIL PROTECTED] wrote: ... I completely disagree. We have two separate frameworks in the kernel, one to enforce generic additional security stuff, and one to track tasks. When I need a feature

[Devel] [PATCH] cgroups: implement device whitelist (v4)

2008-03-17 Thread Serge E. Hallyn
Implement a cgroup to track and enforce open and mknod restrictions on device files. A device cgroup associates a device access whitelist with each cgroup. A whitelist entry has 4 fields. 'type' is a (all), c (char), or b (block). 'all' means it applies to all types and all major and minor

[Devel] Re: [PATCH] cgroups: implement device whitelist (v4)

2008-03-17 Thread Li Zefan
Serge E. Hallyn wrote: Implement a cgroup to track and enforce open and mknod restrictions on device files. A device cgroup associates a device access whitelist with each cgroup. A whitelist entry has 4 fields. 'type' is a (all), c (char), or b (block). 'all' means it applies to all types