* GET_TIMEOUT too small for hotplugging in usb.c (2.4.20)
@ 2003-01-03 22:46 A E Lawrence
0 siblings, 0 replies; only message in thread
From: A E Lawrence @ 2003-01-03 22:46 UTC (permalink / raw)
To: linux-kernel
[-- Attachment #1: Type: text/plain, Size: 136 bytes --]
There seem to be several threads in the archives which report problems that
the small timeout seems to explain. Details attached.
ael
[-- Attachment #2: usb_TIMEOUT.txt --]
[-- Type: text/plain, Size: 1126 bytes --]
GET_TIMEOUT in usb.c requires a larger value to permit hotplugging
------------------------------------------------------------------
drivers/usb/usb.c from 2.4.20 (no version number in file)
#ifdef CONFIG_USB_LONG_TIMEOUT
#define GET_TIMEOUT 20 <== This was 4
#else
#define GET_TIMEOUT 3
#endif
#define SET_TIMEOUT 3
Hotplugging my usb Epson scanner ceased to work sometime ago. I have just
traced this to a short GET_TIMEOUT in usb.c as above.
It is well known that Epson scanners take a long time to respond.
Mine takes around 12 seconds: the explanation seems to be that it switchs
on the transparency light source and waits for it to stabilize before answering
enquiries. This is covered in scanner.c which uses extended timeouts for
Epsons.
However, even with CONFIG_USB_LONG_TIMEOUT set, the timeout in usbcore is only
4 seconds, so the hotplug script does not get called to load that scanner
module. So the longer value of GET_TIMEOUT needs to be increased to at least
15 seconds, perhaps much longer to cover other devices that may be even slower?
Or should it become a module parameter?
A.E.Lawrence
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2003-01-03 22:37 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-01-03 22:46 GET_TIMEOUT too small for hotplugging in usb.c (2.4.20) A E Lawrence
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®