From: "H. Peter Anvin" <hpa@zytor.com>
To: Suresh Siddha <suresh.b.siddha@intel.com>
Cc: "Maciej W. Rozycki" <macro@linux-mips.org>,
"ebiederm@xmission.com" <ebiederm@xmission.com>,
"yinghai@kernel.org" <yinghai@kernel.org>,
"mingo@elte.hu" <mingo@elte.hu>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [patch 2/2] x86, irq: use 0x20 for the IRQ_MOVE_CLEANUP_VECTOR instead of 0x1f
Date: Mon, 01 Feb 2010 13:49:54 -0800 [thread overview]
Message-ID: <4B674C82.9050703@zytor.com> (raw)
In-Reply-To: <1265059497.2802.178.camel@sbs-t61.sc.intel.com>
On 02/01/2010 01:24 PM, Suresh Siddha wrote:
>
> As we are using the code from 2.6.28 and no one noticed/complained about
> this issue for more than 1.5 years, probably the pentium APIC issue is
> not wide-spread.
>
I *think* it's applicable to all CPUs Pentium III or earlier (but not
Pentium 4 -- I'm unsure about the Pentium M.) I don't know about
non-Intel CPUs; I have a vague memory of the Transmeta Efficeon (the
only Transmeta chip with an APIC) *not* having this limitation.
The exact reference is SDM vol 3A 10.8.4, page 10-41 [rev 033US Dec 2009]:
For the P6 family and Pentium processors, the IRR and ISR registers can
queue no more than two interrupts per priority level, and will reject
other interrupts that are received within the same priority level.
However, section 10.8.2 bullet 3 on page 10-38 (and the flowchart on
page 10-37) indicate that such an interrupt is returned to the IOAPIC
for a later retry, i.e. it's not lost. As such, it's not clear to me
from reading the SDM that there is actually a problem here...
-hpa
next prev parent reply other threads:[~2010-02-01 21:52 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-01-14 0:19 [patch 1/2] x86, vmi: Fix vmi_get_timer_vector() to use IRQ0_VECTOR Suresh Siddha
2010-01-14 0:19 ` [patch 2/2] x86, irq: use 0x20 for the IRQ_MOVE_CLEANUP_VECTOR instead of 0x1f Suresh Siddha
2010-01-18 19:36 ` [tip:x86/apic] x86, irq: Use " tip-bot for Suresh Siddha
2010-01-24 5:52 ` [patch 2/2] x86, irq: use " Maciej W. Rozycki
2010-01-24 8:12 ` H. Peter Anvin
2010-01-31 7:19 ` Maciej W. Rozycki
2010-02-01 21:24 ` Suresh Siddha
2010-02-01 21:49 ` H. Peter Anvin [this message]
2010-02-21 5:20 ` Maciej W. Rozycki
2010-02-21 5:37 ` H. Peter Anvin
2010-02-21 14:09 ` Alan Cox
2010-01-18 19:36 ` [tip:x86/apic] x86, vmi: Fix vmi_get_timer_vector() to use IRQ0_VECTOR tip-bot for Suresh Siddha
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=4B674C82.9050703@zytor.com \
--to=hpa@zytor.com \
--cc=ebiederm@xmission.com \
--cc=linux-kernel@vger.kernel.org \
--cc=macro@linux-mips.org \
--cc=mingo@elte.hu \
--cc=suresh.b.siddha@intel.com \
--cc=yinghai@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.