On Thu, 25 Aug 2016, Rich Felker wrote: > assumption that is was just a bug. Now that Mark Rutland has explained > it well (and with your additional explanation below in your email), I > see what the motivation was, but I still think it could be done in a > less-confusing and more-consistent way that doesn't assume ARM-like > irq architecture. It's not only ARM. Some MIPS Octeon stuff has the same layout and requirements to use a single irq number for interrupts which are delivered on a per cpu basis. Patches are welcome :) > > If your particular hardware has the old scheme of seperate interrupt numbers > > for per cpu interrupts, then you can simply use the normal interrupt scheme > > and request a seperate interrupt per cpu. > > Nominally it uses the same range of hardware interrupt numbers for all > (presently both) cpus, but some of them get delivered to a specific > cpu associated with the event (presently, IPI and timer; IPI is on a > fixed number at synthesis time but timer is runtime configurable) > while others are conceptually deliverable to either cpu (presently > only delivered to cpu0, but that's treated as an implementation > detail). If I understand correctly, then this is the classic scheme: CPU0 IPI0 IRQ-N CPU1 IPI1 IRQ-M These and the timers or whatever are strict per cpu and therefor not routable. Regular device interrupts can be routed to any CPU by setting the affinity. Correct? > It currently works requesting the irq with flags that ensure the > handler runs on the same cpu it was delivered on, without using any > other percpu irq framework. Which special flag are you referring to? I'm not aware of one. IRQF_PER_CPU is just telling the core that this is a non routable per cpu interrupt. It's excluded from affinity setting and also on cpu hot unplug the per cpu interrupts are not touched and nothing tries to reroute them to one of the still online cpus. Regarding the interrupt handler. It runs on the CPU on which the interrupt is delivered and there is nothing you can influence with a flag. > If you have concerns about ways this could break and want me to make the > drivers do something else, I'm open to suggestions. If I understand the hardware halfways right, then using request_irq() with IRQF_PER_CPU for these special interrupts is completely correct. The handler either uses this_cpu_xxx() for accessing the per cpu data related to the interrupt or you can hand in a percpu pointer as dev_id to request_irq() which then is handed to the interrupt function as a cookie. Hope that helps. Thanks, tglx -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html