On 2/17/2021 4:01 AM, Harlan Stenn wrote:
> Hi Dhruv,
>
> On 2/17/2021 3:52 AM, Dhruv Dhody wrote:
>> Hi Harlan,
>>
>>>>> leaf poll {
>>>>> type uint8;
>>>>> This seems wrong to me. Looking at s.7.3
>>>>> Poll: 8-bit signed integer representing the maximum interval between
>>>>> successive messages, in log2 seconds. Suggested default limits for
>>>>> minimum and maximum poll intervals are 6 and 10, respectively.
>>>>> ie signed integer and not uint
>>>>>
>>>>
>>>> int8 is better, updated.
>>>
>>> Why "default limits" and not "default values"?
>>
>> The text with the term "default limits" is from RFC 5905 (section 7.3).
>
> Different context.
>
> I just re-read -13 and it does not use "limits", which is good.
>
> I am wondering who will be confused if we see 6/10 used, and then
> somebody notices that the iburst flag is in play so the initiating side
> polls every 2 seconds (poll=2) up to 8 times even though minpoll is 6.
Perhaps changing:
description
"The minimum poll interval used in this association.";
to:
description
"The minimum poll interval normally used in this association.";
>> In the YANG model, we have the leaves minpoll and maxpoll with default
>> values as 6 and 10 the leaf poll is the current polling interval.
>
> Yes, thanks.
>
>> Thanks!
>> Dhruv
>>
>
--
Harlan Stenn <stenn@xxxxxxxxxx>
http://networktimefoundation.org - be a member!
--
last-call mailing list
last-call@xxxxxxxx
https://www.ietf.org/mailman/listinfo/last-call
