[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-id: <4569F957.4090100@shaw.ca>
Date: Sun, 26 Nov 2006 14:30:15 -0600
From: Robert Hancock <hancockr@...w.ca>
To: "Rafael J. Wysocki" <rjw@...k.pl>
Cc: linux-kernel <linux-kernel@...r.kernel.org>,
Alan <alan@...rguk.ukuu.org.uk>, Pavel Machek <pavel@....cz>
Subject: Re: [patch] PM: suspend/resume debugging should depend on
SOFTWARE_SUSPEND
Rafael J. Wysocki wrote:
>> btw, I have some code almost ready for sata_nv to add proper
>> suspend/resume support. Unfortunately I have trouble testing it, since
>> STR doesn't work on my machine since, guess what - the video doesn't
>> come back! It doesn't even take the monitor out of standby mode. None of
>> the acpi_sleep options seem to work, and vbetool appears to helpfully
>> segfault on any operation so that's out.
>
> I guess it's x86-64? Which version of vbetool?
Yes, it's x86-64, with whatever version of vbetool comes with Fedora Core 5.
>
>> This is an NVIDIA SLI setup so that probably makes things a bit more
>> complicated.
>
> Ouch.
>
>> In any case, it should be better than what we have right now for
>> suspend/resume support in sata_nv, namely the "do nothing, won't work
>> (at least not for CK804 and later)" implementation..
>
> I think I can test it if you send me the patch.
>
> Greetings,
> Rafael
Here it is attached. This patch needs to be applied to 2.6.19-rc6-mm1 on
top of the other one I just submitted, "[PATCH -mm] sata_nv: fix ATAPI
in ADMA mode". I don't think it's too horribly broken since it did what
it should have on some aborted suspend-to-disk attempts. (It looks like
STD doesn't work either, it complains there is no swap available even
though there is.. maybe because there's 2GB of RAM and 2GB of swap? It
should still fit though, I would think, as not all RAM is in use..) In
any case, if it works out OK then I can submit this formally.
--
Robert Hancock Saskatoon, SK, Canada
To email, remove "nospam" from hancockr@...pamshaw.ca
Home Page: http://www.roberthancock.com/
View attachment "sata_nv-add-suspend-resume-support.patch" of type "text/plain" (14955 bytes)
Powered by blists - more mailing lists