Page MenuHomeFreeBSD
Differential D25084

bridge tests: Avoid building a switching loop
ClosedPublic
Actions

Authored by kp on May 31 2020, 7:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Feb 7, 9:02 PM
Unknown Object (File)
Sat, Feb 7, 6:20 PM
Unknown Object (File)
Sat, Feb 7, 5:06 PM
Unknown Object (File)
Sat, Jan 31, 10:52 AM
Unknown Object (File)
Jan 7 2026, 9:15 PM
Unknown Object (File)
Jan 6 2026, 7:39 PM
Unknown Object (File)
Dec 28 2025, 6:00 AM
Unknown Object (File)
Dec 25 2025, 3:36 PM
View All Files
Subscribers
ae
imp
lwhsu
melifaro

Details

Reviewers
melifaro
Group Reviewers
network
Commits
rGfd5828b62bdb: bridge tests: Avoid building a switching loop
rS361701: bridge tests: Avoid building a switching loop
Summary

Enable STP before bringing the bridges up. This avoids a switching loop,
which has a tendency to drown out progress in userspace processes,
especially on single-core systems.

Only check that we have indeed shut down one of the looped interfaces

PR: 246448

Diff Detail

Lint
Lint Warnings
SeverityLocationCodeMessage
Warningtests/sys/net/if_bridge_test.sh:CHMOD1Invalid Executable
Unit
No Test Coverage
Build Status
Buildable 31427
Build 29046: arc lint + arc unit

Event Timeline

kp created this revision.May 31 2020, 7:20 PM
Herald added subscribers: melifaro, ae. · View Herald TranscriptMay 31 2020, 7:20 PM
Harbormaster completed remote builds in B31410: Diff 72483.May 31 2020, 7:20 PM
kp requested review of this revision.May 31 2020, 7:20 PM
lwhsu added a subscriber: lwhsu.Jun 1 2020, 3:11 PM
kp updated this revision to Diff 72534.Jun 1 2020, 3:39 PM

We no longer require jq

Harbormaster completed remote builds in B31427: Diff 72534.Jun 1 2020, 3:39 PM
melifaro accepted this revision as: melifaro.Jun 1 2020, 3:51 PM

LGTM.

Minor nit: any chance jails a and b can be named bridge_stp_a and bridge_stp_b to avoid potential clashes with other tests?

This revision is now accepted and ready to land.Jun 1 2020, 3:51 PM
This revision was landed with ongoing or failed builds.Jun 1 2020, 7:26 PM
Closed by commit rS361701: bridge tests: Avoid building a switching loop (authored by kp). · Explain Why
This revision was automatically updated to reflect the committed changes.
kp added a commit: rS361701: bridge tests: Avoid building a switching loop.
Herald added a subscriber: imp. · View Herald TranscriptJun 1 2020, 7:26 PM
kp added a comment.Jun 1 2020, 7:26 PM
In D25084#552683, @melifaro wrote:

Minor nit: any chance jails a and b can be named bridge_stp_a and bridge_stp_b to avoid potential clashes with other tests?

Yeah, but that should be its own commit.

kp added a commit: rGfd5828b62bdb: bridge tests: Avoid building a switching loop.Mar 2 2021, 3:19 PM

Revision Contents
Changeset List

PathSize
tests/
sys/
net/
17 lines

Diff 72534

View Options

tests/sys/net/if_bridge_test.sh

Loading...