ath9k: resume aggregation immediately after a hardware reset

PATCH New Felix Fietkau 1 participant 1 message Oct 23, 2010 Qualcomm Atheros ATH9K wireless driver

Share & tools

Felix Fietkau

author

2010-10-23 15:45 UTC #0 lore raw

PATCH 1/1 ath9k: resume aggregation immediately after a hardware reset

Since aggregation is usually triggered by tx completion, a hardware reset (because of beacon stuck, tx hang or baseband hang) can significantly delay the transmission of the next AMPDU (until the next tx completion event).
Fix this by rescheduling aggregation after such a reset.

Signed-off-by Felix Fietkau <[email protected]>

Cc [email protected]

---

drivers/net/wireless/ath/ath9k/main.c +2 -2

@@ -587,7 +587,7 @@ void ath_hw_check(struct work_struct *wo
 
 		msleep(1);
 	}
-	ath_reset(sc, false);
+	ath_reset(sc, true);
 
 out:
 	ath9k_ps_restore(sc);

@@ -605,7 +605,7 @@ void ath9k_tasklet(unsigned long data)
 	ath9k_ps_wakeup(sc);
 
 	if (status & ATH9K_INT_FATAL) {
-		ath_reset(sc, false);
+		ath_reset(sc, true);
 		ath9k_ps_restore(sc);
 		return;
 	}

drivers/net/wireless/ath/ath9k/xmit.c +1 -1

@@ -2163,7 +2163,7 @@ static void ath_tx_complete_poll_work(st
 		ath_print(ath9k_hw_common(sc->sc_ah), ATH_DBG_RESET,
 			  "tx hung, resetting the chip\n");
 		ath9k_ps_wakeup(sc);
-		ath_reset(sc, false);
+		ath_reset(sc, true);
 		ath9k_ps_restore(sc);
 	}

drivers/net/wireless/ath/ath9k/beacon.c +1 -1

@@ -370,7 +370,7 @@ void ath_beacon_tasklet(unsigned long da
 			ath_print(common, ATH_DBG_BSTUCK,
 				  "beacon is officially stuck\n");
 			sc->sc_flags |= SC_OP_TSF_RESET;
-			ath_reset(sc, false);
+			ath_reset(sc, true);
 		}
 
 		return;