From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932371AbWGJJNx (ORCPT ); Mon, 10 Jul 2006 05:13:53 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932374AbWGJJNw (ORCPT ); Mon, 10 Jul 2006 05:13:52 -0400 Received: from caramon.arm.linux.org.uk ([212.18.232.186]:7697 "EHLO caramon.arm.linux.org.uk") by vger.kernel.org with ESMTP id S932371AbWGJJNw (ORCPT ); Mon, 10 Jul 2006 05:13:52 -0400 Date: Mon, 10 Jul 2006 10:13:40 +0100 From: Russell King To: Ingo Molnar Cc: David Brownell , Linux Kernel list , tglx@linutronix.de, mingo@redhat.com, Andrew Victor , Alessandro Zummo Subject: Re: [patch 2.6.18-rc1] genirq: {en,dis}able_irq_wake() need refcounting too Message-ID: <20060710091340.GA4400@flint.arm.linux.org.uk> Mail-Followup-To: Ingo Molnar , David Brownell , Linux Kernel list , tglx@linutronix.de, mingo@redhat.com, Andrew Victor , Alessandro Zummo References: <200607091458.52298.david-b@pacbell.net> <20060710085849.GA6016@elte.hu> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20060710085849.GA6016@elte.hu> User-Agent: Mutt/1.4.1i Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Jul 10, 2006 at 10:58:49AM +0200, Ingo Molnar wrote: > > * David Brownell wrote: > > > It's not just "normal" mode operation that needs refcounting for the > > {en,dis}able_irq() calls ... "wakeup" mode calls need it too, for the > > very same reasons. > > > > This patch adds that refcounting. I expect that some ARM drivers will > > be triggering the new warning, but this call isn't yet widely used. > > (Which is probably why the bug has lingered this long...) > > Acked-by: Ingo Molnar > > we should also add disable_irq_wake() / enable_irq_wake() APIs and start > migrating most ARM users over to the new APIs, agreed? That makes the > APIs more symmetric and the code more readable too. That _is_ the API anyway. set_irq_wake() was never intended to be called directly from drivers. -- Russell King Linux kernel 2.6 ARM Linux - http://www.arm.linux.org.uk/ maintainer of: 2.6 Serial core