Messages in this thread | | | Date | Wed, 12 Jul 2006 18:30:49 -0700 | From | Andrew Morton <> | Subject | Re: [PATCH] Convert idr's internal locking to _irqsave variant |
| |
On Wed, 12 Jul 2006 13:45:12 -0700 Roland Dreier <rdreier@cisco.com> wrote:
> Currently, the code in lib/idr.c uses a bare spin_lock(&idp->lock) to > do internal locking. This is a nasty trap for code that might call > idr functions from different contexts; for example, it seems perfectly > reasonable to call idr_get_new() from process context and idr_remove() > from interrupt context -- but with the current locking this would lead > to a potential deadlock. > > The simplest fix for this is to just convert the idr locking to use > spin_lock_irqsave(). > > In particular, this fixes a very complicated locking issue detected by > lockdep, involving the ib_ipoib driver's priv->lock and dev->_xmit_lock, > which get involved with the ib_sa module's query_idr.lock.
Sigh. It was always a mistake (of the kernel programming 101 type) to put any locking at all in the idr code. At some stage we need to weed it all out and move it to callers.
Your fix is yet more fallout from that mistake. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |