Re: [PATCH 3/11] USB: interface PM state
David Brownell <[email protected]>
| Newsgroups | gmane.linux.usb.devel |
|---|---|
| Message-ID | <[email protected]> |
On Thursday 05 April 2007 1:04 pm, Alan Stern wrote: > This patch (as880) strives to keep the PM core's idea of a USB > interface's power state in synch with usbcore's own idea. In the end > this doesn't really matter, but it's better to be consistent. ISTR trying this originally and watching it break things in some of the test scenarios I tried. Quite a surprise actually. And as I recall, it interacted with the issue in your patch #7/11 ... What was your test matrix? - USB_SUSPEND on and off? - All host controller drivers? - Devices with remote wakeup enabled? HCDs? Both, using it? - ... I remember identifying at least half a dozen factors that needed testing to make sure all code paths got hit ... some worked this way, some didn't. My test matrix covered each factor on/off, and various populat combinations. Retesting after each bugfix was a PITA ... 20 minutes per test, if they succeeded; sigh. I think that USB_SUSPEND was the key factor I saw, or remote wakeup. - Dave ------------------------------------------------------------------------- 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