On 2020-12-19 00:53, Peter Oh wrote:

On 12/18/20 7:24 AM, Karthikeyan Periyasamy wrote:

quoted

Tx descriptor search index field should be updated with hw peer id
and not by AST Hash. Incorrect search index causes throughput 
degradation
in all the platforms. so updated the search index field with hw peer 
id,
which is a common change applicable for all the platforms.

Tested-on: IPQ8074 hw2.0 AHB 
WLAN.HK.2.4.0.1-01492-QCAHKSWPL_SILICONZ-1

diff --git a/drivers/net/wireless/ath/ath11k/peer.c 

b/drivers/net/wireless/ath/ath11k/peer.c
index b69e7eb..f49abefa 100644

--- a/drivers/net/wireless/ath/ath11k/peer.c
+++ b/drivers/net/wireless/ath/ath11k/peer.c

@@ -309,7 +310,11 @@ int ath11k_peer_create(struct ath11k *ar, struct 

ath11k_vif *arvif,
    	peer->pdev_idx = ar->pdev_idx;
  	peer->sta = sta;
-	arvif->ast_hash = peer->ast_hash;
+
+	if (arvif->vif->type == NL80211_IFTYPE_STATION) {
+		arvif->ast_hash = peer->ast_hash;
+		arvif->ast_idx = peer->hw_peer_id;
+	}

How about non STATION type?

no need of configuring the ast_idx, ast_hash field for non station type. 
Its recommended by HW/FW team.

Thanks,
Karthikeyan P