On 2/3/21 5:08 PM, Karthikeyan periyasamy wrote:
On 2021-02-04 00:59, Peter Oh wrote:
On 1/26/21 6:46 PM, Karthikeyan periyasamy wrote:
On 2020-12-19 00:53, Peter Oh wrote:
On 12/18/20 7:24 AM, Karthikeyan Periyasamy wrote:
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.
Can you add this comment in commit message?
sure
Also could you include the answer to "what conditions you observed the
throughput degradation?" in commit message too?
In all the scenario will see degradation. Its a general improvement.
It sounds good and it will better if you share the number before and
after the patch.
Thanks,
Peter
