On 2021-01-06 11:51, Toke Høiland-Jørgensen wrote: > Ryder Lee <ryder.lee@xxxxxxxxxxxx> writes: > >> The selected txq should be scheduled unconditionally if >> NL80211_EXT_FEATURE_AIRTIME_FAIRNESS is not set by driver. >> >> Also put the sta to the end of the active_txqs list if >> deficit is negative then move on to the next txq. > > Why is this needed? If the feature is not set, no airtime should ever be > accounted to the station, and so sta->airtime[txqi->txq.ac].deficit will > always be 0 - so you're just adding another check that doesn't actually > change the behaviour, aren't you? I think it might make sense to keep airtime reporting even when airtime fairness is disabled at run time, so this patch makes sense to me. Instead of this patch, the right place to deal with this would probably be ieee80211_sta_register_airtime. - Felix
