1*85630469SLior David // SPDX-License-Identifier: ISC
202525a79SVladimir Kondratiev /*
3af3db60aSLazar Alexei * Copyright (c) 2014-2017 Qualcomm Atheros, Inc.
4bf0353a6SAhmad Masri * Copyright (c) 2018-2019, The Linux Foundation. All rights reserved.
502525a79SVladimir Kondratiev */
602525a79SVladimir Kondratiev
7b4490f42SVladimir Kondratiev #include "wil6210.h"
8b4490f42SVladimir Kondratiev #include "txrx.h"
9b4490f42SVladimir Kondratiev
10b4490f42SVladimir Kondratiev #define SEQ_MODULO 0x1000
11b4490f42SVladimir Kondratiev #define SEQ_MASK 0xfff
12b4490f42SVladimir Kondratiev
seq_less(u16 sq1,u16 sq2)13b4490f42SVladimir Kondratiev static inline int seq_less(u16 sq1, u16 sq2)
14b4490f42SVladimir Kondratiev {
15b4490f42SVladimir Kondratiev return ((sq1 - sq2) & SEQ_MASK) > (SEQ_MODULO >> 1);
16b4490f42SVladimir Kondratiev }
17b4490f42SVladimir Kondratiev
seq_inc(u16 sq)18b4490f42SVladimir Kondratiev static inline u16 seq_inc(u16 sq)
19b4490f42SVladimir Kondratiev {
20b4490f42SVladimir Kondratiev return (sq + 1) & SEQ_MASK;
21b4490f42SVladimir Kondratiev }
22b4490f42SVladimir Kondratiev
seq_sub(u16 sq1,u16 sq2)23b4490f42SVladimir Kondratiev static inline u16 seq_sub(u16 sq1, u16 sq2)
24b4490f42SVladimir Kondratiev {
25b4490f42SVladimir Kondratiev return (sq1 - sq2) & SEQ_MASK;
26b4490f42SVladimir Kondratiev }
27b4490f42SVladimir Kondratiev
reorder_index(struct wil_tid_ampdu_rx * r,u16 seq)28b4490f42SVladimir Kondratiev static inline int reorder_index(struct wil_tid_ampdu_rx *r, u16 seq)
29b4490f42SVladimir Kondratiev {
30b4490f42SVladimir Kondratiev return seq_sub(seq, r->ssn) % r->buf_size;
31b4490f42SVladimir Kondratiev }
32b4490f42SVladimir Kondratiev
wil_release_reorder_frame(struct net_device * ndev,struct wil_tid_ampdu_rx * r,int index)335bd60982SLior David static void wil_release_reorder_frame(struct net_device *ndev,
34b4490f42SVladimir Kondratiev struct wil_tid_ampdu_rx *r,
35b4490f42SVladimir Kondratiev int index)
36b4490f42SVladimir Kondratiev {
37b4490f42SVladimir Kondratiev struct sk_buff *skb = r->reorder_buf[index];
38b4490f42SVladimir Kondratiev
39b4490f42SVladimir Kondratiev if (!skb)
40b4490f42SVladimir Kondratiev goto no_frame;
41b4490f42SVladimir Kondratiev
42b4490f42SVladimir Kondratiev /* release the frame from the reorder ring buffer */
43b4490f42SVladimir Kondratiev r->stored_mpdu_num--;
44b4490f42SVladimir Kondratiev r->reorder_buf[index] = NULL;
45b4490f42SVladimir Kondratiev wil_netif_rx_any(skb, ndev);
46b4490f42SVladimir Kondratiev
47b4490f42SVladimir Kondratiev no_frame:
48b4490f42SVladimir Kondratiev r->head_seq_num = seq_inc(r->head_seq_num);
49b4490f42SVladimir Kondratiev }
50b4490f42SVladimir Kondratiev
wil_release_reorder_frames(struct net_device * ndev,struct wil_tid_ampdu_rx * r,u16 hseq)515bd60982SLior David static void wil_release_reorder_frames(struct net_device *ndev,
52b4490f42SVladimir Kondratiev struct wil_tid_ampdu_rx *r,
53b4490f42SVladimir Kondratiev u16 hseq)
54b4490f42SVladimir Kondratiev {
55b4490f42SVladimir Kondratiev int index;
56b4490f42SVladimir Kondratiev
57cf42c4e5SVladimir Kondratiev /* note: this function is never called with
58cf42c4e5SVladimir Kondratiev * hseq preceding r->head_seq_num, i.e it is always true
59cf42c4e5SVladimir Kondratiev * !seq_less(hseq, r->head_seq_num)
60cf42c4e5SVladimir Kondratiev * and thus on loop exit it should be
61cf42c4e5SVladimir Kondratiev * r->head_seq_num == hseq
62cf42c4e5SVladimir Kondratiev */
63cf42c4e5SVladimir Kondratiev while (seq_less(r->head_seq_num, hseq) && r->stored_mpdu_num) {
64b4490f42SVladimir Kondratiev index = reorder_index(r, r->head_seq_num);
655bd60982SLior David wil_release_reorder_frame(ndev, r, index);
66b4490f42SVladimir Kondratiev }
67cf42c4e5SVladimir Kondratiev r->head_seq_num = hseq;
68b4490f42SVladimir Kondratiev }
69b4490f42SVladimir Kondratiev
wil_reorder_release(struct net_device * ndev,struct wil_tid_ampdu_rx * r)705bd60982SLior David static void wil_reorder_release(struct net_device *ndev,
71b4490f42SVladimir Kondratiev struct wil_tid_ampdu_rx *r)
72b4490f42SVladimir Kondratiev {
73b4490f42SVladimir Kondratiev int index = reorder_index(r, r->head_seq_num);
74b4490f42SVladimir Kondratiev
75b4490f42SVladimir Kondratiev while (r->reorder_buf[index]) {
765bd60982SLior David wil_release_reorder_frame(ndev, r, index);
77b4490f42SVladimir Kondratiev index = reorder_index(r, r->head_seq_num);
78b4490f42SVladimir Kondratiev }
79b4490f42SVladimir Kondratiev }
80b4490f42SVladimir Kondratiev
81bd33273bSVladimir Kondratiev /* called in NAPI context */
wil_rx_reorder(struct wil6210_priv * wil,struct sk_buff * skb)82b4490f42SVladimir Kondratiev void wil_rx_reorder(struct wil6210_priv *wil, struct sk_buff *skb)
83bd33273bSVladimir Kondratiev __acquires(&sta->tid_rx_lock) __releases(&sta->tid_rx_lock)
84b4490f42SVladimir Kondratiev {
855bd60982SLior David struct wil6210_vif *vif;
865bd60982SLior David struct net_device *ndev;
871bd82ee0SDedy Lansky int tid, cid, mid, mcast, retry;
887be13fc3SGidon Studinski u16 seq;
897be13fc3SGidon Studinski struct wil_sta_info *sta;
90ec81b5adSDedy Lansky struct wil_tid_ampdu_rx *r;
91b4490f42SVladimir Kondratiev u16 hseq;
92b4490f42SVladimir Kondratiev int index;
93b4490f42SVladimir Kondratiev
947be13fc3SGidon Studinski wil->txrx_ops.get_reorder_params(wil, skb, &tid, &cid, &mid, &seq,
951bd82ee0SDedy Lansky &mcast, &retry);
967be13fc3SGidon Studinski sta = &wil->sta[cid];
977be13fc3SGidon Studinski
98e4373d8eSVladimir Kondratiev wil_dbg_txrx(wil, "MID %d CID %d TID %d Seq 0x%03x mcast %01x\n",
99e4373d8eSVladimir Kondratiev mid, cid, tid, seq, mcast);
100e4373d8eSVladimir Kondratiev
1015bd60982SLior David vif = wil->vifs[mid];
1025bd60982SLior David if (unlikely(!vif)) {
1035bd60982SLior David wil_dbg_txrx(wil, "invalid VIF, mid %d\n", mid);
1045bd60982SLior David dev_kfree_skb(skb);
1055bd60982SLior David return;
1065bd60982SLior David }
1075bd60982SLior David ndev = vif_to_ndev(vif);
1085bd60982SLior David
109bd33273bSVladimir Kondratiev spin_lock(&sta->tid_rx_lock);
110ec81b5adSDedy Lansky
111ec81b5adSDedy Lansky r = sta->tid_rx[tid];
112b4490f42SVladimir Kondratiev if (!r) {
113b4490f42SVladimir Kondratiev wil_netif_rx_any(skb, ndev);
114bd33273bSVladimir Kondratiev goto out;
115b4490f42SVladimir Kondratiev }
116b4490f42SVladimir Kondratiev
1171bd82ee0SDedy Lansky if (unlikely(mcast)) {
1181bd82ee0SDedy Lansky if (retry && seq == r->mcast_last_seq) {
1191bd82ee0SDedy Lansky r->drop_dup_mcast++;
1201bd82ee0SDedy Lansky wil_dbg_txrx(wil, "Rx drop: dup mcast seq 0x%03x\n",
1211bd82ee0SDedy Lansky seq);
1221bd82ee0SDedy Lansky dev_kfree_skb(skb);
1231bd82ee0SDedy Lansky goto out;
1241bd82ee0SDedy Lansky }
1251bd82ee0SDedy Lansky r->mcast_last_seq = seq;
1261bd82ee0SDedy Lansky wil_netif_rx_any(skb, ndev);
1271bd82ee0SDedy Lansky goto out;
1281bd82ee0SDedy Lansky }
1291bd82ee0SDedy Lansky
13091a8edccSVladimir Kondratiev r->total++;
131b4490f42SVladimir Kondratiev hseq = r->head_seq_num;
132b4490f42SVladimir Kondratiev
133c888cdd4SVladimir Kondratiev /** Due to the race between WMI events, where BACK establishment
134c888cdd4SVladimir Kondratiev * reported, and data Rx, few packets may be pass up before reorder
135c888cdd4SVladimir Kondratiev * buffer get allocated. Catch up by pretending SSN is what we
136c888cdd4SVladimir Kondratiev * see in the 1-st Rx packet
137ff7c5c3bSVladimir Kondratiev *
138ff7c5c3bSVladimir Kondratiev * Another scenario, Rx get delayed and we got packet from before
139ff7c5c3bSVladimir Kondratiev * BACK. Pass it to the stack and wait.
140c888cdd4SVladimir Kondratiev */
141c888cdd4SVladimir Kondratiev if (r->first_time) {
142c888cdd4SVladimir Kondratiev r->first_time = false;
143c888cdd4SVladimir Kondratiev if (seq != r->head_seq_num) {
144ff7c5c3bSVladimir Kondratiev if (seq_less(seq, r->head_seq_num)) {
145ff7c5c3bSVladimir Kondratiev wil_err(wil,
146ff7c5c3bSVladimir Kondratiev "Error: frame with early sequence 0x%03x, should be 0x%03x. Waiting...\n",
147ff7c5c3bSVladimir Kondratiev seq, r->head_seq_num);
148ff7c5c3bSVladimir Kondratiev r->first_time = true;
149ff7c5c3bSVladimir Kondratiev wil_netif_rx_any(skb, ndev);
150ff7c5c3bSVladimir Kondratiev goto out;
151ff7c5c3bSVladimir Kondratiev }
152ff7c5c3bSVladimir Kondratiev wil_err(wil,
153ff7c5c3bSVladimir Kondratiev "Error: 1-st frame with wrong sequence 0x%03x, should be 0x%03x. Fixing...\n",
154ff7c5c3bSVladimir Kondratiev seq, r->head_seq_num);
155c888cdd4SVladimir Kondratiev r->head_seq_num = seq;
156c888cdd4SVladimir Kondratiev r->ssn = seq;
157c888cdd4SVladimir Kondratiev }
158c888cdd4SVladimir Kondratiev }
159c888cdd4SVladimir Kondratiev
160b4490f42SVladimir Kondratiev /* frame with out of date sequence number */
161b4490f42SVladimir Kondratiev if (seq_less(seq, r->head_seq_num)) {
162d5b1c32fSVladimir Kondratiev r->ssn_last_drop = seq;
16391a8edccSVladimir Kondratiev r->drop_old++;
16491a8edccSVladimir Kondratiev wil_dbg_txrx(wil, "Rx drop: old seq 0x%03x head 0x%03x\n",
16591a8edccSVladimir Kondratiev seq, r->head_seq_num);
166b4490f42SVladimir Kondratiev dev_kfree_skb(skb);
167b4490f42SVladimir Kondratiev goto out;
168b4490f42SVladimir Kondratiev }
169b4490f42SVladimir Kondratiev
170b4490f42SVladimir Kondratiev /*
171b4490f42SVladimir Kondratiev * If frame the sequence number exceeds our buffering window
172b4490f42SVladimir Kondratiev * size release some previous frames to make room for this one.
173b4490f42SVladimir Kondratiev */
174b4490f42SVladimir Kondratiev if (!seq_less(seq, r->head_seq_num + r->buf_size)) {
175b4490f42SVladimir Kondratiev hseq = seq_inc(seq_sub(seq, r->buf_size));
176b4490f42SVladimir Kondratiev /* release stored frames up to new head to stack */
1775bd60982SLior David wil_release_reorder_frames(ndev, r, hseq);
178b4490f42SVladimir Kondratiev }
179b4490f42SVladimir Kondratiev
180b4490f42SVladimir Kondratiev /* Now the new frame is always in the range of the reordering buffer */
181b4490f42SVladimir Kondratiev
182b4490f42SVladimir Kondratiev index = reorder_index(r, seq);
183b4490f42SVladimir Kondratiev
184b4490f42SVladimir Kondratiev /* check if we already stored this frame */
185b4490f42SVladimir Kondratiev if (r->reorder_buf[index]) {
18691a8edccSVladimir Kondratiev r->drop_dup++;
18791a8edccSVladimir Kondratiev wil_dbg_txrx(wil, "Rx drop: dup seq 0x%03x\n", seq);
188b4490f42SVladimir Kondratiev dev_kfree_skb(skb);
189b4490f42SVladimir Kondratiev goto out;
190b4490f42SVladimir Kondratiev }
191b4490f42SVladimir Kondratiev
192b4490f42SVladimir Kondratiev /*
193b4490f42SVladimir Kondratiev * If the current MPDU is in the right order and nothing else
194b4490f42SVladimir Kondratiev * is stored we can process it directly, no need to buffer it.
195b4490f42SVladimir Kondratiev * If it is first but there's something stored, we may be able
196b4490f42SVladimir Kondratiev * to release frames after this one.
197b4490f42SVladimir Kondratiev */
198b4490f42SVladimir Kondratiev if (seq == r->head_seq_num && r->stored_mpdu_num == 0) {
199b4490f42SVladimir Kondratiev r->head_seq_num = seq_inc(r->head_seq_num);
200b4490f42SVladimir Kondratiev wil_netif_rx_any(skb, ndev);
201b4490f42SVladimir Kondratiev goto out;
202b4490f42SVladimir Kondratiev }
203b4490f42SVladimir Kondratiev
204b4490f42SVladimir Kondratiev /* put the frame in the reordering buffer */
205b4490f42SVladimir Kondratiev r->reorder_buf[index] = skb;
206b4490f42SVladimir Kondratiev r->stored_mpdu_num++;
2075bd60982SLior David wil_reorder_release(ndev, r);
208b4490f42SVladimir Kondratiev
209b4490f42SVladimir Kondratiev out:
210bd33273bSVladimir Kondratiev spin_unlock(&sta->tid_rx_lock);
211b4490f42SVladimir Kondratiev }
212b4490f42SVladimir Kondratiev
213a8313341SVladimir Kondratiev /* process BAR frame, called in NAPI context */
wil_rx_bar(struct wil6210_priv * wil,struct wil6210_vif * vif,u8 cid,u8 tid,u16 seq)2145bd60982SLior David void wil_rx_bar(struct wil6210_priv *wil, struct wil6210_vif *vif,
2155bd60982SLior David u8 cid, u8 tid, u16 seq)
216a8313341SVladimir Kondratiev {
217a8313341SVladimir Kondratiev struct wil_sta_info *sta = &wil->sta[cid];
2185bd60982SLior David struct net_device *ndev = vif_to_ndev(vif);
219a8313341SVladimir Kondratiev struct wil_tid_ampdu_rx *r;
220a8313341SVladimir Kondratiev
221a8313341SVladimir Kondratiev spin_lock(&sta->tid_rx_lock);
222a8313341SVladimir Kondratiev
223a8313341SVladimir Kondratiev r = sta->tid_rx[tid];
224a8313341SVladimir Kondratiev if (!r) {
225a8313341SVladimir Kondratiev wil_err(wil, "BAR for non-existing CID %d TID %d\n", cid, tid);
226a8313341SVladimir Kondratiev goto out;
227a8313341SVladimir Kondratiev }
228a8313341SVladimir Kondratiev if (seq_less(seq, r->head_seq_num)) {
229a8313341SVladimir Kondratiev wil_err(wil, "BAR Seq 0x%03x preceding head 0x%03x\n",
230a8313341SVladimir Kondratiev seq, r->head_seq_num);
231a8313341SVladimir Kondratiev goto out;
232a8313341SVladimir Kondratiev }
2335bd60982SLior David wil_dbg_txrx(wil, "BAR: CID %d MID %d TID %d Seq 0x%03x head 0x%03x\n",
2345bd60982SLior David cid, vif->mid, tid, seq, r->head_seq_num);
2355bd60982SLior David wil_release_reorder_frames(ndev, r, seq);
236a8313341SVladimir Kondratiev
237a8313341SVladimir Kondratiev out:
238a8313341SVladimir Kondratiev spin_unlock(&sta->tid_rx_lock);
239a8313341SVladimir Kondratiev }
240a8313341SVladimir Kondratiev
wil_tid_ampdu_rx_alloc(struct wil6210_priv * wil,int size,u16 ssn)241b4490f42SVladimir Kondratiev struct wil_tid_ampdu_rx *wil_tid_ampdu_rx_alloc(struct wil6210_priv *wil,
242b4490f42SVladimir Kondratiev int size, u16 ssn)
243b4490f42SVladimir Kondratiev {
244b4490f42SVladimir Kondratiev struct wil_tid_ampdu_rx *r = kzalloc(sizeof(*r), GFP_KERNEL);
2458fe59627SVladimir Kondratiev
246b4490f42SVladimir Kondratiev if (!r)
247b4490f42SVladimir Kondratiev return NULL;
248b4490f42SVladimir Kondratiev
249b4490f42SVladimir Kondratiev r->reorder_buf =
250b4490f42SVladimir Kondratiev kcalloc(size, sizeof(struct sk_buff *), GFP_KERNEL);
25137f8d26dSDedy Lansky if (!r->reorder_buf) {
252b4490f42SVladimir Kondratiev kfree(r);
253b4490f42SVladimir Kondratiev return NULL;
254b4490f42SVladimir Kondratiev }
255b4490f42SVladimir Kondratiev
256b4490f42SVladimir Kondratiev r->ssn = ssn;
257b4490f42SVladimir Kondratiev r->head_seq_num = ssn;
258b4490f42SVladimir Kondratiev r->buf_size = size;
259b4490f42SVladimir Kondratiev r->stored_mpdu_num = 0;
260c888cdd4SVladimir Kondratiev r->first_time = true;
2611bd82ee0SDedy Lansky r->mcast_last_seq = U16_MAX;
262b4490f42SVladimir Kondratiev return r;
263b4490f42SVladimir Kondratiev }
264b4490f42SVladimir Kondratiev
wil_tid_ampdu_rx_free(struct wil6210_priv * wil,struct wil_tid_ampdu_rx * r)265b4490f42SVladimir Kondratiev void wil_tid_ampdu_rx_free(struct wil6210_priv *wil,
266b4490f42SVladimir Kondratiev struct wil_tid_ampdu_rx *r)
267b4490f42SVladimir Kondratiev {
268dea16eddSHamad Kadmany int i;
269dea16eddSHamad Kadmany
270b4490f42SVladimir Kondratiev if (!r)
271b4490f42SVladimir Kondratiev return;
272dea16eddSHamad Kadmany
273dea16eddSHamad Kadmany /* Do not pass remaining frames to the network stack - it may be
274dea16eddSHamad Kadmany * not expecting to get any more Rx. Rx from here may lead to
275dea16eddSHamad Kadmany * kernel OOPS since some per-socket accounting info was already
276dea16eddSHamad Kadmany * released.
277dea16eddSHamad Kadmany */
278dea16eddSHamad Kadmany for (i = 0; i < r->buf_size; i++)
279dea16eddSHamad Kadmany kfree_skb(r->reorder_buf[i]);
280dea16eddSHamad Kadmany
281b4490f42SVladimir Kondratiev kfree(r->reorder_buf);
282b4490f42SVladimir Kondratiev kfree(r);
283b4490f42SVladimir Kondratiev }
2843277213fSVladimir Kondratiev
2853277213fSVladimir Kondratiev /* ADDBA processing */
wil_agg_size(struct wil6210_priv * wil,u16 req_agg_wsize)2863277213fSVladimir Kondratiev static u16 wil_agg_size(struct wil6210_priv *wil, u16 req_agg_wsize)
2873277213fSVladimir Kondratiev {
2881b99197dSMaya Erez u16 max_agg_size = min_t(u16, wil->max_agg_wsize, wil->max_ampdu_size /
2893277213fSVladimir Kondratiev (mtu_max + WIL_MAX_MPDU_OVERHEAD));
2903277213fSVladimir Kondratiev
2913277213fSVladimir Kondratiev if (!req_agg_wsize)
2923277213fSVladimir Kondratiev return max_agg_size;
2933277213fSVladimir Kondratiev
2943277213fSVladimir Kondratiev return min(max_agg_size, req_agg_wsize);
2953277213fSVladimir Kondratiev }
2963277213fSVladimir Kondratiev
297361486b2SMaya Erez /* Block Ack - Rx side (recipient) */
wil_addba_rx_request(struct wil6210_priv * wil,u8 mid,u8 cid,u8 tid,u8 dialog_token,__le16 ba_param_set,__le16 ba_timeout,__le16 ba_seq_ctrl)298bf0353a6SAhmad Masri int wil_addba_rx_request(struct wil6210_priv *wil, u8 mid, u8 cid, u8 tid,
299bf0353a6SAhmad Masri u8 dialog_token, __le16 ba_param_set,
3003277213fSVladimir Kondratiev __le16 ba_timeout, __le16 ba_seq_ctrl)
301bd33273bSVladimir Kondratiev __acquires(&sta->tid_rx_lock) __releases(&sta->tid_rx_lock)
3023277213fSVladimir Kondratiev {
303361486b2SMaya Erez u16 param_set = le16_to_cpu(ba_param_set);
304361486b2SMaya Erez u16 agg_timeout = le16_to_cpu(ba_timeout);
305361486b2SMaya Erez u16 seq_ctrl = le16_to_cpu(ba_seq_ctrl);
3063277213fSVladimir Kondratiev struct wil_sta_info *sta;
307c5b3a658SAlexei Avshalom Lazar u16 agg_wsize;
3083277213fSVladimir Kondratiev /* bit 0: A-MSDU supported
3093277213fSVladimir Kondratiev * bit 1: policy (should be 0 for us)
3103277213fSVladimir Kondratiev * bits 2..5: TID
3113277213fSVladimir Kondratiev * bits 6..15: buffer size
3123277213fSVladimir Kondratiev */
313361486b2SMaya Erez u16 req_agg_wsize = WIL_GET_BITS(param_set, 6, 15);
3147be13fc3SGidon Studinski bool agg_amsdu = wil->use_enhanced_dma_hw &&
3157be13fc3SGidon Studinski wil->use_rx_hw_reordering &&
3167be13fc3SGidon Studinski test_bit(WMI_FW_CAPABILITY_AMSDU, wil->fw_capabilities) &&
3177be13fc3SGidon Studinski wil->amsdu_en && (param_set & BIT(0));
318361486b2SMaya Erez int ba_policy = param_set & BIT(1);
319361486b2SMaya Erez u16 ssn = seq_ctrl >> 4;
320382afc3dSVladimir Kondratiev struct wil_tid_ampdu_rx *r;
321361486b2SMaya Erez int rc = 0;
3223277213fSVladimir Kondratiev
323bd33273bSVladimir Kondratiev might_sleep();
3243277213fSVladimir Kondratiev
3253277213fSVladimir Kondratiev /* sanity checks */
326ddf7afddSAhmad Masri if (cid >= wil->max_assoc_sta) {
3273277213fSVladimir Kondratiev wil_err(wil, "BACK: invalid CID %d\n", cid);
328361486b2SMaya Erez rc = -EINVAL;
329361486b2SMaya Erez goto out;
3303277213fSVladimir Kondratiev }
3313277213fSVladimir Kondratiev
3323277213fSVladimir Kondratiev sta = &wil->sta[cid];
3333277213fSVladimir Kondratiev if (sta->status != wil_sta_connected) {
3343277213fSVladimir Kondratiev wil_err(wil, "BACK: CID %d not connected\n", cid);
335361486b2SMaya Erez rc = -EINVAL;
336361486b2SMaya Erez goto out;
3373277213fSVladimir Kondratiev }
3383277213fSVladimir Kondratiev
3393277213fSVladimir Kondratiev wil_dbg_wmi(wil,
340ff7c5c3bSVladimir Kondratiev "ADDBA request for CID %d %pM TID %d size %d timeout %d AMSDU%s policy %d token %d SSN 0x%03x\n",
341361486b2SMaya Erez cid, sta->addr, tid, req_agg_wsize, agg_timeout,
342361486b2SMaya Erez agg_amsdu ? "+" : "-", !!ba_policy, dialog_token, ssn);
3433277213fSVladimir Kondratiev
3443277213fSVladimir Kondratiev /* apply policies */
34552a45702SLior David if (req_agg_wsize == 0) {
34652a45702SLior David wil_dbg_misc(wil, "Suggest BACK wsize %d\n",
3471b99197dSMaya Erez wil->max_agg_wsize);
3481b99197dSMaya Erez agg_wsize = wil->max_agg_wsize;
34952a45702SLior David } else {
350c5b3a658SAlexei Avshalom Lazar agg_wsize = min_t(u16, wil->max_agg_wsize, req_agg_wsize);
35152a45702SLior David }
3523277213fSVladimir Kondratiev
3537be13fc3SGidon Studinski rc = wil->txrx_ops.wmi_addba_rx_resp(wil, mid, cid, tid, dialog_token,
354c5b3a658SAlexei Avshalom Lazar WLAN_STATUS_SUCCESS, agg_amsdu,
355c5b3a658SAlexei Avshalom Lazar agg_wsize, agg_timeout);
356c5b3a658SAlexei Avshalom Lazar if (rc) {
357c5b3a658SAlexei Avshalom Lazar wil_err(wil, "do not apply ba, rc(%d)\n", rc);
358361486b2SMaya Erez goto out;
359361486b2SMaya Erez }
3603277213fSVladimir Kondratiev
3613277213fSVladimir Kondratiev /* apply */
362686ab4f5SMaya Erez if (!wil->use_rx_hw_reordering) {
363382afc3dSVladimir Kondratiev r = wil_tid_ampdu_rx_alloc(wil, agg_wsize, ssn);
364bd33273bSVladimir Kondratiev spin_lock_bh(&sta->tid_rx_lock);
3653277213fSVladimir Kondratiev wil_tid_ampdu_rx_free(wil, sta->tid_rx[tid]);
366382afc3dSVladimir Kondratiev sta->tid_rx[tid] = r;
367bd33273bSVladimir Kondratiev spin_unlock_bh(&sta->tid_rx_lock);
368686ab4f5SMaya Erez }
369361486b2SMaya Erez
370361486b2SMaya Erez out:
371361486b2SMaya Erez return rc;
3723277213fSVladimir Kondratiev }
3733277213fSVladimir Kondratiev
374361486b2SMaya Erez /* BACK - Tx side (originator) */
wil_addba_tx_request(struct wil6210_priv * wil,u8 ringid,u16 wsize)375361486b2SMaya Erez int wil_addba_tx_request(struct wil6210_priv *wil, u8 ringid, u16 wsize)
3763277213fSVladimir Kondratiev {
377361486b2SMaya Erez u8 agg_wsize = wil_agg_size(wil, wsize);
378361486b2SMaya Erez u16 agg_timeout = 0;
37910590c6aSGidon Studinski struct wil_ring_tx_data *txdata = &wil->ring_tx_data[ringid];
380361486b2SMaya Erez int rc = 0;
3813a124ed6SVladimir Kondratiev
3823a124ed6SVladimir Kondratiev if (txdata->addba_in_progress) {
3833a124ed6SVladimir Kondratiev wil_dbg_misc(wil, "ADDBA for vring[%d] already in progress\n",
384361486b2SMaya Erez ringid);
385361486b2SMaya Erez goto out;
3863a124ed6SVladimir Kondratiev }
3873a124ed6SVladimir Kondratiev if (txdata->agg_wsize) {
3883a124ed6SVladimir Kondratiev wil_dbg_misc(wil,
389361486b2SMaya Erez "ADDBA for vring[%d] already done for wsize %d\n",
390361486b2SMaya Erez ringid, txdata->agg_wsize);
391361486b2SMaya Erez goto out;
3923a124ed6SVladimir Kondratiev }
3933a124ed6SVladimir Kondratiev txdata->addba_in_progress = true;
394e00243faSLior David rc = wmi_addba(wil, txdata->mid, ringid, agg_wsize, agg_timeout);
395361486b2SMaya Erez if (rc) {
396af3db60aSLazar Alexei wil_err(wil, "wmi_addba failed, rc (%d)", rc);
3973a124ed6SVladimir Kondratiev txdata->addba_in_progress = false;
3983a124ed6SVladimir Kondratiev }
3993a124ed6SVladimir Kondratiev
400361486b2SMaya Erez out:
401361486b2SMaya Erez return rc;
4023a124ed6SVladimir Kondratiev }
403