Re: [PATCH net-next v3 2/3] net: stmmac: add tc-mqprio qdisc offload

Lorenzo Bianconi <[email protected]>
Newsgroups org.infradead.lists.linux-arm-kernel,org.kernel.vger.netdev
Message-ID <aoEcjrBy6h7E22dI@lore-qca>
> On Sat, 08 Aug 2026 16:49:58 +0200 Lorenzo Bianconi wrote:
> > Subject: [PATCH net-next v3 2/3] net: stmmac: add tc-mqprio qdisc offload
> > 
> > Implement offload of the tc-mqprio qdisc in the stmmac driver. The MTL TX
> > scheduler is switched to strict priority, and the PSTQX/PSTC priority
> > bitmask of each TX queue is programmed from the set of frame priorities
> > mapped to the owning traffic class (qopt->prio_tc_map).
> > 
> > The offload requires the DCB feature and a 1:1 TC to TX queue mapping,
> > with the queue ranges bounded by the number of queues the MAC implements.
> > Configurations with AVB queues are rejected, since forcing strict
> > priority conflicts with the CBS algorithm.
> 
> Clashiko has a bunch of comments, I assume at least one of them is
> actionable. High level - you say "add tc-mqprio qdisc offload"
> but the driver already has a bunch of mqprio code. Background
> would be useful. You also say "1:1 TC to TX queue mapping", if there's
> only one queue per priority why offload mqprio and not prio?
> At least a note in the commit message would be useful.

ack, thx. I will address them in v4.

Regards,
Lorenzo
signature.asc (application/pgp-signature, 228 B)
-----BEGIN PGP SIGNATURE-----

iHUEABYKAB0WIQTquNwa3Txd3rGGn7Y6cBh0uS2trAUCaoEcjgAKCRA6cBh0uS2t
rPLaAPwNhvlwtI5cH91/8Oj5jzqnm4RKoWFYOWz0EWyv0Vw1xwD/QjanQNDBanDr
aT49OJ+Y1YE70hdbgSPIwPHmy9KRHAc=
=PVBu
-----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.