Forráskód Böngészése

qcacmn: OFDMA counter get increased when num_msdu 0

Sometimes we receive num_msdu as 0 with ru_tones and ru_start
which cause OFDMA counter to get increament

Change-Id: Ib099a2143799551266e94bc9b8d281237092a46e
nobelj 6 éve
szülő
commit
7966633c14
2 módosított fájl, 10 hozzáadás és 13 törlés
  1. 0 4
      dp/inc/cdp_txrx_stats_struct.h
  2. 10 9
      dp/wifi3.0/dp_htt.c

+ 0 - 4
dp/inc/cdp_txrx_stats_struct.h

@@ -54,10 +54,6 @@
 #define MAX_RECEPTION_TYPES 4
 
 #define MAX_TRANSMIT_TYPES	9
-#define SU_TX			0
-#define MUMIMO_TX		1
-#define MUOFDMA_TX		2
-#define MUMIMO_OFDMA_TX		3
 
 #define MAX_USER_POS		8
 #define MAX_MU_GROUP_ID		64

+ 10 - 9
dp/wifi3.0/dp_htt.c

@@ -179,7 +179,8 @@ static void dp_tx_stats_update(struct dp_soc *soc, struct dp_peer *peer,
 	if (soc->process_tx_status)
 		return;
 
-	if (ppdu->mu_group_id <= MAX_MU_GROUP_ID && ppdu->ppdu_type != SU_TX) {
+	if (ppdu->mu_group_id <= MAX_MU_GROUP_ID &&
+	    ppdu->ppdu_type != HTT_PPDU_STATS_PPDU_TYPE_SU) {
 		if (unlikely(!(ppdu->mu_group_id & (MAX_MU_GROUP_ID - 1))))
 			QDF_TRACE(QDF_MODULE_ID_TXRX, QDF_TRACE_LEVEL_ERROR,
 				  "mu_group_id out of bound!!\n");
@@ -188,28 +189,28 @@ static void dp_tx_stats_update(struct dp_soc *soc, struct dp_peer *peer,
 				     (ppdu->user_pos + 1));
 	}
 
-	if (ppdu->ppdu_type == MUOFDMA_TX ||
-	    ppdu->ppdu_type == MUMIMO_OFDMA_TX) {
+	if (ppdu->ppdu_type == HTT_PPDU_STATS_PPDU_TYPE_MU_OFDMA ||
+	    ppdu->ppdu_type == HTT_PPDU_STATS_PPDU_TYPE_MU_MIMO_OFDMA) {
 		DP_STATS_UPD(peer, tx.ru_tones, ppdu->ru_tones);
 		DP_STATS_UPD(peer, tx.ru_start, ppdu->ru_start);
 		switch (ppdu->ru_tones) {
 		case RU_26:
-			DP_STATS_INC(peer, tx.ru_loc[0], 1);
+			DP_STATS_INC(peer, tx.ru_loc[0], num_msdu);
 		break;
 		case RU_52:
-			DP_STATS_INC(peer, tx.ru_loc[1], 1);
+			DP_STATS_INC(peer, tx.ru_loc[1], num_msdu);
 		break;
 		case RU_106:
-			DP_STATS_INC(peer, tx.ru_loc[2], 1);
+			DP_STATS_INC(peer, tx.ru_loc[2], num_msdu);
 		break;
 		case RU_242:
-			DP_STATS_INC(peer, tx.ru_loc[3], 1);
+			DP_STATS_INC(peer, tx.ru_loc[3], num_msdu);
 		break;
 		case RU_484:
-			DP_STATS_INC(peer, tx.ru_loc[4], 1);
+			DP_STATS_INC(peer, tx.ru_loc[4], num_msdu);
 		break;
 		case RU_996:
-			DP_STATS_INC(peer, tx.ru_loc[5], 1);
+			DP_STATS_INC(peer, tx.ru_loc[5], num_msdu);
 		break;
 		}
 	}