From: "Siddha, Suresh B" <suresh.b.siddha@intel.com>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>,
Pavel Machek <pavel@ucw.cz>,
Andrew Morton <akpm@linux-foundation.org>,
Andi Kleen <ak@suse.de>,
linux-kernel@vger.kernel.org, Neil Brown <neilb@suse.de>,
Ingo Molnar <mingo@elte.hu>,
"Siddha, Suresh B" <suresh.b.siddha@intel.com>,
asit.k.mallick@intel.com
Subject: Re: [PATCH] x86: Document the hotplug code is incompatible with x86 irq handling
Date: Tue, 12 Jun 2007 14:56:59 -0700 [thread overview]
Message-ID: <20070612215659.GB17143@linux-os.sc.intel.com> (raw)
In-Reply-To: <200706122252.10886.rjw@sisk.pl>
On Tue, Jun 12, 2007 at 10:52:09PM +0200, Rafael J. Wysocki wrote:
> On Tuesday, 12 June 2007 20:19, Eric W. Biederman wrote:
> > Because you are calling unfixably broken code. That should be a decent
> > incentive to do something else won't it?
>
> Can you please tell me _what_ else can be done?
>
> > IOAPICs do not support what the code is doing here. There is lots of
> > practical evidence including bad experiences and practical tests that
> > support this.
>
> Well, AFAICS, Suresh has tried to debug one failing case recently without
> any consistent conclusions. I don't know of any other test cases (links,
> please?).
Rafael, Darrick Wong's issue looks different and hence I was motivated to
look and fix if it was a SW issue. For now, I am not able to comprehend
what is happening on Darrick Wong's system. Need more help from Darrick
as he has the golden failing system.
Meanwhile I talked to our hardware folks about the irq migration in general.
One good news is that future versions of chipsets will have an Interrupt
Remapping feature(for more details please refer to
http://download.intel.com/technology/computing/vptech/Intel(r)_VT_for_Direct_IO.pdf)
where in we can reliably migrate the irq to someother cpu in the process
context.
And for the existing platforms, chipset folks don't see a reason why the
Eric's algorithm(specified below) should fail.
Eric's algorithm for level triggered(edge triggered should be easier than
level triggered):
- Mask the irq in process context.
- Poll RIRR until an ack of for the irq was not pending.
- Migrate the irq.
Eric had a problem of stuck remote IRR on E75xx chipset with this algorithm
and my next step is to reproduce this issue on this platform and understand
the behavior.
thanks,
suresh
next prev parent reply other threads:[~2007-06-12 22:01 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-31 13:32 [PATCH] x86: Document the hotplug code is incompatible with x86 irq handling Eric W. Biederman
2007-06-07 14:01 ` Pavel Machek
2007-06-12 18:19 ` Eric W. Biederman
2007-06-12 20:52 ` Rafael J. Wysocki
2007-06-12 21:56 ` Siddha, Suresh B [this message]
2007-06-12 22:16 ` Rafael J. Wysocki
2007-06-12 22:24 ` Siddha, Suresh B
2007-06-12 22:58 ` Rafael J. Wysocki
2007-06-22 17:27 ` Eric W. Biederman
[not found] <fa.tUMR7tAB+jMgtfyl/LJ7U9QMgBs@ifi.uio.no>
2007-05-31 14:34 ` Robert Hancock
2007-05-31 15:47 ` Eric W. Biederman
2007-05-31 20:12 ` Rafael J. Wysocki
2007-06-01 19:48 ` Eric W. Biederman
2007-06-01 20:06 ` Rafael J. Wysocki
2007-06-01 20:29 ` Eric W. Biederman
2007-06-01 20:44 ` Rafael J. Wysocki
2007-06-01 20:34 ` Rafael J. Wysocki
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=20070612215659.GB17143@linux-os.sc.intel.com \
--to=suresh.b.siddha@intel.com \
--cc=ak@suse.de \
--cc=akpm@linux-foundation.org \
--cc=asit.k.mallick@intel.com \
--cc=ebiederm@xmission.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=neilb@suse.de \
--cc=pavel@ucw.cz \
--cc=rjw@sisk.pl \
/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