linux-kernel - Re: Linux 2.6.21-rc6

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20070414195611.GC806@yamamaya.is-a-geek.org>
Date:	Sat, 14 Apr 2007 21:56:11 +0200
From:	Tobias Diedrich <ranma@...edrich.de>
To:	"Rafael J. Wysocki" <rjw@...k.pl>
Cc:	Adrian Bunk <bunk@...sta.de>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	pavel@...e.cz, linux-pm@...ts.osdl.org,
	Maxim Levitsky <maximlevitsky@...il.com>
Subject: Re: Linux 2.6.21-rc6

Rafael J. Wysocki wrote:
> 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 <rjw@...k.pl>
> > > > > 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 device_suspend() is
> > > > >     called before disable_nonboot_cpus() and platform_finish() is called 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.html).
> > > > >     
> > > > >     The changes here only affect the built-in swsusp.
> > > > >     
> > > > >     [alexey.y.starikovskiy@...ux.intel.com: fix LED blinking during image load]
> > > > >     Signed-off-by: Rafael J. Wysocki <rjw@...k.pl>
> > > > >     Acked-by: Pavel Machek <pavel@....cz>
> > > > >     Cc: Greg KH <greg@...ah.com>
> > > > >     Cc: Nigel Cunningham <nigel@...pend2.net>
> > > > >     Cc: Patrick Mochel <mochel@...italimplant.org>
> > > > >     Cc: Alexey Starikovskiy <alexey.y.starikovskiy@...ux.intel.com>
> > > > >     Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> > > > >     Signed-off-by: Linus Torvalds <torvalds@...ux-foundation.org>
> > > > > 
> > > > > :040000 040000 7eca5b3a8f9606bc4f2ff41192ec8c9d4ca90d18 8313b674e1d1bdf6849350af06d28a89b3bb3054 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 forgot
> > > 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 hardware
> is certainly different.  Which suspend mode do you use?  If that's "platform",
> can you try to use "shutdown" or "reboot" and see if that helps?

Sure.
shutdown/reboot works fine, only platform is broken.

-- 
Tobias						PGP: http://9ac7e0bc.uguu.de
-
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/