Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S964838AbWHHLDo (ORCPT ); Tue, 8 Aug 2006 07:03:44 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S964833AbWHHLDo (ORCPT ); Tue, 8 Aug 2006 07:03:44 -0400 Received: from brick.kernel.dk ([62.242.22.158]:36893 "EHLO kernel.dk") by vger.kernel.org with ESMTP id S964829AbWHHLDm (ORCPT ); Tue, 8 Aug 2006 07:03:42 -0400 Date: Tue, 8 Aug 2006 13:04:47 +0200 From: Jens Axboe To: "Rafael J. Wysocki" Cc: Jiri Slaby , Jason Lunz , Andrew Morton , linux-kernel@vger.kernel.org, andre@linux-ide.org, pavel@suse.cz, linux-pm@osdl.org, linux-ide@vger.kernel.org Subject: Re: swsusp regression [Was: 2.6.18-rc3-mm2] Message-ID: <20060808110447.GM4025@suse.de> References: <20060806030809.2cfb0b1e.akpm@osdl.org> <44D8626F.4020101@gmail.com> <20060808104353.GK4025@suse.de> <200608081259.15966.rjw@sisk.pl> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <200608081259.15966.rjw@sisk.pl> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2505 Lines: 53 On Tue, Aug 08 2006, Rafael J. Wysocki wrote: > On Tuesday 08 August 2006 12:43, Jens Axboe wrote: > > On Tue, Aug 08 2006, Jiri Slaby wrote: > > > Rafael J. Wysocki wrote: > > > >On Monday 07 August 2006 18:23, Jason Lunz wrote: > > > >>In gmane.linux.kernel, you wrote: > > > >>>>ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.18-rc3/2.6.18-rc3-mm2/ > > > >>>I tried it and guess what :)... swsusp doesn't work :@. > > > >>> > > > >>>This time I was able to dump process states with sysrq-t: > > > >>>http://www.fi.muni.cz/~xslaby/sklad/ide2.gif > > > >>> > > > >>>My guess is ide2/2.0 dies (hpt370 driver), since last thing kernel > > > >>>prints is suspending device 2.0 > > > >>Does it go away if you revert this? > > > >>ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/patches/2.6/2.6.18-rc3/2.6.18-rc3-mm2/broken-out/ide-reprogram-disk-pio-timings-on-resume.patch > > > >> > > > >>That should only affect resume, not suspend, but it does mess around > > > >>with ide power management. Is this maybe happening on the *second* > > > >>suspend? > > > >> > > > >>>-hdc: ATAPI 63X DVD-ROM DVD-R CD-R/RW drive, 2048kB Cache, UDMA(33) > > > >>>+hdc: ATAPI CD-ROM drive, 0kB Cache, UDMA(33) > > > >>This looks suspicious. -mm does have several ide-fix-hpt3xx patches. > > > > > > > >I found that git-block.patch broke the suspend for me. Still have no idea > > > >what's up with it. > > > > > > I suspect elevator changes. The wait_for_completion is not woken in > > > ide-io by ll_rw_blk. But I don't understand block layer too much. > > > > The ide changes are far more likely, it's probably missing a completion. > > Actually I think the commit f74bf2e6b415588e562fdcfdd454d587eb33cd46 > (Remove ->waiting member from struct request) is wrong, because > generic_ide_suspend() uses the end_of_io member of rq to pass the PM data > to ide_do_drive_cmd() where the pointer gets overwritten by &wait (must_wait > is "true", because action == ide_wait). Previously &wait was stored in > rq->waiting and it didn't overwrite the PM data. Indeed, that looks broken now. That must be what is screwing it up. With the former patch applied, did cdrom detection still look funny to you? I'll concoct a fix for that breakage. -- Jens Axboe - 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/