qca-wifi: Rectify nbuf copy expand in TX Capture feature
Replace skb_copy_expand with qdf_nbuf_copy_expand to correctly get offset for expand length. Additionally, loop over all elements in nbuf_ppdu_desc_list instead of only checking once. Change-Id: If9aed3c7c97354d912923ca25c8c52f1db27dac1 CRs-Fixed: 2620672
This commit is contained in:

committed by
Gerrit - the friendly Code Review server

parent
7732480801
commit
2a21448971
@@ -2125,13 +2125,16 @@ dp_tx_mon_proc_pending_ppdus(struct dp_pdev *pdev, struct dp_tx_tid *tx_tid,
|
||||
/* Find missing mpdus from current schedule list */
|
||||
ppdu_cnt = 0;
|
||||
|
||||
if (!nbuf_ppdu_desc_list[ppdu_cnt]) {
|
||||
while (!nbuf_ppdu_desc_list[ppdu_cnt]) {
|
||||
ppdu_cnt++;
|
||||
if (ppdu_cnt < ppdu_desc_cnt)
|
||||
continue;
|
||||
break;
|
||||
}
|
||||
|
||||
if (ppdu_cnt == ppdu_desc_cnt)
|
||||
break;
|
||||
|
||||
ppdu_desc = (struct cdp_tx_completion_ppdu *)qdf_nbuf_data(
|
||||
pend_ppdu);
|
||||
|
||||
@@ -2577,8 +2580,8 @@ insert_mgmt_buf_to_queue:
|
||||
(struct cdp_tx_completion_ppdu *)
|
||||
qdf_nbuf_data(nbuf_retry_ppdu);
|
||||
tmp_mgmt_ctl_nbuf =
|
||||
skb_copy_expand(mgmt_ctl_nbuf,
|
||||
0, 0, GFP_ATOMIC);
|
||||
qdf_nbuf_copy_expand(mgmt_ctl_nbuf,
|
||||
0, 0);
|
||||
|
||||
dp_update_tx_cap_info(pdev, nbuf_retry_ppdu,
|
||||
&tx_capture_info, true);
|
||||
@@ -2801,7 +2804,6 @@ dp_check_ppdu_and_deliver(struct dp_pdev *pdev,
|
||||
uint32_t num_mpdu;
|
||||
uint16_t start_seq, seq_no = 0;
|
||||
int i;
|
||||
uint32_t len;
|
||||
qdf_nbuf_t mpdu_nbuf;
|
||||
struct dp_peer *peer;
|
||||
uint8_t type;
|
||||
@@ -2858,9 +2860,6 @@ dp_check_ppdu_and_deliver(struct dp_pdev *pdev,
|
||||
|
||||
num_mpdu = ppdu_desc->user[0].mpdu_success;
|
||||
|
||||
/* get length */
|
||||
len = qdf_nbuf_queue_len(&ppdu_desc->mpdu_q);
|
||||
|
||||
/* ba_size is updated in BA bitmap TLVs, which are not received
|
||||
* in case of non-QoS TID.
|
||||
*/
|
||||
|
Reference in New Issue
Block a user