]> git.baikalelectronics.ru Git - kernel.git/commit
Bluetooth: Fix deadlock in the ERTM logic
authorGustavo F. Padovan <padovan@profusion.mobi>
Tue, 21 Sep 2010 19:31:11 +0000 (16:31 -0300)
committerGustavo F. Padovan <padovan@profusion.mobi>
Thu, 30 Sep 2010 15:19:35 +0000 (12:19 -0300)
commitf849544b7cac336f89e157480a2991e1eac76790
treefd8fbcb76608bce78062c6a74ff8e8b65b3e34ae
parentc16c795eeb4cf94732b366456fd0a40409bda709
Bluetooth: Fix deadlock in the ERTM logic

The Enhanced Retransmission Mode(ERTM) is a realiable mode of operation
of the Bluetooth L2CAP layer. Think on it like a simplified version of
TCP.
The problem we were facing here was a deadlock. ERTM uses a backlog
queue to queue incomimg packets while the user is helding the lock. At
some moment the sk_sndbuf can be exceeded and we can't alloc new skbs
then the code sleep with the lock to wait for memory, that stalls the
ERTM connection once we can't read the acknowledgements packets in the
backlog queue to free memory and make the allocation of outcoming skb
successful.

This patch actually affect all users of bt_skb_send_alloc(), i.e., all
L2CAP modes and SCO.

We are safe against socket states changes or channels deletion while the
we are sleeping wait memory. Checking for the sk->sk_err and
sk->sk_shutdown make the code safe, since any action that can leave the
socket or the channel in a not usable state set one of the struct
members at least. Then we can check both of them when getting the lock
again and return with the proper error if something unexpected happens.

Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
Signed-off-by: Ulisses Furquim <ulisses@profusion.mobi>
include/net/bluetooth/bluetooth.h