From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-x236.google.com (mail-wm0-x236.google.com [IPv6:2a00:1450:400c:c09::236]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by lists.bufferbloat.net (Postfix) with ESMTPS id BE9923B260 for ; Fri, 6 May 2016 01:27:13 -0400 (EDT) Received: by mail-wm0-x236.google.com with SMTP id n129so43959065wmn.1 for ; Thu, 05 May 2016 22:27:13 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tieto.com; s=google; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-transfer-encoding; bh=mQzgaGwp8i40/ieIU8Rd59znMHhzxMzGMaPByw2wcw0=; b=Z5P7RVEUD4Zstho5fgz5N47p6RkX127kr8GBGAzhlZMWfRVI1V5zAJPyFnvW6EqG62 4PZDBGozuKE0fQk+F1w2S2QurtCQ1yk2DM3gA8jYMaqaywrVmsJJjPVxVr5AB3c3ysxV ty5RW8ZOGWf5JwyxUVgwmP/Uql2txfa++Dl/o= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-transfer-encoding; bh=mQzgaGwp8i40/ieIU8Rd59znMHhzxMzGMaPByw2wcw0=; b=Pc19+m0QmWcIWGngDxrcdDsuBJc6kJpzxIAbGNDxTvVAu/o+yHkHRHieJbYeNrD4O7 0xElBUZYSPu0erz+0T1n9DN+u3ZPWxh5bqcSbi0XOTLzXJ9F//EatsCar6pKnkjNVZT6 yKOrFg+CrlgeZEjLoRzkbZR7SrQmQ4H4bjFm8/lBKIrt/Wp0eJe9ecfQjQWO+85BPAjc 1qwCIc+En7PynMezWw8yVncw5+fTqcvnipTqNDERDKGBuYW7ZUx67tu9j2DOg2Nd7MVa OGbkc+9XUeLaLI+hqbSSPJWCFX5BeDcdih91Y5d8tPhgqK6j+YPQwLyOycczMuxQB6XH lwwQ== X-Gm-Message-State: AOPr4FUxWb7XQFI6V5ziM7LCfur01Gr54CYvw+XbZOiRKQMx/6EgZrU2jbGFe+Vv0Qu7m+Lu4DFwnstrePTTLq8nmKsrudRJ6SES8eo/aiw4QJYZEzuzplzc+JJ97lxvNyq+PedD9G5mQOi+g6+xRgpbHR8gGvgDr234hA== MIME-Version: 1.0 X-Received: by 10.194.171.194 with SMTP id aw2mr20464483wjc.113.1462512432520; Thu, 05 May 2016 22:27:12 -0700 (PDT) Received: by 10.194.65.6 with HTTP; Thu, 5 May 2016 22:27:12 -0700 (PDT) In-Reply-To: References: <1460636302-31161-1-git-send-email-michal.kazior@tieto.com> <1462446039-1070-1-git-send-email-michal.kazior@tieto.com> <1462446039-1070-6-git-send-email-michal.kazior@tieto.com> Date: Fri, 6 May 2016 07:27:12 +0200 Message-ID: From: Michal Kazior To: Dave Taht Cc: linux-wireless , Johannes Berg , make-wifi-fast@lists.bufferbloat.net, "codel@lists.bufferbloat.net" , Avery Pennarun Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable X-DomainID: tieto.com Subject: Re: [Make-wifi-fast] [PATCHv4 5/5] mac80211: add debug knobs for codel 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: Fri, 06 May 2016 05:27:13 -0000 On 5 May 2016 at 17:21, Dave Taht wrote: > On Thu, May 5, 2016 at 4:00 AM, Michal Kazior w= rote: >> This adds a few debugfs entries to make it easier >> to test, debug and experiment. > > I might argue in favor of moving all these (inc the fq ones) into > their own dir, maybe "aqm" or "sqm". > > The mixture of read only stats and configuration vars is a bit confusing. > > Also in my testing of the previous patch, actually seeing the stats > get updated seemed to be highly async or inaccurate. For example, it > was obvious from the captures themselves that codel_ce_mark-ing was > happening, but the actual numbers out of wack with the mark seen or > fq_backlog seen. (I can go back to revisit this) That's kind of expected since all of these bits are exposed as separate debugfs entries/files. To avoid that it'd be necessary to provide a single debugfs entry/file whose contents are generated on open() while holding local->fq.lock. But then you could argue it should contain all per-sta-tid info as well (backlog, flows, drops) as well instead of having them in netdev*/stations/*/txqs. Hmm.. Micha=C5=82