UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

Bluetooth: Add l2cap_state_change_and_error() 

l2cap_state_change_and_error() introduces the ability to update a
l2cap_user with changes in channel's state and error code with just one
call. The main reason for this is to avoid race conditions between and
setting the state and then the error. Otherwise we would need to release
the lock between both operations.

This is another step of an ongoing work to make l2cap_core.c totally
independent from l2cap's struct sock.

Signed-off-by: Gustavo Padovan <gustavo.padovan@collabora.co.uk>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
This commit is contained in:
Gustavo Padovan 2013-10-15 19:24:46 -03:00 committed by Marcel Holtmann
parent 53f5212121
commit f8e73017c1
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
net/bluetooth/l2cap_core.c
View File

@ -241,9 +241,15 @@ static void l2cap_state_change(struct l2cap_chan *chan, int state)
release_sock(sk); release_sock(sk);
} }
static inline void __l2cap_chan_set_err(struct l2cap_chan *chan, int err) static inline void l2cap_state_change_and_error(struct l2cap_chan *chan,
int state, int err)
{ {
struct sock *sk = chan->sk;
lock_sock(sk);
chan->state = state;
chan->ops->state_change(chan, chan->state, err); chan->ops->state_change(chan, chan->state, err);
release_sock(sk);
} }
static inline void l2cap_chan_set_err(struct l2cap_chan *chan, int err) static inline void l2cap_chan_set_err(struct l2cap_chan *chan, int err)
@ -251,7 +257,7 @@ static inline void l2cap_chan_set_err(struct l2cap_chan *chan, int err)
struct sock *sk = chan->sk; struct sock *sk = chan->sk;
lock_sock(sk); lock_sock(sk);
__l2cap_chan_set_err(chan, err); chan->ops->state_change(chan, chan->state, err);
release_sock(sk); release_sock(sk);
} }
@ -1228,7 +1234,6 @@ static inline int l2cap_mode_supported(__u8 mode, __u32 feat_mask)
static void l2cap_send_disconn_req(struct l2cap_chan *chan, int err) static void l2cap_send_disconn_req(struct l2cap_chan *chan, int err)
{ {
struct sock *sk = chan->sk;
struct l2cap_conn *conn = chan->conn; struct l2cap_conn *conn = chan->conn;
struct l2cap_disconn_req req; struct l2cap_disconn_req req;
@ -1251,10 +1256,7 @@ static void l2cap_send_disconn_req(struct l2cap_chan *chan, int err)
l2cap_send_cmd(conn, l2cap_get_ident(conn), L2CAP_DISCONN_REQ, l2cap_send_cmd(conn, l2cap_get_ident(conn), L2CAP_DISCONN_REQ,
sizeof(req), &req); sizeof(req), &req);
lock_sock(sk); l2cap_state_change_and_error(chan, BT_DISCONN, err);
__l2cap_state_change(chan, BT_DISCONN);
__l2cap_chan_set_err(chan, err);
release_sock(sk);
} }
/* ---- L2CAP connections ---- */ /* ---- L2CAP connections ---- */