From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756655Ab0EKRtP (ORCPT ); Tue, 11 May 2010 13:49:15 -0400 Received: from mho-02-ewr.mailhop.org ([204.13.248.72]:62547 "EHLO mho-02-ewr.mailhop.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754832Ab0EKRtN (ORCPT ); Tue, 11 May 2010 13:49:13 -0400 X-Mail-Handler: MailHop Outbound by DynDNS X-Originating-IP: 69.181.193.102 X-Report-Abuse-To: abuse@dyndns.com (see http://www.dyndns.com/services/mailhop/outbound_abuse.html for abuse reporting information) X-MHO-User: U2FsdGVkX18NoadW0NX5Pv8aD/bSUsxK Date: Tue, 11 May 2010 10:48:58 -0700 From: Tony Lindgren To: Matthew Garrett Cc: "Rafael J. Wysocki" , Kevin Hilman , Arve =?utf-8?B?SGrDuG5uZXbDpWc=?= , linux-pm@lists.linux-foundation.org, linux-kernel@vger.kernel.org, Alan Stern , Tejun Heo , Oleg Nesterov , Paul Walmsley , magnus.damm@gmail.com, mark gross , Arjan van de Ven , Geoff Smith , Brian Swetland Subject: Re: [PATCH 0/8] Suspend block api (version 6) Message-ID: <20100511174857.GC13931@atomide.com> References: <87632vhbs8.fsf@deeprootsystems.com> <201005102225.52431.rjw@sisk.pl> <20100511161227.GD13600@atomide.com> <20100511161448.GA16148@srcf.ucam.org> <20100511163632.GE13600@atomide.com> <20100511164554.GA17016@srcf.ucam.org> <20100511165821.GA13931@atomide.com> <20100511170348.GA17443@srcf.ucam.org> <20100511172442.GB13931@atomide.com> <20100511173036.GB17868@srcf.ucam.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20100511173036.GB17868@srcf.ucam.org> User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Matthew Garrett [100511 10:25]: > On Tue, May 11, 2010 at 10:24:43AM -0700, Tony Lindgren wrote: > > > For the failed suspend path in the kernel, currently the kernel would > > unwind back all the drivers because of the failed driver, but that path > > should be possible to optimize. > > If you think it's possible to make this work then feel free to. But at > the point where you're adding code to every driver's suspend function to > determine whether or not it's got any pending events that userspace > hasn't consumed yet, and adding code to every bit of userspace to allow > it to indicate whether or not it's busy consuming events or just busy > drawing 3D bouncing cattle, I think you've reinvented suspend blocks. Sorry, I have a working system that idles nicely and stays up on batteries for a long time while running. I don't need to implement anything like this :) Cheers, Tony