Nitin, Ron, Kini and Jan:

 

Did you replace the nexthop-address and not remove this text? 

 

 

   <nexthop-address> ::= (<IPv4> <ipv4-address>) |

                         (<IPV6> <ipv6-address>) |

                         (<IEEE_MAC> <IEEE_MAC_ADDRESS>)

 

The paper only his this definition once?

 

Sue 

 

 

From: i2rs [mailto:[email protected]] On Behalf Of Susan Hares
Sent: Sunday, April 20, 2014 6:00 PM
To: [email protected]
Cc: 'Jeffrey Haas'; [email protected]; 'Alia Atlas'
Subject: Re: [i2rs] draft-ietf-i2rs-rib-info-model-02

 

Nitin, Ron, Kini and Jan:

 

I forgot to ask question on the grammar: 

 

<special-nexthop> :: = <DISCARD> | <DISCARD_WITH_ERROR> | (<RECEIVE>
[COS_VALUE] [<rate-limiter>]) 

 

Rater limited is not defined.  Is there a reason why this would be in a
standard document without a definition?  Did I miss something on the list? 

 

Sue 

 

From: i2rs [mailto:[email protected]] On Behalf Of Susan Hares
Sent: Sunday, April 20, 2014 5:31 PM
To: [email protected]
Cc: 'Jeffrey Haas'; [email protected]; Alia Atlas
Subject: [i2rs] draft-ietf-i2rs-rib-info-model-02

 

Nitin, Ron, Kini, Jan: 

 

1)      Is Section 7 of your document normative or informative? 

 

2)      Your grammar seems wordy/inconsistent in the repetition of the next
hop below 

 

Your RIB grammar on page 17 states: 

 

<nexthop-list> ::= <special-nexthop> | 

                                ((nexthop-list-member>) | 

                                 (<nexthop-list-member> . | <nexthop-list>))

 

<nexthop-list-member> ::= (<nexthop-chain> | 

 
<nexthop-chain-identifier>)

 
[<nexthop-list-member-attributes>]

 

<nexthop-chain>::= (<nexthop> .) 

 

Questions: Why do you have nexthop-list-member listed twice?  Why list
<nexthop-list> twice? Is this readability or technical matter? 

 

Why not: 

<nexthop-list>::= ((<special-nexthop> | (nexthop-list-member) . ) . 

 

Why not: 

<nexthop-list-member> ::= (((<nexthop-chain-identifier> | (<nexthops> .
))[<nexthop-list-member-attributes]) ). 

 

Were you trying to name the chains? 

 

3)      Two variables seem orphaned: 

 

Multicast-source-ipv4-address ::= <IPv4_ADDRESS> <IPV4_PREFIX_LENGTH>

Multicast-source-ipv6-address ::= <IPv6_ADDRESS> <IPv6_PREFIX_LENGTH>

 

         Did I miss someplace where they attached to the normative section
6. 

           You indicated the PIM paths can be chains (section 7.3), but you
do not give the normative section. 

 

Sue Hares 

 

_______________________________________________
i2rs mailing list
[email protected]
https://www.ietf.org/mailman/listinfo/i2rs

Reply via email to