From f89a6806d90fd11e0e1e5e922ef95332ad8bfeb8 Mon Sep 17 00:00:00 2001
From: Christian Poessinger <christian@poessinger.com>
Date: Thu, 16 Jun 2022 21:20:39 +0200
Subject: qos: T4284: first implementation introducing a new vyos.qos module

---
 .../include/interface/mirror.xml.i                 |  18 +-
 .../include/interface/redirect.xml.i               |   4 +-
 .../include/qos/bandwidth-auto.xml.i               |  43 ++++
 interface-definitions/include/qos/bandwidth.xml.i  |  28 ++-
 .../include/qos/class-match-ipv4-address.xml.i     |  19 ++
 .../include/qos/class-match-ipv6-address.xml.i     |  14 ++
 .../include/qos/class-match.xml.i                  | 177 +++++++++++++++++
 .../include/qos/class-police-exceed.xml.i          |  66 ++++++
 .../include/qos/class-priority.xml.i               |  15 ++
 interface-definitions/include/qos/dscp.xml.i       | 143 -------------
 .../include/qos/limiter-actions.xml.i              |  66 ------
 interface-definitions/include/qos/match-dscp.xml.i | 142 +++++++++++++
 interface-definitions/include/qos/match.xml.i      | 221 ---------------------
 interface-definitions/include/qos/max-length.xml.i |   8 +-
 interface-definitions/include/qos/queue-type.xml.i |  17 +-
 .../include/version/qos-version.xml.i              |   2 +-
 16 files changed, 530 insertions(+), 453 deletions(-)
 create mode 100644 interface-definitions/include/qos/bandwidth-auto.xml.i
 create mode 100644 interface-definitions/include/qos/class-match-ipv4-address.xml.i
 create mode 100644 interface-definitions/include/qos/class-match-ipv6-address.xml.i
 create mode 100644 interface-definitions/include/qos/class-match.xml.i
 create mode 100644 interface-definitions/include/qos/class-police-exceed.xml.i
 create mode 100644 interface-definitions/include/qos/class-priority.xml.i
 delete mode 100644 interface-definitions/include/qos/dscp.xml.i
 delete mode 100644 interface-definitions/include/qos/limiter-actions.xml.i
 create mode 100644 interface-definitions/include/qos/match-dscp.xml.i
 delete mode 100644 interface-definitions/include/qos/match.xml.i

(limited to 'interface-definitions/include')

diff --git a/interface-definitions/include/interface/mirror.xml.i b/interface-definitions/include/interface/mirror.xml.i
index 2959551f0..74a172b50 100644
--- a/interface-definitions/include/interface/mirror.xml.i
+++ b/interface-definitions/include/interface/mirror.xml.i
@@ -1,23 +1,31 @@
 <!-- include start from interface/mirror.xml.i -->
 <node name="mirror">
   <properties>
-    <help>Incoming/outgoing packet mirroring destination</help>
+    <help>Mirror ingress/egress packets</help>
   </properties>
   <children>
     <leafNode name="ingress">
       <properties>
-        <help>Mirror the ingress traffic of the interface to the destination interface</help>
+        <help>Mirror ingress traffic to destination interface</help>
         <completionHelp>
-            <script>${vyos_completion_dir}/list_interfaces.py</script>
+          <script>${vyos_completion_dir}/list_interfaces.py</script>
         </completionHelp>
+        <valueHelp>
+          <format>txt</format>
+          <description>Destination interface name</description>
+        </valueHelp>
       </properties>
     </leafNode>
     <leafNode name="egress">
       <properties>
-        <help>Mirror the egress traffic of the interface to the destination interface</help>
+        <help>Mirror egress traffic to destination interface</help>
         <completionHelp>
-            <script>${vyos_completion_dir}/list_interfaces.py</script>
+          <script>${vyos_completion_dir}/list_interfaces.py</script>
         </completionHelp>
+        <valueHelp>
+          <format>txt</format>
+          <description>Destination interface name</description>
+        </valueHelp>
       </properties>
     </leafNode>
   </children>
diff --git a/interface-definitions/include/interface/redirect.xml.i b/interface-definitions/include/interface/redirect.xml.i
index 8df8957ac..b01e486ce 100644
--- a/interface-definitions/include/interface/redirect.xml.i
+++ b/interface-definitions/include/interface/redirect.xml.i
@@ -1,13 +1,13 @@
 <!-- include start from interface/redirect.xml.i -->
 <leafNode name="redirect">
   <properties>
-    <help>Incoming packet redirection destination</help>
+    <help>Redirect incoming packet to destination</help>
     <completionHelp>
       <script>${vyos_completion_dir}/list_interfaces.py</script>
     </completionHelp>
     <valueHelp>
       <format>txt</format>
-      <description>Interface name</description>
+      <description>Destination interface name</description>
     </valueHelp>
     <constraint>
       #include <include/constraint/interface-name.xml.in>
diff --git a/interface-definitions/include/qos/bandwidth-auto.xml.i b/interface-definitions/include/qos/bandwidth-auto.xml.i
new file mode 100644
index 000000000..3780b7444
--- /dev/null
+++ b/interface-definitions/include/qos/bandwidth-auto.xml.i
@@ -0,0 +1,43 @@
+<!-- include start from qos/bandwidth-auto.xml.i -->
+<leafNode name="bandwidth">
+  <properties>
+    <help>Available bandwidth for this policy</help>
+    <completionHelp>
+      <list>auto</list>
+    </completionHelp>
+    <valueHelp>
+      <format>auto</format>
+      <description>Rate matches interface speed</description>
+    </valueHelp>
+    <valueHelp>
+      <format>&lt;number&gt;</format>
+      <description>Bits per second</description>
+    </valueHelp>
+    <valueHelp>
+      <format>&lt;number&gt;bit</format>
+      <description>Bits per second</description>
+    </valueHelp>
+    <valueHelp>
+      <format>&lt;number&gt;kbit</format>
+      <description>Kilobits per second</description>
+    </valueHelp>
+    <valueHelp>
+      <format>&lt;number&gt;mbit</format>
+      <description>Megabits per second</description>
+    </valueHelp>
+    <valueHelp>
+      <format>&lt;number&gt;gbit</format>
+      <description>Gigabits per second</description>
+    </valueHelp>
+    <valueHelp>
+      <format>&lt;number&gt;tbit</format>
+      <description>Terabits per second</description>
+    </valueHelp>
+    <constraint>
+      <validator name="numeric" argument="--positive"/>
+      <regex>\d+(bit|kbit|mbit|gbit|tbit)</regex>
+    </constraint>
+  </properties>
+  <defaultValue>auto</defaultValue>
+</leafNode>
+<!-- include end -->
diff --git a/interface-definitions/include/qos/bandwidth.xml.i b/interface-definitions/include/qos/bandwidth.xml.i
index 82af22f42..62ea93b67 100644
--- a/interface-definitions/include/qos/bandwidth.xml.i
+++ b/interface-definitions/include/qos/bandwidth.xml.i
@@ -1,15 +1,35 @@
 <!-- include start from qos/bandwidth.xml.i -->
 <leafNode name="bandwidth">
   <properties>
-    <help>Traffic-limit used for this class</help>
+    <help>Available bandwidth for this policy</help>
     <valueHelp>
       <format>&lt;number&gt;</format>
-      <description>Rate in kbit (kilobit per second)</description>
+      <description>Bits per second</description>
     </valueHelp>
     <valueHelp>
-      <format>&lt;number&gt;&lt;suffix&gt;</format>
-      <description>Rate with scaling suffix (mbit, mbps, ...)</description>
+      <format>&lt;number&gt;bit</format>
+      <description>Bits per second</description>
     </valueHelp>
+    <valueHelp>
+      <format>&lt;number&gt;kbit</format>
+      <description>Kilobits per second</description>
+    </valueHelp>
+    <valueHelp>
+      <format>&lt;number&gt;mbit</format>
+      <description>Megabits per second</description>
+    </valueHelp>
+    <valueHelp>
+      <format>&lt;number&gt;gbit</format>
+      <description>Gigabits per second</description>
+    </valueHelp>
+    <valueHelp>
+      <format>&lt;number&gt;tbit</format>
+      <description>Terabits per second</description>
+    </valueHelp>
+    <constraint>
+      <validator name="numeric" argument="--positive"/>
+      <regex>\d+(bit|kbit|mbit|gbit|tbit)</regex>
+    </constraint>
   </properties>
 </leafNode>
 <!-- include end -->
diff --git a/interface-definitions/include/qos/class-match-ipv4-address.xml.i b/interface-definitions/include/qos/class-match-ipv4-address.xml.i
new file mode 100644
index 000000000..8e84c988a
--- /dev/null
+++ b/interface-definitions/include/qos/class-match-ipv4-address.xml.i
@@ -0,0 +1,19 @@
+<!-- include start from qos/class-match-ipv4-address.xml.i -->
+<leafNode name="address">
+  <properties>
+    <help>IPv4 destination address for this match</help>
+    <valueHelp>
+      <format>ipv4</format>
+      <description>IPv4 address</description>
+    </valueHelp>
+    <valueHelp>
+      <format>ipv4net</format>
+      <description>IPv4 prefix</description>
+    </valueHelp>
+    <constraint>
+      <validator name="ipv4-address"/>
+      <validator name="ipv4-prefix"/>
+    </constraint>
+  </properties>
+</leafNode>
+<!-- include end -->
diff --git a/interface-definitions/include/qos/class-match-ipv6-address.xml.i b/interface-definitions/include/qos/class-match-ipv6-address.xml.i
new file mode 100644
index 000000000..fd7388127
--- /dev/null
+++ b/interface-definitions/include/qos/class-match-ipv6-address.xml.i
@@ -0,0 +1,14 @@
+<!-- include start from qos/class-match-ipv6-address.xml.i -->
+<leafNode name="address">
+  <properties>
+    <help>IPv6 destination address for this match</help>
+    <valueHelp>
+      <format>ipv6net</format>
+      <description>IPv6 address and prefix length</description>
+    </valueHelp>
+    <constraint>
+      <validator name="ipv6"/>
+    </constraint>
+  </properties>
+</leafNode>
+<!-- include end -->
diff --git a/interface-definitions/include/qos/class-match.xml.i b/interface-definitions/include/qos/class-match.xml.i
new file mode 100644
index 000000000..d9c35731d
--- /dev/null
+++ b/interface-definitions/include/qos/class-match.xml.i
@@ -0,0 +1,177 @@
+<!-- include start from qos/class-match.xml.i -->
+<tagNode name="match">
+  <properties>
+    <help>Class matching rule name</help>
+    <constraint>
+      <regex>[^-].*</regex>
+    </constraint>
+    <constraintErrorMessage>Match queue name cannot start with hyphen (-)</constraintErrorMessage>
+  </properties>
+  <children>
+    #include <include/generic-description.xml.i>
+    <node name="ether">
+      <properties>
+        <help>Ethernet header match</help>
+      </properties>
+      <children>
+        <leafNode name="destination">
+          <properties>
+            <help>Ethernet destination address for this match</help>
+            <valueHelp>
+              <format>macaddr</format>
+              <description>MAC address to match</description>
+            </valueHelp>
+            <constraint>
+              <validator name="mac-address"/>
+            </constraint>
+          </properties>
+        </leafNode>
+        <leafNode name="protocol">
+          <properties>
+            <help>Ethernet protocol for this match</help>
+            <!-- this refers to /etc/protocols -->
+            <completionHelp>
+              <list>all 802.1Q 802_2 802_3 aarp aoe arp atalk dec ip ipv6 ipx lat localtalk rarp snap x25</list>
+            </completionHelp>
+            <valueHelp>
+              <format>u32:0-65535</format>
+              <description>Ethernet protocol number</description>
+            </valueHelp>
+            <valueHelp>
+              <format>txt</format>
+              <description>Ethernet protocol name</description>
+            </valueHelp>
+            <valueHelp>
+              <format>all</format>
+              <description>Any protocol</description>
+            </valueHelp>
+            <valueHelp>
+              <format>ip</format>
+              <description>Internet IP (IPv4)</description>
+            </valueHelp>
+            <valueHelp>
+              <format>ipv6</format>
+              <description>Internet IP (IPv6)</description>
+            </valueHelp>
+            <valueHelp>
+              <format>arp</format>
+              <description>Address Resolution Protocol</description>
+            </valueHelp>
+            <valueHelp>
+              <format>atalk</format>
+              <description>Appletalk</description>
+            </valueHelp>
+            <valueHelp>
+              <format>ipx</format>
+              <description>Novell Internet Packet Exchange</description>
+            </valueHelp>
+            <valueHelp>
+              <format>802.1Q</format>
+              <description>802.1Q VLAN tag</description>
+            </valueHelp>
+            <constraint>
+              <validator name="ip-protocol"/>
+            </constraint>
+          </properties>
+        </leafNode>
+        <leafNode name="source">
+          <properties>
+            <help>Ethernet source address for this match</help>
+            <valueHelp>
+              <format>macaddr</format>
+              <description>MAC address to match</description>
+            </valueHelp>
+            <constraint>
+              <validator name="mac-address"/>
+            </constraint>
+          </properties>
+        </leafNode>
+      </children>
+    </node>
+    #include <include/generic-interface.xml.i>
+    <node name="ip">
+      <properties>
+        <help>Match IP protocol header</help>
+      </properties>
+      <children>
+        <node name="destination">
+          <properties>
+            <help>Match on destination port or address</help>
+          </properties>
+          <children>
+            #include <include/qos/class-match-ipv4-address.xml.i>
+            #include <include/port-number.xml.i>
+          </children>
+        </node>
+        #include <include/qos/match-dscp.xml.i>
+        #include <include/qos/max-length.xml.i>
+        #include <include/ip-protocol.xml.i>
+        <node name="source">
+          <properties>
+            <help>Match on source port or address</help>
+          </properties>
+          <children>
+            #include <include/qos/class-match-ipv4-address.xml.i>
+            #include <include/port-number.xml.i>
+          </children>
+        </node>
+        #include <include/qos/tcp-flags.xml.i>
+      </children>
+    </node>
+    <node name="ipv6">
+      <properties>
+        <help>Match IPv6 protocol header</help>
+      </properties>
+      <children>
+        <node name="destination">
+          <properties>
+            <help>Match on destination port or address</help>
+          </properties>
+          <children>
+            #include <include/qos/class-match-ipv6-address.xml.i>
+            #include <include/port-number.xml.i>
+          </children>
+        </node>
+        #include <include/qos/match-dscp.xml.i>
+        #include <include/qos/max-length.xml.i>
+        #include <include/ip-protocol.xml.i>
+        <node name="source">
+          <properties>
+            <help>Match on source port or address</help>
+          </properties>
+          <children>
+            #include <include/qos/class-match-ipv6-address.xml.i>
+            #include <include/port-number.xml.i>
+          </children>
+        </node>
+        #include <include/qos/tcp-flags.xml.i>
+      </children>
+    </node>
+    <leafNode name="mark">
+      <properties>
+        <help>Match on mark applied by firewall</help>
+        <valueHelp>
+          <format>txt</format>
+          <description>FW mark to match</description>
+        </valueHelp>
+        <constraint>
+          <validator name="numeric" argument="--range 0x0-0xffff"/>
+        </constraint>
+      </properties>
+    </leafNode>
+    <leafNode name="vif">
+      <properties>
+        <help>Virtual Local Area Network (VLAN) ID for this match</help>
+        <valueHelp>
+          <format>u32:0-4095</format>
+          <description>Virtual Local Area Network (VLAN) tag </description>
+        </valueHelp>
+        <constraint>
+          <validator name="numeric" argument="--range 0-4095"/>
+        </constraint>
+        <constraintErrorMessage>VLAN ID must be between 0 and 4095</constraintErrorMessage>
+      </properties>
+    </leafNode>
+  </children>
+</tagNode>
+<!-- include end -->
diff --git a/interface-definitions/include/qos/class-police-exceed.xml.i b/interface-definitions/include/qos/class-police-exceed.xml.i
new file mode 100644
index 000000000..ee2ce16a8
--- /dev/null
+++ b/interface-definitions/include/qos/class-police-exceed.xml.i
@@ -0,0 +1,66 @@
+<!-- include start from qos/police.xml.i -->
+<leafNode name="exceed">
+  <properties>
+    <help>Default action for packets exceeding the limiter</help>
+    <completionHelp>
+      <list>continue drop ok reclassify pipe</list>
+    </completionHelp>
+    <valueHelp>
+      <format>continue</format>
+      <description>Do not do anything, just continue with the next action in line</description>
+    </valueHelp>
+    <valueHelp>
+      <format>drop</format>
+      <description>Drop the packet immediately</description>
+    </valueHelp>
+    <valueHelp>
+      <format>ok</format>
+      <description>Accept the packet</description>
+    </valueHelp>
+    <valueHelp>
+      <format>reclassify</format>
+      <description>Treat the packet as non-matching to the filter this action is attached to and continue with the next filter in line (if any)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>pipe</format>
+      <description>Pass the packet to the next action in line</description>
+    </valueHelp>
+    <constraint>
+      <regex>(continue|drop|ok|reclassify|pipe)</regex>
+    </constraint>
+  </properties>
+  <defaultValue>drop</defaultValue>
+</leafNode>
+<leafNode name="not-exceed">
+  <properties>
+    <help>Default action for packets not exceeding the limiter</help>
+    <completionHelp>
+      <list>continue drop ok reclassify pipe</list>
+    </completionHelp>
+    <valueHelp>
+      <format>continue</format>
+      <description>Do not do anything, just continue with the next action in line</description>
+    </valueHelp>
+    <valueHelp>
+      <format>drop</format>
+      <description>Drop the packet immediately</description>
+    </valueHelp>
+    <valueHelp>
+      <format>ok</format>
+      <description>Accept the packet</description>
+    </valueHelp>
+    <valueHelp>
+      <format>reclassify</format>
+      <description>Treat the packet as non-matching to the filter this action is attached to and continue with the next filter in line (if any)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>pipe</format>
+      <description>Pass the packet to the next action in line</description>
+    </valueHelp>
+    <constraint>
+      <regex>(continue|drop|ok|reclassify|pipe)</regex>
+    </constraint>
+  </properties>
+  <defaultValue>ok</defaultValue>
+</leafNode>
+<!-- include end -->
diff --git a/interface-definitions/include/qos/class-priority.xml.i b/interface-definitions/include/qos/class-priority.xml.i
new file mode 100644
index 000000000..3fd848c93
--- /dev/null
+++ b/interface-definitions/include/qos/class-priority.xml.i
@@ -0,0 +1,15 @@
+<!-- include start from qos/class-priority.xml.i -->
+<leafNode name="priority">
+  <properties>
+    <help>Priority for rule evaluation</help>
+    <valueHelp>
+      <format>u32:0-20</format>
+      <description>Priority for match rule evaluation</description>
+    </valueHelp>
+    <constraint>
+      <validator name="numeric" argument="--range 0-20"/>
+    </constraint>
+    <constraintErrorMessage>Priority must be between 0 and 20</constraintErrorMessage>
+  </properties>
+</leafNode>
+<!-- include end -->
diff --git a/interface-definitions/include/qos/dscp.xml.i b/interface-definitions/include/qos/dscp.xml.i
deleted file mode 100644
index bb90850ac..000000000
--- a/interface-definitions/include/qos/dscp.xml.i
+++ /dev/null
@@ -1,143 +0,0 @@
-<!-- include start from qos/dscp.xml.i -->
-<leafNode name="dscp">
-  <properties>
-    <help>Match on Differentiated Services Codepoint (DSCP)</help>
-    <completionHelp>
-      <list>default reliability throughput lowdelay priority immediate flash flash-override critical internet network AF11 AF12 AF13 AF21 AF22 AF23 AF31 AF32 AF33 AF41 AF42 AF43 CS1 CS2 CS3 CS4 CS5 CS6 CS7 EF</list>
-    </completionHelp>
-    <valueHelp>
-      <format>u32:0-63</format>
-      <description>Differentiated Services Codepoint (DSCP) value </description>
-    </valueHelp>
-    <valueHelp>
-      <format>default</format>
-      <description>match DSCP (000000)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>reliability</format>
-      <description>match DSCP (000001)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>throughput</format>
-      <description>match DSCP (000010)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>lowdelay</format>
-      <description>match DSCP (000100)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>priority</format>
-      <description>match DSCP (001000)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>immediate</format>
-      <description>match DSCP (010000)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>flash</format>
-      <description>match DSCP (011000)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>flash-override</format>
-      <description>match DSCP (100000)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>critical</format>
-      <description>match DSCP (101000)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>internet</format>
-      <description>match DSCP (110000)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>network</format>
-      <description>match DSCP (111000)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF11</format>
-      <description>High-throughput data</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF12</format>
-      <description>High-throughput data</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF13</format>
-      <description>High-throughput data</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF21</format>
-      <description>Low-latency data</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF22</format>
-      <description>Low-latency data</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF23</format>
-      <description>Low-latency data</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF31</format>
-      <description>Multimedia streaming</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF32</format>
-      <description>Multimedia streaming</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF33</format>
-      <description>Multimedia streaming</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF41</format>
-      <description>Multimedia conferencing</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF42</format>
-      <description>Multimedia conferencing</description>
-    </valueHelp>
-    <valueHelp>
-      <format>AF43</format>
-      <description>Multimedia conferencing</description>
-    </valueHelp>
-    <valueHelp>
-      <format>CS1</format>
-      <description>Low-priority data</description>
-    </valueHelp>
-    <valueHelp>
-      <format>CS2</format>
-      <description>OAM</description>
-    </valueHelp>
-    <valueHelp>
-      <format>CS3</format>
-      <description>Broadcast video</description>
-    </valueHelp>
-    <valueHelp>
-      <format>CS4</format>
-      <description>Real-time interactive</description>
-    </valueHelp>
-    <valueHelp>
-      <format>CS5</format>
-      <description>Signaling</description>
-    </valueHelp>
-    <valueHelp>
-      <format>CS6</format>
-      <description>Network control</description>
-    </valueHelp>
-    <valueHelp>
-      <format>CS7</format>
-      <description></description>
-    </valueHelp>
-    <valueHelp>
-      <format>EF</format>
-      <description>Expedited Forwarding</description>
-    </valueHelp>
-    <constraint>
-      <validator name="numeric" argument="--range 0-63"/>
-      <regex>(default|reliability|throughput|lowdelay|priority|immediate|flash|flash-override|critical|internet|network|AF11|AF12|AF13|AF21|AF22|AF23|AF31|AF32|AF33|AF41|AF42|AF43|CS1|CS2|CS3|CS4|CS5|CS6|CS7|EF)</regex>
-    </constraint>
-    <constraintErrorMessage>Priority must be between 0 and 63</constraintErrorMessage>
-  </properties>
-</leafNode>
-<!-- include end -->
diff --git a/interface-definitions/include/qos/limiter-actions.xml.i b/interface-definitions/include/qos/limiter-actions.xml.i
deleted file mode 100644
index a993423aa..000000000
--- a/interface-definitions/include/qos/limiter-actions.xml.i
+++ /dev/null
@@ -1,66 +0,0 @@
-<!-- include start from qos/limiter-actions.xml.i -->
-<leafNode name="exceed-action">
-  <properties>
-    <help>Default action for packets exceeding the limiter (default: drop)</help>
-    <completionHelp>
-      <list>continue drop ok reclassify pipe</list>
-    </completionHelp>
-    <valueHelp>
-      <format>continue</format>
-      <description>Don't do anything, just continue with the next action in line</description>
-    </valueHelp>
-    <valueHelp>
-      <format>drop</format>
-      <description>Drop the packet immediately</description>
-    </valueHelp>
-    <valueHelp>
-      <format>ok</format>
-      <description>Accept the packet</description>
-    </valueHelp>
-    <valueHelp>
-      <format>reclassify</format>
-      <description>Treat the packet as non-matching to the filter this action is attached to and continue with the next filter in line (if any)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>pipe</format>
-      <description>Pass the packet to the next action in line</description>
-    </valueHelp>
-    <constraint>
-      <regex>(continue|drop|ok|reclassify|pipe)</regex>
-    </constraint>
-  </properties>
-  <defaultValue>drop</defaultValue>
-</leafNode>
-<leafNode name="notexceed-action">
-  <properties>
-    <help>Default action for packets not exceeding the limiter (default: ok)</help>
-    <completionHelp>
-      <list>continue drop ok reclassify pipe</list>
-    </completionHelp>
-    <valueHelp>
-      <format>continue</format>
-      <description>Don't do anything, just continue with the next action in line</description>
-    </valueHelp>
-    <valueHelp>
-      <format>drop</format>
-      <description>Drop the packet immediately</description>
-    </valueHelp>
-    <valueHelp>
-      <format>ok</format>
-      <description>Accept the packet</description>
-    </valueHelp>
-    <valueHelp>
-      <format>reclassify</format>
-      <description>Treat the packet as non-matching to the filter this action is attached to and continue with the next filter in line (if any)</description>
-    </valueHelp>
-    <valueHelp>
-      <format>pipe</format>
-      <description>Pass the packet to the next action in line</description>
-    </valueHelp>
-    <constraint>
-      <regex>(continue|drop|ok|reclassify|pipe)</regex>
-    </constraint>
-  </properties>
-  <defaultValue>ok</defaultValue>
-</leafNode>
-<!-- include end -->
diff --git a/interface-definitions/include/qos/match-dscp.xml.i b/interface-definitions/include/qos/match-dscp.xml.i
new file mode 100644
index 000000000..1323fc033
--- /dev/null
+++ b/interface-definitions/include/qos/match-dscp.xml.i
@@ -0,0 +1,142 @@
+<!-- include start from qos/match-dscp.xml.i -->
+<leafNode name="dscp">
+  <properties>
+    <help>Match on Differentiated Services Codepoint (DSCP)</help>
+    <completionHelp>
+      <list>default reliability throughput lowdelay priority immediate flash flash-override critical internet network af11 af12 af13 af21 af22 af23 af31 af32 af33 af41 af42 af43 cs1 cs2 cs3 cs4 cs5 cs6 cs7 ef</list>
+    </completionHelp>
+    <valueHelp>
+      <format>u32:0-63</format>
+      <description>Differentiated Services Codepoint (DSCP) value </description>
+    </valueHelp>
+    <valueHelp>
+      <format>default</format>
+      <description>match DSCP (000000)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>reliability</format>
+      <description>match DSCP (000001)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>throughput</format>
+      <description>match DSCP (000010)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>lowdelay</format>
+      <description>match DSCP (000100)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>priority</format>
+      <description>match DSCP (001000)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>immediate</format>
+      <description>match DSCP (010000)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>flash</format>
+      <description>match DSCP (011000)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>flash-override</format>
+      <description>match DSCP (100000)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>critical</format>
+      <description>match DSCP (101000)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>internet</format>
+      <description>match DSCP (110000)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>network</format>
+      <description>match DSCP (111000)</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af11</format>
+      <description>High-throughput data</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af12</format>
+      <description>High-throughput data</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af13</format>
+      <description>High-throughput data</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af21</format>
+      <description>Low-latency data</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af22</format>
+      <description>Low-latency data</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af23</format>
+      <description>Low-latency data</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af31</format>
+      <description>Multimedia streaming</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af32</format>
+      <description>Multimedia streaming</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af33</format>
+      <description>Multimedia streaming</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af41</format>
+      <description>Multimedia conferencing</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af42</format>
+      <description>Multimedia conferencing</description>
+    </valueHelp>
+    <valueHelp>
+      <format>af43</format>
+      <description>Multimedia conferencing</description>
+    </valueHelp>
+    <valueHelp>
+      <format>cs1</format>
+      <description>Low-priority data</description>
+    </valueHelp>
+    <valueHelp>
+      <format>cs2</format>
+      <description>OAM</description>
+    </valueHelp>
+    <valueHelp>
+      <format>cs3</format>
+      <description>Broadcast video</description>
+    </valueHelp>
+    <valueHelp>
+      <format>cs4</format>
+      <description>Real-time interactive</description>
+    </valueHelp>
+    <valueHelp>
+      <format>cs5</format>
+      <description>Signaling</description>
+    </valueHelp>
+    <valueHelp>
+      <format>cs6</format>
+      <description>Network control</description>
+    </valueHelp>
+    <valueHelp>
+      <format>cs7</format>
+      <description></description>
+    </valueHelp>
+    <valueHelp>
+      <format>ef</format>
+      <description>Expedited Forwarding</description>
+    </valueHelp>
+    <constraint>
+      <validator name="numeric" argument="--range 0-63"/>
+      <regex>(default|reliability|throughput|lowdelay|priority|immediate|flash|flash-override|critical|internet|network|af11|af12|af13|af21|af22|af23|af31|af32|af33|af41|af42|af43|cs1|cs2|cs3|cs4|cs5|cs6|cs7|ef)</regex>
+    </constraint>
+  </properties>
+</leafNode>
+<!-- include end -->
diff --git a/interface-definitions/include/qos/match.xml.i b/interface-definitions/include/qos/match.xml.i
deleted file mode 100644
index 7d89e4460..000000000
--- a/interface-definitions/include/qos/match.xml.i
+++ /dev/null
@@ -1,221 +0,0 @@
-<!-- include start from qos/match.xml.i -->
-<tagNode name="match">
-  <properties>
-    <help>Class matching rule name</help>
-    <constraint>
-      <regex>[^-].*</regex>
-    </constraint>
-    <constraintErrorMessage>Match queue name cannot start with hyphen (-)</constraintErrorMessage>
-  </properties>
-  <children>
-    #include <include/generic-description.xml.i>
-    <node name="ether">
-      <properties>
-        <help>Ethernet header match</help>
-      </properties>
-      <children>
-        <leafNode name="destination">
-          <properties>
-            <help>Ethernet destination address for this match</help>
-            <valueHelp>
-              <format>macaddr</format>
-              <description>MAC address to match</description>
-            </valueHelp>
-            <constraint>
-              <validator name="mac-address"/>
-            </constraint>
-          </properties>
-        </leafNode>
-        <leafNode name="protocol">
-          <properties>
-            <help>Ethernet protocol for this match</help>
-            <!-- this refers to /etc/protocols -->
-            <completionHelp>
-              <list>all 802.1Q 802_2 802_3 aarp aoe arp atalk dec ip ipv6 ipx lat localtalk rarp snap x25</list>
-            </completionHelp>
-            <valueHelp>
-              <format>u32:0-65535</format>
-              <description>Ethernet protocol number</description>
-            </valueHelp>
-            <valueHelp>
-              <format>txt</format>
-              <description>Ethernet protocol name</description>
-            </valueHelp>
-            <valueHelp>
-              <format>all</format>
-              <description>Any protocol</description>
-            </valueHelp>
-            <valueHelp>
-              <format>ip</format>
-              <description>Internet IP (IPv4)</description>
-            </valueHelp>
-            <valueHelp>
-              <format>ipv6</format>
-              <description>Internet IP (IPv6)</description>
-            </valueHelp>
-            <valueHelp>
-              <format>arp</format>
-              <description>Address Resolution Protocol</description>
-            </valueHelp>
-            <valueHelp>
-              <format>atalk</format>
-              <description>Appletalk</description>
-            </valueHelp>
-            <valueHelp>
-              <format>ipx</format>
-              <description>Novell Internet Packet Exchange</description>
-            </valueHelp>
-            <valueHelp>
-              <format>802.1Q</format>
-              <description>802.1Q VLAN tag</description>
-            </valueHelp>
-            <constraint>
-              <validator name="ip-protocol"/>
-            </constraint>
-          </properties>
-        </leafNode>
-        <leafNode name="source">
-          <properties>
-            <help>Ethernet source address for this match</help>
-            <valueHelp>
-              <format>macaddr</format>
-              <description>MAC address to match</description>
-            </valueHelp>
-            <constraint>
-              <validator name="mac-address"/>
-            </constraint>
-          </properties>
-        </leafNode>
-      </children>
-    </node>
-    #include <include/generic-interface.xml.i>
-    <node name="ip">
-      <properties>
-        <help>Match IP protocol header</help>
-      </properties>
-      <children>
-        <node name="destination">
-          <properties>
-            <help>Match on destination port or address</help>
-          </properties>
-          <children>
-            <leafNode name="address">
-              <properties>
-                <help>IPv4 destination address for this match</help>
-                <valueHelp>
-                  <format>ipv4net</format>
-                  <description>IPv4 address and prefix length</description>
-                </valueHelp>
-                <constraint>
-                  <validator name="ipv4"/>
-                </constraint>
-              </properties>
-            </leafNode>
-            #include <include/port-number.xml.i>
-          </children>
-        </node>
-        #include <include/qos/dscp.xml.i>
-        #include <include/qos/max-length.xml.i>
-        #include <include/ip-protocol.xml.i>
-        <node name="source">
-          <properties>
-            <help>Match on source port or address</help>
-          </properties>
-          <children>
-            <leafNode name="address">
-              <properties>
-                <help>IPv4 source address for this match</help>
-                <valueHelp>
-                  <format>ipv4net</format>
-                  <description>IPv4 address and prefix length</description>
-                </valueHelp>
-                <constraint>
-                  <validator name="ipv4"/>
-                </constraint>
-              </properties>
-            </leafNode>
-            #include <include/port-number.xml.i>
-          </children>
-        </node>
-        #include <include/qos/tcp-flags.xml.i>
-      </children>
-    </node>
-    <node name="ipv6">
-      <properties>
-        <help>Match IPv6 protocol header</help>
-      </properties>
-      <children>
-        <node name="destination">
-          <properties>
-            <help>Match on destination port or address</help>
-          </properties>
-          <children>
-            <leafNode name="address">
-              <properties>
-                <help>IPv6 destination address for this match</help>
-                <valueHelp>
-                  <format>ipv6net</format>
-                  <description>IPv6 address and prefix length</description>
-                </valueHelp>
-                <constraint>
-                  <validator name="ipv6"/>
-                </constraint>
-              </properties>
-            </leafNode>
-            #include <include/port-number.xml.i>
-          </children>
-        </node>
-        #include <include/qos/dscp.xml.i>
-        #include <include/qos/max-length.xml.i>
-        #include <include/ip-protocol.xml.i>
-        <node name="source">
-          <properties>
-            <help>Match on source port or address</help>
-          </properties>
-          <children>
-            <leafNode name="address">
-              <properties>
-                <help>IPv6 source address for this match</help>
-                <valueHelp>
-                  <format>ipv6net</format>
-                  <description>IPv6 address and prefix length</description>
-                </valueHelp>
-                <constraint>
-                  <validator name="ipv6"/>
-                </constraint>
-              </properties>
-            </leafNode>
-            #include <include/port-number.xml.i>
-          </children>
-        </node>
-        #include <include/qos/tcp-flags.xml.i>
-      </children>
-    </node>
-    <leafNode name="mark">
-      <properties>
-        <help>Match on mark applied by firewall</help>
-        <valueHelp>
-          <format>txt</format>
-          <description>FW mark to match</description>
-        </valueHelp>
-        <constraint>
-          <validator name="numeric" argument="--range 0x0-0xffff"/>
-        </constraint>
-      </properties>
-    </leafNode>
-    <leafNode name="vif">
-      <properties>
-        <help>Virtual Local Area Network (VLAN) ID for this match</help>
-        <valueHelp>
-          <format>u32:0-4095</format>
-          <description>Virtual Local Area Network (VLAN) tag </description>
-        </valueHelp>
-        <constraint>
-          <validator name="numeric" argument="--range 0-4095"/>
-        </constraint>
-        <constraintErrorMessage>VLAN ID must be between 0 and 4095</constraintErrorMessage>
-      </properties>
-    </leafNode>
-  </children>
-</tagNode>
-<!-- include end -->
diff --git a/interface-definitions/include/qos/max-length.xml.i b/interface-definitions/include/qos/max-length.xml.i
index 4cc20f8c4..64cdd02ec 100644
--- a/interface-definitions/include/qos/max-length.xml.i
+++ b/interface-definitions/include/qos/max-length.xml.i
@@ -1,15 +1,15 @@
 <!-- include start from qos/max-length.xml.i -->
 <leafNode name="max-length">
   <properties>
-    <help>Maximum packet length (ipv4)</help>
+    <help>Maximum packet length</help>
     <valueHelp>
-      <format>u32:0-65535</format>
+      <format>u32:1-65535</format>
       <description>Maximum packet/payload length</description>
     </valueHelp>
     <constraint>
-      <validator name="numeric" argument="--range 0-65535"/>
+      <validator name="numeric" argument="--range 1-65535"/>
     </constraint>
-    <constraintErrorMessage>Maximum IPv4 total packet length is 65535</constraintErrorMessage>
+    <constraintErrorMessage>Maximum packet length is 65535</constraintErrorMessage>
   </properties>
 </leafNode>
 <!-- include end -->
diff --git a/interface-definitions/include/qos/queue-type.xml.i b/interface-definitions/include/qos/queue-type.xml.i
index 634f61024..c7d4cde82 100644
--- a/interface-definitions/include/qos/queue-type.xml.i
+++ b/interface-definitions/include/qos/queue-type.xml.i
@@ -3,28 +3,31 @@
   <properties>
     <help>Queue type for default traffic</help>
     <completionHelp>
-      <list>fq-codel fair-queue drop-tail random-detect</list>
+      <list>drop-tail fair-queue fq-codel priority random-detect</list>
     </completionHelp>
     <valueHelp>
-      <format>fq-codel</format>
-      <description>Fair Queue Codel</description>
+      <format>drop-tail</format>
+      <description>First-In-First-Out (FIFO)</description>
     </valueHelp>
     <valueHelp>
       <format>fair-queue</format>
       <description>Stochastic Fair Queue (SFQ)</description>
     </valueHelp>
     <valueHelp>
-      <format>drop-tail</format>
-      <description>First-In-First-Out (FIFO)</description>
+      <format>fq-codel</format>
+      <description>Fair Queue Codel</description>
+    </valueHelp>
+    <valueHelp>
+      <format>priority</format>
+      <description>Priority queuing</description>
     </valueHelp>
     <valueHelp>
       <format>random-detect</format>
       <description>Random Early Detection (RED)</description>
     </valueHelp>
     <constraint>
-      <regex>(fq-codel|fair-queue|drop-tail|random-detect)</regex>
+      <regex>(drop-tail|fair-queue|fq-codel|priority|random-detect)</regex>
     </constraint>
   </properties>
-  <defaultValue>drop-tail</defaultValue>
 </leafNode>
 <!-- include end -->
diff --git a/interface-definitions/include/version/qos-version.xml.i b/interface-definitions/include/version/qos-version.xml.i
index e4d139349..c67e61e91 100644
--- a/interface-definitions/include/version/qos-version.xml.i
+++ b/interface-definitions/include/version/qos-version.xml.i
@@ -1,3 +1,3 @@
 <!-- include start from include/version/qos-version.xml.i -->
-<syntaxVersion component='qos' version='1'></syntaxVersion>
+<syntaxVersion component='qos' version='2'></syntaxVersion>
 <!-- include end -->
-- 
cgit v1.2.3