Commit 44651b85 authored by Gustavo F. Padovan's avatar Gustavo F. Padovan Committed by Marcel Holtmann

Bluetooth: Don't set control bits to zero first

We can set the SAR bits in the control field directly.
Signed-off-by: default avatarGustavo F. Padovan <padovan@profusion.mobi>
Reviewed-by: default avatarJoão Paulo Rechi Vita <jprvita@profusion.mobi>
Signed-off-by: default avatarMarcel Holtmann <marcel@holtmann.org>
parent 01760bdd
...@@ -1636,16 +1636,15 @@ static inline int l2cap_sar_segment_sdu(struct sock *sk, struct msghdr *msg, siz ...@@ -1636,16 +1636,15 @@ static inline int l2cap_sar_segment_sdu(struct sock *sk, struct msghdr *msg, siz
__skb_queue_tail(&sar_queue, skb); __skb_queue_tail(&sar_queue, skb);
len -= pi->remote_mps; len -= pi->remote_mps;
size += pi->remote_mps; size += pi->remote_mps;
control = 0;
while (len > 0) { while (len > 0) {
size_t buflen; size_t buflen;
if (len > pi->remote_mps) { if (len > pi->remote_mps) {
control |= L2CAP_SDU_CONTINUE; control = L2CAP_SDU_CONTINUE;
buflen = pi->remote_mps; buflen = pi->remote_mps;
} else { } else {
control |= L2CAP_SDU_END; control = L2CAP_SDU_END;
buflen = len; buflen = len;
} }
...@@ -1658,7 +1657,6 @@ static inline int l2cap_sar_segment_sdu(struct sock *sk, struct msghdr *msg, siz ...@@ -1658,7 +1657,6 @@ static inline int l2cap_sar_segment_sdu(struct sock *sk, struct msghdr *msg, siz
__skb_queue_tail(&sar_queue, skb); __skb_queue_tail(&sar_queue, skb);
len -= buflen; len -= buflen;
size += buflen; size += buflen;
control = 0;
} }
skb_queue_splice_tail(&sar_queue, TX_QUEUE(sk)); skb_queue_splice_tail(&sar_queue, TX_QUEUE(sk));
if (sk->sk_send_head == NULL) if (sk->sk_send_head == NULL)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment