From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Rafael J. Wysocki" Subject: Re: [PATCH] PM: Introduce core framework for run-time PM of I/O devices (rev. 3) Date: Tue, 23 Jun 2009 19:00:03 +0200 Message-ID: <200906231900.04660.rjw@sisk.pl> References: <200906230121.18932.rjw@sisk.pl> Mime-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Return-path: Received: from ogre.sisk.pl ([217.79.144.158]:38207 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755043AbZFWQ7X (ORCPT ); Tue, 23 Jun 2009 12:59:23 -0400 In-Reply-To: <200906230121.18932.rjw@sisk.pl> Content-Disposition: inline Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Alan Stern Cc: linux-pm@lists.linux-foundation.org, Oliver Neukum , Magnus Damm , ACPI Devel Maling List , Ingo Molnar , LKML , Greg KH , Arjan van de Ven On Tuesday 23 June 2009, Rafael J. Wysocki wrote: > Hi, > > Below is a new revision of the patch introducing the run-time PM framework. > > The most visible changes from the last version: > > * I realized that if child_count is atomic, we can drop the parent locking from > all of the functions, so I did that. > > * Introduced pm_runtime_put() that decrements the resume counter and queues > up an idle notification if the counter went down to 0 (and wasn't 0 previously). > Using asynchronous notification makes it possible to call pm_runtime_put() > from interrupt context, if necessary. > > * Changed the meaning of the RPM_WAKE bit slightly (it is now also used for > disabling run-time PM for a device along with the resume counter). > > Please let me know if I've overlooked anything. :-) Well, I found quite a few problems myself, mostly related to disabling-enabling of the run-time PM and to RPM_WAKE. Updated patch will be sent out later today. Best, Rafael