From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: Linux 2.6.21-rc6 Date: Sat, 14 Apr 2007 20:28:03 +0200 Message-ID: <200704142028.04694.rjw@sisk.pl> References: <20070414123154.GA2863@melchior.yamamaya.is-a-geek.org> <20070414130045.GE3691@stusta.de> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <20070414130045.GE3691@stusta.de> Content-Disposition: inline List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-pm-bounces@lists.linux-foundation.org Errors-To: linux-pm-bounces@lists.linux-foundation.org To: Adrian Bunk , Tobias Diedrich Cc: linux-pm@lists.osdl.org, Linux Kernel Mailing List , Maxim Levitsky List-Id: linux-pm@vger.kernel.org On Saturday, 14 April 2007 15:00, Adrian Bunk wrote: > On Sat, Apr 14, 2007 at 02:31:54PM +0200, Tobias Diedrich wrote: > > Tobias Diedrich wrote: > > > > ed746e3b18f4df18afa3763155972c5835f284c5 is first bad commit > > > > commit ed746e3b18f4df18afa3763155972c5835f284c5 > > > > Author: Rafael J. Wysocki > > > > Date: Sat Feb 10 01:43:32 2007 -0800 > > > > = > > > > [PATCH] swsusp: Change code ordering in disk.c > > > > = > > > > Change the ordering of code in kernel/power/disk.c so that devi= ce_suspend() is > > > > called before disable_nonboot_cpus() and platform_finish() is c= alled after > > > > enable_nonboot_cpus() and before device_resume(), as indicated = by the recent > > > > discussion on Linux-PM (cf. > > > > http://lists.osdl.org/pipermail/linux-pm/2006-November/004164.h= tml). > > > > = > > > > The changes here only affect the built-in swsusp. > > > > = > > > > [alexey.y.starikovskiy@linux.intel.com: fix LED blinking during= image load] > > > > Signed-off-by: Rafael J. Wysocki > > > > Acked-by: Pavel Machek > > > > Cc: Greg KH > > > > Cc: Nigel Cunningham > > > > Cc: Patrick Mochel > > > > Cc: Alexey Starikovskiy > > > > Signed-off-by: Andrew Morton > > > > Signed-off-by: Linus Torvalds > > > > = > > > > :040000 040000 7eca5b3a8f9606bc4f2ff41192ec8c9d4ca90d18 8313b674e1d= 1bdf6849350af06d28a89b3bb3054 M kernel > > > > = > > > > = > > > > Now, the remaining test is to try reverting this commit from -rc6. = :) > > > = > > > Doesn't apply cleanly against -rc6, but fixes the problem when > > > reverted from -rc1. > > = > > Now, this was already reported in > > http://lkml.org/lkml/2007/3/16/126 > > and I even flagged that message in my local folder, but apparently forg= ot > > to follow up on it... *sigh* > = > Unless I misunderstood something, all of the problems Maxim described in = > this email are fixed for him in -rc6. > = > But it's quite possible that you are running into a different issue = > exposed by this commit. Yes, it's likely. Tobias, I'm unable to reproduce the problem with your .config, but my hardw= are is certainly different. Which suspend mode do you use? If that's "platfor= m", can you try to use "shutdown" or "reboot" and see if that helps? Rafael -- = If you don't have the time to read, you don't have the time or the tools to write. - Stephen King