Browse Source

[sqm-scripts/luci-app-sqm] Document how to disable shaping on a per direction basis

sqm-scripts for a long time interprets a "Down- or Upload speed" of zero as
an indication that the shaper should be disabled. Note that really shaping
an individual direction down  o zero will make the link effectively dead
for tcp (think reverse ACK traffic). Son instead of allowing the user to
configure something broken, 0 was "over-loaded" to denote no shaping
since several years, but that information has not been documented visibly
to the users. This commit aims at fixing that oversight.

Signed-off-by: Sebastian Moeller <moeller0@gmx.de>
lilik-openwrt-22.03
Toke Høiland-Jørgensen 10 years ago
parent
commit
771ce26a71
2 changed files with 3 additions and 3 deletions
  1. +1
    -1
      net/luci-app-sqm/Makefile
  2. +2
    -2
      net/luci-app-sqm/files/sqm-cbi.lua

+ 1
- 1
net/luci-app-sqm/Makefile View File

@ -9,7 +9,7 @@ include $(TOPDIR)/rules.mk
PKG_NAME:=luci-app-sqm
PKG_VERSION:=3
PKG_RELEASE:=1
PKG_RELEASE:=2
PKG_LICENSE:=GPLv2
LUCI_DIR:=/usr/lib/lua/luci


+ 2
- 2
net/luci-app-sqm/files/sqm-cbi.lua View File

@ -52,11 +52,11 @@ end
n.rmempty = false
dl = s:taboption("tab_basic", Value, "download", translate("Download speed (kbit/s) (ingress):"))
dl = s:taboption("tab_basic", Value, "download", translate("Download speed (kbit/s) (ingress) set to 0 to selectively disable ingress shaping:"))
dl.datatype = "and(uinteger,min(0))"
dl.rmempty = false
ul = s:taboption("tab_basic", Value, "upload", translate("Upload speed (kbit/s) (egress):"))
ul = s:taboption("tab_basic", Value, "upload", translate("Upload speed (kbit/s) (egress) set to 0 to selectively disable egress shaping:"))
ul.datatype = "and(uinteger,min(0))"
ul.rmempty = false


Loading…
Cancel
Save