Re: [PATCH 00/16] Bluetooth: L2CAP: fix and annotate l2cap_conn::chan_l locking
From: Luiz Augusto von Dentz
Date: Tue Sep 01 2026 - 15:33:25 EST
Hi Pauli,
On Tue, Sep 1, 2026 at 3:11 PM <patchwork-bot+bluetooth@xxxxxxxxxx> wrote:
>
> Hello:
>
> This series was applied to bluetooth/bluetooth-next.git (master)
> by Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>:
>
> On Sat, 29 Aug 2026 17:19:55 +0300 you wrote:
> > The intent in current code is that:
> >
> > - l2cap_conn::chan_l read/write is guarded by l2cap_conn::lock
> >
> > - l2cap_chan_del called with held l2cap_chan::lock (l2cap_sock.c uses
> > this)
> >
> > [...]
>
> Here is the summary with links:
> - [01/16] Bluetooth: L2CAP: take chan->lock for l2cap_chan_add/ready/del
> https://git.kernel.org/bluetooth/bluetooth-next/c/ca52f4764c87
> - [02/16] Bluetooth: L2CAP: add l2cap_chan_close_unlocked() and locking helpers
> https://git.kernel.org/bluetooth/bluetooth-next/c/368dc7fcaced
> - [03/16] Bluetooth: L2CAP: fix race condition in l2cap_sock_shutdown()
> https://git.kernel.org/bluetooth/bluetooth-next/c/86b377359826
> - [04/16] Bluetooth: 6lowpan: use l2cap_chan_close_unlocked()
> https://git.kernel.org/bluetooth/bluetooth-next/c/4e64bd587b64
> - [05/16] Bluetooth: L2CAP: remove unused l2cap_chan_close()
> https://git.kernel.org/bluetooth/bluetooth-next/c/c5123fddfef1
> - [06/16] Bluetooth: 6lowpan: avoid concurrent peer_del() in bt_6lowpan_disconnect
> https://git.kernel.org/bluetooth/bluetooth-next/c/02122fd80029
> - [07/16] Bluetooth: L2CAP: hold conn->lock for __l2cap_ecred_conn_rsp_defer
> https://git.kernel.org/bluetooth/bluetooth-next/c/62df59925278
> - [08/16] Bluetooth: L2CAP: hold l2cap_conn::lock in l2cap_connect_cfm()
> https://git.kernel.org/bluetooth/bluetooth-next/c/604e2f6d1e95
> - [09/16] Bluetooth: L2CAP: add annotations for l2cap_chan list locking
> https://git.kernel.org/bluetooth/bluetooth-next/c/886931f0c7e6
> - [10/16] Bluetooth: L2CAP: take lock for l2cap_chan_del in l2cap_ecred_rsp_defer
> (no matching commit)
> - [11/16] Bluetooth: L2CAP: hold chan in l2cap_ecred_conn_rsp()
> https://git.kernel.org/bluetooth/bluetooth-next/c/26836086682a
> - [12/16] Bluetooth: L2CAP: annotate locking for l2cap_chan_del()
> (no matching commit)
> - [13/16] Bluetooth: L2CAP: annotate locking for l2cap_ops callbacks
> (no matching commit)
> - [14/16] Bluetooth: L2CAP: make concurrent l2cap_set_timer() refcounting safe
> https://git.kernel.org/bluetooth/bluetooth-next/c/761224d13f8a
> - [15/16] Bluetooth: L2CAP: remove conditional locking from l2cap_connect()
> https://git.kernel.org/bluetooth/bluetooth-next/c/2c127ee79a0b
> - [16/16] Bluetooth: L2CAP: refuse __l2cap_chan_add if chan already has conn
> (no matching commit)
>
> You are awesome, thank you!
> --
> Deet-doot-dot, I am a bot.
> https://korg.docs.kernel.org/patchwork/pwbot.html
>
I've run into some conflicts; this is why some changes were not
applied. Could you please look into it and rebase them?
--
Luiz Augusto von Dentz