Search Linux Wireless

Re: [PATCH] ath11k: Update tx descriptor search index properly

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2021-02-04 09:07, Peter Oh wrote:
On 2/3/21 7:17 PM, Karthikeyan periyasamy wrote:
On 2021-02-04 08:38, Peter Oh wrote:
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.

I have seen 10% increase in WDS scenario.

Thanks,
Karthikeyan P

Thank you for the sharing.

BTW, all such information should go to commit message.

Peter

I update in the commit message

Thanks,
Karthikeyan P



[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Wireless Regulations]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux