[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <47BC9AF4.10807@gmail.com>
Date: Thu, 21 Feb 2008 00:26:12 +0300
From: Alexey Starikovskiy <aystarik@...il.com>
To: "Rafael J. Wysocki" <rjw@...k.pl>
CC: Linus Torvalds <torvalds@...ux-foundation.org>,
suspend-devel@...ts.sourceforge.net,
Jeff Chua <jeff.chua.linux@...il.com>,
Dave Airlie <airlied@...ux.ie>, Greg KH <gregkh@...e.de>,
lkml <linux-kernel@...r.kernel.org>, linux-acpi@...r.kernel.org,
Jesse Barnes <jesse.barnes@...el.com>
Subject: Re: [Suspend-devel] 2.6.25-rc2 System no longer powers off after
suspend-to-disk. Screen becomes green.
Rafael J. Wysocki wrote:
> On Wednesday, 20 of February 2008, Linus Torvalds wrote:
>
>> On Wed, 20 Feb 2008, Rafael J. Wysocki wrote:
>>
>>> I think we should export the target sleep state somehow.
>>>
>> Yeah. By *not* using "->suspend()" for freezing or hibernate.
>>
>> Please, Rafael - just make the f*cking suspend-to-disk use other routines
>> already.
>>
>
> Okay, I think I'll just start sending patches for that, but rather not earlier
> than in the 2.6.27 time frame. No one else works on that and I've been busy
> with other things recently. Besides, I'm not even a full time kernel
> developer ...
>
>
Rafael,
If I can help, please say so.
Regards,
Alex.
>> 99% of all hardware needs to do exactly *nothing* on suspend-to-disk, and the
>> ones that really do need things tend to need to not do a whole lot.
>>
>> For example, the "freeze" action for USB (which is one of the hardest
>> things to suspend) should literally be something like just setting the
>> controller STOP bit, and waiting for it to have stopped. The "unfreeze"
>> should be to just clear the stop bit, while the "restart" should be just a
>> controller reset to use the current memory image.
>>
>> NONE OF THIS HAS ABSOLUTELY ANYTHING TO DO WITH SUSPEND.
>>
>> It never did. I've told people so for years. Maybe actually seeing the
>> problems will make people realize.
>>
>
> I think so.
>
>
>> So please, we shouldn't call "->suspend[_late]" or "->resume[_early]" at
>> all. Not with PMSG_FREEZE, not with PMSG_*anything*.
>>
>> Can we please get this fixed some day?
>>
>
> Yes, we can (hopefully).
>
> Thanks,
> Rafael
> -
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists