Commit-ID:  7703b08cc93b3586f9eb733f3a2b10bed634a5cf
Gitweb:     http://git.kernel.org/tip/7703b08cc93b3586f9eb733f3a2b10bed634a5cf
Author:     David Daney <david.da...@cavium.com>
AuthorDate: Thu, 17 Aug 2017 17:53:31 -0700
Committer:  Thomas Gleixner <t...@linutronix.de>
CommitDate: Fri, 18 Aug 2017 11:21:41 +0200

genirq: Add handle_fasteoi_{level,edge}_irq flow handlers

Follow-on patch for gpio-thunderx uses a irqdomain hierarchy which
requires slightly different flow handlers, add them to chip.c which
contains most of the other flow handlers.  Make these conditionally
compiled based on CONFIG_IRQ_FASTEOI_HIERARCHY_HANDLERS.

Signed-off-by: David Daney <david.da...@cavium.com>
Signed-off-by: Thomas Gleixner <t...@linutronix.de>
Cc: Mark Rutland <mark.rutl...@arm.com>
Cc: Alexandre Courbot <gnu...@gmail.com>
Cc: Marc Zyngier <marc.zyng...@arm.com>
Cc: Linus Walleij <linus.wall...@linaro.org>
Cc: linux-g...@vger.kernel.org
Link: 
http://lkml.kernel.org/r/1503017616-3252-3-git-send-email-david.da...@cavium.com

---
 include/linux/irq.h |   2 +
 kernel/irq/Kconfig  |   4 ++
 kernel/irq/chip.c   | 106 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 112 insertions(+)

diff --git a/include/linux/irq.h b/include/linux/irq.h
index d2d54379..d4728bf 100644
--- a/include/linux/irq.h
+++ b/include/linux/irq.h
@@ -568,6 +568,8 @@ extern int irq_chip_compose_msi_msg(struct irq_data *data, 
struct msi_msg *msg);
 extern int irq_chip_pm_get(struct irq_data *data);
 extern int irq_chip_pm_put(struct irq_data *data);
 #ifdef CONFIG_IRQ_DOMAIN_HIERARCHY
+extern void handle_fasteoi_ack_irq(struct irq_desc *desc);
+extern void handle_fasteoi_mask_irq(struct irq_desc *desc);
 extern void irq_chip_enable_parent(struct irq_data *data);
 extern void irq_chip_disable_parent(struct irq_data *data);
 extern void irq_chip_ack_parent(struct irq_data *data);
diff --git a/kernel/irq/Kconfig b/kernel/irq/Kconfig
index 1d06af7..a117adf 100644
--- a/kernel/irq/Kconfig
+++ b/kernel/irq/Kconfig
@@ -73,6 +73,10 @@ config IRQ_DOMAIN_HIERARCHY
        bool
        select IRQ_DOMAIN
 
+# Support for hierarchical fasteoi+edge and fasteoi+level handlers
+config IRQ_FASTEOI_HIERARCHY_HANDLERS
+       bool
+
 # Generic IRQ IPI support
 config GENERIC_IRQ_IPI
        bool
diff --git a/kernel/irq/chip.c b/kernel/irq/chip.c
index 6514f07..2395898 100644
--- a/kernel/irq/chip.c
+++ b/kernel/irq/chip.c
@@ -1092,6 +1092,112 @@ void irq_cpu_offline(void)
 }
 
 #ifdef CONFIG_IRQ_DOMAIN_HIERARCHY
+
+#ifdef CONFIG_IRQ_FASTEOI_HIERARCHY_HANDLERS
+/**
+ *     handle_fasteoi_ack_irq - irq handler for edge hierarchy
+ *     stacked on transparent controllers
+ *
+ *     @desc:  the interrupt description structure for this irq
+ *
+ *     Like handle_fasteoi_irq(), but for use with hierarchy where
+ *     the irq_chip also needs to have its ->irq_ack() function
+ *     called.
+ */
+void handle_fasteoi_ack_irq(struct irq_desc *desc)
+{
+       struct irq_chip *chip = desc->irq_data.chip;
+
+       raw_spin_lock(&desc->lock);
+
+       if (!irq_may_run(desc))
+               goto out;
+
+       desc->istate &= ~(IRQS_REPLAY | IRQS_WAITING);
+
+       /*
+        * If its disabled or no action available
+        * then mask it and get out of here:
+        */
+       if (unlikely(!desc->action || irqd_irq_disabled(&desc->irq_data))) {
+               desc->istate |= IRQS_PENDING;
+               mask_irq(desc);
+               goto out;
+       }
+
+       kstat_incr_irqs_this_cpu(desc);
+       if (desc->istate & IRQS_ONESHOT)
+               mask_irq(desc);
+
+       /* Start handling the irq */
+       desc->irq_data.chip->irq_ack(&desc->irq_data);
+
+       preflow_handler(desc);
+       handle_irq_event(desc);
+
+       cond_unmask_eoi_irq(desc, chip);
+
+       raw_spin_unlock(&desc->lock);
+       return;
+out:
+       if (!(chip->flags & IRQCHIP_EOI_IF_HANDLED))
+               chip->irq_eoi(&desc->irq_data);
+       raw_spin_unlock(&desc->lock);
+}
+EXPORT_SYMBOL_GPL(handle_fasteoi_ack_irq);
+
+/**
+ *     handle_fasteoi_mask_irq - irq handler for level hierarchy
+ *     stacked on transparent controllers
+ *
+ *     @desc:  the interrupt description structure for this irq
+ *
+ *     Like handle_fasteoi_irq(), but for use with hierarchy where
+ *     the irq_chip also needs to have its ->irq_mask_ack() function
+ *     called.
+ */
+void handle_fasteoi_mask_irq(struct irq_desc *desc)
+{
+       struct irq_chip *chip = desc->irq_data.chip;
+
+       raw_spin_lock(&desc->lock);
+       mask_ack_irq(desc);
+
+       if (!irq_may_run(desc))
+               goto out;
+
+       desc->istate &= ~(IRQS_REPLAY | IRQS_WAITING);
+
+       /*
+        * If its disabled or no action available
+        * then mask it and get out of here:
+        */
+       if (unlikely(!desc->action || irqd_irq_disabled(&desc->irq_data))) {
+               desc->istate |= IRQS_PENDING;
+               mask_irq(desc);
+               goto out;
+       }
+
+       kstat_incr_irqs_this_cpu(desc);
+       if (desc->istate & IRQS_ONESHOT)
+               mask_irq(desc);
+
+       preflow_handler(desc);
+       handle_irq_event(desc);
+
+       cond_unmask_eoi_irq(desc, chip);
+
+       raw_spin_unlock(&desc->lock);
+       return;
+out:
+       if (!(chip->flags & IRQCHIP_EOI_IF_HANDLED))
+               chip->irq_eoi(&desc->irq_data);
+       raw_spin_unlock(&desc->lock);
+}
+EXPORT_SYMBOL_GPL(handle_fasteoi_mask_irq);
+
+#endif /* CONFIG_IRQ_FASTEOI_HIERARCHY_HANDLERS */
+
 /**
  * irq_chip_enable_parent - Enable the parent interrupt (defaults to unmask if
  * NULL)

Reply via email to