Dear Authors:

The "acl-base" identity defined in your draft is empty (i.e. only with a 
description) . Then you define "ipv4-acl" to be "acl-base". So basically you 
inherited the comments twice.

identity acl-base {
description
"Base Access Control List type for all Access Control List type
identifiers.";
}
identity ipv4-acl {
base acl:acl-base;
description
"ACL that primarily matches on fields from the IPv4 header
(e.g. IPv4 destination address) and layer 4 headers (e.g. TCP
destination port). An acl of type ipv4-acl does not contain
matches on fields in the ethernet header or the IPv6 header.";
}
identity ipv6-acl {
base acl:acl-base;
description
"ACL that primarily matches on fields from the IPv6 header
(e.g. IPv6 destination address) and layer 4 headers (e.g. TCP
destination port). An acl of type ipv6-acl does not contain
matches on fields in the ethernet header or the IPv4 header.";
}


You really don't need to define the "acl-base". What is the impact if defining 
the "ipv4-acl" and "ipv6-acl" as follows?

identity ipv4-acl {
   description
   "ACL that primarily matches on fields from the IPv4 header
   (e.g. IPv4 destination address) and layer 4 headers (e.g. TCP
   destination port). An acl of type ipv4-acl does not contain
   matches on fields in the ethernet header or the IPv6 header.";
}
identity ipv6-acl {
   description
   "ACL that primarily matches on fields from the IPv6 header
   (e.g. IPv6 destination address) and layer 4 headers (e.g. TCP
   destination port). An acl of type ipv6-acl does not contain
   matches on fields in the ethernet header or the IPv4 header.";
}


Thanks, Linda Dunbar
_______________________________________________
netmod mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/netmod

Reply via email to