Page MenuHomeFreeBSD
Differential D28303

altq tests: Basic ALTQ test
ClosedPublic
Actions

Authored by kp on Jan 23 2021, 10:52 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Sep 8, 9:13 PM
Unknown Object (File)
Tue, Sep 8, 6:19 PM
Unknown Object (File)
Sun, Sep 6, 7:10 PM
Unknown Object (File)
Sat, Sep 5, 3:35 AM
Unknown Object (File)
Sat, Sep 5, 12:20 AM
Unknown Object (File)
Fri, Sep 4, 6:57 PM
Unknown Object (File)
Fri, Sep 4, 4:20 PM
Unknown Object (File)
Fri, Sep 4, 10:30 AM
View All Files
Subscribers
donner
farrokhi
imp
melifaro

Details

Reviewers
donner
Group Reviewers
network
Commits
rG2400173b9377: altq tests: Basic ALTQ test
rGe02597d7ce67: altq tests: Basic ALTQ test
rG16b3833344ad: altq tests: Basic ALTQ test
Summary

Activate ALTQ_HFSC, crudely check if it really limits bandwidth as we'd expect.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 36469
Build 33358: arc lint + arc unit

Event Timeline

kp created this revision.Jan 23 2021, 10:52 AM
Herald added subscribers: melifaro, farrokhi, imp. · View Herald TranscriptJan 23 2021, 10:52 AM
kp requested review of this revision.Jan 23 2021, 10:52 AM
Harbormaster completed remote builds in B36429: Diff 82763.Jan 23 2021, 10:52 AM
donner added a subscriber: donner.Jan 23 2021, 1:05 PM
donner added inline comments.
tests/sys/netpfil/pf/altq.sh
26

It's expected, that this command always succeed, so there is no check needed. Correct?

38

There is a race condition between the "saturate" ping and the test. A busy system might have several seconds between those commands.

Would it be possible to rephrase the check, so that the ping is sending i.e. 10 packets and we expect 3 of them back?

kp updated this revision to Diff 82852.Jan 25 2021, 3:16 PM

Adjust to new altq.foo sysctl

Harbormaster completed remote builds in B36469: Diff 82852.Jan 25 2021, 3:17 PM
donner accepted this revision as: donner.Jan 25 2021, 3:23 PM
This revision is now accepted and ready to land.Jan 25 2021, 3:23 PM
Closed by commit rG16b3833344ad: altq tests: Basic ALTQ test (authored by kp). · Explain WhyJan 25 2021, 7:59 PM
This revision was automatically updated to reflect the committed changes.
kp added a commit: rG16b3833344ad: altq tests: Basic ALTQ test.
kp added a commit: rGe02597d7ce67: altq tests: Basic ALTQ test.Jul 31 2021, 1:04 PM
kp added a commit: rG2400173b9377: altq tests: Basic ALTQ test.

Revision Contents
Changeset List

PathSize
tests/
sys/
netpfil/
pf/
3 lines
49 lines
24 lines

Diff 82852

View Options

tests/sys/netpfil/pf/Makefile

Loading...
View Options

tests/sys/netpfil/pf/altq.sh

Loading...
View Options

tests/sys/netpfil/pf/utils.subr

Loading...