On Fri, 2011-08-19 at 10:44 -0400, Christopher Faylor wrote: > On Fri, Aug 19, 2011 at 01:52:53PM +0200, Corinna Vinschen wrote: > >On Aug 18 20:54, Yaakov (Cygwin/X) wrote: > >> 2011-08-18 Yaakov Selkowitz <yselkowitz@...> > >> > >> * devices.h (fh_devices): Define DEV_MISC_MAJOR, DEV_MEM_MAJOR, > >> DEV_SOUND_MAJOR. Use throughout. > >> * fhandler_proc.cc (proc_tab): Add /proc/devices and /proc/misc > >> virtual files. > >> (format_proc_devices): New function. > >> (format_proc_misc): New function. > > > >Yes, I think that's ok. > > Isn't it somehow possible to just iterate over dev_storage and generate > this automatically rather than hard-coding the names of devices?
I don't think so. For the most part, /proc/devices doesn't list individual devices but only groups thereof, and there is the "misc" major device which is only descriptive and not an actual device name. There is also the matter of distinguishing between block and character devices. As for /proc/misc, technically it could be done as you describe, but is it worth the price of iterating over a 2581-member array to find the two matching cases? If the misc devices would vary based on configuration as on Linux, I would see your point, but as we only and always have these two, I'm not so sure. Let me know how you want to proceed. Yaakov
