From: Keir Fraser <keir.fraser@eu.citrix.com>
To: Jan Beulich <jbeulich@novell.com>, xen-devel@lists.xensource.com
Subject: Re: [PATCH] x86: change IO-APIC ack method default for single IO-APIC systems
Date: Wed, 21 Jan 2009 14:35:54 +0000 [thread overview]
Message-ID: <C59CE34A.1AB6%keir.fraser@eu.citrix.com> (raw)
In-Reply-To: <49773D8A.76E4.0078.0@novell.com>
On 21/01/2009 14:21, "Jan Beulich" <jbeulich@novell.com> wrote:
> "ioapic_ack=old". With that knowledge I think it is now reasonable to
> include that patch in -unstable, as the introduction of the 'new' ack
> method was only to address issues with certain chipsets silently
> setting up alternative IRQ routes when RTEs in secondary IO-APICs got
> masked.
I don't specifically recall that this issue required two IO-APICs. In fact I
think it did not. It was actually something to do with the chipset trying to
distinguish between an OS using 'legacy' routing versus 'mp-bios' routing,
via a rather distasteful IO-APIC hack. Unfortunately the hack was not that
uncommon and I don't think those chipsets had more than one IO-APIC.
Overall I think ack_type new has worked quite well. I was actually about to
remove the old ack_type! (But now I won't ;-) I'm not inclined to take this
patch though.
-- Keir
next prev parent reply other threads:[~2009-01-21 14:35 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-01-21 14:21 [PATCH] x86: change IO-APIC ack method default for single IO-APIC systems Jan Beulich
2009-01-21 14:23 ` Jiang, Yunhong
2009-01-21 14:33 ` [PATCH] x86: change IO-APIC ack method default forsingle " Jan Beulich
2009-01-21 14:35 ` Jiang, Yunhong
2009-01-21 14:37 ` [PATCH] x86: change IO-APIC ack method default for single " Keir Fraser
2009-01-21 14:44 ` [PATCH] x86: change IO-APIC ack method default forsingle " Jan Beulich
2009-01-21 14:35 ` Keir Fraser [this message]
2009-01-21 14:41 ` [PATCH] x86: change IO-APIC ack method default for single " Jiang, Yunhong
2009-01-21 14:51 ` [PATCH] x86: change IO-APIC ack method default forsingle " Jan Beulich
2009-01-21 16:32 ` Keir Fraser
2009-01-21 16:44 ` [PATCH] x86: change IO-APIC ack method defaultforsingle " Jan Beulich
2009-01-22 3:09 ` [PATCH] x86: change IO-APIC ack method default forsingle " Tian, Kevin
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=C59CE34A.1AB6%keir.fraser@eu.citrix.com \
--to=keir.fraser@eu.citrix.com \
--cc=jbeulich@novell.com \
--cc=xen-devel@lists.xensource.com \
/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.