Messages in this thread | | | From | Limin Gu <> | Subject | Re: [RFC][PATCH] jobfs - new virtual filesystem for job kernel/user interface | Date | Wed, 8 Dec 2004 18:09:49 -0800 (PST) |
| |
> > * Limin Gu (limin@dbear.engr.sgi.com) wrote: > > I am looking for your comments on the attached draft, it is the job patch > > for 2.6.9. I have posted job patch for older kernel before, but in this patch > > I have replaced the /proc/job binary ioctl calls with a new small virtual > > filesystem (jobfs). > > Limin, glad to see you got to this.
Chris,
Thanks for your detailed review, I will work through your comments later. I will make the code look better.
Do you have any major concern about the overall design of jobfs? Do you find any major flaw or big no no where you read the code? For example, using existing procfs other than creating a new virtual filesystem. It seems that people are moving things out of /proc, not adding more, also procfs does not provide mkdir/chown operations, that are the reasons I chose a custom fs over procfs.
Another question is where do we mount the jobfs. The mount point does not show up in the kernel code, but what are the choices, /job, /dev/job, or other places? Is there any guideline about this?
Thanks!
--Limin
- 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/
| |