Re: [PATCH net-next] net: stmmac: stm32: Set TSO/TBS Tx queues default settings

Kurt Kanzenbach <[email protected]>
Newsgroups gmane.linux.network,gmane.linux.ports.arm.kernel
Message-ID <[email protected]>
On Mon Aug 17 2026, Jakub Kicinski wrote:
> On Wed, 12 Aug 2026 08:28:57 +0200 Kurt Kanzenbach wrote:
>> TSO and TBS cannot coexist. Use the first queue with TSO and the rest for
>> TBS. Tx queues with TBS can support etf qdisc hw offload. This is done
>> similar to dwmac-imx and dwmac-intel.
>
> Can you explain your use case?

I want to use ETF Qdisc with hardware offload, which is currently not
possible on the stm32mp2.

> And how many queues the device has in total?

The stm32mp2 has four Tx and two Rx queues.

>
> You say "TSO and TBS cannot coexist" but can any queue on your
> platform be configured to support either feature?

I think so. The data sheet says: "Do not enable time-based scheduling for
channels on which the TSO feature is enabled.". But, I didn't find any
limitations on what queue/channel can enable TBS.

> If yes why are we configuring this statically instead of making
> appropriate configuration based on qdisc or some other uAPI knob?

It seems like a static configuration in the driver today. I basically
followed the same convention as dwmac-imx, dwmac-intel, dwmac-mediatek
and dwmac-qcom-ethqos. Only dwmac-socfpga does it differently.

Tx Launch Time requires a different DMA descriptor layout. Currently
tc_setup_etf() just returns -EINVAL if the DMA configuration is not
setup appropriately. I guess a dynamic configuration requires to change
the DMA config and perform a full release/open cycle.

Thanks,
Kurt
signature.asc (application/pgp-signature, 861 B)
-----BEGIN PGP SIGNATURE-----

iQJHBAEBCgAxFiEEvLm/ssjDfdPf21mSwZPR8qpGc4IFAmqEAkQTHGt1cnRAbGlu
dXRyb25peC5kZQAKCRDBk9HyqkZzgl24D/oD5pIbsJKmttKF5pgORFPdAeBIldyk
SJDlmLRbsKEvzsNC9upW0HDi48XGkXrQCaJjHwY8WLLbgvaBZhM7mDpfnI0GRXyc
XAyGdrQeO/BKPZ13oKJGWzu3IgV09rHW/tb4hT9wQDsXCNV88756qGey1h5pmrdT
rVqiP3muzIbHgXxZsomFPhe7eMOjttuYGAPRq2zFhXMCto8lqtn5GzdfxKXEnTxF
RMGp6gKZ/1onXl55AB/+E1DeUKilOkUyuc9b7SFCu0Czvtre0DllIO3/MwfX+hd9
+MUB1PP8bZ3CaN93N2d3pLyEZdY/NeqWOTmqnZforsvA5AXq8cexXNTz/PK+h35K
1N4gfK2szhFNyUmQS4joUj9j44PP6BunJIKHpvRZ2leVDb9JnJP0bprCy6b848H0
ifNkmYMIWKGdV+peW0ckNIfdg492rz5KZjLpzVZBwhH2j3XkRKiPZv1hz6i9ztgg
oncYVsNyXoCfUscIpCUfPOAy9DgIVvVyg7jW4Zj8wuG+atrIv0qybOLOD3xjLf6b
85nbZ18cyhXNFWyYxD0jHE8NkKe6CJu5QbfPSiPLYJtf2GrZPmp7YsQFaBRLU3HG
aUOXjDcgp7V1/SLyTjMpTDSsDIacT4E97SMnFLGHrGCxKpUC+PHB4tmi3u0W4WYY
TY9LCZK5MwqJvg==
=TQFM
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.