[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAGRyCJGAMGxW04_XQjrUforZ6G7Y4gcR=CvkzZDiP0vqHnB-pg@mail.gmail.com>
Date: Thu, 1 Dec 2022 11:48:32 +0100
From: Daniele Palmas <dnlplm@...il.com>
To: Dave Taht <dave.taht@...il.com>
Cc: David Miller <davem@...emloft.net>,
Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>,
Eric Dumazet <edumazet@...gle.com>,
Subash Abhinov Kasiviswanathan <quic_subashab@...cinc.com>,
Sean Tranchetti <quic_stranche@...cinc.com>,
Jonathan Corbet <corbet@....net>,
Alexander Lobakin <alexandr.lobakin@...el.com>,
Gal Pressman <gal@...dia.com>,
Bjørn Mork <bjorn@...k.no>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
netdev@...r.kernel.org
Subject: Re: [PATCH net-next v2 0/3] add tx packets aggregation to ethtool and rmnet
Hello Dave,
Il giorno mer 30 nov 2022 alle ore 16:04 Dave Taht
<dave.taht@...il.com> ha scritto:
>
> On Wed, Nov 30, 2022 at 5:15 AM Daniele Palmas <dnlplm@...il.com> wrote:
> >
> > Hello maintainers and all,
> >
> > this patchset implements tx qmap packets aggregation in rmnet and generic
> > ethtool support for that.
> >
> > Some low-cat Thread-x based modems are not capable of properly reaching the maximum
> > allowed throughput both in tx and rx during a bidirectional test if tx packets
> > aggregation is not enabled.
> >
> > I verified this problem with rmnet + qmi_wwan by using a MDM9207 Cat. 4 based modem
> > (50Mbps/150Mbps max throughput). What is actually happening is pictured at
> > https://drive.google.com/file/d/1gSbozrtd9h0X63i6vdkNpN68d-9sg8f9/view
>
> Thank you for documenting which device this is. Is it still handing in
> 150ms of bufferbloat in good conditions,
> and 25 seconds or so in bad?
>
New Flent test results available at
https://drive.google.com/drive/folders/1-rpeuM2Dg9rVdYCP0M84K4Ook5kcZTWc?usp=share_link
>From what I can understand, it seems to me a bit better, but not
completely sure how much is directly related to the changes of v2.
Regards,
Daniele
Powered by blists - more mailing lists