[lkml]   [2012]   [Mar]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v2 0/3] cgroup: add xattr support
Tejun Heo wrote:
> Hello, Li.
> Sorry about the delay. Was on vacation.

Me too. ;)

> On Thu, Mar 01, 2012 at 02:16:57PM +0800, Li Zefan wrote:
>> This can be used to attach metadata to a cgroup, which is required
>> by systemd.
>> As lazy as I am, here just quoted from Lennart for the detailed use
>> cases:
> I much prefer this version but would prefer to deal with this in the
> next merge window as we're already on rc6. On a cursory look...

I'm fine with this, especially I'm busy with other stuff.

> * I'm not sure kmem_ is a good prefix. It sucks that xattr values are
> stored in kmem w/o size limit and we probably want to improve on
> both fronts in the future. Wouldn't it be better to name them
> simple_*() and put them in fs/libfs.c?


> * I don't think we want CONFIG option for cgroupfs xattr. Let's just
> add a mount option.


> Thanks.

 \ /
  Last update: 2012-03-08 10:03    [W:0.075 / U:3.804 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site