Re: [PATCH 01/13] PM: Add wake lock api.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 2009-02-12 at 18:21 -0800, Arve Hjønnevåg wrote:> On Thu, Feb 12, 2009 at 5:10 PM, Matthew Garrett <mjg59@xxxxxxxxxxxxx> wrote:> > I dislike the kernel-side use of wakelocks. They're basically equivalent> > to a device returning -EBUSY during the suspend phase, which is> > something that can be done without any kernel modifications.> > I don't get why people object to wakelocks supporting timeouts, but> think drivers returning -EBUSY to abort suspend is ok. If suspend> fails, the higher level code has to periodically retry until it can> succeed. This means that the device is awake for longer than it need> to, and you are repeatedly wasting time freezing all tasks and> suspending a set of drivers before you get to the driver that is> preventing suspend.
I for one don't think drivers returning -EBUSY is okay. Once a user asksfor suspend to ram or hibernation, nothing but improper configuration(in the hibernation case) should stop them getting it. Even if they askto suspend to ram in the middle of writing a cd, they should get thesuspend to ram. But, of course, I'm just one voice.
Regards,
Nigel
_______________________________________________linux-pm mailing listlinux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx://lists.linux-foundation.org/mailman/listinfo/linux-pm


[Index of Archives]     [Linux ACPI]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [CPU Freq]     [Kernel Newbies]     [Fedora Kernel]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux