genirq: Work around __irq_set_handler vs stacked domains ordering issues
authorMarc Zyngier <marc.zyngier@arm.com>
Sat, 15 Nov 2014 10:49:13 +0000 (10:49 +0000)
committerThomas Gleixner <tglx@linutronix.de>
Sun, 23 Nov 2014 12:01:47 +0000 (13:01 +0100)
commitf86eff222fabe30da5c536ef2b51bd98d14cfe3b
tree747aa489c789c5618f016e81cec68251b84d3360
parentafb7da83b9f476728623130703acb553d7c7c4d9
genirq: Work around __irq_set_handler vs stacked domains ordering issues

With the introduction of stacked domains, we have the issue that,
depending on where in the stack this is called, __irq_set_handler
will succeed or fail: If this is called from the inner irqchip,
__irq_set_handler() will fail, as it will look at the outer domain
as the (desc->irq_data.chip == &no_irq_chip) test fails (we haven't
set the top level yet).

This patch implements the following: "If there is at least one
valid irqchip in the domain, it will probably sort itself out".
This is clearly not ideal, but it is far less confusing then
crashing because the top-level domain is not up yet.

[ tglx: Added comment and a protection against chained interrupts in
   that context ]

Signed-off-by: Marc Zyngier <marc.zyngier@arm.com>
Cc: Yingjoe Chen <yingjoe.chen@mediatek.com>
Cc: Bjorn Helgaas <bhelgaas@google.com>
Cc: linux-arm-kernel@lists.infradead.org
Cc: Jiang Liu <jiang.liu@linux.intel.com>
Link: http://lkml.kernel.org/r/1416048553-29289-3-git-send-email-marc.zyngier@arm.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
kernel/irq/chip.c