This prevents tx hangs or hiccups if pull-push
supporting firmware defines per-txq thresholds or
switches modes dynamically.

Fixes: 299468782d94 ("ath10k: implement wake_tx_queue")
Signed-off-by: Michal Kazior <michal.kaz...@tieto.com>
---
 drivers/net/wireless/ath/ath10k/mac.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/net/wireless/ath/ath10k/mac.c 
b/drivers/net/wireless/ath/ath10k/mac.c
index 74dd010a9188..4d51b92e67e0 100644
--- a/drivers/net/wireless/ath/ath10k/mac.c
+++ b/drivers/net/wireless/ath/ath10k/mac.c
@@ -3772,7 +3772,8 @@ void ath10k_mac_tx_push_pending(struct ath10k *ar)
 
                /* Prevent aggressive sta/tid taking over tx queue */
                max = 16;
-               while (max--) {
+               ret = 0;
+               while (ath10k_mac_tx_can_push(hw, txq) && max--) {
                        ret = ath10k_mac_tx_push_txq(hw, txq);
                        if (ret < 0)
                                break;
@@ -4015,14 +4016,13 @@ static void ath10k_mac_op_wake_tx_queue(struct 
ieee80211_hw *hw,
        struct ath10k *ar = hw->priv;
        struct ath10k_txq *artxq = (void *)txq->drv_priv;
 
-       if (ath10k_mac_tx_can_push(hw, txq)) {
-               spin_lock_bh(&ar->txqs_lock);
-               if (list_empty(&artxq->list))
-                       list_add_tail(&artxq->list, &ar->txqs);
-               spin_unlock_bh(&ar->txqs_lock);
+       spin_lock_bh(&ar->txqs_lock);
+       if (list_empty(&artxq->list))
+               list_add_tail(&artxq->list, &ar->txqs);
+       spin_unlock_bh(&ar->txqs_lock);
 
+       if (ath10k_mac_tx_can_push(hw, txq))
                tasklet_schedule(&ar->htt.txrx_compl_task);
-       }
 
        ath10k_htt_tx_txq_update(hw, txq);
 }
-- 
2.1.4


_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

Reply via email to