From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from s3.sipsolutions.net ([144.76.43.62]:44404 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726340AbeIERDr (ORCPT ); Wed, 5 Sep 2018 13:03:47 -0400 Message-ID: <1536150812.3528.15.camel@sipsolutions.net> (sfid-20180905_143347_255643_20AC8DB6) Subject: Re: [PATCH] mac80211: TDLS: fix skb queue/priority assignment From: Johannes Berg To: Toke =?ISO-8859-1?Q?H=F8iland-J=F8rgensen?= , linux-wireless@vger.kernel.org Cc: Felix Fietkau Date: Wed, 05 Sep 2018 14:33:32 +0200 In-Reply-To: <87efe8yvju.fsf@toke.dk> References: <20180905113402.7665-1-johannes@sipsolutions.net> <1536147453.3528.11.camel@sipsolutions.net> <87k1o05g04.fsf@toke.dk> <1536147683.3528.12.camel@sipsolutions.net> <1536147865.3528.14.camel@sipsolutions.net> <87efe8yvju.fsf@toke.dk> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Wed, 2018-09-05 at 14:32 +0200, Toke Høiland-Jørgensen wrote: > Johannes Berg writes: > > > On Wed, 2018-09-05 at 13:41 +0200, Johannes Berg wrote: > > > On Wed, 2018-09-05 at 13:40 +0200, Toke Høiland-Jørgensen wrote: > > > > > > > > Guess we'll have to deal with everything else if we ever move management > > > > frames onto the TXQ path as well... > > > > > > Depends on whether we care for management frame priorities or not ... so > > > far we haven't really. > > > > Actually, for the most part we have implemented that properly. Except > > for the TXQ I added for bufferable management ... oh well, I think we're > > the only user thereof now. > > > > I'm not sure we want to have a TXQ per TID for management, that seems > > overkill. But I'm also not sure how to solve this otherwise ... > > Graft it to an existing TXQ, similar to how the fragments queue is used > now? Saves a TXQ at the expense of having to special-case it... The problem isn't so much how we handle it in mac80211 for the queueing, but how we deal with things like A-MSDU and how we present it to the driver ... for iwlwifi at least we'd really like to have only data frames so we can map it directly to the hardware queue ... johannes