Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758220Ab0G3Ka1 (ORCPT ); Fri, 30 Jul 2010 06:30:27 -0400 Received: from gate.crashing.org ([63.228.1.57]:59269 "EHLO gate.crashing.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756346Ab0G3KaK (ORCPT ); Fri, 30 Jul 2010 06:30:10 -0400 Subject: Re: [PATCH 1/4] irq: Add new IRQ flag IRQF_NO_SUSPEND From: Benjamin Herrenschmidt To: Ian Campbell Cc: linux-kernel@vger.kernel.org, Thomas Gleixner , Jeremy Fitzhardinge , Dmitry Torokhov , Paul Mackerras , Grant Likely , xen-devel@lists.xensource.com, linux-input@vger.kernel.org, linuxppc-dev@ozlabs.org, devicetree-discuss@lists.ozlabs.org In-Reply-To: <1280398595-29708-1-git-send-email-ian.campbell@citrix.com> References: <1280398573.24292.1684.camel@zakaz.uk.xensource.com> <1280398595-29708-1-git-send-email-ian.campbell@citrix.com> Content-Type: text/plain; charset="UTF-8" Date: Fri, 30 Jul 2010 20:29:38 +1000 Message-ID: <1280485778.1902.9.camel@pasglop> Mime-Version: 1.0 X-Mailer: Evolution 2.28.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2752 Lines: 71 On Thu, 2010-07-29 at 11:16 +0100, Ian Campbell wrote: > A small number of users of IRQF_TIMER are using it for the implied no > suspend behaviour on interrupts which are not timer interrupts. > > Therefore add a new IRQF_NO_SUSPEND flag, rename IRQF_TIMER to > __IRQF_TIMER and redefine IRQF_TIMER in terms of these new flags. > > Signed-off-by: Ian Campbell > Cc: Thomas Gleixner > Cc: Jeremy Fitzhardinge > Cc: Dmitry Torokhov Acked-by: Benjamin Herrenschmidt > Cc: Paul Mackerras > Cc: Grant Likely > Cc: xen-devel@lists.xensource.com > Cc: linux-input@vger.kernel.org > Cc: linuxppc-dev@ozlabs.org > Cc: devicetree-discuss@lists.ozlabs.org > --- > include/linux/interrupt.h | 7 ++++++- > kernel/irq/manage.c | 2 +- > 2 files changed, 7 insertions(+), 2 deletions(-) > > diff --git a/include/linux/interrupt.h b/include/linux/interrupt.h > index c233113..a0384a4 100644 > --- a/include/linux/interrupt.h > +++ b/include/linux/interrupt.h > @@ -53,16 +53,21 @@ > * IRQF_ONESHOT - Interrupt is not reenabled after the hardirq handler finished. > * Used by threaded interrupts which need to keep the > * irq line disabled until the threaded handler has been run. > + * IRQF_NO_SUSPEND - Do not disable this IRQ during suspend > + * > */ > #define IRQF_DISABLED 0x00000020 > #define IRQF_SAMPLE_RANDOM 0x00000040 > #define IRQF_SHARED 0x00000080 > #define IRQF_PROBE_SHARED 0x00000100 > -#define IRQF_TIMER 0x00000200 > +#define __IRQF_TIMER 0x00000200 > #define IRQF_PERCPU 0x00000400 > #define IRQF_NOBALANCING 0x00000800 > #define IRQF_IRQPOLL 0x00001000 > #define IRQF_ONESHOT 0x00002000 > +#define IRQF_NO_SUSPEND 0x00004000 > + > +#define IRQF_TIMER (__IRQF_TIMER | IRQF_NO_SUSPEND) > > /* > * Bits used by threaded handlers: > diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c > index e149748..c3003e9 100644 > --- a/kernel/irq/manage.c > +++ b/kernel/irq/manage.c > @@ -216,7 +216,7 @@ static inline int setup_affinity(unsigned int irq, struct irq_desc *desc) > void __disable_irq(struct irq_desc *desc, unsigned int irq, bool suspend) > { > if (suspend) { > - if (!desc->action || (desc->action->flags & IRQF_TIMER)) > + if (!desc->action || (desc->action->flags & IRQF_NO_SUSPEND)) > return; > desc->status |= IRQ_SUSPENDED; > } -- 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/