]> bbs.cooldavid.org Git - net-next-2.6.git/commitdiff
Lower USB storage settling delay to something more reasonable
authorLinus Torvalds <torvalds@linux-foundation.org>
Fri, 26 Feb 2010 18:03:22 +0000 (10:03 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 26 Feb 2010 18:03:22 +0000 (10:03 -0800)
The five-second delay can be rather annoying, and makes the system
appear much less responsive when you connect a USB drive.

It's also not entirely clear that it is needed - the settling delay has
at least historically been an issue on some Apple iPods, for example,
and some devices have been reported to need even more than the old 5s
delay.

But before we penalize them all, let's see how bad it really is.  Some
of the reasons for long delays seem to be actual historical kernel bugs
that should probably never have been papered over with a delay in the
first place (there's a Ubuntu bug report for 2.6.20 about a NULL pointer
dereference unless 'delay_use' is 8 or more, for example).

It also looks like some distros have already shipped with delay_use=0,
so the five second default may well be totally historical.

In other words: "Let's see if anybody screams".

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
drivers/usb/storage/usb.c

index e9f995486ec1b1fa46021ed0bd3202044d0a425a..bbeeb92a213101d7eeab4314c91532da707247f7 100644 (file)
@@ -78,7 +78,7 @@ MODULE_AUTHOR("Matthew Dharm <mdharm-usb@one-eyed-alien.net>");
 MODULE_DESCRIPTION("USB Mass Storage driver for Linux");
 MODULE_LICENSE("GPL");
 
-static unsigned int delay_use = 5;
+static unsigned int delay_use = 1;
 module_param(delay_use, uint, S_IRUGO | S_IWUSR);
 MODULE_PARM_DESC(delay_use, "seconds to delay before using a new device");