Subject: [Diffserv] Comment on DIFFSERV-MIB (draft-ietf-diffserv-mib-09.txt)

From: Frank Strauss <[email-masked]>
Date: 01 Jun 2001 19:36:03 +0200

Hi!

Although I don't have the big picture of the MIB, I stumbled on the
fact that the columns of the diffServSixTupleClfrTable are sorted in a
strange way. This is in contradiction with the DESCRIPTION of
INET-ADDRESS-MIB::InetAddress (`The InetAddressType object which
defines the context MUST be registered immediately before the object
which uses the InetAddress textual convention.').  I propose the
following changes.

Since the MIB is part of an I-D and thus the patch might not fit other
people's MIB files, I've put a complete version at
http://www.ibr.cs.tu-bs.de/~strauss/DIFFSERV-MIB.

 -frank

--- DIFFSERV-MIB.orig	Fri Jun  1 18:21:22 2001
+++ DIFFSERV-MIB	Fri Jun  1 19:25:38 2001
@@ -520,11 +520,11 @@
 DiffServSixTupleClfrEntry ::= SEQUENCE {
     diffServSixTupleClfrId           Unsigned32,
     diffServSixTupleClfrDstAddrType  InetAddressType,
-    diffServSixTupleClfrDstPrefixLength InetAddressPrefixLength,
     diffServSixTupleClfrDstAddr      InetAddress,
+    diffServSixTupleClfrDstPrefixLength InetAddressPrefixLength,
     diffServSixTupleClfrSrcAddrType  InetAddressType,
-    diffServSixTupleClfrSrcPrefixLength InetAddressPrefixLength,
     diffServSixTupleClfrSrcAddr      InetAddress,
+    diffServSixTupleClfrSrcPrefixLength InetAddressPrefixLength,
     diffServSixTupleClfrDscp         DscpOrAny,
     diffServSixTupleClfrProtocol     Unsigned32,
     diffServSixTupleClfrDstL4PortMin InetPortNumber,
@@ -554,11 +554,17 @@
        entry."
     ::= { diffServSixTupleClfrEntry 2 }
 
-diffServSixTupleClfrDstPrefixLength OBJECT-TYPE
-
-
-
+diffServSixTupleClfrDstAddr OBJECT-TYPE
+    SYNTAX         InetAddress
+    MAX-ACCESS     read-create
+    STATUS         current
+    DESCRIPTION
+       "The IP address to match against the packet's destination IP
+       address.  diffServSixTupleClfrDstPrefixLength indicates the
+       number of bits that are relevant."
+    ::= { diffServSixTupleClfrEntry 3 }
 
+diffServSixTupleClfrDstPrefixLength OBJECT-TYPE
     SYNTAX         InetAddressPrefixLength
     UNITS          "bits"
     MAX-ACCESS     read-create
@@ -571,7 +577,7 @@
        indicates the use of a CIDR Prefix.  IPv6 is similar, except that
        prefix lengths range from 0..128."
     DEFVAL         { 0 }
-    ::= { diffServSixTupleClfrEntry 3 }
+    ::= { diffServSixTupleClfrEntry 4 }
 
 diffServSixTupleClfrSrcAddrType OBJECT-TYPE
     SYNTAX         InetAddressType
@@ -579,17 +585,17 @@
     STATUS         current
     DESCRIPTION
        "The type of IP source address used by this classifier entry."
-    ::= { diffServSixTupleClfrEntry 4 }
+    ::= { diffServSixTupleClfrEntry 5 }
 
-diffServSixTupleClfrDstAddr OBJECT-TYPE
+diffServSixTupleClfrSrcAddr OBJECT-TYPE
     SYNTAX         InetAddress
     MAX-ACCESS     read-create
     STATUS         current
     DESCRIPTION
-       "The IP address to match against the packet's destination IP
-       address.  diffServSixTupleClfrDstPrefixLength indicates the
-       number of bits that are relevant."
-    ::= { diffServSixTupleClfrEntry 5 }
+       "The IP address to match against the packet's source IP address.
+       diffServSixTupleClfrSrcPrefixLength indicates the number of bits
+       that are relevant."
+    ::= { diffServSixTupleClfrEntry 6 }
 
 diffServSixTupleClfrSrcPrefixLength OBJECT-TYPE
     SYNTAX         InetAddressPrefixLength
@@ -604,20 +610,6 @@
        indicates the use of a CIDR Prefix.  IPv6 is similar, except that
        prefix lengths range from 0..128."
     DEFVAL         { 0 }
-    ::= { diffServSixTupleClfrEntry 6 }
-
-
-
-
-
-diffServSixTupleClfrSrcAddr OBJECT-TYPE
-    SYNTAX         InetAddress
-    MAX-ACCESS     read-create
-    STATUS         current
-    DESCRIPTION
-       "The IP address to match against the packet's source IP address.
-       diffServSixTupleClfrSrcPrefixLength indicates the number of bits
-       that are relevant."
     ::= { diffServSixTupleClfrEntry 7 }
 
 diffServSixTupleClfrDscp OBJECT-TYPE

_______________________________________________
diffserv mailing list
[email-masked]
http://www1.ietf.org/mailman/listinfo/diffserv
Archive: http://www2.ietf.org/mail-archive/working-groups/diffserv/current/maillist.html