Michael> For example, mcast_restart_thread performs list walk with Michael> no locking. Isnt this a problem? Do you mean ipoib_mcast_restart_task()? If so it seems all priv->multicast_list accesses are inside priv->lock. - R.