From: Dave Taht <dave.taht@gmail.com> To: Kalle Valo <kvalo@codeaurora.org> Cc: "Sven Eckelmann" <sven@narfation.org>, ath10k <ath10k@lists.infradead.org>, "Linus Lüssing" <linus.luessing@c0d3.blue>, "Simon Wunderlich" <sw@simonwunderlich.de>, "Linux Kernel Network Developers" <netdev@vger.kernel.org>, linux-wireless <linux-wireless@vger.kernel.org>, LKML <linux-kernel@vger.kernel.org>, "Ben Greear" <greearb@candelatech.com>, "David S . Miller" <davem@davemloft.net>, "Linus Lüssing" <ll@simonwunderlich.de>, mail@adrianschmutzler.de Subject: Re: [PATCH] ath10k: increase rx buffer size to 2048 Date: Tue, 28 Apr 2020 05:27:25 -0700 [thread overview] Message-ID: <CAA93jw6xoh=Nu3-OcfU5cnO5rct+QGqRf_Tnwx7-BpO8Fhrakw@mail.gmail.com> (raw) In-Reply-To: <87blnblsyv.fsf@codeaurora.org> On Tue, Apr 28, 2020 at 5:06 AM Kalle Valo <kvalo@codeaurora.org> wrote: > > Sven Eckelmann <sven@narfation.org> writes: > > > On Wednesday, 1 April 2020 09:00:49 CEST Sven Eckelmann wrote: > >> On Wednesday, 5 February 2020 20:10:43 CEST Linus Lüssing wrote: > >> > From: Linus Lüssing <ll@simonwunderlich.de> > >> > > >> > Before, only frames with a maximum size of 1528 bytes could be > >> > transmitted between two 802.11s nodes. > >> > > >> > For batman-adv for instance, which adds its own header to each frame, > >> > we typically need an MTU of at least 1532 bytes to be able to transmit > >> > without fragmentation. > >> > > >> > This patch now increases the maxmimum frame size from 1528 to 1656 > >> > bytes. > >> [...] > >> > >> @Kalle, I saw that this patch was marked as deferred [1] but I couldn't find > >> any mail why it was done so. It seems like this currently creates real world > >> problems - so would be nice if you could explain shortly what is currently > >> blocking its acceptance. > > > > Ping? > > Sorry for the delay, my plan was to first write some documentation about > different hardware families but haven't managed to do that yet. > > My problem with this patch is that I don't know what hardware and > firmware versions were tested, so it needs analysis before I feel safe > to apply it. The ath10k hardware families are very different that even > if a patch works perfectly on one ath10k hardware it could still break > badly on another one. > > What makes me faster to apply ath10k patches is to have comprehensive > analysis in the commit log. This shows me the patch author has > considered about all hardware families, not just the one he is testing > on, and that I don't need to do the analysis myself. I have been struggling to get the ath10k to sing and dance using various variants of the firmware, on this bug over here: https://forum.openwrt.org/t/aql-and-the-ath10k-is-lovely/ The puzzling thing is the loss of bidirectional throughput at codel target 20, and getting WAY more (but less than I expected) at codel target 5. This doesn't quite have bearing the size of the rx ring, except that in my experiments the rx ring is rather small!! and yet I get way more performance out of it.... (still, as you'll see from the bug, it's WAY better than it used to be) is NAPI in this driver? I'm afraid to look. > -- > https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches -- Make Music, Not War Dave Täht CTO, TekLibre, LLC http://www.teklibre.com Tel: 1-831-435-0729
WARNING: multiple messages have this Message-ID (diff)
From: Dave Taht <dave.taht@gmail.com> To: Kalle Valo <kvalo@codeaurora.org> Cc: "Linus Lüssing" <linus.luessing@c0d3.blue>, "Simon Wunderlich" <sw@simonwunderlich.de>, "Linux Kernel Network Developers" <netdev@vger.kernel.org>, linux-wireless <linux-wireless@vger.kernel.org>, LKML <linux-kernel@vger.kernel.org>, ath10k <ath10k@lists.infradead.org>, mail@adrianschmutzler.de, "Ben Greear" <greearb@candelatech.com>, "David S . Miller" <davem@davemloft.net>, "Sven Eckelmann" <sven@narfation.org>, "Linus Lüssing" <ll@simonwunderlich.de> Subject: Re: [PATCH] ath10k: increase rx buffer size to 2048 Date: Tue, 28 Apr 2020 05:27:25 -0700 [thread overview] Message-ID: <CAA93jw6xoh=Nu3-OcfU5cnO5rct+QGqRf_Tnwx7-BpO8Fhrakw@mail.gmail.com> (raw) In-Reply-To: <87blnblsyv.fsf@codeaurora.org> On Tue, Apr 28, 2020 at 5:06 AM Kalle Valo <kvalo@codeaurora.org> wrote: > > Sven Eckelmann <sven@narfation.org> writes: > > > On Wednesday, 1 April 2020 09:00:49 CEST Sven Eckelmann wrote: > >> On Wednesday, 5 February 2020 20:10:43 CEST Linus Lüssing wrote: > >> > From: Linus Lüssing <ll@simonwunderlich.de> > >> > > >> > Before, only frames with a maximum size of 1528 bytes could be > >> > transmitted between two 802.11s nodes. > >> > > >> > For batman-adv for instance, which adds its own header to each frame, > >> > we typically need an MTU of at least 1532 bytes to be able to transmit > >> > without fragmentation. > >> > > >> > This patch now increases the maxmimum frame size from 1528 to 1656 > >> > bytes. > >> [...] > >> > >> @Kalle, I saw that this patch was marked as deferred [1] but I couldn't find > >> any mail why it was done so. It seems like this currently creates real world > >> problems - so would be nice if you could explain shortly what is currently > >> blocking its acceptance. > > > > Ping? > > Sorry for the delay, my plan was to first write some documentation about > different hardware families but haven't managed to do that yet. > > My problem with this patch is that I don't know what hardware and > firmware versions were tested, so it needs analysis before I feel safe > to apply it. The ath10k hardware families are very different that even > if a patch works perfectly on one ath10k hardware it could still break > badly on another one. > > What makes me faster to apply ath10k patches is to have comprehensive > analysis in the commit log. This shows me the patch author has > considered about all hardware families, not just the one he is testing > on, and that I don't need to do the analysis myself. I have been struggling to get the ath10k to sing and dance using various variants of the firmware, on this bug over here: https://forum.openwrt.org/t/aql-and-the-ath10k-is-lovely/ The puzzling thing is the loss of bidirectional throughput at codel target 20, and getting WAY more (but less than I expected) at codel target 5. This doesn't quite have bearing the size of the rx ring, except that in my experiments the rx ring is rather small!! and yet I get way more performance out of it.... (still, as you'll see from the bug, it's WAY better than it used to be) is NAPI in this driver? I'm afraid to look. > -- > https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches -- Make Music, Not War Dave Täht CTO, TekLibre, LLC http://www.teklibre.com Tel: 1-831-435-0729 _______________________________________________ ath10k mailing list ath10k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath10k
next prev parent reply other threads:[~2020-04-28 12:27 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-02-05 19:10 [PATCH] ath10k: increase rx buffer size to 2048 Linus Lüssing 2020-02-05 19:10 ` Linus Lüssing 2020-04-01 7:00 ` Sven Eckelmann 2020-04-01 7:00 ` Sven Eckelmann 2020-04-25 11:14 ` Sven Eckelmann 2020-04-25 11:14 ` Sven Eckelmann 2020-04-28 12:01 ` Kalle Valo 2020-04-28 12:01 ` Kalle Valo 2020-04-28 12:27 ` Dave Taht [this message] 2020-04-28 12:27 ` Dave Taht 2020-04-28 14:27 ` Ben Greear 2020-04-28 14:27 ` Ben Greear 2021-01-18 16:13 ` Kalle Valo 2021-01-18 16:13 ` Kalle Valo
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to='CAA93jw6xoh=Nu3-OcfU5cnO5rct+QGqRf_Tnwx7-BpO8Fhrakw@mail.gmail.com' \ --to=dave.taht@gmail.com \ --cc=ath10k@lists.infradead.org \ --cc=davem@davemloft.net \ --cc=greearb@candelatech.com \ --cc=kvalo@codeaurora.org \ --cc=linus.luessing@c0d3.blue \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-wireless@vger.kernel.org \ --cc=ll@simonwunderlich.de \ --cc=mail@adrianschmutzler.de \ --cc=netdev@vger.kernel.org \ --cc=sven@narfation.org \ --cc=sw@simonwunderlich.de \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.