Browse Source

qcacmn: Modify the assert condition for stale frames

Stale frame's global time stamp is expected to be smaller than
all the frames in reo list, unless the frames in reo list are
marked as parallel rx wrt to the frames already delivered to
the upper layer.

CRs-Fixed: 3305693
Change-Id: I5a8222ddbf26ddef718bf38b307ef5f524ffef8c
Edayilliam Jayadev 2 years ago
parent
commit
43b4691c62
1 changed files with 3 additions and 3 deletions
  1. 3 3
      umac/cmn_services/mgmt_txrx/core/src/wlan_mgmt_txrx_rx_reo.c

+ 3 - 3
umac/cmn_services/mgmt_txrx/core/src/wlan_mgmt_txrx_rx_reo.c

@@ -2513,6 +2513,9 @@ mgmt_rx_reo_update_list(struct mgmt_rx_reo_list *reo_list,
 			break;
 		}
 
+		qdf_assert_always(!frame_desc->is_stale ||
+				  cur_entry->is_parallel_rx);
+
 		list_insertion_pos++;
 
 		status = mgmt_rx_reo_update_wait_count(
@@ -2526,9 +2529,6 @@ mgmt_rx_reo_update_list(struct mgmt_rx_reo_list *reo_list,
 			      ~MGMT_RX_REO_STATUS_WAIT_FOR_FRAME_ON_OTHER_LINKS;
 	}
 
-	if (frame_desc->is_stale)
-		qdf_assert_always(!list_insertion_pos);
-
 	if (frame_desc->type == MGMT_RX_REO_FRAME_DESC_HOST_CONSUMED_FRAME &&
 	    !frame_desc->is_stale && frame_desc->reo_required) {
 		if (least_greater_entry_found) {