Message ID | 20120614103112.GA15685@bloggs.ozlabs.ibm.com (mailing list archive) |
---|---|
State | Superseded |
Delegated to: | Paul Mackerras |
Headers | show |
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 <paulus@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
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
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 <paulus@samba.org> --- arch/powerpc/include/asm/hw_irq.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)