* powerpc: Mark low level irq handlers NO_THREAD
@ 2013-08-12 14:48 Sebastian Andrzej Siewior
2013-08-12 21:31 ` Benjamin Herrenschmidt
2013-08-12 23:11 ` Scott Wood
0 siblings, 2 replies; 3+ messages in thread
From: Sebastian Andrzej Siewior @ 2013-08-12 14:48 UTC (permalink / raw)
To: Vitaly Bordug
Cc: Marcelo Tosatti, Thomas Gleixner, linuxppc-dev,
Sebastian Andrzej Siewior
From: Thomas Gleixner <tglx@linutronix.de>
These low level handlers cannot be threaded. Mark them NO_THREAD
Reported-by: leroy christophe <christophe.leroy@c-s.fr>
Tested-by: leroy christophe <christophe.leroy@c-s.fr>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
---
This patch has been posted on Feb 13, 2013 and nobody responded back then.
arch/powerpc/platforms/8xx/m8xx_setup.c | 1 +
arch/powerpc/sysdev/cpm1.c | 1 +
2 files changed, 2 insertions(+)
--- a/arch/powerpc/platforms/8xx/m8xx_setup.c
+++ b/arch/powerpc/platforms/8xx/m8xx_setup.c
@@ -43,6 +43,7 @@ static irqreturn_t timebase_interrupt(in
static struct irqaction tbint_irqaction = {
.handler = timebase_interrupt,
+ .flags = IRQF_NO_THREAD,
.name = "tbint",
};
--- a/arch/powerpc/sysdev/cpm1.c
+++ b/arch/powerpc/sysdev/cpm1.c
@@ -120,6 +120,7 @@ static irqreturn_t cpm_error_interrupt(i
static struct irqaction cpm_error_irqaction = {
.handler = cpm_error_interrupt,
+ .flags = IRQF_NO_THREAD,
.name = "error",
};
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: powerpc: Mark low level irq handlers NO_THREAD
2013-08-12 14:48 powerpc: Mark low level irq handlers NO_THREAD Sebastian Andrzej Siewior
@ 2013-08-12 21:31 ` Benjamin Herrenschmidt
2013-08-12 23:11 ` Scott Wood
1 sibling, 0 replies; 3+ messages in thread
From: Benjamin Herrenschmidt @ 2013-08-12 21:31 UTC (permalink / raw)
To: Sebastian Andrzej Siewior; +Cc: Marcelo Tosatti, Thomas Gleixner, linuxppc-dev
On Mon, 2013-08-12 at 16:48 +0200, Sebastian Andrzej Siewior wrote:
> From: Thomas Gleixner <tglx@linutronix.de>
>
> These low level handlers cannot be threaded. Mark them NO_THREAD
>
> Reported-by: leroy christophe <christophe.leroy@c-s.fr>
> Tested-by: leroy christophe <christophe.leroy@c-s.fr>
> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> ---
>
> This patch has been posted on Feb 13, 2013 and nobody responded back then.
And it was merged in 3.11 no ? :-)
Cheers,
Ben.
> arch/powerpc/platforms/8xx/m8xx_setup.c | 1 +
> arch/powerpc/sysdev/cpm1.c | 1 +
> 2 files changed, 2 insertions(+)
>
> --- a/arch/powerpc/platforms/8xx/m8xx_setup.c
> +++ b/arch/powerpc/platforms/8xx/m8xx_setup.c
> @@ -43,6 +43,7 @@ static irqreturn_t timebase_interrupt(in
>
> static struct irqaction tbint_irqaction = {
> .handler = timebase_interrupt,
> + .flags = IRQF_NO_THREAD,
> .name = "tbint",
> };
>
> --- a/arch/powerpc/sysdev/cpm1.c
> +++ b/arch/powerpc/sysdev/cpm1.c
> @@ -120,6 +120,7 @@ static irqreturn_t cpm_error_interrupt(i
>
> static struct irqaction cpm_error_irqaction = {
> .handler = cpm_error_interrupt,
> + .flags = IRQF_NO_THREAD,
> .name = "error",
> };
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: powerpc: Mark low level irq handlers NO_THREAD
2013-08-12 14:48 powerpc: Mark low level irq handlers NO_THREAD Sebastian Andrzej Siewior
2013-08-12 21:31 ` Benjamin Herrenschmidt
@ 2013-08-12 23:11 ` Scott Wood
1 sibling, 0 replies; 3+ messages in thread
From: Scott Wood @ 2013-08-12 23:11 UTC (permalink / raw)
To: Sebastian Andrzej Siewior; +Cc: Marcelo Tosatti, Thomas Gleixner, linuxppc-dev
On Mon, 2013-08-12 at 16:48 +0200, Sebastian Andrzej Siewior wrote:
> From: Thomas Gleixner <tglx@linutronix.de>
>
> These low level handlers cannot be threaded. Mark them NO_THREAD
>
> Reported-by: leroy christophe <christophe.leroy@c-s.fr>
> Tested-by: leroy christophe <christophe.leroy@c-s.fr>
> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> ---
>
> This patch has been posted on Feb 13, 2013 and nobody responded back then.
It went in v3.11-rc1.
-Scott
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-08-12 23:11 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-12 14:48 powerpc: Mark low level irq handlers NO_THREAD Sebastian Andrzej Siewior
2013-08-12 21:31 ` Benjamin Herrenschmidt
2013-08-12 23:11 ` Scott Wood
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.