On Tue, Oct 18, 2005 at 10:13:33AM -0700, Greg KH wrote: > > I'm cross-posting to l-k because someone I know was making sounds at > > a notion of #ifdef CONFIG_COMPAT. But I think this solutions is superior > > to adding anything outside of devio.c. > > Why not put this in fs/compat_ioctl.c where the other usbfs 32bit ioctls > are? > There are a few exception like submit urb and reap urb is in devio.c because it contain user space pointer which need some state if put under compat_ioctl.c.
I want to know why it is better in devio.c in this case as well. Another comment regarding the change, the USBDEVFS_IOCTL is passing an ioctl buffer inside another ioctl wrapper. I am a little nervous no check have been done to what ioctl it is passing. Most of the case you don't need to convert the buffer, but what if there is some ioctl need to do something special. On the safe side, I am expecting a big switch to list all the ioctl we known can safely pass, can grow the list when needed. BTW, the other common case is the disconnect ioctl. Chris ------------------------------------------------------- This SF.Net email is sponsored by: Power Architecture Resource Center: Free content, downloads, discussions, and more. http://solutions.newsforge.com/ibmarch.tmpl _______________________________________________ [email protected] To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
