On Sun, 12 Nov 2006, David Brownell wrote: > > Or are you trying to say that the original device_may_wakeup() value would > > be 0 if the bug were detected? > > The latter: device_may_wakeup() never returns true. There are three paths > for that: > > (a) userspace workaround, which is the regression that was reported; > (b) the AMD 756 workaround, and > (c) that board-specific quirk code. > > Of course (c) hasn't been submitted yet because it didn't work ... evidently > because of the regression where device_may_wakeup(root_hub) was ignored.
Well, I would argue that part of the problem has to do with the use of device_may_wakeup. It is tied to a sysfs API and therefore administrative in nature, but now you say it's also being used to record hardware quirks. > > If you think autostop should also check for device_may_wakeup(), I'll make > > it do so. Remember though that autostop is intended to work even when > > CONFIG_PM is off. > > The original autosuspend logic would never kick in without PM; after all, > it's purely a power saving mechanism! And testing device_may_wakeup() will > be restoring that behavior, since without PM that's always false. It would restore that behavior, and it would be silly way of doing so. There are better ways to prevent autostop without PM, such as making ohci_rh_suspend() and ohci_rh_resume() depend on CONFIG_PM! However it was always my intention that autostop should operate without PM. It's not only about saving power, it also is about reducing load on system resources -- primarily DMA, although this may be a lot less severe with OHCI than with UHCI. Does OHCI do any DMA at all when no devices are plugged in and the schedule is empty? My quick impression from the spec is that it does not, in which case there is no point in keeping autostop when CONFIG_PM is off. Alan Stern ------------------------------------------------------------------------- Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642 _______________________________________________ linux-usb-devel@lists.sourceforge.net To unsubscribe, use the last form field at: https://lists.sourceforge.net/lists/listinfo/linux-usb-devel
