Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757291AbYCZX7p (ORCPT ); Wed, 26 Mar 2008 19:59:45 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754646AbYCZX7f (ORCPT ); Wed, 26 Mar 2008 19:59:35 -0400 Received: from ogre.sisk.pl ([217.79.144.158]:41982 "EHLO ogre.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754626AbYCZX7e (ORCPT ); Wed, 26 Mar 2008 19:59:34 -0400 From: "Rafael J. Wysocki" To: Alexey Starikovskiy Subject: Re: [RFC][PATCH 1/3] PM: Introduce new top level suspend and hibernation callbacks (rev. 4) Date: Thu, 27 Mar 2008 00:59:13 +0100 User-Agent: KMail/1.9.6 (enterprise 20070904.708012) Cc: pm list , ACPI Devel Maling List , Alan Stern , Greg KH , Len Brown , LKML , David Brownell , Pavel Machek , Benjamin Herrenschmidt , Oliver Neukum References: <200803262353.30566.rjw@sisk.pl> <200803270003.51570.rjw@sisk.pl> <47EADBD0.6040400@suse.de> In-Reply-To: <47EADBD0.6040400@suse.de> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200803270059.14417.rjw@sisk.pl> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1476 Lines: 31 On Thursday, 27 of March 2008, Alexey Starikovskiy wrote: > Rafael J. Wysocki wrote: > > This is the 4th revision of the patch. > > + * struct pm_noirq_ops - device PM callbacks executed with interrupts disabled > > + * > > + * The following callbacks included in 'struct pm_noirq_ops' are executed with > > + * the nonboot CPUs switched off and with interrupts disabled on the only > > + * functional CPU. They also are executed with the PM core list of devices > > + * locked, so they must NOT unregister any devices. > > + * > > + * @suspend_noirq: Complete the operations of ->suspend() by carrying out any > > + * actions required for suspending the device that need interrupts to be > > + * disabled > IMHO, no need to add _noirq in both struct and struct members. > pm_noirq->suspend_noirq does not look good... I added the _noirq part to the names of the callbacks, since otherwise the second struct looks confusingly similar to the first one. I also could have put the _noirq callbacks into the first struct, but that would have been wasteful, since device types and device classes don't use them. Also, the majority of bus types won't use them and the vast majority of drivers won't use them as well. Thanks, Rafael -- 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/