The patch titled usbfs: struct async is a fixed size structure has been added to the -mm tree. Its filename is usbfs-struct-async-is-a-fixed-size-structure.patch *** Remember to use Documentation/SubmitChecklist when testing your code *** See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find out what to do about this The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/ ------------------------------------------------------ Subject: usbfs: struct async is a fixed size structure From: Pete Zaitcev <zaitcev@xxxxxxxxxx> The ISO descriptors are allocated separately in proc_submiturb for a fetch from user mode, then tucked at the end of URB. This seems like a dead code. Signed-off-by: Pete Zaitcev <zaitcev@xxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- drivers/usb/core/devio.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff -puN drivers/usb/core/devio.c~usbfs-struct-async-is-a-fixed-size-structure drivers/usb/core/devio.c --- a/drivers/usb/core/devio.c~usbfs-struct-async-is-a-fixed-size-structure +++ a/drivers/usb/core/devio.c @@ -210,9 +210,9 @@ err: static struct async *alloc_async(unsigned int numisoframes) { - unsigned int assize = sizeof(struct async) + numisoframes * sizeof(struct usb_iso_packet_descriptor); - struct async *as = kzalloc(assize, GFP_KERNEL); + struct async *as; + as = kzalloc(sizeof(struct async), GFP_KERNEL); if (!as) return NULL; as->urb = usb_alloc_urb(numisoframes, GFP_KERNEL); _ Patches currently in -mm which might be from zaitcev@xxxxxxxxxx are usb-libusual-locking-cleanup.patch usbfs-struct-async-is-a-fixed-size-structure.patch - To unsubscribe from this list: send the line "unsubscribe mm-commits" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html