From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michael Tremer <michael.tremer@ipfire.org>
To: development@lists.ipfire.org
Subject: [PATCH 15/17] QoS: Drop support for setting TOS bits per class
Date: Mon, 14 Oct 2019 16:46:25 +0000
Message-ID: <20191014164627.18516-15-michael.tremer@ipfire.org>
In-Reply-To: <20191014164627.18516-1-michael.tremer@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============6454740815340777909=="
List-Id: <development.lists.ipfire.org>

--===============6454740815340777909==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

This is useless since no ISP will evaluate those settings
any more and it has a rather large impact on throughput.

Signed-off-by: Michael Tremer <michael.tremer(a)ipfire.org>
---
 config/qos/makeqosscripts.pl | 24 ------------------------
 html/cgi-bin/qos.cgi         | 12 ------------
 2 files changed, 36 deletions(-)

diff --git a/config/qos/makeqosscripts.pl b/config/qos/makeqosscripts.pl
index 9a57aad28..4f66b20b4 100644
--- a/config/qos/makeqosscripts.pl
+++ b/config/qos/makeqosscripts.pl
@@ -128,7 +128,6 @@ case "\$1" in
 		echo "[iptables]"
 		iptables -t mangle -n -L QOS-OUT -v -x 2> /dev/null
 		iptables -t mangle -n -L QOS-INC -v -x 2> /dev/null
-		iptables -t mangle -n -L QOS-TOS -v -x 2> /dev/null
 		exit 0
 	  ;;
 	esac
@@ -191,9 +190,7 @@ print <<END
=20
 	### ADD QOS-OUT CHAIN TO THE MANGLE TABLE IN IPTABLES
 	iptables -t mangle -N QOS-OUT
-	iptables -t mangle -N QOS-TOS
 	iptables -t mangle -I POSTROUTING -o $qossettings{'RED_DEV'} -j QOS-OUT
-	iptables -t mangle -A POSTROUTING -o $qossettings{'RED_DEV'} -j QOS-TOS
=20
 	### Don't change mark on traffic for the ipsec tunnel
 	iptables -t mangle -A QOS-OUT -m mark --mark 50 -j RETURN
@@ -401,7 +398,6 @@ print <<END
 	### ADD QOS-INC CHAIN TO THE MANGLE TABLE IN IPTABLES
 	iptables -t mangle -N QOS-INC
 	iptables -t mangle -A FORWARD -i $qossettings{'RED_DEV'} -j QOS-INC
-	iptables -t mangle -A FORWARD -i $qossettings{'RED_DEV'} -j QOS-TOS
=20
 	### SET TOS
 END
@@ -509,22 +505,6 @@ print <<END
 	### REDUNDANT: SET ALL NONMARKED PACKETS TO DEFAULT CLASS
 	iptables -t mangle -A QOS-INC -j CLASSIFY --set-class 2:$qossettings{'DEFCL=
ASS_INC'}
=20
-	### SETTING TOS BITS
-END
-;
-	foreach $classentry (sort @classes)
-	{
-		@classline =3D split( /\;/, $classentry );
-		$qossettings{'CLASS'} =3D $classline[1];
-		$qossettings{'TOS'} =3D abs $classline[7] * 2;
-		if ($qossettings{'TOS'} ne "0") {
-			print "\tiptables -t mangle -A QOS-TOS -m mark --mark $qossettings{'CLASS=
'} -j TOS --set-tos $qossettings{'TOS'}\n";
-			print "\tiptables -t mangle -A QOS-TOS -m mark --mark $qossettings{'CLASS=
'} -j RETURN\n";
-		}
-	}
-
-print <<END
-
 	## STARTING COLLECTOR
 	/usr/local/bin/qosd $qossettings{'RED_DEV'} >/dev/null 2>&1
 	/usr/local/bin/qosd $qossettings{'IMQ_DEV'} >/dev/null 2>&1
@@ -553,15 +533,11 @@ print <<END
=20
 	# REMOVE & FLUSH CHAINS
 	iptables -t mangle --delete POSTROUTING -o $qossettings{'RED_DEV'} -j QOS-O=
UT >/dev/null 2>&1
-	iptables -t mangle --delete POSTROUTING -o $qossettings{'RED_DEV'} -j QOS-T=
OS >/dev/null 2>&1
 	iptables -t mangle --delete FORWARD -i $qossettings{'RED_DEV'} -j QOS-INC >=
/dev/null 2>&1
-	iptables -t mangle --delete FORWARD -i $qossettings{'RED_DEV'} -j QOS-TOS >=
/dev/null 2>&1
 	iptables -t mangle --flush  QOS-OUT >/dev/null 2>&1
 	iptables -t mangle --delete-chain QOS-OUT >/dev/null 2>&1
 	iptables -t mangle --flush  QOS-INC >/dev/null 2>&1
 	iptables -t mangle --delete-chain QOS-INC >/dev/null 2>&1
-	iptables -t mangle --flush  QOS-TOS >/dev/null 2>&1
-	iptables -t mangle --delete-chain QOS-TOS >/dev/null 2>&1
=20
 	rmmod sch_htb >/dev/null 2>&1
=20
diff --git a/html/cgi-bin/qos.cgi b/html/cgi-bin/qos.cgi
index 320eef67d..8211a3ca0 100644
--- a/html/cgi-bin/qos.cgi
+++ b/html/cgi-bin/qos.cgi
@@ -883,18 +883,6 @@ END
 		<tr><td width=3D'33%' align=3D'right'>Ceilburst:
 		    <td width=3D'33%' align=3D'left'><input type=3D'text' size=3D'20' name=
=3D'CBURST' maxlength=3D'8' value=3D"$qossettings{'CBURST'}" />
 		    <td width=3D'33%' align=3D'center'>&nbsp;
-END
-;
-			$selected{'TOS'}{$qossettings{'TOS'}} =3D "selected=3D'selected'";
-print <<END
-		<tr><td width=3D'33%' align=3D'right'>TOS-Bit:
-		    <td width=3D'33%' align=3D'left'><select name=3D'TOS'>
-				<option value=3D'0' $selected{'TOS'}{'0'}>$Lang::tr{'disabled'} (0)</opt=
ion>
-				<option value=3D'8' $selected{'TOS'}{'8'}>$Lang::tr{'min delay'} (8)</op=
tion>
-				<option value=3D'4' $selected{'TOS'}{'4'}>$Lang::tr{'max throughput'} (4=
)</option>
-				<option value=3D'2' $selected{'TOS'}{'2'}>$Lang::tr{'max reliability'} (=
2)</option>
-				<option value=3D'1' $selected{'TOS'}{'1'}>$Lang::tr{'min costs'} (1)</op=
tion></select>
-		    <td width=3D'33%' align=3D'center'>&nbsp;
 		<tr><td width=3D'33%' align=3D'right'>$Lang::tr{'remark'}:
 		    <td width=3D'66%' colspan=3D'2' align=3D'left'><input type=3D'text' na=
me=3D'REMARK' size=3D'40' maxlength=3D'40' value=3D"$qossettings{'REMARK'}" />
 		<tr><td width=3D'33%' align=3D'right'><img src=3D'/blob.gif' alt=3D'*' />&=
nbsp;$Lang::tr{'required field'}
--=20
2.20.1


--===============6454740815340777909==--