[lkml]   [2012]   [Mar]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v2 0/3] cgroup: add xattr support
Hello, Li.

Sorry about the delay. Was on vacation.

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 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.



 \ /
  Last update: 2012-03-04 19:07    [W:0.114 / U:5.788 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site