Johannes Berg <johannes@xxxxxxxxxxxxxxxx> writes: > On Wed, 2018-09-05 at 13:34 +0200, Johannes Berg wrote: >> From: Johannes Berg <johannes.berg@xxxxxxxxx> >> >> If the TDLS setup happens over a connection to an AP that >> doesn't have QoS, we nevertheless assign a non-zero TID >> (skb->priority) and queue mapping, which may confuse us or >> drivers later. >> >> Fix it by just assigning the special skb->priority and then >> using ieee80211_select_queue() just like other data frames >> would go through. >> >> Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx> >> --- >> OK, this addresses the case I was worried about - perhaps >> better take this than the txq patch? (This is just you talking to yourself, right? :)) > I really only found this. All the other cases aren't doing real data > frames, only null-data and management. So I guess for those we're OK, > and this was the only special case after all. SGTM. Guess we'll have to deal with everything else if we ever move management frames onto the TXQ path as well... -Toke