On Tue, 12 Dec 2006, Dominik Brodowski wrote:

> > I'll have to study the code some more.  Which kernel did you say you are 
> > using?
> 
> Latest Linus' git. Today's snd-usb-audio test was with head
> 4259cb25d436a79bf6b07d8075423573567c211d (plus an completely unrelated
> pcmcia patch, but the related modules didn't even load).
> 
> > > case B) snd-usb-audio device connected to "right" USB port:
> > > 
> > > The flip occurs on or after IRQ status 0004.
> > > 
> > > 
> > > (unfortunately, without extended USB debug messages)
> > 
> > And in this example there wasn't even a suspend-resume pair to confuse 
> > the issue, which suggests that the same thing might end up happening even 
> > without the patch git-bisect identified.  Can you try running exactly the 
> > same test, but with that patch reverted?
> 
> Actually, there _was_ a suspend and resume going on, it just didn't show up
> in the logs for I had CONFIG_USB_DEBUG disabled for that test accidentally. 
> 
> http://userweb.kernel.org/~brodo/dmesg-autosuspend-2b.txt
> 
> from today confirms it, and I also ran the pre-autosuspend kernel ( its head
> is 8c03356a559ced6fa78931f498193f776d67e445 ) to re-check that it is an issue
> which appeared with the autosuspend patch.

Okay, I was fooled by the lack of debugging info.  And clearly the suspend 
or resume routine is implicated.

This suggests we check the value of the intr_enable register at the entry 
and exit of both ehci_bus_suspend() and ehci_bus_resume() in ehci_hub.c.  
You can add the appropriate printk statements easily.

Hmmm...  Perhaps the final writel() at the end of ehci_bus_resume() isn't
getting sent through.  The mere act of reading it back might be enough to
change the behavior.

On the other hand, your latest log suggests that the STS_FLR bit gets set 
during the ehci_bus_suspend() routine, not the resume routine.  So it will 
be best to check at the beginning and end of both routines.

Alan Stern


-------------------------------------------------------------------------
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
_______________________________________________
linux-usb-devel@lists.sourceforge.net
To unsubscribe, use the last form field at:
https://lists.sourceforge.net/lists/listinfo/linux-usb-devel

Reply via email to