Messages in this thread |  | | | Subject | Re: [ckrm-tech] [PATCH 0/5] Allow more than PAGESIZE data read in configfs | | From | Chandra Seetharaman <> | | Date | Mon, 16 Oct 2006 12:33:41 -0700 |
| |
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@us.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@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |