Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758437AbYBYVud (ORCPT ); Mon, 25 Feb 2008 16:50:33 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753978AbYBYVuY (ORCPT ); Mon, 25 Feb 2008 16:50:24 -0500 Received: from gprs189-60.eurotel.cz ([160.218.189.60]:43326 "EHLO amd.ucw.cz" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753091AbYBYVuX (ORCPT ); Mon, 25 Feb 2008 16:50:23 -0500 Date: Mon, 25 Feb 2008 22:50:49 +0100 From: Pavel Machek To: "Michael S. Tsirkin" Cc: "Rafael J. Wysocki" , Andrew Morton , LKML , Alexey Starikovskiy , Len Brown , linux-acpi@vger.kernel.org, Jeff Garzik Subject: Re: broken suspend in .2.6.25-rc3 on T61p (was Re: new regression in 2.6.25-rc3: no keyboard/lid acpi events on thinkpad T61p) Message-ID: <20080225215049.GD2659@elf.ucw.cz> References: <8f53421d0802251119q470a79c1p31048ef136271f84@mail.gmail.com> <20080225114611.d91791f6.akpm@linux-foundation.org> <8f53421d0802251245j5439b37die840e34bb293f995@mail.gmail.com> <200802252157.10412.rjw@sisk.pl> <20080225212615.GA2659@elf.ucw.cz> <8f53421d0802251331n31b37db8gb644dc9aa2d553d4@mail.gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8f53421d0802251331n31b37db8gb644dc9aa2d553d4@mail.gmail.com> X-Warning: Reading this can be dangerous to your mental health. User-Agent: Mutt/1.5.17 (2007-11-01) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1171 Lines: 38 Hi! > > > commit 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 > > > Author: Pavel Machek > > > Date: Thu Feb 21 13:56:55 2008 +0100 > > > > > > power_state: get rid of write-only variable in SATA > > > > This is pretty unlikely to be it. Can you double check that this patch > > really breaks something? > > I did and it seems to: just reverting > 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2 fixes resume for me. > 635adc28087ced0c843d2ecb6d4ae474d0e611cd which is > 559bbe6cbd0d8c68d40076a5f7dc98e3bf5864b2~1 > also resumes fine. Hmm, I guess that should teach me about "simple cleanups". Do you use any of: ata/sata_inic162x.c ata/sata_nv.c ata/sata_sil24.c by chance? (Ok, the patch is very safe to revert, it was "cleanup", it fixes nothing). Pavel -- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/