Re: [patch]ratelimit debounce error messages

Oliver Neukum <[email protected]>
Newsgroups gmane.linux.usb.devel
Organization Novell
Message-ID <[email protected]>
Am Donnerstag, 1. März 2007 20:13 schrieb Alan Stern:
> On Thu, 1 Mar 2007, Oliver Neukum wrote:
> 
> > Am Dienstag, 27. Februar 2007 17:03 schrieb Alan Stern:
> > 
> > > Oliver, I've noticed in the past that khubd tries too hard to enumerate a
> > > new connection.  The new scheme and the old scheme are both tried twice,
> > > and each try has two attempts to read the device descriptor, where each 
> > > attempt can loop twice, each loop with up to three calls to 
> > > usb_control_msg() and a device reset, with timeouts as long as 1 second.
> > > 
> > > In really bad cases this can tie things up for close to a minute.  We
> > > ought to be able to do better.  Some of the loops should be eliminated and
> > > some of the timeouts decreased.
> > 
> > Something like this?
> 
> Actually I had in mind something simpler, like getting rid of the
> GET_DESCRIPTOR_TRIES loop for i (or reducing it to one attempt using the
> new scheme and one attempt using the old scheme) and getting rid of the 
> SET_ADDRESS_TRIES loop for j, both in hub_port_init().

I would prefer to try very hard once. This is what this patch does.

	Regards
		Oliver

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
_______________________________________________
[email protected]
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.