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 | [email protected]
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
