On Wed, 2013-10-09 at 23:54 +0200, Helge Deller wrote: > Signed-off-by: Helge Deller <deller@xxxxxx> > > diff --git a/arch/parisc/kernel/irq.c b/arch/parisc/kernel/irq.c > index 2e6443b..c439c05 100644 > --- a/arch/parisc/kernel/irq.c > +++ b/arch/parisc/kernel/irq.c > @@ -529,8 +529,8 @@ void do_cpu_irq_mask(struct pt_regs *regs) > cpumask_t dest; > #endif > > - old_regs = set_irq_regs(regs); > local_irq_disable(); > + old_regs = set_irq_regs(regs); I don't quite understand why. set_irq_regs is just saving the current regs pointer. The design intent is to call it first thing in the interrupt routine but because of the way we use them, it makes no difference whether you do it before or after disabling interrupts because it's stacked. What was the reason for wanting to change it to a non-standard calling pattern? James -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
