diff options
| author |   Thomas Gleixner <tglx@linutronix.de> | 2025-03-13 15:31:15 +0100 |
|---|---|---|
| committer | Thomas Gleixner <tglx@linutronix.de> | 2025-04-07 09:43:19 +0200 |
| commit | 06f2f68a670aae28b825065439301831e74da880 (patch) | |
| tree | 9e7fc433a3c343cd5ca4b04617e46daf15c46efb | |
| parent | irqchip: Add EcoNet EN751221 INTC (diff) | |
| download | wireguard-linux-06f2f68a670aae28b825065439301831e74da880.tar.xz wireguard-linux-06f2f68a670aae28b825065439301831e74da880.zip | |
genirq/generic-chip: Make locking unconditional
The SMP conditional wrappers around raw_spin_[un]lock() have no real
value. On !SMP kernels the lock operations are NOOPs except for a
preempt_disable/enable() pair on PREEMPT enabled kernels, which are not
really worth to optimize for. Aside of that this evades lockdep on !SMP
kernels.
Remove the !SMP stubs and make it unconditional.
No functional change.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: https://lore.kernel.org/all/20250313142524.011345765@linutronix.de
| -rw-r--r-- | include/linux/irq.h | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/include/linux/irq.h b/include/linux/irq.h index dd5df1e2d032..500772943207 100644 --- a/include/linux/irq.h +++ b/include/linux/irq.h @@ -1222,7 +1222,6 @@ static inline struct irq_chip_type *irq_data_get_chip_type(struct irq_data *d) #define IRQ_MSK(n) (u32)((n) < 32 ? ((1 << (n)) - 1) : UINT_MAX) -#ifdef CONFIG_SMP static inline void irq_gc_lock(struct irq_chip_generic *gc) { raw_spin_lock(&gc->lock); @@ -1232,10 +1231,6 @@ static inline void irq_gc_unlock(struct irq_chip_generic *gc) { raw_spin_unlock(&gc->lock); } -#else -static inline void irq_gc_lock(struct irq_chip_generic *gc) { } -static inline void irq_gc_unlock(struct irq_chip_generic *gc) { } -#endif /* * The irqsave variants are for usage in non interrupt code. Do not use |