mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: David Howells <dhowells@redhat.com>
To: Linus Torvalds <torvalds@osdl.org>
Cc: Ingo Molnar <mingo@elte.hu>, Andrew Morton <akpm@osdl.org>,
	David Howells <dhowells@redhat.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
	Dmitry Torokhov <dtor@mail.ru>, Greg KH <greg@kroah.com>,
	David Brownell <david-b@pacbell.net>,
	Alan Stern <stern@rowland.harvard.edu>
Subject: Re: [PATCH 3/3] IRQ: Maintain regs pointer globally rather than passing to IRQ handlers
Date: Tue, 03 Oct 2006 11:21:21 +0100	[thread overview]
Message-ID: <10735.1159870881@warthog.cambridge.redhat.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0610021349090.3952@g5.osdl.org>

Linus Torvalds <torvalds@osdl.org> wrote:

> So far, when this has come up, the gains it gives have not been worth the 
> pain. I don't quite see why FRV is so broken that it would matter 20% 
> worth, and I suspect that number was somehow really not real, but more a 
> matter of "this small code snippet that is part of the irq delivery and 
> isn't really measurable improves by 20%", which is a different thing.

What appears to make up the difference is the loop in handle_IRQ_event().
That has to resurrect the arguments for the IRQ handler after calling the
previous IRQ handler.

FRV is just the easiest place for me to measure things like this.  Trying to
do so on i386 would be tricky, and Xen wouldn't help as it could affect the
measurement of time - though it might permit me to count the intructions
instead.  I might be able to do so on my power5 box, I suppose, but again,
like Xen, that's virtualised, and I'm not sure what affect that'd have.

So, I'm sure this will affect other archs, but it's much harder for me to
measure those.

But, you're also right: this is a statistic, and I'm sure you know the old
saying about those...

David

  parent reply	other threads:[~2006-10-03 10:23 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-02 16:20 [PATCH 1/3] IRQ: Typedef the IRQ flow handler function type David Howells
2006-10-02 16:20 ` [PATCH 2/3] IRQ: Typedef the IRQ " David Howells
     [not found] ` <20061002162053.17763.26032.stgit@warthog.cambridge.redhat.com>
2006-10-02 20:21   ` [PATCH 3/3] IRQ: Maintain regs pointer globally rather than passing to IRQ handlers Andrew Morton
2006-10-02 20:18     ` Ingo Molnar
2006-10-02 20:54       ` Linus Torvalds
2006-10-02 21:01         ` Andrew Morton
2006-10-02 21:12           ` Linus Torvalds
2006-10-02 21:19             ` Andi Kleen
2006-10-02 21:46               ` Linus Torvalds
2006-10-02 21:47                 ` Ingo Molnar
2006-10-02 21:59                 ` Andi Kleen
2006-10-02 22:33                   ` Linus Torvalds
2006-10-03 10:43           ` Ingo Molnar
     [not found]           ` <fa.v9OUIBlFjbmpdm2jHjUOj/6fm5Y@ifi.uio.no>
2006-10-07 18:44             ` Bill Waddington
2006-10-02 21:12         ` David Miller
2006-10-02 21:18           ` Thomas Gleixner
2006-10-02 22:59         ` Karsten Wiese
2006-10-03  0:36         ` Dave Airlie
2006-10-05  8:01         ` David Woodhouse
2006-10-03 10:21       ` David Howells [this message]
2006-10-02 20:43     ` Dmitry Torokhov
2006-10-02 20:56       ` Andrew Morton
2006-10-02 23:52       ` Greg KH
2006-10-02 20:46     ` David Brownell
2006-10-02 20:58       ` Andrew Morton
2006-10-02 21:34       ` Alan Stern
2006-10-02 23:00         ` David Brownell
2006-10-03 18:03           ` Alan Stern
2006-10-06  3:45             ` David Brownell
2006-10-06 17:42               ` Alan Stern
2006-10-03 10:01     ` David Howells
2006-10-03 10:30     ` David Howells
2006-10-05 14:22   ` David Howells
2006-10-05 19:46     ` Andrew Morton
2006-10-05 20:19       ` Thomas Gleixner
2006-10-05 20:20       ` Dmitry Torokhov
2006-10-05 20:34       ` Greg KH
2006-10-05 23:35     ` Linus Torvalds
2006-10-06  1:31       ` [PATCH] powerpc: irq change build breaks Olof Johansson
2006-10-06  4:22         ` Benjamin Herrenschmidt
2006-10-06 16:42       ` [PATCH 3/3] IRQ: Maintain regs pointer globally rather than passing to IRQ handlers Russell King
2006-10-06 18:01         ` Linus Torvalds
2006-10-07  2:54           ` Matthew Wilcox
2006-10-07 14:44             ` Matthew Wilcox
2006-10-06 18:53       ` [PATCH] fix mesh compile errors after irq changes Olaf Hering
2006-10-06 19:09         ` Geert Uytterhoeven
2006-10-06 20:34       ` [PATCH] powerpc: fixup " Olaf Hering
2006-10-06 20:52         ` [PATCH] powerpc: spu " Olaf Hering
2006-10-06 21:06           ` [PATCH] ppc: PReP " Olaf Hering
2006-10-07  0:19         ` [PATCH] powerpc: " Paul Mackerras
2006-10-07 12:25           ` Paul Mackerras
2006-10-06  0:52     ` [PATCH 3/3] IRQ: Maintain regs pointer globally rather than passing to IRQ handlers Jeff Garzik
2006-10-06 11:25       ` Alan Cox
2006-10-06 11:15         ` Jeff Garzik
2006-10-06 11:11           ` Ingo Molnar
2006-10-06 11:27             ` Jeff Garzik
2006-10-06 11:25               ` Ingo Molnar
2006-10-06 14:07                 ` Dmitry Torokhov
2006-10-06 14:16                   ` Jeff Garzik
2006-10-06 15:18               ` [PATCH, RAW] IRQ: Maintain irq number " Jeff Garzik
2006-10-06 15:20                 ` Jeff Garzik
2006-10-06 15:47                 ` Linus Torvalds
2006-10-06 16:21                   ` Dmitry Torokhov
2006-10-06 16:40                     ` Linus Torvalds
2006-10-06 16:38                   ` Jeff Garzik
2006-10-06  8:03     ` [PATCH 3/3] IRQ: Maintain regs pointer " Gregor Jasny
     [not found] <fa.FU9k10MvHKEiGBkmyRa0N7lIvX4@ifi.uio.no>
     [not found] ` <fa.YmeJPP3GwSahgI09Gcaha4kqm84@ifi.uio.no>
     [not found]   ` <fa.qbSmIOXP3NtOgNMHs5oazelaSJs@ifi.uio.no>
     [not found]     ` <fa.AB8rZ1kwd3vQ1HCbYfV1438E4A0@ifi.uio.no>

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=10735.1159870881@warthog.cambridge.redhat.com \
    --to=dhowells@redhat.com \
    --cc=akpm@osdl.org \
    --cc=david-b@pacbell.net \
    --cc=dtor@mail.ru \
    --cc=greg@kroah.com \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=stern@rowland.harvard.edu \
    --cc=tglx@linutronix.de \
    --cc=torvalds@osdl.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

Powered by JetHome