public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Thomas Gleixner <tglx@linutronix.de>
Cc: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
	"Rusty Russell" <rusty@rustcorp.com.au>,
	"David Brownell" <dbrownell@users.sourceforge.net>,
	"Eric Miao" <eric.y.miao@gmail.com>,
	"Peter Zijlstra" <a.p.zijlstra@chello.nl>,
	"John Stultz" <johnstul@us.ibm.com>,
	"Nicolas Pitre" <nico@marvell.com>,
	"Jamie Lokier" <jamie@shareable.org>,
	LKML <linux-kernel@vger.kernel.org>,
	"Remy Bohmer" <linux@bohmer.net>,
	"Hugh Dickins" <hugh.dickins@tiscali.co.uk>,
	linux-arm-kernel@lists.infradead.org,
	"Andrea Gallo" <andrea.gallo@stericsson.com>,
	"Andrew Morton" <akpm@linux-foundation.org>,
	"Linus Torvalds" <torvalds@linux-foundation.org>,
	"Ingo Molnar" <mingo@elte.hu>,
	"Alan Cox" <alan@lxorguk.ukuu.org.uk>
Subject: Re: Get rid of IRQF_DISABLED - (was [PATCH] genirq: warn about IRQF_SHARED|IRQF_DISABLED)
Date: Mon, 30 Nov 2009 14:39:56 +0000	[thread overview]
Message-ID: <20091130143956.GA8227@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <20091130143703.GA7028@n2100.arm.linux.org.uk>

On Mon, Nov 30, 2009 at 02:37:03PM +0000, Russell King - ARM Linux wrote:
> Now, at the risk of covering old ground, how about we have two separate
> irqaction lists, one for handlers to be called with irqs disabled and
> one for handlers with irqs enabled.  We run the irqs-disabled list
> first, naturally with irqs disabled.  If, at the end of that run (or
> maybe after each handler), IRQs have ended being enabled, print some
> diagnostics.  (We're going to need something like this to ensure that
> drivers interrupt handlers don't enable IRQs themselves.)  Then enable
> IRQs and run the irqs-enabled chain.

Oh, and the other interesting thing to do may be to have a way of
measuring how much time irq handlers run for, so that handlers taking
an excessive time (more than 0.5ms or so - thinking about the 1000Hz
timer rate found on some arches) can be targetted.

  reply	other threads:[~2009-11-30 14:40 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20091127195857.GB28193@n2100.arm.linux.org.uk>
2009-11-27 21:10 ` [PATCH] warn about shared irqs requesting IRQF_DISABLED registered with setup_irq Uwe Kleine-König
2009-11-27 22:18   ` Thomas Gleixner
2009-11-28 20:03     ` Uwe Kleine-König
2009-11-28 21:50       ` Thomas Gleixner
2009-11-28 22:13         ` David Brownell
2009-11-29  2:31         ` Jamie Lokier
2009-11-29 10:26           ` Uwe Kleine-König
2009-11-29 15:18             ` Jamie Lokier
2009-11-29 15:27               ` Russell King - ARM Linux
2009-11-30 20:39                 ` Jamie Lokier
2009-11-30  9:28               ` Uwe Kleine-König
2009-11-30  9:54               ` Thomas Gleixner
2009-11-28 22:09       ` David Brownell
2009-11-30 10:47   ` [PATCH] genirq: warn about IRQF_SHARED|IRQF_DISABLED at the right place Uwe Kleine-König
2009-11-30 13:54     ` Get rid of IRQF_DISABLED - (was [PATCH] genirq: warn about IRQF_SHARED|IRQF_DISABLED) Thomas Gleixner
2009-11-30 14:03       ` Peter Zijlstra
2009-11-30 14:24         ` Thomas Gleixner
2009-11-30 14:47           ` Alan Cox
2009-11-30 15:01             ` Russell King - ARM Linux
2009-11-30 15:32               ` Alan Cox
2009-11-30 15:43                 ` Russell King - ARM Linux
2009-11-30 20:15               ` Andrew Victor
2009-11-30 20:53               ` David Brownell
2009-11-30 20:38             ` David Brownell
2009-12-01  1:42               ` Andy Walls
2009-11-30 19:59           ` Benjamin Herrenschmidt
2009-11-30 21:31             ` Thomas Gleixner
2009-11-30 21:42               ` Benjamin Herrenschmidt
2009-11-30 21:54                 ` Thomas Gleixner
2009-11-30 14:37       ` Russell King - ARM Linux
2009-11-30 14:39         ` Russell King - ARM Linux [this message]
2009-11-30 17:48           ` Thomas Gleixner
2009-11-30 14:51         ` Alan Cox
2009-11-30 21:59           ` Thomas Gleixner
2009-11-30 23:30             ` Alan Cox
2009-11-30 15:38         ` Nicolas Pitre
2009-11-30 17:46         ` Thomas Gleixner
2009-11-30 19:51       ` Uwe Kleine-König
2009-11-30 21:23         ` Thomas Gleixner
2009-11-30 20:21     ` [PATCH] genirq: warn about IRQF_SHARED|IRQF_DISABLED at the right place David Brownell
2009-11-30 20:27       ` Uwe Kleine-König
2010-01-12 15:42     ` [RESEND PATCH] " Uwe Kleine-König

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20091130143956.GA8227@n2100.arm.linux.org.uk \
    --to=linux@arm.linux.org.uk \
    --cc=a.p.zijlstra@chello.nl \
    --cc=akpm@linux-foundation.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=andrea.gallo@stericsson.com \
    --cc=dbrownell@users.sourceforge.net \
    --cc=eric.y.miao@gmail.com \
    --cc=hugh.dickins@tiscali.co.uk \
    --cc=jamie@shareable.org \
    --cc=johnstul@us.ibm.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@bohmer.net \
    --cc=mingo@elte.hu \
    --cc=nico@marvell.com \
    --cc=rusty@rustcorp.com.au \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    --cc=u.kleine-koenig@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox