Messages in this thread | | | Date | Fri, 3 Aug 2007 02:47:56 +0100 | From | Matthew Garrett <> | Subject | Re: [PATCH] USB: Only enable autosuspend by default on certain device classes |
| |
On Thu, Aug 02, 2007 at 06:15:05PM -0700, Greg KH wrote:
> Well, if you do this, then you can pretty much delete the whole quirk > table we have, right?
At the moment, yes.
> And personally, I want to do better than Windows XP when it comes to > power management. This patch is only going to suspend a very tiny > subset of devices, including a whole bunch of ones that do not even have > drivers in Linux, causing our power footprint to be bigger than needed.
I agree. I'd much rather see us suspending devices whenever possible - it's just that I have concerns over the scalability of the blacklist, given the number of devices that seem to have issues.
> Also, we have udev rules for SANE that disables their autosuspend > settings, which handles the majority of the devices we have seen with > problems.
Several printers seem to have the issue as well, and the blacklist seems to contain some odd miscellaneous devices like the Blackberry. The main concern I have is that kernel developers just don't tend to be the sort of people that use webcams, printers or scanners, so we're relying on normal users to go to the effort of reporting that their device has stopped working.
> So I really don't want to accept this patch. But, what problems are you > seeing with our current suspend logic that you feel we need to be this > harsh?
It's definitely a brute force approach, but it's one that means that we get the low hanging fruit (ie, pretty much anything that's likely to be plugged into a laptop) while massively reducing the probability of breaking anyone's system. Saving some power is a nice win, but breaking someone's printer is a pretty big loss.
-- Matthew Garrett | mjg59@srcf.ucam.org - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |