diff options
author | Alan Stern <stern@rowland.harvard.edu> | 2008-06-30 13:39:59 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-07-21 15:16:51 -0700 |
commit | f756cbd458ab71c996a069cb3928fb1e2d7cd9cc (patch) | |
tree | 4caadc9f8a55e662d144535b855ad442af19fc50 /drivers/net/gianfar.c | |
parent | 86c57edf60f5c98adb496880f56cd0e5a3423153 (diff) | |
download | blackbird-op-linux-f756cbd458ab71c996a069cb3928fb1e2d7cd9cc.tar.gz blackbird-op-linux-f756cbd458ab71c996a069cb3928fb1e2d7cd9cc.zip |
usb-storage: revert DMA-alignment change for Wireless USB
This patch (as1110) reverts an earlier patch meant to help with
Wireless USB host controllers. These controllers can have bulk
maxpacket values larger than 512, which puts unusual constraints on
the sizes of scatter-gather list elements. However it turns out that
the block layer does not provide the support we need to enforce these
constraints; merely changing the DMA alignment mask doesn't help.
Hence there's no reason to keep the original patch. The Wireless USB
problem will have to be solved a different way.
In addition, there is a reason to get rid of the earlier patch. By
dereferencing a pointer stored in the ep_in array of struct
usb_device, the current code risks an invalid memory access when it
runs concurrently with device removal. The members of that array are
cleared before the driver's disconnect method is called, so it should
not try to use them.
Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/net/gianfar.c')
0 files changed, 0 insertions, 0 deletions