On Wed, Nov 25, 2015 at 10:29:53AM -0500, Alan Stern wrote: > I want to see a modified version of your patch. Several things need to > be changed or fixed, but the major change needs to be the way memory is > allocated. It should be done as part of the mmap system call, not as a > separate ioctl. > > There are numerous smaller issues: The allocated memory needs to be > charged against usbfs_memory_usage, the memory should be allocated > using dma_zalloc_coherent instead of kmalloc, proc_do_submiturb should > check that the buffer starts anywhere within the allocated memory > rather than just at the beginning, and so on. > > I'd make these changes myself, but I just don't have enough free time > now. If anybody else would like to work on them, please feel free. I can take a stab at fixing some of these. Perhaps not all of them, but maybe it will be a useful starting point. /* Steinar */ -- Homepage: https://www.sesse.net/ -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
