Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932080AbVI2Hen (ORCPT ); Thu, 29 Sep 2005 03:34:43 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932083AbVI2Hen (ORCPT ); Thu, 29 Sep 2005 03:34:43 -0400 Received: from pentafluge.infradead.org ([213.146.154.40]:61345 "EHLO pentafluge.infradead.org") by vger.kernel.org with ESMTP id S932080AbVI2Hem (ORCPT ); Thu, 29 Sep 2005 03:34:42 -0400 Date: Thu, 29 Sep 2005 08:34:37 +0100 From: Christoph Hellwig To: Jeff Garzik Cc: Joshua Kwan , linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org, axboe@suse.de, torvalds@osdl.org, randy_dunlap Subject: Re: SATA suspend/resume (was Re: [PATCH] updated version of Jens' SATA suspend-to-ram patch) Message-ID: <20050929073437.GC9669@infradead.org> Mail-Followup-To: Christoph Hellwig , Jeff Garzik , Joshua Kwan , linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org, axboe@suse.de, torvalds@osdl.org, randy_dunlap References: <20050923163334.GA13567@triplehelix.org> <433B79D8.9080305@pobox.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <433B79D8.9080305@pobox.com> User-Agent: Mutt/1.4.2.1i X-SRS-Rewrite: SMTP reverse-path rewritten from by pentafluge.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1796 Lines: 43 On Thu, Sep 29, 2005 at 01:21:28AM -0400, Jeff Garzik wrote: > Ah hah! I found the other SCSI suspend patch: > > http://lwn.net/Articles/97453/ > > Anybody (Joshua?) up for reconciling and testing the two? > > The main change from Jens/Joshua's patch is that we use SCSI's > sd_shutdown() to call sync cache, eliminating the need for > ata_flush_cache(), since the SCSI layer would now perform that. > > For bonus points, > > 1) sd should call START STOP UNIT on suspend, which eliminates the need > for ata_standby_drive(), and completely encompasses the suspend process > in the SCSI layer. > > 2) sd should call START STOP UNIT on resume -- and as a SUPER BONUS, the > combination of these two changes ensures that there are no queue > synchronization issues, the likes of which would require hacks like > Jens' while-loop patch. > > None of these are huge changes requiring a lot of thinking/planning... > > Finally, ideally, we should be issuing a hardware or software reset on > suspend. I like this one much more than the other patch aswell, because suspsending is an ULDD operation, not an LLDD one, and this fits the layering model much better. The only complaints here are cosmetics: - generic_scsi_suspend/generic_scsi_resume are misnamed, they should probably be scsi_device_suspend/resume. - while we're at it they could probably move to scsi_sysfs.c to keep them static in one file - they're just a tiny bit of glue anyway. - get rid of all the CONFIG_PM ifdefs - it just clutters thing up far too much. - 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/