On an SMP machine we must use spin_lock_irqsave
and not spin_lock_irq
from interrupt context.
Why would we want to save the flags (which contain the IF)?
Is there another interrupt routine that could interrupt us?
spin_lock_irqsave
is basically used to save the interrupt state before taking the spin lock, this is because spin lock disables the interrupt, when the lock is taken in interrupt context, and re-enables it when while unlocking. The interrupt state is saved so that it should reinstate the interrupts again.
Example:
spin_lock_irq
will disable the interrupt x and take the the lockspin_unlock_irq
will enable the interrupt x.So in the 3rd step above after releasing the lock we will have interrupt x enabled which was earlier disabled before the lock was acquired.
So only when you are sure that interrupts are not disabled only then you should spin_lock_irq
otherwise you should always use spin_lock_irqsave
.