Re: [PATCH v3 0/3] cgroup: add xattr support

2012-08-07 Thread Aristeu Rozanski
(Added Lennart to the discussion so he can help explaining how systemd will use the xattrs) On Fri, Jul 20, 2012 at 11:04:59AM -0700, Tejun Heo wrote: > Hello, Hugh, Aristeu. > > On Fri, Jul 20, 2012 at 01:59:59PM -0400, Aristeu Rozanski wrote: > > hm, not sure that's what Tejun meant. tmpfs uses

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-24 Thread Aristeu Rozanski
On Tue, Jul 24, 2012 at 11:28:09AM -0700, Tejun Heo wrote: > On Mon, Jul 23, 2012 at 02:12:52PM -0400, Aristeu Rozanski wrote: > > > Will "lsattr -R" in the cgroupfs mountpoint do enough to judge how > > > much memory is being used in this way? I expect not, but I'm > > > unfamliar with it: you ma

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-24 Thread Tejun Heo
Hello, On Mon, Jul 23, 2012 at 02:12:52PM -0400, Aristeu Rozanski wrote: > > Will "lsattr -R" in the cgroupfs mountpoint do enough to judge how > > much memory is being used in this way? I expect not, but I'm > > unfamliar with it: you may need to show counts elsewhere. > > that's for ext{2,3,4}

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-23 Thread Aristeu Rozanski
On Sun, Jul 22, 2012 at 12:12:07PM -0700, Hugh Dickins wrote: > Please, do not do this. > > It may be fun to implement, but not to review and maintain. > > If we're going to start supporting swappable kernel memory, tmpfs > xattrs is not the right place to start, and libfs xattrs certainly not: >

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-22 Thread Hugh Dickins
On Fri, 20 Jul 2012, Aristeu Rozanski wrote: > On Wed, Jul 18, 2012 at 06:11:32PM -0700, Hugh Dickins wrote: > > > But why do we need something completely new? Can't we hijack some > > > inodes used by tmpfs and use them for xattr storage? ie. Would it be > > > difficult to use tmpfs as backend s

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-20 Thread Tejun Heo
Hello, Hugh. On Wed, Jul 18, 2012 at 06:11:32PM -0700, Hugh Dickins wrote: > So systemd is already doing it as I suggested, but finds that awkward in > some respects, and wants more help from the kernel in synchronization. Yeah, pretty much. > Stuffing an unbounded amount of data into cgroupfs x

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-20 Thread Tejun Heo
Hello, Hugh, Aristeu. On Fri, Jul 20, 2012 at 01:59:59PM -0400, Aristeu Rozanski wrote: > hm, not sure that's what Tejun meant. tmpfs uses anonymous memory for the file > contents, so reuse that infrastructure to allocate space for the extended > attributes the same way, instead of using kmem. >

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-20 Thread Aristeu Rozanski
Hugh, On Wed, Jul 18, 2012 at 06:11:32PM -0700, Hugh Dickins wrote: > > But why do we need something completely new? Can't we hijack some > > inodes used by tmpfs and use them for xattr storage? ie. Would it be > > difficult to use tmpfs as backend storage for on-memory xattr? With > > that, we

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-18 Thread Hugh Dickins
On Wed, 18 Jul 2012, Tejun Heo wrote: > On Wed, Jul 18, 2012 at 01:02:03PM -0700, Hugh Dickins wrote: > > > I don't know. I > > > really hope it used anonymous page cache instead of kmem tho. Hugh, > > > would something like that be difficult? > > > > Yes, it would be difficult. > > > > You don

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-18 Thread Tejun Heo
Hello, Hugh. On Wed, Jul 18, 2012 at 01:02:03PM -0700, Hugh Dickins wrote: > > I don't know. I > > really hope it used anonymous page cache instead of kmem tho. Hugh, > > would something like that be difficult? > > Yes, it would be difficult. > > You don't use the word "swappable", but I take

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-18 Thread Hugh Dickins
On Tue, 17 Jul 2012, Tejun Heo wrote: > On Mon, Jul 02, 2012 at 10:29:25AM -0400, Aristeu Rozanski wrote: > > This series are a refreshed version of a patchset submitted by Li Zefan back > > in march: > > https://lkml.org/lkml/2012/3/1/13 > > > > With Li's permission, I refreshed the patches t

Re: [PATCH v3 0/3] cgroup: add xattr support

2012-07-17 Thread Tejun Heo
Hello, On Mon, Jul 02, 2012 at 10:29:25AM -0400, Aristeu Rozanski wrote: > This series are a refreshed version of a patchset submitted by Li Zefan back > in march: > https://lkml.org/lkml/2012/3/1/13 > > With Li's permission, I refreshed the patches to apply over the latest > upstream > an