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-----