Return-Path: Date: Thu, 9 Feb 2012 16:40:39 +0200 From: Emeltchenko Andrei To: Marcel Holtmann Cc: linux-bluetooth@vger.kernel.org Subject: Re: [RFCv3 03/16] Bluetooth: Do not use sk lock in get_chan functions Message-ID: <20120209144038.GC22374@aemeltch-MOBL1> References: <1328797057-26331-1-git-send-email-Andrei.Emeltchenko.news@gmail.com> <1328797057-26331-4-git-send-email-Andrei.Emeltchenko.news@gmail.com> <1328797725.28848.22.camel@aeonflux> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1328797725.28848.22.camel@aeonflux> Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Marcel, On Thu, Feb 09, 2012 at 03:28:45PM +0100, Marcel Holtmann wrote: > Hi Andrei, > > > When needed we take explicit lock. > > > > Signed-off-by: Andrei Emeltchenko > > --- > > net/bluetooth/l2cap_core.c | 6 ++---- > > 1 files changed, 2 insertions(+), 4 deletions(-) > > > > diff --git a/net/bluetooth/l2cap_core.c b/net/bluetooth/l2cap_core.c > > index ae08944..dcf4792 100644 > > --- a/net/bluetooth/l2cap_core.c > > +++ b/net/bluetooth/l2cap_core.c > > @@ -105,9 +105,8 @@ static struct l2cap_chan *l2cap_get_chan_by_scid(struct l2cap_conn *conn, u16 ci > > > > mutex_lock(&conn->chan_lock); > > c = __l2cap_get_chan_by_scid(conn, cid); > > - if (c) > > - lock_sock(c->sk); > > mutex_unlock(&conn->chan_lock); > > + > > return c; > > } > > so this is dangerous since now we are creating an imbalance with the > socket locks. Are we fixing that one later on? Yes, that will be fixed later on. Do you think that needs to be merged with the other code? Best regards Andrei Emeltchenko