On Thu, Sep 29, 2011 at 10:34:09AM +0100, Russell King - ARM Linux wrote:

On Wed, Sep 28, 2011 at 05:06:44PM +0800, Shawn Guo wrote:

quoted

+#ifdef CONFIG_SMP
+		else if (irqnr < 16) {
+			writel_relaxed(irqstat, gic_cpu_base_addr +
+						GIC_CPU_EOI);
+			do_IPI(irqnr, regs);
+		}
+#endif
+#ifdef CONFIG_LOCAL_TIMERS
+		else if (irqnr == 29) {
+			writel_relaxed(irqstat, gic_cpu_base_addr +
+						GIC_CPU_EOI);
+			do_local_timer(regs);
+		}
+#endif

As I've said for similar patches. neither of these two functions are
designed to be called from another C function (because they're marked
__exception or __exception_irq_entry).  Both of these markers tell the
unwinder that a struct pt_regs is located directly above the functions
stack frame.

Can you please share your position on Marc's PPI and GIC
MULTI_IRQ_HANDLER series?  Are you possibly going to merge them in the
coming merge window?  If you are, I would directly move my imx6q onto
those series and save the local gic_handle_irq().  Otherwise, I may
have to add a wrapper for do_IPI() and do_local_timer() to work around
the pt_regs issue you pointed out here.

-- 
Regards,
Shawn