From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.codeaurora.org (smtp.codeaurora.org [198.145.29.96]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.bufferbloat.net (Postfix) with ESMTPS id A61183B2A4 for ; Thu, 9 Nov 2017 20:06:47 -0500 (EST) Received: by smtp.codeaurora.org (Postfix, from userid 1000) id D67BA60866; Fri, 10 Nov 2017 01:06:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1510276006; bh=XtXCVpVupiOJLY6DBhZuWX513UVm2eBsfN6xhO/vEAk=; h=From:To:Cc:Subject:References:Date:In-Reply-To:From; b=gQdBVabj+i7jzAVa+VMpmBDBpoG4BUnyHOkpcnpSTC6FuB2paeMBAH+bGAul90ZtF LcN0IXNLvVJz5AgUxAjbIZqqfoYzbavIHgxizcJooG92XjyrGPTialX01O0Tfzu3JG KqBggACN+6TGBXSdrL+RqvnNQ1h9ezmTtNKQ4mrU= X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on pdx-caf-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.8 required=2.0 tests=ALL_TRUSTED,BAYES_00, DKIM_SIGNED,T_DKIM_INVALID autolearn=no autolearn_force=no version=3.4.0 Received: from purkki.adurom.net (purkki.adurom.net [80.68.90.206]) (using TLSv1.2 with cipher DHE-RSA-AES128-SHA (128/128 bits)) (No client certificate requested) (Authenticated sender: kvalo@smtp.codeaurora.org) by smtp.codeaurora.org (Postfix) with ESMTPSA id 6F802607A1; Fri, 10 Nov 2017 01:06:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=codeaurora.org; s=default; t=1510276006; bh=XtXCVpVupiOJLY6DBhZuWX513UVm2eBsfN6xhO/vEAk=; h=From:To:Cc:Subject:References:Date:In-Reply-To:From; b=gQdBVabj+i7jzAVa+VMpmBDBpoG4BUnyHOkpcnpSTC6FuB2paeMBAH+bGAul90ZtF LcN0IXNLvVJz5AgUxAjbIZqqfoYzbavIHgxizcJooG92XjyrGPTialX01O0Tfzu3JG KqBggACN+6TGBXSdrL+RqvnNQ1h9ezmTtNKQ4mrU= DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 6F802607A1 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=kvalo@codeaurora.org From: Kalle Valo To: Dave Taht Cc: Toke =?utf-8?Q?H=C3=B8iland-J=C3=B8rgensen?= , Rajkumar Manoharan , "make-wifi-fast\@lists.bufferbloat.net" , "linux-wireless\@vger.kernel.org" , ath10k@lists.infradead.org References: <20171109071945.11033-1-toke@toke.dk> <6112ae948ba2409da27709d01da567eb@NALASEXR01H.na.qualcomm.com> <87k1yzq9i8.fsf@toke.dk> Date: Fri, 10 Nov 2017 03:06:41 +0200 In-Reply-To: (Dave Taht's message of "Thu, 9 Nov 2017 16:29:03 -0800") Message-ID: <87tvy3lz6m.fsf@purkki.adurom.net> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.4 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Make-wifi-fast] [PATCH] ath10k: Re-enable TXQs for all devices 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, 10 Nov 2017 01:06:47 -0000 Adding ath10k list to get this discussion to the list archive. Dave Taht writes: > On Thu, Nov 9, 2017 at 4:10 PM, Toke H=C3=B8iland-J=C3=B8rgensen wrote: >> Rajkumar Manoharan writes: >> >>>> Commit 4ca1807815aa6801aaced7fdefa9edacc2521767 disables the use of the >>>> mac80211 TXQs for some devices because of a theoretical throughput >>>> regression. We have not seen this regression for a while now, so it sh= ould be >>>> safe to re-enable TXQs. >>>> >>>> Signed-off-by: Toke H=C3=B8iland-J=C3=B8rgensen >>>> --- >>>> This has been in LEDE trunk for a couple of months now with good resul= ts. >>>> >>> Toke, >>> >>> Good to know that the performance drop is not seen with the chips that = does not >>> have push-pull support. The issue was originally reported with ap152 + = qca988x >>> by community [1]. Hope this combination is also considered in LEDE. >> >> Ah, was that the original bug report? Thank you, I have not been able to >> find that anywhere! >> >> The issue that seems to point to has been fixed a while ago; I'll send >> and updated patch with a better commit message (also forgot to cc the >> ath10k list, I see). >> >> -Toke > > Hmm. I remember that thread. I thought we'd basically resolved that > issue (45% of the time spent in fq_codel_drop under udp flood), > back then, with eric adding the batch drop fix to fq_codel itself: > > See commit: https://osdn.net/projects/android-x86/scm/git/kernel/commits/= 9d18562a227874289fda8ca5d117d8f503f1dcca > > which fixed up the problem beautifully: > > https://lists.bufferbloat.net/pipermail/make-wifi-fast/2016-May/000590.ht= ml > > So if we've been carrying this darn patch for the ath10k vs something > that we'd actually fixed elsewhere in the stack, for over a year, > sigh. --=20 Kalle Valo