aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBenjamin Herrenschmidt <benh@kernel.crashing.org>2017-12-12 16:22:52 +1100
committerStewart Smith <stewart@linux.vnet.ibm.com>2017-12-12 18:39:43 -0600
commit9cd69214a129c36016fd3294f5219fa47a50404c (patch)
tree30bc693e9289edd9a51f4a699144155c4ed4ad29
parent230c6a9faba869d35e4c991306aab575872e5a0e (diff)
downloadskiboot-9cd69214a129c36016fd3294f5219fa47a50404c.zip
skiboot-9cd69214a129c36016fd3294f5219fa47a50404c.tar.gz
skiboot-9cd69214a129c36016fd3294f5219fa47a50404c.tar.bz2
xive: Do not return a trigger page for an escalation interrupt
This is bogus, we don't support them. (Thankfully the callers didn't actually try to use this on escalation interrupts). Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> Tested-by: Oliver O'Halloran <oohall@gmail.com> Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com> (cherry picked from commit 3e7f04a7398a5dcb38ff11d0b11f630630a3bb74) Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
-rw-r--r--hw/xive.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/hw/xive.c b/hw/xive.c
index cd748fc..f708cf3 100644
--- a/hw/xive.c
+++ b/hw/xive.c
@@ -2092,8 +2092,8 @@ void *xive_get_trigger_port(uint32_t girq)
return NULL;
if (GIRQ_IS_ESCALATION(girq)) {
- /* Page 2 of the EQ MMIO space is the escalate irq */
- return x->eq_mmio + idx * 0x20000 + 0x10000;
+ /* There is no trigger page for escalation interrupts */
+ return NULL;
} else {
/* Make sure it's an IPI on that chip */
if (girq < x->int_base ||