diff for duplicates of <87mutue4y8.fsf@toke.dk> diff --git a/a/1.txt b/N1/1.txt index dab90fa..964ea9d 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -2,22 +2,24 @@ Wen Gong <wgong@qti.qualcomm.com> writes: >> -----Original Message----- >> From: ath10k <ath10k-bounces@lists.infradead.org> On Behalf Of Toke ->> Høiland-Jørgensen +>> H=C3=B8iland-J=C3=B8rgensen >> Sent: Wednesday, August 8, 2018 6:44 PM >> To: Wen Gong <wgong@codeaurora.org>; ath10k@lists.infradead.org; >> johannes@sipsolutions.net >> Cc: linux-wireless@vger.kernel.org ->> Subject: Re: [PATCH v2 2/2] ath10k: Set sk_pacing_shift to 6 for 11AC WiFi +>> Subject: Re: [PATCH v2 2/2] ath10k: Set sk_pacing_shift to 6 for 11AC Wi= +Fi >> chips ->> +>>=20 >> Wen Gong <wgong@codeaurora.org> writes: ->> +>>=20 >> > Upstream kernel has an interface to help adjust sk_pacing_shift to >> > help improve TCP UL throughput. >> > The sk_pacing_shift is 8 in mac80211, this is based on test with 11N >> > WiFi chips with ath9k. For QCA6174/QCA9377 PCI 11AC chips, the 11AC >> > VHT80 TCP UL throughput testing result shows 6 is the optimal. ->> > Overwrite the sk_pacing_shift to 6 in ath10k driver for QCA6174/9377 PCI. +>> > Overwrite the sk_pacing_shift to 6 in ath10k driver for QCA6174/9377 P= +CI. >> > >> > Tested with QCA6174 PCI with firmware >> > WLAN.RM.4.4.1-00109-QCARMSWPZ-1, but this will also affect QCA9377 @@ -46,25 +48,30 @@ Wen Gong <wgong@qti.qualcomm.com> writes: >> > default(262144)+5 Streams 598 >> > changed(2621440)+1 Stream 598 >> > changed(2621440)+2 Streams 601 ->> ->> You still haven't provided any latency numbers for these tests, which makes ->> it impossible to verify that setting sk_pacing_shift to 6 is the right tradeoff. ->> ->> As I said before, from your numbers I suspect the right setting is actually 7, +>>=20 +>> You still haven't provided any latency numbers for these tests, which ma= +kes +>> it impossible to verify that setting sk_pacing_shift to 6 is the right t= +radeoff. +>>=20 +>> As I said before, from your numbers I suspect the right setting is actua= +lly 7, >> which would be 10-20ms less latency under load; way more important than >> ~50 Mbps... ->> +>>=20 > Hi Toke, > Could you give the command line for the latency test? > https://flent.org/intro.html#quick-start > I used the command but test failed: -> flent tcp_download -p 1 -l 60 -H 192.168.1.5 -t text-to-be-included-in-plot -o file1.png +> flent tcp_download -p 1 -l 60 -H 192.168.1.5 -t text-to-be-included-in-pl= +ot -o file1.png > error loading plotter: unable to find plot configuration "1" Try something like: -flent -H 192.168.1.5 -t "sk_pacing_shift 7" tcp_nup --test-parameter upload_streams=1 +flent -H 192.168.1.5 -t "sk_pacing_shift 7" tcp_nup --test-parameter upload= +_streams=3D1 This will note the value of sk_pacing_shift you are testing in the data @@ -85,8 +92,3 @@ are also very welcome to send me the .flent.gz data files and I'll take a look to make sure everything looks reasonable :) -Toke - -_______________________________________________ -ath10k mailing list -ath10k@lists.infradead.org -http://lists.infradead.org/mailman/listinfo/ath10k diff --git a/a/content_digest b/N1/content_digest index ac2c3c0..9dc5986 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -7,31 +7,33 @@ "Date\0Fri, 10 Aug 2018 15:17:35 +0200\0" "To\0Wen Gong <wgong@qti.qualcomm.com>" Wen Gong <wgong@codeaurora.org> - ath10k@lists.infradead.org <ath10k@lists.infradead.org> - " johannes@sipsolutions.net <johannes@sipsolutions.net>\0" - "Cc\0linux-wireless@vger.kernel.org <linux-wireless@vger.kernel.org>\0" + " ath10k\\@lists.infradead.org <ath10k@lists.infradead.org>" + " johannes\\@sipsolutions.net <johannes@sipsolutions.net>\0" + "Cc\0linux-wireless\\@vger.kernel.org <linux-wireless@vger.kernel.org>\0" "\00:1\0" "b\0" "Wen Gong <wgong@qti.qualcomm.com> writes:\n" "\n" ">> -----Original Message-----\n" ">> From: ath10k <ath10k-bounces@lists.infradead.org> On Behalf Of Toke\n" - ">> H\303\270iland-J\303\270rgensen\n" + ">> H=C3=B8iland-J=C3=B8rgensen\n" ">> Sent: Wednesday, August 8, 2018 6:44 PM\n" ">> To: Wen Gong <wgong@codeaurora.org>; ath10k@lists.infradead.org;\n" ">> johannes@sipsolutions.net\n" ">> Cc: linux-wireless@vger.kernel.org\n" - ">> Subject: Re: [PATCH v2 2/2] ath10k: Set sk_pacing_shift to 6 for 11AC WiFi\n" + ">> Subject: Re: [PATCH v2 2/2] ath10k: Set sk_pacing_shift to 6 for 11AC Wi=\n" + "Fi\n" ">> chips\n" - ">> \n" + ">>=20\n" ">> Wen Gong <wgong@codeaurora.org> writes:\n" - ">> \n" + ">>=20\n" ">> > Upstream kernel has an interface to help adjust sk_pacing_shift to\n" ">> > help improve TCP UL throughput.\n" ">> > The sk_pacing_shift is 8 in mac80211, this is based on test with 11N\n" ">> > WiFi chips with ath9k. For QCA6174/QCA9377 PCI 11AC chips, the 11AC\n" ">> > VHT80 TCP UL throughput testing result shows 6 is the optimal.\n" - ">> > Overwrite the sk_pacing_shift to 6 in ath10k driver for QCA6174/9377 PCI.\n" + ">> > Overwrite the sk_pacing_shift to 6 in ath10k driver for QCA6174/9377 P=\n" + "CI.\n" ">> >\n" ">> > Tested with QCA6174 PCI with firmware\n" ">> > WLAN.RM.4.4.1-00109-QCARMSWPZ-1, but this will also affect QCA9377\n" @@ -60,25 +62,30 @@ ">> > default(262144)+5 Streams 598\n" ">> > changed(2621440)+1 Stream 598\n" ">> > changed(2621440)+2 Streams 601\n" - ">> \n" - ">> You still haven't provided any latency numbers for these tests, which makes\n" - ">> it impossible to verify that setting sk_pacing_shift to 6 is the right tradeoff.\n" - ">> \n" - ">> As I said before, from your numbers I suspect the right setting is actually 7,\n" + ">>=20\n" + ">> You still haven't provided any latency numbers for these tests, which ma=\n" + "kes\n" + ">> it impossible to verify that setting sk_pacing_shift to 6 is the right t=\n" + "radeoff.\n" + ">>=20\n" + ">> As I said before, from your numbers I suspect the right setting is actua=\n" + "lly 7,\n" ">> which would be 10-20ms less latency under load; way more important than\n" ">> ~50 Mbps...\n" - ">> \n" + ">>=20\n" "> Hi Toke,\n" "> Could you give the command line for the latency test?\n" "> https://flent.org/intro.html#quick-start\n" "> I used the command but test failed:\n" - "> flent tcp_download -p 1 -l 60 -H 192.168.1.5 -t text-to-be-included-in-plot -o file1.png\n" + "> flent tcp_download -p 1 -l 60 -H 192.168.1.5 -t text-to-be-included-in-pl=\n" + "ot -o file1.png\n" "> error loading plotter: unable to find plot configuration \"1\"\n" "\n" "Try something like:\n" "\n" "\n" - "flent -H 192.168.1.5 -t \"sk_pacing_shift 7\" tcp_nup --test-parameter upload_streams=1\n" + "flent -H 192.168.1.5 -t \"sk_pacing_shift 7\" tcp_nup --test-parameter upload=\n" + "_streams=3D1\n" "\n" "\n" "This will note the value of sk_pacing_shift you are testing in the data\n" @@ -98,11 +105,6 @@ "are also very welcome to send me the .flent.gz data files and I'll take\n" "a look to make sure everything looks reasonable :)\n" "\n" - "-Toke\n" - "\n" - "_______________________________________________\n" - "ath10k mailing list\n" - "ath10k@lists.infradead.org\n" - http://lists.infradead.org/mailman/listinfo/ath10k + -Toke -3217a7b7c7ccc6291feddcf801f40fbc50ba23d24ed13404978536d5f1f65e26 +73ec2850439c7585d0476629cfd4611b74f9c9319380ec693367d7b67cb7b83c
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.