Hi Gustavo, > We can lock the parent lock only inside the new_connection() call, > then we just use the l2cap_chan_lock() in core code. > > Signed-off-by: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx> > --- > net/bluetooth/l2cap_core.c | 16 ++++++---------- > net/bluetooth/l2cap_sock.c | 9 ++++++++- > 2 files changed, 14 insertions(+), 11 deletions(-) I am fine with this patch under the premise that 2/8 is properly explained. And here you need to also get into a bit more detail. With any kind of locking changes, I expect a proper commit message explaining why this is safe. Regards Marcel -- 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
