Re: [PATCH v5 06/27] irq_domain/powerpc: eliminate irq_map; use irq_alloc_desc() instead
From: Thomas Gleixner <hidden>
Date: 2012-04-06 11:28:42
Also in:
linux-arm-kernel, linux-devicetree, lkml
Possibly related (same subject, not in this thread)
- 2012-04-19 · Re: [PATCH v5 06/27] irq_domain/powerpc: eliminate irq_map; use irq_alloc_desc() instead · Grant Likely <hidden>
- 2012-04-11 · Re: [PATCH v5 06/27] irq_domain/powerpc: eliminate irq_map; use irq_alloc_desc() instead · Thomas Gleixner <hidden>
- 2012-04-11 · Re: [PATCH v5 06/27] irq_domain/powerpc: eliminate irq_map; use irq_alloc_desc() instead · Benjamin Herrenschmidt <benh@kernel.crashing.org>
- 2012-04-11 · Re: [PATCH v5 06/27] irq_domain/powerpc: eliminate irq_map; use irq_alloc_desc() instead · Grant Likely <hidden>
- 2012-04-11 · Re: [PATCH v5 06/27] irq_domain/powerpc: eliminate irq_map; use irq_alloc_desc() instead · Benjamin Herrenschmidt <benh@kernel.crashing.org>
On Fri, 6 Apr 2012, Andreas Schwab wrote:
Thomas Gleixner [off-list ref] writes:quoted
And it looks like the irqdomain code is the real culprit. void irq_set_virq_count(unsigned int count) { pr_debug("irq: Trying to set virq count to %d\n", count); BUG_ON(count < NUM_ISA_INTERRUPTS); if (count < NR_IRQS) irq_virq_count = count; } That looks simply wrong.....There is a single use of irq_set_virq_count, which is only relevant to the PS3.
Though irq_virq_count is statically initialized to NR_IRQS and it's used in the code more than once.