On Fri, 2006-08-04 at 14:41 -0700, Matt Helsley wrote:
> On Fri, 2006-08-04 at 14:29 -0700, Chandra Seetharaman wrote:
> > On Fri, 2006-08-04 at 13:52 -0700, Dave Hansen wrote:
> > > On Fri, 2006-08-04 at 13:11 -0700, Chandra Seetharaman wrote:
> > > > I think it was Joel. His point was that files in configfs was intended
> > > > to be used as attr=value type of usage, and data worth of PAGE_SIZE is
> > > > good enough for that kind of usage.
> > > 
> > > Is there a real reason that we can't do this at the directory level?
> > > 
> > > Instead of:
> > > 
> > >   $ cat /configfs/rg0/tasks
> > >   1 2 3 4
> > > 
> > > This:
> > > 
> > >   $ ls /configfs/rg0/tasks/
> > >   1 2 3 4
> > > 
> > 
> > It might solve the problem in hand. But, as a user, if i see a file i
> > would expect some data in that file. There is no relevant data to be
> > displayed here.
> 
> IMHO There's nothing wrong with having empty files.
> 
> If it really disturbs you that much perhaps we could make the file named
> <pid> a symlink to /proc/<pid>.
> 

I do not think configfs supports linking files from other filesystems

> > > It might even make some semantic sense for moving tasks between resource
> > > groups to use sys_rename().
> > > 
> > > -- Dave
> 
>       Rename does have some significant advantages. It would show up via
> inotify for instance.

But, it comes with additional coding logic too. If we have a file for
each task, then whenever a task enters/exits a resource group we have to
call configfs function to create/delete the file, which is unnecessary.

In the current implementation, we walk thru the list _only_ when user
reads the "members" file. 
> 
> Cheers,
>       -Matt Helsley
> 
> 
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share your
> opinions on IT & business topics through brief surveys -- and earn cash
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> ckrm-tech mailing list
> https://lists.sourceforge.net/lists/listinfo/ckrm-tech
-- 

----------------------------------------------------------------------
    Chandra Seetharaman               | Be careful what you choose....
              - [EMAIL PROTECTED]   |      .......you may get it.
----------------------------------------------------------------------



-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys -- and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
ckrm-tech mailing list
https://lists.sourceforge.net/lists/listinfo/ckrm-tech

Reply via email to