On 07/28/2014 02:11 PM, Felix Fietkau wrote: > On 2014-07-28 22:54, Ben Greear wrote: >> On 07/23/2014 06:40 AM, Felix Fietkau wrote: >>> If an aggregation session fails, frames still end up in the driver queue >>> with IEEE80211_TX_CTL_AMPDU set. >>> This causes tx for the affected station/tid to stall, since >>> ath_tx_get_tid_subframe returning packets to send. >>> >>> Fix this by clearing IEEE80211_TX_CTL_AMPDU as long as no aggregation >>> session is running. >> >> Did this and your previous patch about 'pending tx frames accounting' >> fix the issues reported? > Yes. > >> They at least do not fix the problems I see where tx appears to hang, >> but my setup is pretty 'special'.... > Interesting. Maybe you should ask Antonio for an updated version of the > patch that he used to debug this issue. If you give me the output of it > while it's locked up, I might be able to figure out what's going on in > your setup. What is the easiest upstream tree for you to debug? Thanks, Ben > > - Felix > -- > To unsubscribe from this list: send the line "unsubscribe linux-wireless" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- Ben Greear <greearb@xxxxxxxxxxxxxxx> Candela Technologies Inc http://www.candelatech.com -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
