]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
genirq: Use irqd_get_trigger_type to compare the trigger type for shared IRQs
authorHans de Goede <hdegoede@redhat.com>
Sat, 15 Apr 2017 10:08:31 +0000 (12:08 +0200)
committerThomas Gleixner <tglx@linutronix.de>
Sat, 15 Apr 2017 13:42:43 +0000 (15:42 +0200)
commit382bd4de61827dbaaf5fb4fb7b1f4be4a86505e7
treec009717481f1691701c7b970e8d3cbfd9f67a139
parentd1b25279c1a3fe99a14b3afc6d590e218f854b10
genirq: Use irqd_get_trigger_type to compare the trigger type for shared IRQs

When requesting a shared irq with IRQF_TRIGGER_NONE then the irqaction
flags get filled with the trigger type from the irq_data:

        if (!(new->flags & IRQF_TRIGGER_MASK))
                new->flags |= irqd_get_trigger_type(&desc->irq_data);

On the first setup_irq() the trigger type in irq_data is NONE when the
above code executes, then the irq is started up for the first time and
then the actual trigger type gets established, but that's too late to fix
up new->flags.

When then a second user of the irq requests the irq with IRQF_TRIGGER_NONE
its irqaction's triggertype gets set to the actual trigger type and the
following check fails:

        if (!((old->flags ^ new->flags) & IRQF_TRIGGER_MASK))

Resulting in the request_irq failing with -EBUSY even though both
users requested the irq with IRQF_SHARED | IRQF_TRIGGER_NONE

Fix this by comparing the new irqaction's trigger type to the trigger type
stored in the irq_data which correctly reflects the actual trigger type
being used for the irq.

Suggested-by: Thomas Gleixner <tglx@linutronix.de>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Acked-by: Marc Zyngier <marc.zyngier@arm.com>
Link: http://lkml.kernel.org/r/20170415100831.17073-1-hdegoede@redhat.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
kernel/irq/manage.c