From: Jes Sorensen <jes@sgi.com>
To: linux-ia64@vger.kernel.org
Subject: Re: PMU and timer interrupts
Date: Fri, 10 Feb 2006 23:26:34 +0000 [thread overview]
Message-ID: <yq0d5hu3idx.fsf@jaguar.mkp.net> (raw)
In-Reply-To: <20060209171648.GA26865@frankl.hpl.hp.com>
>>>>> "Ken" = Chen, Kenneth W <kenneth.w.chen@intel.com> writes:
Ken> By hardware, upon an interrupt, both psr.i and psr.ic bit are off
Ken> when the control goes into the ivt:interrupt(). In there,
Ken> pt_regs is saved and only psr.ic bit is turned on. It eventually
Ken> calls down to handle_IRQ_event(). Since SA_INTERRUPT flag is on
Ken> for timer interrupt, kernel doesn't enable irq when invoking the
Ken> handler.
Ken> In fact, most of the interrupts having SA_INTERRUPT flag set. So
Ken> the blind spots you worried about for PMU is a lot wider. For
Ken> example, on my system, serial, e1000, mpt fusion, qla2300
Ken> interrupts all have that flag set. Looks like kernel executes
Ken> these handlers with irq disabled all the way through.
Ken,
I wonder if that isn't mostly bugs on the driver authors side?
SA_INTERRUPT was really introduced to handle UART interrupts, whereas
I think all the other drivers you mention ought to be able to handle
it correctly.
Cheers,
Jes
next prev parent reply other threads:[~2006-02-10 23:26 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-02-09 17:16 PMU and timer interrupts Stephane Eranian
2006-02-09 18:16 ` Jack Steiner
2006-02-09 18:20 ` Luck, Tony
2006-02-09 18:39 ` Stephane Eranian
2006-02-09 19:10 ` Jack Steiner
2006-02-09 19:11 ` Stephane Eranian
2006-02-10 13:35 ` Stephane Eranian
2006-02-10 13:54 ` Jes Sorensen
2006-02-10 20:33 ` Chen, Kenneth W
2006-02-10 21:22 ` Stephane Eranian
2006-02-10 23:04 ` Chen, Kenneth W
2006-02-10 23:21 ` Stephane Eranian
2006-02-10 23:26 ` Jes Sorensen [this message]
2006-02-13 10:38 ` Stephane Eranian
2006-02-13 10:53 ` Jes Sorensen
2006-02-13 20:01 ` Stephane Eranian
2006-02-13 20:12 ` Stephane Eranian
2006-02-13 20:13 ` Chen, Kenneth W
2006-02-13 20:32 ` Chen, Kenneth W
2006-02-13 21:26 ` Keith Owens
2006-02-13 22:49 ` Chen, Kenneth W
2006-02-13 23:46 ` Stephane Eranian
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=yq0d5hu3idx.fsf@jaguar.mkp.net \
--to=jes@sgi.com \
--cc=linux-ia64@vger.kernel.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