Re: [PATCH] irq: fix crash due to op-less irq domains

[Linus Walleij]

On Wed, Dec 14, 2011 at 11:14 PM, Rob Herring <robherring2@xxxxxxxxx> wrote:
> Linus,
>
> On 12/14/2011 02:55 PM, Linus Walleij wrote:
>> From: Linus Walleij <linus.walleij@xxxxxxxxxx>
>>
>> IRQ domains without ops does not work anymore after commit
>> "irq: support domains with non-zero hwirq base", since the
>> check dereferences domain->ops->to_irq without checking of
>> domain->ops are NULL. This makes U300 (and probably most other
>> systems using the PL190 VIC) boot again.
>
> I thought the VIC series had the fix for this, but it seems not. NULL
> ops is considered invalid based on the code comments. The correct fix is
> this patch:
>
> irqdomain: export irq_domain_simple_ops for !CONFIG_OF
> http://www.spinics.net/lists/arm-kernel/msg150290.html
>
> I believe this is going in with some iMX changes.

Can we please move this patch over to whatever branch the patch that
breaks my system is on, so I can create a proper fix for the
non-devicetree VIC users then?

Or apply an exact copy of the patch. Git will cope.

(Among the systems broken are likely the ARM Versatile too BTW,
not just obscure things like my U300.)

> But I guess the VIC is
> going to need a fix to use irq_domain_simple_ops.

If I just have the baseline in linux-next, I can fix it, but then I need
that patch in linux-next first, now all non-devicetree VIC systems are
just broken with no way for me to fix them :-(

Thanks,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/