[PATCH] Add /proc/devices

Yaakov (Cygwin/X) yselkowitz@users.sourceforge.net
Fri Aug 19 19:50:00 GMT 2011

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

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.


More information about the Cygwin-patches mailing list