[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1161027221.6389.135.camel@linuxchandra>
Date: Mon, 16 Oct 2006 12:33:41 -0700
From: Chandra Seetharaman <sekharan@...ibm.com>
To: Joel Becker <Joel.Becker@...cle.com>
Cc: Matt Helsley <matthltc@...ibm.com>,
Linux Kernel <linux-kernel@...r.kernel.org>,
Greg KH <gregkh@...e.de>, Andrew Morton <akpm@...l.org>,
CKRM-Tech <ckrm-tech@...ts.sourceforge.net>
Subject: Re: [ckrm-tech] [PATCH 0/5] Allow more than PAGESIZE data read in
configfs
On Fri, 2006-10-13 at 17:09 -0700, Joel Becker wrote:
> On Fri, Oct 13, 2006 at 04:37:38PM -0700, Matt Helsley wrote:
> > > Sure it works. You have one per resource group. In
> > > resource_group_make_object(), you sysfs_mkdir() the sysfs file. There
> >
> > That's the easy part. Next we need to make the pid attribute whenever a
> > new task is created. And delete it when the task dies. And move it
> > around whenever it changes groups. Is there rename() support in /sys? If
> > not, would changes to allow rename() be acceptable (I'm worried it would
> > impact alot of assumptions made in the existing code)?
>
> No, you don't create a pid attribute per task. The sysfs file
> is literally your large attribute. So, instead of echoing a new pid to
> "/sys/kernel/config/ckrm/group1/pids", you echo to
> "/sys/ckrm/group1/pids". To display them all, you just cat
> "/sys/ckrm/group1/pids". It's exactly like the file you want in
> configfs, just located in a place where it is allowed.
>>From what I see, sysfs also has the PAGESIZE limitation. If that _is_
the case, then moving to sysfs does not help us any. Correct me if I am
wrong.
Won't we have the same arguments that we have now ?
<snip>
>
--
----------------------------------------------------------------------
Chandra Seetharaman | Be careful what you choose....
- sekharan@...ibm.com | .......you may get it.
----------------------------------------------------------------------
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists