On 3 Jun 2026, at 5:35 pm, Nikolay Aleksandrov <[email protected]> wrote: > So looking at how this is implemented, why not put most of it in TC? > It is testing for skb class, for tc qdisc, I don't see a reason for it > to be in the bridge at all. You can filter the mcast groups and simulate > the "reserved" flag, it has to be set manually anyway. Adding new > tests in the bridge software fast-path just for this is a waste.
This makes sense. Maybe it’s better to use flower+TCAM to implement this, and have our SRP daemon manage flower entries in addition to MDB entries. The mv88e6xxx driver’s TCAM support is recent (hi Cedric!) and does not yet appear to support the 6352 or matching on PCP/DA, but one presumes this is possible to add. Alternatively, responding to your comments, the bridge could mark frames with a “stream reserved” DA with a new tc_skb_ext bit, which would be surfaced to flower similarly to l2_miss. The rest of the policy would live in the flower rule. MDB_FLAGS_STREAM_RESERVED would remain but IFLA_BRPORT_FILTER_STREAM_RESERVED could go (we would dynamically enable the equivalent in mv88e6xxx by refcounting MDB_FLAGS_STREAM_RESERVED entries). Luke
