Messages in this thread |  | | | From | "Rafael J. Wysocki" <> | | Subject | Re: [PATCH 1/3] add flag IRQF_NO_SUSPEND in 'struct irqaction' | | Date | Fri, 17 Jul 2009 03:52:11 +0200 |
| |
On Friday 17 July 2009, Guanqun Lu wrote: > We currently only bypass IRQF_TIMER in '__disable_irq', > but Xen specific IRQs should not be disabled either. > This commit adds a new flag to accompolish this goal > without being mixed up with IRQF_TIMER flag.
For some obscure reasons [2/3] didn't reach my inbox. The other 2 patches look fine to me.
Best, Rafael
> Signed-off-by: Guanqun Lu <guanqun.lu@intel.com> > --- > include/linux/interrupt.h | 1 + > kernel/irq/manage.c | 3 ++- > 2 files changed, 3 insertions(+), 1 deletions(-) > > diff --git a/include/linux/interrupt.h b/include/linux/interrupt.h > index 2721f07..99264c3 100644 > --- a/include/linux/interrupt.h > +++ b/include/linux/interrupt.h > @@ -58,6 +58,7 @@ > #define IRQF_PERCPU 0x00000400 > #define IRQF_NOBALANCING 0x00000800 > #define IRQF_IRQPOLL 0x00001000 > +#define IRQF_NO_SUSPEND 0x00002000 > > /* > * Bits used by threaded handlers: > diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c > index 50da676..3dc4e74 100644 > --- a/kernel/irq/manage.c > +++ b/kernel/irq/manage.c > @@ -192,7 +192,8 @@ 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_TIMER | IRQF_NO_SUSPEND))) > return; > desc->status |= IRQ_SUSPENDED; > }
|  |