Hi Johan, On to, 2014-10-02 at 10:16 +0300, johan.hedberg@xxxxxxxxx wrote: > From: Johan Hedberg <johan.hedberg@xxxxxxxxx> > > The L2CAP connection's channel list lock (conn->chan_lock) must never be > taken while already holding a channel lock (chan->lock) in order to > avoid lock-inversion and lockdep warnings. So far the l2cap_chan_connect > function has acquired the chan->lock early in the function and then > later called l2cap_chan_add(conn, chan) which will try to take the > conn->chan_lock. This violates the correct order of taking the locks and > may lead to the following type of lockdep warnings: > patch worked ok so no more lockdep warning. Patch also looked good so ack from me. Tested-by: Jukka Rissanen <jukka.rissanen@xxxxxxxxxxxxxxx> Acked-by: Jukka Rissanen <jukka.rissanen@xxxxxxxxxxxxxxx> Cheers, Jukka -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
