Messages in this thread |  | | | Date | Tue, 17 Jul 2012 13:41:26 -0700 | | From | Tejun Heo <> | | Subject | Re: [PATCH v3 0/3] cgroup: add xattr support |
| |
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 > and added the modifications suggested by others in the thread: > - using a mount option instead of config option to enable the xattr support > - reinitialize the list in kmem_xattrs_free() > - renamed functions to simple_xattr_*() > > Signed-off-by: Li Zefan <lizefan@huawei.com> > Signed-off-by: Aristeu Rozanski <aris@redhat.com>
I raised this point before but I'm worried about directly exposing kernel memory through xattr interface to userland. Maybe it's okay as long as !root users are kept from creating them. I don't know. I really hope it used anonymous page cache instead of kmem tho. Hugh, would something like that be difficult?
Thanks.
-- tejun
|  |