Skip to content

Commit

Permalink
Bluetooth: Fix double acking I-Frames when sending pending I-Frames
Browse files Browse the repository at this point in the history
Pending I-Frame(s) are considered as acknowledgement. To void double
acking (via I-Frame and later via RR) clear ack timer when sending
first pending I-Frame.

Signed-off-by: Szymon Janc <szymon.janc@tieto.com>
Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
Acked-by: Marcel Holtmann <marcel@holtmann.org>
Signed-off-by: Johan Hedberg <johan.hedberg@intel.com>
  • Loading branch information
Szymon Janc authored and Johan Hedberg committed Feb 13, 2012
1 parent 8ed7a0a commit 930fa4a
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion net/bluetooth/l2cap_core.c
Original file line number Diff line number Diff line change
Expand Up @@ -1455,7 +1455,9 @@ static int l2cap_ertm_send(struct l2cap_chan *chan)

if (bt_cb(skb)->retries == 1) {
chan->unacked_frames++;
nsent++;

if (!nsent++)
__clear_ack_timer(chan);
}

chan->frames_sent++;
Expand Down

0 comments on commit 930fa4a

Please sign in to comment.