From: Jason Cooper <jason@lakedaemon.net>
To: Paul Burton <paul.burton@imgtec.com>
Cc: linux-mips@linux-mips.org, Ralf Baechle <ralf@linux-mips.org>,
Marc Zyngier <marc.zyngier@arm.com>,
Thomas Gleixner <tglx@linutronix.de>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 17/26] irqchip: mips-cpu: Introduce IPI IRQ domain support
Date: Thu, 1 Sep 2016 19:46:26 +0000 [thread overview]
Message-ID: <20160901194626.GH10637@io.lakedaemon.net> (raw)
In-Reply-To: <20160830172929.16948-18-paul.burton@imgtec.com>
Hi Paul,
On Tue, Aug 30, 2016 at 06:29:20PM +0100, Paul Burton wrote:
> Introduce support for registering an IPI IRQ domain suitable for use by
> systems using the MIPS MT (multithreading) ASE within a single core.
> This will allow for such systems to be supported generically, without
> the current kludge of IPI code split between the MIPS arch & the malta
> board support code.
>
> Signed-off-by: Paul Burton <paul.burton@imgtec.com>
> ---
>
> Changes in v2: None
>
> drivers/irqchip/Kconfig | 2 +
> drivers/irqchip/irq-mips-cpu.c | 128 ++++++++++++++++++++++++++++++++++++++---
> 2 files changed, 122 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/irqchip/Kconfig b/drivers/irqchip/Kconfig
> index 7f87289..8af8704 100644
> --- a/drivers/irqchip/Kconfig
> +++ b/drivers/irqchip/Kconfig
> @@ -127,7 +127,9 @@ config IMGPDC_IRQ
> config IRQ_MIPS_CPU
> bool
> select GENERIC_IRQ_CHIP
> + select GENERIC_IRQ_IPI if SYS_SUPPORTS_MULTITHREADING
> select IRQ_DOMAIN
> + select IRQ_DOMAIN_HIERARCHY if GENERIC_IRQ_IPI
>
> config CLPS711X_IRQCHIP
> bool
> diff --git a/drivers/irqchip/irq-mips-cpu.c b/drivers/irqchip/irq-mips-cpu.c
> index 338de92..8108257 100644
> --- a/drivers/irqchip/irq-mips-cpu.c
> +++ b/drivers/irqchip/irq-mips-cpu.c
> @@ -17,15 +17,14 @@
> /*
> * Almost all MIPS CPUs define 8 interrupt sources. They are typically
> * level triggered (i.e., cannot be cleared from CPU; must be cleared from
> - * device). The first two are software interrupts which we don't really
> - * use or support. The last one is usually the CPU timer interrupt if
> - * counter register is present or, for CPUs with an external FPU, by
> - * convention it's the FPU exception interrupt.
> + * device).
> *
> - * Don't even think about using this on SMP. You have been warned.
> + * The first two are software interrupts (i.e. not exposed as pins) which
> + * may be used for IPIs in multi-threaded single-core systems.
> *
> - * This file exports one global function:
> - * void mips_cpu_irq_init(void);
> + * The last one is usually the CPU timer interrupt if the counter register
> + * is present, or for old CPUs with an external FPU by convention it's the
> + * FPU exception interrupt.
> */
> #include <linux/init.h>
> #include <linux/interrupt.h>
> @@ -40,6 +39,7 @@
> #include <asm/setup.h>
>
> static struct irq_domain *irq_domain;
> +static struct irq_domain *ipi_domain;
>
> static inline void unmask_mips_irq(struct irq_data *d)
> {
> @@ -90,6 +90,29 @@ static void mips_mt_cpu_irq_ack(struct irq_data *d)
> mask_mips_irq(d);
> }
>
> +#ifdef CONFIG_GENERIC_IRQ_IPI
Looking this over, you're adding a *lot* of #ifdef's into the code. Why
not put all the ipi code in a separate file, say irq-mips-ipi.c?
Patches 15 and 16 look fine.
thx,
Jason.
> +
> +static void mips_mt_send_ipi(struct irq_data *d, unsigned int cpu)
> +{
> + irq_hw_number_t hwirq = irqd_to_hwirq(d);
> + unsigned long flags;
> + int vpflags;
> +
> + local_irq_save(flags);
> +
> + /* We can only send IPIs to VPEs within the local core */
> + BUG_ON(cpu_data[cpu].core != current_cpu_data.core);
> +
> + vpflags = dvpe();
> + settc(cpu_vpe_id(&cpu_data[cpu]));
> + write_vpe_c0_cause(read_vpe_c0_cause() | (C_SW0 << hwirq));
> + evpe(vpflags);
> +
> + local_irq_restore(flags);
> +}
> +
> +#endif /* CONFIG_GENERIC_IRQ_IPI */
> +
> static struct irq_chip mips_mt_cpu_irq_controller = {
> .name = "MIPS",
> .irq_startup = mips_mt_cpu_irq_startup,
> @@ -100,6 +123,9 @@ static struct irq_chip mips_mt_cpu_irq_controller = {
> .irq_eoi = unmask_mips_irq,
> .irq_disable = mask_mips_irq,
> .irq_enable = unmask_mips_irq,
> +#ifdef CONFIG_GENERIC_IRQ_IPI
> + .ipi_send_single= mips_mt_send_ipi,
> +#endif
> };
>
> asmlinkage void __weak plat_irq_dispatch(void)
> @@ -116,7 +142,10 @@ asmlinkage void __weak plat_irq_dispatch(void)
> pending >>= CAUSEB_IP;
> while (pending) {
> irq = fls(pending) - 1;
> - virq = irq_linear_revmap(irq_domain, irq);
> + if (IS_ENABLED(CONFIG_GENERIC_IRQ_IPI) && irq < 2)
> + virq = irq_linear_revmap(ipi_domain, irq);
> + else
> + virq = irq_linear_revmap(irq_domain, irq);
> do_IRQ(virq);
> pending &= ~BIT(irq);
> }
> @@ -147,6 +176,82 @@ static const struct irq_domain_ops mips_cpu_intc_irq_domain_ops = {
> .xlate = irq_domain_xlate_onecell,
> };
>
> +#ifdef CONFIG_GENERIC_IRQ_IPI
> +
> +struct cpu_ipi_domain_state {
> + DECLARE_BITMAP(allocated, 2);
> +};
> +
> +static int mips_cpu_ipi_alloc(struct irq_domain *domain, unsigned int virq,
> + unsigned int nr_irqs, void *arg)
> +{
> + struct cpu_ipi_domain_state *state = domain->host_data;
> + unsigned int i, hwirq;
> + int ret;
> +
> + for (i = 0; i < nr_irqs; i++) {
> + hwirq = find_first_zero_bit(state->allocated, 2);
> + if (hwirq == 2)
> + return -EBUSY;
> + bitmap_set(state->allocated, hwirq, 1);
> +
> + ret = irq_domain_set_hwirq_and_chip(domain, virq + i, hwirq,
> + &mips_mt_cpu_irq_controller,
> + NULL);
> + if (ret)
> + return ret;
> +
> + ret = irq_set_irq_type(virq + i, IRQ_TYPE_LEVEL_HIGH);
> + if (ret)
> + return ret;
> + }
> +
> + return 0;
> +}
> +
> +static int mips_cpu_ipi_match(struct irq_domain *d, struct device_node *node,
> + enum irq_domain_bus_token bus_token)
> +{
> + bool is_ipi;
> +
> + switch (bus_token) {
> + case DOMAIN_BUS_IPI:
> + is_ipi = d->bus_token == bus_token;
> + return (!node || (to_of_node(d->fwnode) == node)) && is_ipi;
> + default:
> + return 0;
> + }
> +}
> +
> +static const struct irq_domain_ops mips_cpu_ipi_chip_ops = {
> + .alloc = mips_cpu_ipi_alloc,
> + .match = mips_cpu_ipi_match,
> +};
> +
> +static void mips_cpu_register_ipi_domain(struct device_node *of_node)
> +{
> + struct cpu_ipi_domain_state *ipi_domain_state;
> +
> + ipi_domain_state = kzalloc(sizeof(*ipi_domain_state), GFP_KERNEL);
> + if (!ipi_domain_state)
> + panic("Failed to alloc IPI domain state");
> +
> + ipi_domain = irq_domain_add_hierarchy(irq_domain,
> + IRQ_DOMAIN_FLAG_IPI_SINGLE,
> + 2, of_node,
> + &mips_cpu_ipi_chip_ops,
> + ipi_domain_state);
> + if (!ipi_domain)
> + panic("Failed to add MIPS CPU IPI domain");
> + ipi_domain->bus_token = DOMAIN_BUS_IPI;
> +}
> +
> +#else /* !CONFIG_GENERIC_IRQ_IPI */
> +
> +static inline void mips_cpu_register_ipi_domain(struct device_node *of_node) {}
> +
> +#endif /* !CONFIG_GENERIC_IRQ_IPI */
> +
> static void __init __mips_cpu_irq_init(struct device_node *of_node)
> {
> /* Mask interrupts. */
> @@ -158,6 +263,13 @@ static void __init __mips_cpu_irq_init(struct device_node *of_node)
> NULL);
> if (!irq_domain)
> panic("Failed to add irqdomain for MIPS CPU");
> +
> + /*
> + * Only proceed to register the software interrupt IPI implementation
> + * for CPUs which implement the MIPS MT (multi-threading) ASE.
> + */
> + if (cpu_has_mipsmt)
> + mips_cpu_register_ipi_domain(of_node);
> }
>
> void __init mips_cpu_irq_init(void)
> --
> 2.9.3
>
next prev parent reply other threads:[~2016-09-01 21:03 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-30 17:29 [PATCH v2 00/26] MIPS generic kernels, SEAD-3 & Boston support Paul Burton
2016-08-30 17:29 ` [PATCH v2 01/26] MIPS: PCI: Support for CONFIG_PCI_DOMAINS_GENERIC Paul Burton
2016-08-30 17:29 ` [PATCH v2 02/26] MIPS: PCI: Make pcibios_set_cache_line_size an initcall Paul Burton
2016-08-30 17:29 ` [PATCH v2 03/26] MIPS: PCI: Inline pcibios_assign_all_busses Paul Burton
2016-08-30 17:29 ` [PATCH v2 04/26] MIPS: PCI: Split pci.c into pci.c & pci-legacy.c Paul Burton
2016-08-30 17:29 ` [PATCH v2 05/26] MIPS: PCI: Introduce CONFIG_PCI_DRIVERS_LEGACY Paul Burton
2016-08-30 17:29 ` [PATCH v2 06/26] MIPS: PCI: Support generic drivers Paul Burton
2016-08-30 17:29 ` [PATCH v2 07/26] MIPS: Sanitise coherentio semantics Paul Burton
2016-08-30 17:29 ` [PATCH v2 08/26] MIPS: dma-default: Don't check hw_coherentio if device is non-coherent Paul Burton
2016-08-30 17:29 ` [PATCH v2 09/26] MIPS: Support per-device DMA coherence Paul Burton
2016-08-30 17:29 ` [PATCH v2 10/26] MIPS: Print CM error reports upon bus errors Paul Burton
2016-08-30 17:29 ` [PATCH v2 11/26] dt-bindings: Document mti,mips-cpc binding Paul Burton
2016-08-30 17:29 ` [PATCH v2 12/26] MIPS: CPC: Provide a default mips_cpc_default_phys_base Paul Burton
2016-08-31 10:21 ` Sergei Shtylyov
2016-08-30 17:29 ` [PATCH v2 13/26] dt-bindings: Document mti,mips-cdmm binding Paul Burton
2016-08-30 17:29 ` [PATCH v2 14/26] MIPS: CDMM: Allow CDMM base address to be specified via DT Paul Burton
2016-08-30 17:29 ` [PATCH v2 15/26] irqchip: mips-cpu: Replace magic 0x100 with IE_SW0 Paul Burton
2016-08-30 17:29 ` [PATCH v2 16/26] irqchip: mips-cpu: Prepare for non-legacy IRQ domains Paul Burton
2016-08-30 17:29 ` [PATCH v2 17/26] irqchip: mips-cpu: Introduce IPI IRQ domain support Paul Burton
2016-09-01 19:46 ` Jason Cooper [this message]
2016-08-30 17:29 ` [PATCH v2 18/26] MIPS: smp-mt: Use CPU interrupt controller " Paul Burton
2016-08-30 17:29 ` [PATCH v2 19/26] MIPS: Stengthen IPI IRQ domain sanity check Paul Burton
2016-08-30 17:29 ` [PATCH v2 20/26] MIPS: Adjust MIPS64 CAC_BASE to reflect Config.K0 Paul Burton
2016-08-30 17:29 ` [PATCH v2 21/26] MIPS: Support generating Flattened Image Trees (.itb) Paul Burton
2016-08-30 17:29 ` [PATCH v2 22/26] MIPS: generic: Introduce generic DT-based board support Paul Burton
2016-08-30 17:29 ` [PATCH v2 23/26] MIPS: generic: Convert SEAD-3 to a generic board Paul Burton
2016-08-30 17:29 ` [PATCH v2 24/26] dt-bindings: Document img,boston-clock binding Paul Burton
2016-08-30 17:29 ` [PATCH v2 25/26] clk: boston: Add a driver for MIPS Boston board clocks Paul Burton
2016-08-30 17:29 ` [PATCH v2 26/26] MIPS: generic: Support MIPS Boston development boards Paul Burton
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160901194626.GH10637@io.lakedaemon.net \
--to=jason@lakedaemon.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mips@linux-mips.org \
--cc=marc.zyngier@arm.com \
--cc=paul.burton@imgtec.com \
--cc=ralf@linux-mips.org \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).