Search Linux Wireless

Re: [PATCH] mac80211: check ATF flag in ieee80211_next_txq()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Wireless Regulations]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux