Hi Andrei, * Andrei Emeltchenko <Andrei.Emeltchenko.news@xxxxxxxxx> [2012-07-10 15:27:45 +0300]: > From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > > > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx> > --- > include/net/bluetooth/l2cap.h | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/include/net/bluetooth/l2cap.h b/include/net/bluetooth/l2cap.h > index d80e3f0..dfe9374 100644 > --- a/include/net/bluetooth/l2cap.h > +++ b/include/net/bluetooth/l2cap.h > @@ -672,11 +672,17 @@ enum { > > static inline void l2cap_chan_hold(struct l2cap_chan *c) > { > + BT_DBG("chan %p refcnt %d -> %d", c, atomic_read(&c->refcnt), > + atomic_read(&c->refcnt) + 1); > + We don't really need to call atomic_read() twice read. Just print the new value and the log reader has to figure it out. Gustavo -- 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