On 16/04/2014 18:00, Dave Taht wrote: > should I have said "de-protected"? in > > linux-3.14/drivers/net/wireless/ath/ath9k/xmit.c > if (info->flags & IEEE80211_TX_CTL_PS_RESPONSE) { > ath_txq_unlock(sc, txq); > txq = sc->tx.uapsdq; > ^^^^^^ > ath_txq_lock(sc, txq); > } else if (txctl->an && > Isn't the point here that you're potentially switching txq and so need to unlock the old one and then lock the new one? -- Robert Bradley