| Age | Commit message (Collapse) | Author |
|---|
|
Yet another place where id needs to be converted to hex.
Should fix bug 3078.
|
|
This fixes problem which causes QoS not to fail on commit if ceiling
is a percentage.
The ceiling maybe a percentage (not fixed) value so it needs to
be interpreted before passing to tc.
|
|
Internal TC commands use hex for identifiers and one place in perl
script forgot to reformat.
Bugfix for 3142 from Chris Smith <chris.smith@conceptcoders.com>
|
|
ceiling parameter was not being forwarded to kernel.
|
|
If the customer has incorrectly assigned bandwidth and
rate ceilings for QOS, the problem can not be detected until
commit. At that point (possibly during boot), it is better to
print a warning rather than failing.
This addresses Bug 2919.
|
|
This reverts commit ff77155be360216289e1e73af50dc4b82383cc68.
|
|
Revert "More unfrozen stuff"
This reverts commit f1e5d5991c8fc1420d00ef51c826f77b2d8c2bbd.
|
|
Revert "Better validation of numeric input"
This reverts commit e1624917ba2ed574fe350b470762a732a9820cbd.
|
|
Revert "silence ethtool messages when auto bandwidth used"
This reverts commit c1b530e7bfa5c33a8f51f634f906d2412f187e90.
|
|
If device doesn't support full ethtool, then ethtool error messages
were leaking out to console, when QOS bandwidth was detecting interface
speed.
|
|
Fix for Bug 2763 and related problems. Need better handling of numeric
inputs. Real perl programmers don't use a switch, but hash instead.
|
|
Bugfix for 3010
|
|
|
|
Need to cleanup old qdisc/class/filters before applying new one.
Bugfix: 2932
|
|
The generated filtering rules were using wrong id's when
set-dscp was being used. Need to put dsmark rules on root,
and htb rules on start of that tree, and the id needs to
be printed in hex not decimal.
Bugfix: 2947
|
|
Add a bunch of checks.
* Check policy configuration at commit time
* Add check for duplicate names
* Allow _ in policy name
Bugfix: 2907
|
|
Bugfix: 2919
But configuration level in error message
|
|
Having a hardcoded switch statement is harder to update than
using a hash.
|
|
Script was using old value, and default be undefined not 0.
|
|
Fix parsing problems for update-interface that caused problems
during boot.
Bugfix: 2914
|
|
The function validate_name is no longer used.
|
|
The delete-policy option requires two arguments (for consistency),
and the validate-name option is no longer used.
|
|
For compatiablity use DSCP values in the 0..63 range and scale
in the script, rather than uses raw header values. This makes configuration
more familar to IOS users.
|
|
No functional change just change listName to listPolicy to make
it clearer what is going on.
|
|
If no suffix is given, the bandwidth value should be in "bits/sec"
(not bytes/sec), to be compatiable with other routers.
|
|
This adds ability to choose the underlying queue type for traffic shaper
classes:
drop-tail => fifo
random-detect => red
fair-queue => sfq
priority => prio
For Red, the values are computed based on the bandwidth so it is much
simpler than raw tc usage.
|
|
This makes dsmark work. Need to round up to power of 2 and also
pass parent down to handle nested case properly.
|
|
If the command to tc doesn't work, then reprint to standard out
for diagnostic. Also, fix use of sudo in delete of qdisc.
|
|
But do it in a manner safe from quoting attacks!
|
|
Matching on 'ip' uses the u32 tc filter and other match types 'interface',
'vif' use the basic tc filter. Since kernel doesn't allow mixing filter
types in the same classifier, need to catch and report the problem rather
than failing in the actual commands.
|
|
add preliminary support for dsmark (still needs testing).
allow class 2 (compute default class as n+1)
|
|
This enforces syntax checks on dscp values.
|
|
Use proper syntax for incoming interface and outgoing vlan match.
Need to catch u32 vs basic match filter conflicts.
|
|
need to lookup ifindex for interface not vlan tag (duh)
and ip field should be initialized only when ip tag is found.
|
|
And add more help text.
|
|
1. checking for name clash shouldn't see self
2. leftover from rate to bandwidth name change
3. leftover debug message.
|
|
Return undefined on bad value, so caller can print message.
|
|
Make qos syntax more familar to IOS users.
|
|
This is preliminary (still needs testing) but get the syntax and basic structure
in place.
|
|
Check names of policys for syntax and uniqueness.
Add hooks for update that does refresh of interface (should fix bug).
Block deletion of policy that is in use.
|
|
fixes http://bugzilla.vyatta.com/show_bug.cgi?id=2738
|
|
If getprotocol or getdsfield is called with undefined value, then
return undefined.
|
|
If commit of new policy fails, then remove all filters/qdisc.
|
|
Try for complete sentences for describing problem.
|
|
Fix a number of perl scoping bugs that kept auto rate processing
from working.
|
|
numeric comparison uses where string comparison should have been used
|
|
|
|
debug is global
|
|
Use exec to ensure running sudo (no shell interpretation).
Change how debug is enabled.
|
|
Forgot to hexify the value. Tc takes classnames in hex.
|
