On Thu, 2012-06-14 at 20:31 +1000, Paul Mackerras wrote: > At present, hard_irq_disable() does nothing because of this code in > include/linux/interrupt.h: > > #ifndef hard_irq_disable > #define hard_irq_disable() do { } while(0) > #endif > > So we need to make our hard_irq_disable() be a macro. > > Signed-off-by: Paul Mackerras <pau...@samba.org>
This was broken by my rewrite of the lazy irq stuff when I turned that from a macro to a function. So it should be CC'ed to stable 3.4. Also I think it's simpler (and we do that commonly) to just do #define hard_irq_disable hard_irq_disable Cheers, Ben. > --- > arch/powerpc/include/asm/hw_irq.h | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > diff --git a/arch/powerpc/include/asm/hw_irq.h > b/arch/powerpc/include/asm/hw_irq.h > index c9aac24..d8f873f 100644 > --- a/arch/powerpc/include/asm/hw_irq.h > +++ b/arch/powerpc/include/asm/hw_irq.h > @@ -93,12 +93,13 @@ static inline bool arch_irqs_disabled(void) > #define __hard_irq_disable() __mtmsrd(local_paca->kernel_msr, 1) > #endif > > -static inline void hard_irq_disable(void) > +static inline void _hard_irq_disable(void) > { > __hard_irq_disable(); > get_paca()->soft_enabled = 0; > get_paca()->irq_happened |= PACA_IRQ_HARD_DIS; > } > +#define hard_irq_disable() _hard_irq_disable() > > /* > * This is called by asynchronous interrupts to conditionally _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev