Felix Fietkau <nbd@xxxxxxxx> writes: > On 2018-09-20 21:05, Rajkumar Manoharan wrote: >> Per frame airtime estimation could be used to track outstanding airtime >> of each txq and can be used to throttle ieee80211_tx_dequeue(). This >> mechanism on its own will get us the queue limiting and latency >> reduction goodness for firmwares with deep queues. And for that it can >> be completely independent of the airtime fairness scheduler, which can >> use the after-tx-compl airtime information to presumably get more >> accurate fairness which includes retransmissions etc. > What about packets that get dequeued from the txq, but get freed by > ieee80211_free_txskb? > I think this may be a bit fragile, since if for any reason accounting > isn't perfect here, tx queues might stall. Yeah, I worry about that as well. I guess we basically have three avenues for fixing this? 1. Make sure packets are not freed after dequeue (probably not feasible). 2. Add a mechanism to get queues unstuck if accounting does deviate (a watchdog-type thing, or tie it to packet enqueue or something?). 3. Tie this back into the airtime fairness scheduler deficit, the same way Kan's original patch did. The problem with (3) is that we lose the ability to use a different source of airtime usage information for the fairness scheduler (without doing weird subtractions in the accounting). In particular, this would mean we can't account for retransmissions. So I'd prefer (2), I think. Anyone else with bright ideas? :) -Toke
