From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nigel Cunningham Subject: Re: S4 sleep woes Date: Mon, 02 Aug 2004 09:32:13 +1000 Sender: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org Message-ID: <1091403133.11580.63.camel@laptop.cunninghams> References: <20040730050615.GA24597@blackham.com.au> <20040731150112.GA480@hell.org.pl> <20040731184636.GG3443@blackham.com.au> Reply-To: ncunningham-jjFNsPSvq+iXDw4h08c5KA@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20040731184636.GG3443-4vSAtV5O1nc0n/F98K4Iww@public.gmane.org> Errors-To: acpi-devel-admin-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , List-Archive: To: Bernard Blackham Cc: ACPI List , Patrick Mochel List-Id: linux-acpi@vger.kernel.org Howdy. On Sun, 2004-08-01 at 04:46, Bernard Blackham wrote: > Ah, there appears to be one, yes (disk_show and disk_store in > kernel/power/disk.c). Looking again, pmdisk appears to handle it > correctly (just a little jump of logic). Guess that means it's a bug > in software suspend 2. :) I wasn't setting pm_disk_mode from suspend 2, but leaving it at it's default. It looks to me (without testing it yet) like the fix is to explicitly set pm_disk_mode to PM_DISK_PLATFORM when suspending and restore the original value afterwards. Sound right, Patrick? Regards, Nigel ------------------------------------------------------- This SF.Net email is sponsored by OSTG. Have you noticed the changes on Linux.com, ITManagersJournal and NewsForge in the past few weeks? Now, one more big change to announce. We are now OSTG- Open Source Technology Group. Come see the changes on the new OSTG site. www.ostg.com