Re: [openib-general] ipoib lockdep warning

From: Roland Dreier
Date: Tue Jul 11 2006 - 18:52:40 EST


No time to really look at this in detail, but I think the issue is a
slightly bogus conversion to netif_tx_lock(). Can you try this patch
and see if things are better?

diff --git a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
index ab40488..ddd1946 100644
--- a/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
+++ b/drivers/infiniband/ulp/ipoib/ipoib_multicast.c
@@ -820,9 +820,8 @@ void ipoib_mcast_restart_task(void *dev_

ipoib_mcast_stop_thread(dev, 0);

- local_irq_save(flags);
netif_tx_lock(dev);
- spin_lock(&priv->lock);
+ spin_lock_irqsave(&priv->lock, flags);

/*
* Unfortunately, the networking core only gives us a list of all of
@@ -893,9 +892,8 @@ void ipoib_mcast_restart_task(void *dev_
}
}

- spin_unlock(&priv->lock);
+ spin_unlock_irqrestore(&priv->lock, flags);
netif_tx_unlock(dev);
- local_irq_restore(flags);

/* We have to cancel outside of the spinlock */
list_for_each_entry_safe(mcast, tmcast, &remove_list, list) {
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/