Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751382Ab1BHErW (ORCPT ); Mon, 7 Feb 2011 23:47:22 -0500 Received: from mail-iy0-f174.google.com ([209.85.210.174]:42104 "EHLO mail-iy0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750852Ab1BHErU (ORCPT ); Mon, 7 Feb 2011 23:47:20 -0500 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=date:from:to:cc:subject:message-id:references:mime-version :content-type:content-disposition:in-reply-to:user-agent; b=EvqrgG+9qtkzsr+87f2YxF2gutM7eGPb8nytM6oT11URojBTrLWSdDeazrmF7jUwXm 8X79EzboxRIPr9Hf5bymYpKRJtUvfnwQ6ufXsrbmSQMUpzv7pX3EneFn53FqBOVFzuGn 61RRM51AxdtnCbIePDDLI2iQzhl6jgzccZCe0= Date: Mon, 7 Feb 2011 20:47:12 -0800 From: Dmitry Torokhov To: "Rafael J. Wysocki" Cc: Len Brown , ACPI Devel Mailing List , LKML , Linux PM mailing list Subject: Re: [PATCH] thermal: Use freezable workqueue Message-ID: <20110208044712.GA906@core.coreip.homeip.net> References: <201102080029.57195.rjw@sisk.pl> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201102080029.57195.rjw@sisk.pl> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1738 Lines: 47 On Tue, Feb 08, 2011 at 12:29:57AM +0100, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki > > If thermal polling is enabled, which for example is the case for > ACPI thermal zones with the _TZP object defined, the thermal driver > uses delayed work items for this purpose. Unfortunately, since > they are queued up using schedule_delayed_work(), the work function > may be executed during system suspend or resume, which is not > desirable. > > To prevent that from happening, use a freezable workqueue for > queuing up delayed work items in the thermal driver. > > Signed-off-by: Rafael J. Wysocki > --- > drivers/thermal/thermal_sys.c | 22 ++++++++++++++++++++-- > 1 file changed, 20 insertions(+), 2 deletions(-) > > Index: linux-2.6/drivers/thermal/thermal_sys.c > =================================================================== > --- linux-2.6.orig/drivers/thermal/thermal_sys.c > +++ linux-2.6/drivers/thermal/thermal_sys.c > @@ -62,6 +62,20 @@ static DEFINE_MUTEX(thermal_list_lock); > > static unsigned int thermal_event_seqnum; > > +static struct workqueue_struct *thermal_wq; > + > +static int __init thermal_start_workqueue(void) > +{ > + thermal_wq = alloc_workqueue("thermal", WQ_FREEZEABLE, 0); Should probably be unbound as well. FWIW, I would not mind if we had a global freezeable workqueue already predefined. I could switch input_polldev and vmw_balloon to it and there probably could be more users... Thanks. -- Dmitry -- 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/