USB: serial drivers need to use larger bulk-in buffers
authorAlan Stern <stern@rowland.harvard.edu>
Wed, 23 Feb 2011 20:28:18 +0000 (15:28 -0500)
committerGreg Kroah-Hartman <gregkh@suse.de>
Wed, 23 Mar 2011 20:03:46 +0000 (13:03 -0700)
commit366a91fbe3eeaf11c7bc380e854c7096bcb976dd
treec58efa89519be35bf2dbc63916a15ad754c3f8a1
parent2ac31a250c7d0cf642cc183bcf55e571ada867f6
USB: serial drivers need to use larger bulk-in buffers

commit 969e3033ae7733a0af8f7742ca74cd16c0857e71 upstream.

When a driver doesn't know how much data a device is going to send,
the buffer size should be at least as big as the endpoint's maxpacket
value.  The serial drivers don't follow this rule; many of them
request only 256-byte bulk-in buffers.  As a result, they suffer
overflow errors if a high-speed device wants to send a lot of data,
because high-speed bulk endpoints are required to have a maxpacket
size of 512.

This patch (as1450) fixes the problem by using the driver's
bulk_in_size value as a minimum, always allocating buffers no smaller
than the endpoint's maxpacket size.

Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Tested-by: Flynn Marquardt <flynn@flynnux.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/serial/usb-serial.c
include/linux/usb/serial.h