Am Mittwoch, 23. Oktober 2002 18:45 schrieb Greg KH: > On Wed, Oct 23, 2002 at 06:37:19PM +0200, Oliver Neukum wrote: > > > Ok, one of my original questions was if any V4L applications used the > > > current /proc interface. If they do, as you say, then moving them to > > > driverfs would not be a simple thing to do at this time. > > > > The driverfs API is not without problems. In fact, it's close to > > opening Pandora's box. > > > > It turns a simple model of open() - IO - close() into something > > much more complex. The recent patch to use it shows that the locking > > issues connected to using an API that makes implementing open() > > optional were not understood at all. > > Ah, such kind words. It just warms my heart. :)
I don't want any of your internal organs to suffer from undue chilling ;-) > It's nice to see that you are taking a look at the driverfs code for > problems. No one has said the locking issues are completely fixed. The problem is that this is not just an issue of code quality. The thing itself is harder to get right. In short, I am not sure whether writing to driverfs is a good idea for simple everyday drivers. You've made something that's inherently hard easy to abuse. In a way, you worked too well. A model of one simple file to be opened per device _is_ simpler than a number of files to control various settings. Plus, how do you control permissions on that thing ? I sent you a patch that should solve the unload problem. Next I'd like to see a lock for attributes and making implementing open mandatory. After that the interfaces should be basically useable, but still I don't think it's for general consumption, ever. Regards Oliver ------------------------------------------------------- This sf.net email is sponsored by: Influence the future of Java(TM) technology. Join the Java Community Process(SM) (JCP(SM)) program now. http://ads.sourceforge.net/cgi-bin/redirect.pl?sunm0002en _______________________________________________ [EMAIL PROTECTED] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel