From: "Arınç ÜNAL" <arinc.unal@arinc9.com>
To: Lorenzo Bianconi <lorenzo@kernel.org>
Cc: Daniel Golle <daniel@makrotopia.org>,
DENG Qingfang <dqfext@gmail.com>,
Sean Wang <sean.wang@mediatek.com>, Andrew Lunn <andrew@lunn.ch>,
Florian Fainelli <f.fainelli@gmail.com>,
Vladimir Oltean <olteanv@gmail.com>,
"David S. Miller" <davem@davemloft.net>,
Eric Dumazet <edumazet@google.com>,
Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
AngeloGioacchino Del Regno
<angelogioacchino.delregno@collabora.com>,
netdev@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org
Subject: Re: [PATCH net-next] net: dsa: mt7530: Add TBF qdisc offload support
Date: Thu, 31 Oct 2024 13:35:39 +0000 (UTC) [thread overview]
Message-ID: <d2776a19-5176-4ce4-9306-273ec7cda0a6@arinc9.com> (raw)
In-Reply-To: <ZyM5CPfQYHc_Eolh@lore-desk>
On 31/10/2024 11:00, Lorenzo Bianconi wrote:
>> On 30/10/2024 22:29, Lorenzo Bianconi wrote:
>>> Introduce port_setup_tc callback in mt7530 dsa driver in order to enable
>>> dsa ports rate shaping via hw Token Bucket Filter (TBF) for hw switched
>>> traffic. Enable hw TBF just for EN7581 SoC for the moment.
>>
>> Is this because you didn't test it on the other models? Let me know if
>> that's the case and I'll test it.
>
> yep, exactly. I have tested it just on EN7581 since I do not have any other
> boards for testing at the moment. If you confirm it works on other SoCs too,
> I can remove the limitation.
Seems to be working fine on MT7530. As we have tested this on the oldest
and newest models that use this switching IP, I'm going to assume it will
work on the other models as well. You can remove the limitation. Also,
please change MT7530_ERLCR_P and MT7530_GERLCR to MT753X_ERLCR_P and
MT753X_GERLCR.
tc qdisc add dev lan4 root tbf rate 10mbit burst 10kb latency 50ms
[ ID] Interval Transfer Bitrate Retr
[ 5] 0.00-5.00 sec 5.88 MBytes 9.85 Mbits/sec 4 sender
[ 5] 0.00-5.00 sec 5.50 MBytes 9.23 Mbits/sec receiver
tc qdisc del dev lan4 root
[ ID] Interval Transfer Bitrate Retr
[ 5] 0.00-5.00 sec 469 MBytes 786 Mbits/sec 0 sender
[ 5] 0.00-5.00 sec 468 MBytes 785 Mbits/sec receiver
tc qdisc add dev lan4 root tbf rate 11mbit burst 10kb latency 50ms
[ ID] Interval Transfer Bitrate Retr
[ 5] 0.00-5.00 sec 6.38 MBytes 10.7 Mbits/sec 6 sender
[ 5] 0.00-5.00 sec 6.00 MBytes 10.1 Mbits/sec receiver
tc qdisc del dev lan4 root
[ ID] Interval Transfer Bitrate Retr
[ 5] 0.00-5.00 sec 467 MBytes 783 Mbits/sec 0 sender
[ 5] 0.00-5.00 sec 466 MBytes 783 Mbits/sec receiver
tc qdisc add dev lan4 root tbf rate 11mbit burst 10kb latency 50ms
tc qdisc replace dev lan4 root tbf rate 10mbit burst 10kb latency 50ms
[ ID] Interval Transfer Bitrate Retr
[ 5] 0.00-5.00 sec 5.88 MBytes 9.85 Mbits/sec 4 sender
[ 5] 0.00-5.00 sec 5.50 MBytes 9.23 Mbits/sec receiver
Arınç
next prev parent reply other threads:[~2024-10-31 13:37 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-30 19:29 [PATCH net-next] net: dsa: mt7530: Add TBF qdisc offload support Lorenzo Bianconi
2024-10-31 7:10 ` Arınç ÜNAL
2024-10-31 8:00 ` Lorenzo Bianconi
2024-10-31 13:35 ` Arınç ÜNAL [this message]
2024-10-31 14:19 ` Lorenzo Bianconi
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=d2776a19-5176-4ce4-9306-273ec7cda0a6@arinc9.com \
--to=arinc.unal@arinc9.com \
--cc=andrew@lunn.ch \
--cc=angelogioacchino.delregno@collabora.com \
--cc=daniel@makrotopia.org \
--cc=davem@davemloft.net \
--cc=dqfext@gmail.com \
--cc=edumazet@google.com \
--cc=f.fainelli@gmail.com \
--cc=kuba@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=lorenzo@kernel.org \
--cc=matthias.bgg@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=olteanv@gmail.com \
--cc=pabeni@redhat.com \
--cc=sean.wang@mediatek.com \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox