On Wed, 29 Jun 2005, Richard B. Johnson wrote:
> >
> > task 1:
> > spin_lock(&non_irq_lock);
> >
> > task 2:
> >
> > spin_lock_irqsave(&some_irq_used_lock);
> > spin_lock(&non_irq_lock);
> >
> > Here we see that task 2 can spin with interrupts off, while the first task
> > is servicing an interrupt, and God forbid if the IRQ handler sends some
> > kind of SMP signal to the CPU running task 2 since that would be a
> > deadlock. Granted, this is a hypothetical situation, but makes using
> > spin_lock with interrupts enabled a little scary.
> >
>
> But it wouldn't deadlock! It would just spin until the guy on
> another CPU that had the lock unlocked.
>
Since Timur specified that spin_lock is used when you know that the lock
is not used in an interrupt, I'll continue as if that was the case.
This is a deadlock, because sending most SMP signals expect a reply when
it is received, and will wait until it gets it. Here's a more detailed
version.
CPU 1: running task 1
spin_lock(&non_irq_lock);
on CPU 2: running task 2
spin_lock_irqsave(&some_irq_used_lock); /* interrupts now off */
spin_lock(&non_irq_lock); /* blocked and spinning waiting for task 1*/
back on CPU 1:
interrupt goes off a preempts task 1 before it releases the
non_irq_lock.
Calls some stupid ISR that needs to send a signal to the other CPU.
Sends signal and waits for a reply. (for example the flush_tlb_others)
Now we have a deadlock. The interrupt on CPU 1 is waiting for a response
after sending an IPI to CPU 2, but CPU 2 is stuck spinning with interrupts
disabled and wont ever respond because the lock it is waiting for is held
by task 1 on CPU 1 that was preempted by the interrupt that will never
return.
This is probably the reason it is not allowed to call most IPIs from
interrupt or bottom half context.
> FYI, spin_lock() is supposed to be used in an interrupt where it
> is already known that the interrupts are OFF so you don't need
> to save/restore flags because you know the condition. IFF the
> ISR were to enable interrupts, with a spin-lock held (bad practice),
> it still wouldn't deadlock, it's just that the entire system could
> potentially degenerate into a poll-mode, spin in the ISR, mode
> with awful performance.
I stated earlier that the only times I use spin_lock is when I already
know that interrupts are off, and that includes ISRs.
-- Steve
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]