* [PATCH] MIPS: add IRQF_TIMER flag for Timer Interrupts
@ 2009-10-08 13:17 Wu Zhangjin
2009-10-08 15:13 ` Thomas Gleixner
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Wu Zhangjin @ 2009-10-08 13:17 UTC (permalink / raw)
To: linux-mips, pm list, linux-kernel
Cc: Thomas Gleixner, Ralf Baechle, Rafael J. Wysocki, Yoichi Yuasa,
Wu Zhangjin
This patch add IRQF_TIMER flag for all Timer interrupts in linux-MIPS,
which will help to not disable the Timer IRQ when suspending to ensure
resuming normally(d6c585a4342a2ff627a29f9aea77c5ed4cd76023) and not
thread them when enabled PREEMPT_RT.
Signed-off-by: Wu Zhangjin <wuzhangjin@gmail.com>
---
arch/mips/jazz/irq.c | 2 +-
arch/mips/kernel/cevt-gt641xx.c | 2 +-
arch/mips/kernel/cevt-r4k.c | 2 +-
arch/mips/kernel/i8253.c | 2 +-
arch/mips/nxp/pnx8550/common/time.c | 2 +-
arch/mips/sgi-ip27/ip27-timer.c | 2 +-
arch/mips/sni/time.c | 2 +-
7 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/arch/mips/jazz/irq.c b/arch/mips/jazz/irq.c
index 7fd170d..7bd32d0 100644
--- a/arch/mips/jazz/irq.c
+++ b/arch/mips/jazz/irq.c
@@ -134,7 +134,7 @@ static irqreturn_t r4030_timer_interrupt(int irq, void *dev_id)
static struct irqaction r4030_timer_irqaction = {
.handler = r4030_timer_interrupt,
- .flags = IRQF_DISABLED,
+ .flags = IRQF_DISABLED | IRQF_TIMER,
.name = "R4030 timer",
};
diff --git a/arch/mips/kernel/cevt-gt641xx.c b/arch/mips/kernel/cevt-gt641xx.c
index 92351e0..f5d265e 100644
--- a/arch/mips/kernel/cevt-gt641xx.c
+++ b/arch/mips/kernel/cevt-gt641xx.c
@@ -113,7 +113,7 @@ static irqreturn_t gt641xx_timer0_interrupt(int irq, void *dev_id)
static struct irqaction gt641xx_timer0_irqaction = {
.handler = gt641xx_timer0_interrupt,
- .flags = IRQF_DISABLED | IRQF_PERCPU,
+ .flags = IRQF_DISABLED | IRQF_PERCPU | IRQF_TIMER,
.name = "gt641xx_timer0",
};
diff --git a/arch/mips/kernel/cevt-r4k.c b/arch/mips/kernel/cevt-r4k.c
index 2652362..b469ad0 100644
--- a/arch/mips/kernel/cevt-r4k.c
+++ b/arch/mips/kernel/cevt-r4k.c
@@ -83,7 +83,7 @@ out:
struct irqaction c0_compare_irqaction = {
.handler = c0_compare_interrupt,
- .flags = IRQF_DISABLED | IRQF_PERCPU,
+ .flags = IRQF_DISABLED | IRQF_PERCPU | IRQF_TIMER,
.name = "timer",
};
diff --git a/arch/mips/kernel/i8253.c b/arch/mips/kernel/i8253.c
index f7d8d5d..ed5c441 100644
--- a/arch/mips/kernel/i8253.c
+++ b/arch/mips/kernel/i8253.c
@@ -98,7 +98,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)
static struct irqaction irq0 = {
.handler = timer_interrupt,
- .flags = IRQF_DISABLED | IRQF_NOBALANCING,
+ .flags = IRQF_DISABLED | IRQF_NOBALANCING | IRQF_TIMER,
.name = "timer"
};
diff --git a/arch/mips/nxp/pnx8550/common/time.c b/arch/mips/nxp/pnx8550/common/time.c
index 18b1927..d987a89 100644
--- a/arch/mips/nxp/pnx8550/common/time.c
+++ b/arch/mips/nxp/pnx8550/common/time.c
@@ -59,7 +59,7 @@ static irqreturn_t pnx8xxx_timer_interrupt(int irq, void *dev_id)
static struct irqaction pnx8xxx_timer_irq = {
.handler = pnx8xxx_timer_interrupt,
- .flags = IRQF_DISABLED | IRQF_PERCPU,
+ .flags = IRQF_DISABLED | IRQF_PERCPU | IRQF_TIMER,
.name = "pnx8xxx_timer",
};
diff --git a/arch/mips/sgi-ip27/ip27-timer.c b/arch/mips/sgi-ip27/ip27-timer.c
index 6d0e59f..d6802d6 100644
--- a/arch/mips/sgi-ip27/ip27-timer.c
+++ b/arch/mips/sgi-ip27/ip27-timer.c
@@ -105,7 +105,7 @@ static irqreturn_t hub_rt_counter_handler(int irq, void *dev_id)
struct irqaction hub_rt_irqaction = {
.handler = hub_rt_counter_handler,
- .flags = IRQF_DISABLED | IRQF_PERCPU,
+ .flags = IRQF_DISABLED | IRQF_PERCPU | IRQF_TIMER,
.name = "hub-rt",
};
diff --git a/arch/mips/sni/time.c b/arch/mips/sni/time.c
index 62df6a5..f3b60e6 100644
--- a/arch/mips/sni/time.c
+++ b/arch/mips/sni/time.c
@@ -67,7 +67,7 @@ static irqreturn_t a20r_interrupt(int irq, void *dev_id)
static struct irqaction a20r_irqaction = {
.handler = a20r_interrupt,
- .flags = IRQF_DISABLED | IRQF_PERCPU,
+ .flags = IRQF_DISABLED | IRQF_PERCPU | IRQF_TIMER,
.name = "a20r-timer",
};
--
1.6.2.1
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH] MIPS: add IRQF_TIMER flag for Timer Interrupts
2009-10-08 13:17 [PATCH] MIPS: add IRQF_TIMER flag for Timer Interrupts Wu Zhangjin
@ 2009-10-08 15:13 ` Thomas Gleixner
2009-10-08 20:26 ` Rafael J. Wysocki
2009-10-09 15:17 ` Atsushi Nemoto
2 siblings, 0 replies; 5+ messages in thread
From: Thomas Gleixner @ 2009-10-08 15:13 UTC (permalink / raw)
To: Wu Zhangjin
Cc: linux-mips, pm list, linux-kernel, Ralf Baechle,
Rafael J. Wysocki, Yoichi Yuasa
On Thu, 8 Oct 2009, Wu Zhangjin wrote:
> This patch add IRQF_TIMER flag for all Timer interrupts in linux-MIPS,
> which will help to not disable the Timer IRQ when suspending to ensure
> resuming normally(d6c585a4342a2ff627a29f9aea77c5ed4cd76023) and not
> thread them when enabled PREEMPT_RT.
>
> Signed-off-by: Wu Zhangjin <wuzhangjin@gmail.com>
Acked-by: Thomas Gleixner <tglx@linutronix.de>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] MIPS: add IRQF_TIMER flag for Timer Interrupts
2009-10-08 13:17 [PATCH] MIPS: add IRQF_TIMER flag for Timer Interrupts Wu Zhangjin
2009-10-08 15:13 ` Thomas Gleixner
@ 2009-10-08 20:26 ` Rafael J. Wysocki
2009-10-09 15:17 ` Atsushi Nemoto
2 siblings, 0 replies; 5+ messages in thread
From: Rafael J. Wysocki @ 2009-10-08 20:26 UTC (permalink / raw)
To: Wu Zhangjin
Cc: linux-mips, pm list, linux-kernel, Thomas Gleixner, Ralf Baechle,
Yoichi Yuasa
On Thursday 08 October 2009, Wu Zhangjin wrote:
> This patch add IRQF_TIMER flag for all Timer interrupts in linux-MIPS,
> which will help to not disable the Timer IRQ when suspending to ensure
> resuming normally(d6c585a4342a2ff627a29f9aea77c5ed4cd76023) and not
> thread them when enabled PREEMPT_RT.
>
> Signed-off-by: Wu Zhangjin <wuzhangjin@gmail.com>
Acked-by: Rafael J. Wysocki <rjw@sisk.pl>
> ---
> arch/mips/jazz/irq.c | 2 +-
> arch/mips/kernel/cevt-gt641xx.c | 2 +-
> arch/mips/kernel/cevt-r4k.c | 2 +-
> arch/mips/kernel/i8253.c | 2 +-
> arch/mips/nxp/pnx8550/common/time.c | 2 +-
> arch/mips/sgi-ip27/ip27-timer.c | 2 +-
> arch/mips/sni/time.c | 2 +-
> 7 files changed, 7 insertions(+), 7 deletions(-)
>
> diff --git a/arch/mips/jazz/irq.c b/arch/mips/jazz/irq.c
> index 7fd170d..7bd32d0 100644
> --- a/arch/mips/jazz/irq.c
> +++ b/arch/mips/jazz/irq.c
> @@ -134,7 +134,7 @@ static irqreturn_t r4030_timer_interrupt(int irq, void *dev_id)
>
> static struct irqaction r4030_timer_irqaction = {
> .handler = r4030_timer_interrupt,
> - .flags = IRQF_DISABLED,
> + .flags = IRQF_DISABLED | IRQF_TIMER,
> .name = "R4030 timer",
> };
>
> diff --git a/arch/mips/kernel/cevt-gt641xx.c b/arch/mips/kernel/cevt-gt641xx.c
> index 92351e0..f5d265e 100644
> --- a/arch/mips/kernel/cevt-gt641xx.c
> +++ b/arch/mips/kernel/cevt-gt641xx.c
> @@ -113,7 +113,7 @@ static irqreturn_t gt641xx_timer0_interrupt(int irq, void *dev_id)
>
> static struct irqaction gt641xx_timer0_irqaction = {
> .handler = gt641xx_timer0_interrupt,
> - .flags = IRQF_DISABLED | IRQF_PERCPU,
> + .flags = IRQF_DISABLED | IRQF_PERCPU | IRQF_TIMER,
> .name = "gt641xx_timer0",
> };
>
> diff --git a/arch/mips/kernel/cevt-r4k.c b/arch/mips/kernel/cevt-r4k.c
> index 2652362..b469ad0 100644
> --- a/arch/mips/kernel/cevt-r4k.c
> +++ b/arch/mips/kernel/cevt-r4k.c
> @@ -83,7 +83,7 @@ out:
>
> struct irqaction c0_compare_irqaction = {
> .handler = c0_compare_interrupt,
> - .flags = IRQF_DISABLED | IRQF_PERCPU,
> + .flags = IRQF_DISABLED | IRQF_PERCPU | IRQF_TIMER,
> .name = "timer",
> };
>
> diff --git a/arch/mips/kernel/i8253.c b/arch/mips/kernel/i8253.c
> index f7d8d5d..ed5c441 100644
> --- a/arch/mips/kernel/i8253.c
> +++ b/arch/mips/kernel/i8253.c
> @@ -98,7 +98,7 @@ static irqreturn_t timer_interrupt(int irq, void *dev_id)
>
> static struct irqaction irq0 = {
> .handler = timer_interrupt,
> - .flags = IRQF_DISABLED | IRQF_NOBALANCING,
> + .flags = IRQF_DISABLED | IRQF_NOBALANCING | IRQF_TIMER,
> .name = "timer"
> };
>
> diff --git a/arch/mips/nxp/pnx8550/common/time.c b/arch/mips/nxp/pnx8550/common/time.c
> index 18b1927..d987a89 100644
> --- a/arch/mips/nxp/pnx8550/common/time.c
> +++ b/arch/mips/nxp/pnx8550/common/time.c
> @@ -59,7 +59,7 @@ static irqreturn_t pnx8xxx_timer_interrupt(int irq, void *dev_id)
>
> static struct irqaction pnx8xxx_timer_irq = {
> .handler = pnx8xxx_timer_interrupt,
> - .flags = IRQF_DISABLED | IRQF_PERCPU,
> + .flags = IRQF_DISABLED | IRQF_PERCPU | IRQF_TIMER,
> .name = "pnx8xxx_timer",
> };
>
> diff --git a/arch/mips/sgi-ip27/ip27-timer.c b/arch/mips/sgi-ip27/ip27-timer.c
> index 6d0e59f..d6802d6 100644
> --- a/arch/mips/sgi-ip27/ip27-timer.c
> +++ b/arch/mips/sgi-ip27/ip27-timer.c
> @@ -105,7 +105,7 @@ static irqreturn_t hub_rt_counter_handler(int irq, void *dev_id)
>
> struct irqaction hub_rt_irqaction = {
> .handler = hub_rt_counter_handler,
> - .flags = IRQF_DISABLED | IRQF_PERCPU,
> + .flags = IRQF_DISABLED | IRQF_PERCPU | IRQF_TIMER,
> .name = "hub-rt",
> };
>
> diff --git a/arch/mips/sni/time.c b/arch/mips/sni/time.c
> index 62df6a5..f3b60e6 100644
> --- a/arch/mips/sni/time.c
> +++ b/arch/mips/sni/time.c
> @@ -67,7 +67,7 @@ static irqreturn_t a20r_interrupt(int irq, void *dev_id)
>
> static struct irqaction a20r_irqaction = {
> .handler = a20r_interrupt,
> - .flags = IRQF_DISABLED | IRQF_PERCPU,
> + .flags = IRQF_DISABLED | IRQF_PERCPU | IRQF_TIMER,
> .name = "a20r-timer",
> };
>
>
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] MIPS: add IRQF_TIMER flag for Timer Interrupts
2009-10-08 13:17 [PATCH] MIPS: add IRQF_TIMER flag for Timer Interrupts Wu Zhangjin
2009-10-08 15:13 ` Thomas Gleixner
2009-10-08 20:26 ` Rafael J. Wysocki
@ 2009-10-09 15:17 ` Atsushi Nemoto
2009-10-09 15:45 ` Wu Zhangjin
2 siblings, 1 reply; 5+ messages in thread
From: Atsushi Nemoto @ 2009-10-09 15:17 UTC (permalink / raw)
To: wuzhangjin; +Cc: linux-mips, linux-pm, linux-kernel, tglx, ralf, rjw, yuasa
On Thu, 8 Oct 2009 21:17:54 +0800, Wu Zhangjin <wuzhangjin@gmail.com> wrote:
> This patch add IRQF_TIMER flag for all Timer interrupts in linux-MIPS,
> which will help to not disable the Timer IRQ when suspending to ensure
> resuming normally(d6c585a4342a2ff627a29f9aea77c5ed4cd76023) and not
> thread them when enabled PREEMPT_RT.
>
> Signed-off-by: Wu Zhangjin <wuzhangjin@gmail.com>
> ---
> arch/mips/jazz/irq.c | 2 +-
> arch/mips/kernel/cevt-gt641xx.c | 2 +-
> arch/mips/kernel/cevt-r4k.c | 2 +-
> arch/mips/kernel/i8253.c | 2 +-
> arch/mips/nxp/pnx8550/common/time.c | 2 +-
> arch/mips/sgi-ip27/ip27-timer.c | 2 +-
> arch/mips/sni/time.c | 2 +-
> 7 files changed, 7 insertions(+), 7 deletions(-)
Maybe cevt-bcm1480.c, cevt-ds1287.c, cevt-sb1250.c, cevt-txx9.c and
i8253.c need same fix?
---
Atsushi Nemoto
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] MIPS: add IRQF_TIMER flag for Timer Interrupts
2009-10-09 15:17 ` Atsushi Nemoto
@ 2009-10-09 15:45 ` Wu Zhangjin
0 siblings, 0 replies; 5+ messages in thread
From: Wu Zhangjin @ 2009-10-09 15:45 UTC (permalink / raw)
To: Atsushi Nemoto; +Cc: linux-mips, linux-pm, linux-kernel, tglx, ralf, rjw, yuasa
On Sat, 2009-10-10 at 00:17 +0900, Atsushi Nemoto wrote:
> On Thu, 8 Oct 2009 21:17:54 +0800, Wu Zhangjin <wuzhangjin@gmail.com> wrote:
> > This patch add IRQF_TIMER flag for all Timer interrupts in linux-MIPS,
> > which will help to not disable the Timer IRQ when suspending to ensure
> > resuming normally(d6c585a4342a2ff627a29f9aea77c5ed4cd76023) and not
> > thread them when enabled PREEMPT_RT.
> >
> > Signed-off-by: Wu Zhangjin <wuzhangjin@gmail.com>
> > ---
> > arch/mips/jazz/irq.c | 2 +-
> > arch/mips/kernel/cevt-gt641xx.c | 2 +-
> > arch/mips/kernel/cevt-r4k.c | 2 +-
> > arch/mips/kernel/i8253.c | 2 +-
> > arch/mips/nxp/pnx8550/common/time.c | 2 +-
> > arch/mips/sgi-ip27/ip27-timer.c | 2 +-
> > arch/mips/sni/time.c | 2 +-
> > 7 files changed, 7 insertions(+), 7 deletions(-)
>
> Maybe cevt-bcm1480.c, cevt-ds1287.c, cevt-sb1250.c, cevt-txx9.c and
> i8253.c need same fix?
>
I got them via "grep timer -ur arch/mips | grep name", so, perhaps there
are also some other timer interrupts missing here.
just get more via "grep Timer -ur arch/mips | grep name", "grep IRQF_
-ur arch/mips/ | grep -v TIMER | grep =", a new patch will be sent out
in the next E-mail.
Regards,
Wu Zhangjin
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2009-10-09 15:46 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-10-08 13:17 [PATCH] MIPS: add IRQF_TIMER flag for Timer Interrupts Wu Zhangjin
2009-10-08 15:13 ` Thomas Gleixner
2009-10-08 20:26 ` Rafael J. Wysocki
2009-10-09 15:17 ` Atsushi Nemoto
2009-10-09 15:45 ` Wu Zhangjin
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).