From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.toke.dk (mail.toke.dk [52.28.52.200]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.bufferbloat.net (Postfix) with ESMTPS id 2512E3CB40 for ; Wed, 21 Feb 2018 06:00:18 -0500 (EST) From: Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=toke.dk; s=20161023; t=1519210816; bh=outWA7Ru7U2reJHM+UFYhj8XFOgnTAV6G4DnEMTgw2E=; h=From:To:Subject:In-Reply-To:References:Date:From; b=PSCzO66dKLnd/+L/3xRcPHVfAbTaJu+TNOQB/U9BjEulLnWTkL3IuABfweWAC/5ZF xYUWN1vycWBZXpuMScwL6obJve7r9jP7nuuPZlCjOza8frFQKbQMZy2jbV6mFwVU5J adwBbp57mhjGfpVC5aMLHY/C09D4HV9svsgiRK0LVg0akS5BL58nztrTnMbkrQhtgy yADAi9bhHL0ME5Fo1nMZ4B5kqsvh1/yIMGLl45fuzDHwOUyuYujjxuF2raWBiwgnjV Kn3oKJIfPNDoJEdMFPSa29pOxLpw9zStkDrNDYZd7ypbayvqgLLsLyZkBpgJTzuVR4 sBy6NJnZem6hg== To: Arend van Spriel , make-wifi-fast@lists.bufferbloat.net, linux-wireless@vger.kernel.org In-Reply-To: <5A8D3445.3040300@broadcom.com> References: <20180219170224.14816-1-toke@toke.dk> <5A8D3445.3040300@broadcom.com> Date: Wed, 21 Feb 2018 12:00:16 +0100 X-Clacks-Overhead: GNU Terry Pratchett Message-ID: <87bmgiboan.fsf@toke.dk> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Make-wifi-fast] [PATCH 0/3] Export TXQ parameters and statistics via nl80211 X-BeenThere: make-wifi-fast@lists.bufferbloat.net X-Mailman-Version: 2.1.20 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 21 Feb 2018 11:00:18 -0000 Arend van Spriel writes: > On 2/19/2018 6:02 PM, Toke H=C3=B8iland-J=C3=B8rgensen wrote: >> This series adds TXQ parameters and statistics that were previously only >> available through debugfs to the nl80211 userspace interface and the >> cfg80211 kernel interface. Patches for iw to print the statistics and >> change the settings are included. > > So what is your motivation for having this exposed through nl80211. > For the average end-user the stats and parameters are fairly fuzzy. Two reasons, basically: 1. Visibility and statistics; this is basically the same information that is available at the qdisc layer (with `tc -s qdisc`), but which has been missing on WiFi interfaces ever sine we switched to the TXQ structure. Having this available has been quite valuable for debugging qdisc setups on wired links, and it's not always feasible to ask users to recompile their kernels with debugfs enabled. 2. Having visibility into the queues from userspace makes it possible to make decisions based upon (e.g.) which stations are currently backlogged. I'm working on a "policy mode" for the airtime fairness scheduler which will use this capability. > So can we expect some manual in which is described what parameter > should be tweaked based on the retrieved statistics. Heh, not sure I'll promise a whole manual, but I am happy to write a blog post (or wiki page if that's better) explaining what these values mean and what insight one might gain from them. > Also do you intend to remove the debugfs method? Seems a bit redundant > to have two mostly identical interfaces in place. Yes, I am planning to do that in a separate patch; I also have some patches pending that changes the airtime scheduler and adds airtime statistics to nl80211. So I am planning to send a cleanup patch once all that is in place (and I've had time to change my tools that are currently parsing debugfs :)). >> Wasn't sure whether to include the updates to nl80211.h in the iw >> patchset, so I didn't :) > > Updating the nl80211.h in iw should be done with a copy from the > kernel so it has to wait for the kernel patch to be applied (which > tree I am unsure). Gotcha. Sort of guessed that from the commit history, but wasn't sure. Thanks for confirming my suspicions :) -Toke