Move bridged TC off the node NIC so the DUT can own its qdisc - #101
Open
Jafaral wants to merge 3 commits into
Open
Move bridged TC off the node NIC so the DUT can own its qdisc#101Jafaral wants to merge 3 commits into
Jafaral wants to merge 3 commits into
Conversation
Host-to-switch netem/tbf was installed on the node NIC and replaced the DUT's root qdisc. Keep the old TX meaning: node attachment constraints shape that node's outbound via an IFB on the switch veth ingress. Switch-attachment constraints still shape switch outbound. P2P stays on the node; warn instead of inventing a switch. Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
Jafaral
force-pushed
the
switch-side-tc
branch
from
September 6, 2026 16:05
3bb66b8 to
391de4a
Compare
Keep TX and RX on the switch so the DUT keeps its qdisc. Switch to: is node RX; from: is node TX (IFB). Schema is generated from the YANG in README.org. Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
Jafaral
force-pushed
the
switch-side-tc
branch
from
September 6, 2026 16:10
391de4a to
09e7ace
Compare
Switch tests were sharing the module munet with leftover sw1-e0/eth0. Node delay/rate/loss now live on the switch IFB, not the node NIC. Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
Jafaral
force-pushed
the
switch-side-tc
branch
from
September 9, 2026 03:31
6a4d4af to
758b1e0
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
delay/rate/lossare applied on the switch, not the node NIC, so the DUT keeps its root qdisc.to: Nis traffic toward that node (node RX, switch egress). Switchfrom: Nis traffic from that node (node TX, IFB on switch ingress). Use two entries for both directions.delaystill means that node’s TX (same pipe asfrom:). Network-root constraints still default toto:(RX). P2P is unchanged; warn if TC stays on the node.Example Config