Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e6c70c23 authored by Karthik Kantamneni's avatar Karthik Kantamneni Committed by Madan Koyyalamudi
Browse files

qcacmn: Skip sending WMI REORDER_QUEUE_SETUP for BAR frames 

When host receives BAR frames with OOR/2K jump in sequence
number it updates REO HW with new starting sequence number
and requests F.W to setup peer reorder queue. But REORDER_QUEUE_SETUP
is already requested to F.W as part of initial add BA in host.

So while handling OOR/2K jump BAR frames don't send
REORDER_QUEUE_SETUP to F.W

Change-Id: Idc3657e901dc792f699b53c45e4ae5133d286891
CRs-Fixed: 2981280
parent 2471eee1

dp/wifi3.0/dp_internal.h

+3 −1
Original line number Diff line number Diff line
@@ -2311,11 +2311,13 @@ dp_get_pdev_from_soc_pdev_id_wifi3(struct dp_soc *soc, 
 * @tid: TID

 * @ba_window_size: BlockAck window size

 * @start_seq: Starting sequence number

 * @bar_update: BAR update triggered

 *

 * Return: QDF_STATUS code

 */

QDF_STATUS dp_rx_tid_update_wifi3(struct dp_peer *peer, int tid, uint32_t

					 ba_window_size, uint32_t start_seq);

					 ba_window_size, uint32_t start_seq,

					 bool bar_update);



/**

 * dp_get_peer_mac_list(): function to get peer mac list of vdev

dp/wifi3.0/dp_peer.c

+11 −9
Original line number Diff line number Diff line
@@ -2325,7 +2325,8 @@ static bool dp_get_peer_vdev_roaming_in_progress(struct dp_peer *peer) 
}



QDF_STATUS dp_rx_tid_update_wifi3(struct dp_peer *peer, int tid, uint32_t

					 ba_window_size, uint32_t start_seq)

					 ba_window_size, uint32_t start_seq,

					 bool bar_update)

{

	struct dp_rx_tid *rx_tid = &peer->rx_tid[tid];

	struct dp_soc *soc = peer->vdev->pdev->soc;
@@ -2357,7 +2358,7 @@ QDF_STATUS dp_rx_tid_update_wifi3(struct dp_peer *peer, int tid, uint32_t 
	if (dp_get_peer_vdev_roaming_in_progress(peer))

		return QDF_STATUS_E_PERM;



	if (soc->cdp_soc.ol_ops->peer_rx_reorder_queue_setup)

	if (soc->cdp_soc.ol_ops->peer_rx_reorder_queue_setup && !bar_update)

		soc->cdp_soc.ol_ops->peer_rx_reorder_queue_setup(

			soc->ctrl_psoc, peer->vdev->pdev->pdev_id,

			peer->vdev->vdev_id, peer->mac_addr.raw,
@@ -2548,7 +2549,7 @@ QDF_STATUS dp_rx_tid_setup_wifi3(struct dp_peer *peer, int tid, 
	rx_tid->ba_win_size = ba_window_size;

	if (rx_tid->hw_qdesc_vaddr_unaligned)

		return dp_rx_tid_update_wifi3(peer, tid, ba_window_size,

			start_seq);

			start_seq, false);

	rx_tid->delba_tx_status = 0;

	rx_tid->ppdu_id_2k = 0;

	rx_tid->num_of_addba_req = 0;
@@ -3297,7 +3298,7 @@ int dp_addba_resp_tx_completion_wifi3(struct cdp_soc_t *cdp_soc, 
	if (status) {

		rx_tid->num_addba_rsp_failed++;

		dp_rx_tid_update_wifi3(peer, tid, 1,

				       IEEE80211_SEQ_MAX);

				       IEEE80211_SEQ_MAX, false);

		rx_tid->ba_status = DP_RX_BA_INACTIVE;

		qdf_spin_unlock_bh(&rx_tid->tid_lock);

		dp_err("RxTid- %d addba rsp tx completion failed", tid);
@@ -3324,7 +3325,8 @@ int dp_addba_resp_tx_completion_wifi3(struct cdp_soc_t *cdp_soc, 


	if (dp_rx_tid_update_wifi3(peer, tid,

				   rx_tid->ba_win_size,

				   rx_tid->startseqnum)) {

				   rx_tid->startseqnum,

				   false)) {

		dp_err("Failed update REO SSN");

	}
@@ -3493,7 +3495,7 @@ int dp_addba_requestprocess_wifi3(struct cdp_soc_t *cdp_soc, 
	rx_tid->num_of_addba_req++;

	if ((rx_tid->ba_status == DP_RX_BA_ACTIVE &&

	     rx_tid->hw_qdesc_vaddr_unaligned)) {

		dp_rx_tid_update_wifi3(peer, tid, 1, IEEE80211_SEQ_MAX);

		dp_rx_tid_update_wifi3(peer, tid, 1, IEEE80211_SEQ_MAX, false);

		rx_tid->ba_status = DP_RX_BA_INACTIVE;

		peer->active_ba_session_cnt--;

		QDF_TRACE(QDF_MODULE_ID_DP, QDF_TRACE_LEVEL_DEBUG,
@@ -3632,7 +3634,7 @@ int dp_delba_process_wifi3(struct cdp_soc_t *cdp_soc, uint8_t *peer_mac, 
	 */

	rx_tid->delba_rcode = reasoncode;

	rx_tid->num_of_delba_req++;

	dp_rx_tid_update_wifi3(peer, tid, 1, IEEE80211_SEQ_MAX);

	dp_rx_tid_update_wifi3(peer, tid, 1, IEEE80211_SEQ_MAX, false);



	rx_tid->ba_status = DP_RX_BA_INACTIVE;

	peer->active_ba_session_cnt--;
@@ -3695,12 +3697,12 @@ int dp_delba_tx_completion_wifi3(struct cdp_soc_t *cdp_soc, uint8_t *peer_mac, 
		rx_tid->delba_tx_status = 0;

	}

	if (rx_tid->ba_status == DP_RX_BA_ACTIVE) {

		dp_rx_tid_update_wifi3(peer, tid, 1, IEEE80211_SEQ_MAX);

		dp_rx_tid_update_wifi3(peer, tid, 1, IEEE80211_SEQ_MAX, false);

		rx_tid->ba_status = DP_RX_BA_INACTIVE;

		peer->active_ba_session_cnt--;

	}

	if (rx_tid->ba_status == DP_RX_BA_IN_PROGRESS) {

		dp_rx_tid_update_wifi3(peer, tid, 1, IEEE80211_SEQ_MAX);

		dp_rx_tid_update_wifi3(peer, tid, 1, IEEE80211_SEQ_MAX, false);

		rx_tid->ba_status = DP_RX_BA_INACTIVE;

	}

	qdf_spin_unlock_bh(&rx_tid->tid_lock);

dp/wifi3.0/dp_rx_err.c

+2 −1
Original line number Diff line number Diff line
@@ -819,7 +819,8 @@ void dp_rx_err_handle_bar(struct dp_soc *soc, 


	status = dp_rx_tid_update_wifi3(peer, tid,

					peer->rx_tid[tid].ba_win_size,

					start_seq_num);

					start_seq_num,

					true);

	if (status != QDF_STATUS_SUCCESS) {

		dp_err_rl("failed to handle bar frame update rx tid");

		DP_STATS_INC(soc, rx.err.bar_handle_fail_count, 1);