public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: tglx@linutronix.de (Thomas Gleixner)
To: linux-arm-kernel@lists.infradead.org
Subject: [patch 2/2] arm: Allow forced irq threading
Date: Thu, 14 Feb 2013 23:04:48 +0100 (CET)	[thread overview]
Message-ID: <alpine.LFD.2.02.1302142256520.22263@ionos> (raw)
In-Reply-To: <20130214214931.GK17833@n2100.arm.linux.org.uk>

On Thu, 14 Feb 2013, Russell King - ARM Linux wrote:

> On Thu, Feb 14, 2013 at 08:53:53PM -0000, Thomas Gleixner wrote:
> > All timer interrupts and the perf interrupt are marked NO_THREAD, so
> > its safe to allow forced interrupt threading.
> 
> What effect does this have on IRQ latency, particularly on the slower CPUs?
> 
> If it adds to the IRQ latency, then it must be conditional; there are
> platforms where IRQ latency is important for things like network devices
> to keep up (particularly the SMC NICs).

It does not affect anything unless you add "threadirqs" to the kernel
command line. This command line argument puts the kernel into forced
irq threading mode. Otherwise it just works as now.

And yes, if enabled on the command line it adds latency due to the
delayed invocation of the handler in the thread.

Though for mainline this is primary a debug mechanism where you can
morph the kernel explosion in the interrupt handler into a takedown of
the handler thread instead of crashing the machine completely.
 
Now on RT we force this and I can tell you that even on slow machines
with SMC NICs the overhead seems to be in a range where these systems
can cope with. Of course, if you tweak the scheduling params of the
irq thread low enough you can run into problematic situations. But
that's not an ARM specific problem, that's a general problem of
configuring a RT system.

Thanks,

	tglx

  reply	other threads:[~2013-02-14 22:04 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-14 20:53 [patch 0/2] ARM: Allow forced irq threading Thomas Gleixner
2013-02-14 20:53 ` [patch 1/2] arm: Mark pmu interupt IRQF_NO_THREAD Thomas Gleixner
2013-02-15  9:51   ` Will Deacon
2013-08-12 19:52     ` Sebastian Andrzej Siewior
2013-08-13 11:11       ` Will Deacon
2013-08-14  9:52         ` Sebastian Andrzej Siewior
2013-02-14 20:53 ` [patch 2/2] arm: Allow forced irq threading Thomas Gleixner
2013-02-14 21:49   ` Russell King - ARM Linux
2013-02-14 22:04     ` Thomas Gleixner [this message]
2013-08-12 20:03       ` [PATCH 2/2 v2] " Sebastian Andrzej Siewior
2013-08-14 10:08         ` Sekhar Nori
2013-08-14 14:43           ` [PATCH 2/2 v3] " Sebastian Andrzej Siewior

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=alpine.LFD.2.02.1302142256520.22263@ionos \
    --to=tglx@linutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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