From: Kenji Kaneshige <kaneshige.kenji@jp.fujitsu.com>
To: linux-ia64@vger.kernel.org
Subject: [RFC][PATCH 0/2] SN platform needs platform_irq_to_vector?
Date: Thu, 02 Aug 2007 04:27:51 +0000 [thread overview]
Message-ID: <1186028871.3875.12.camel@kane-linux> (raw)
Hi Jes-san, Russ-san, Christoph-san and SN platform developers,
I glanced over the SN code, and I found SN platform doesn't have SN
specific irq_to_vector() function, though it has SN specific
local_vector_to_irq() function. I guess it was OK because maybe SN
platform depends on irq = vector and old generic irq_to_vector() was
just returning the same value as irq.
The irq_to_vector() function, however, has been changed by vector
domain support, and currently it can return the different value from
irq, even when VECTOR_DOMAIN_PERCPU is not enabled. This can happen
when an interrupt thorough I/O SAPIC is unregistered and another
interupt is registered. So if my guess (SN platform depends on irq =
vector) is true, something problem might happen on SN platform when
PCI hotplugging or driver reloading.
If it is true that SN platform depends on irq = vector, we need to
implement SN specific irq_to_vector() that returns the same value as
irq, I think. I made sample patches to implement platform specific
irq_to_vector() as follows. Note that [PATCH 2/2] is not tested yet
because I don't have any SN platform unfortunately.
[PATCH 1/2] Add base support for platform_irq_to_vector()
[PATCH 2/2] Implement platform_irq_to_vector() for SN
Could you give me comments?
Thanks,
Kenji Kaneshige
next reply other threads:[~2007-08-02 4:27 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-02 4:27 Kenji Kaneshige [this message]
2007-08-02 7:27 ` [RFC][PATCH 0/2] SN platform needs platform_irq_to_vector? Jes Sorensen
2007-08-02 13:26 ` John Keller
2007-08-02 14:04 ` Kenji Kaneshige
2007-08-02 14:50 ` Russ Anderson
2007-08-02 17:38 ` Keith Rich
2007-08-02 19:32 ` John Keller
2007-08-03 5:22 ` Kenji Kaneshige
2007-08-03 6:04 ` Yasuaki Ishimatsu
2007-08-03 6:20 ` Kenji Kaneshige
2007-08-03 14:58 ` Russ Anderson
2007-08-03 18:03 ` John Keller
2007-08-03 18:19 ` Luck, Tony
2007-08-06 13:59 ` John Keller
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=1186028871.3875.12.camel@kane-linux \
--to=kaneshige.kenji@jp.fujitsu.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 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.