Re: [Gen-art] Gen-ART Last Call review of draft-ietf-manet-olsrv2-dat-metric-08

[Paul Kyzivat <[e-mail]>]

Henning,

On 11/24/15 10:09 AM, Henning Rogge wrote:
> Am 24.11.2015 um 16:00 schrieb Paul Kyzivat:
>> So you are saying they are mutually exclusive: HELLO message by 9.4, all
>> other packets by 9.3. Can you make the text clearer about this. As
>> written it still could be interpreted as applying 9.3 to all packets,
>> including HELLO, and then applying 9.4 to HELLO packets. (Even though
>> this is nonsense if you think about it.)
>
> No, it is NOT exclusive...
>
> there is no Hello packet... there are just RFC5444 packets which might
> contain one (or in theory multiple) NHDP HELLO messages.
>
> Its the same with IP and UDP... every UDP header is contained in an IP
> header. We are talking about two different levels of hierarchy.
>
> DAT is processing both the RFC5444 packet header (which contains the
> packet sequence number) of ALL RFC5444 packets and the message content
> of the HELLO messages.

OK. But my confusion remains, because there seem to be duplicate actions 
in both 9.3 and 9.4 that ought not be done twice. In particular:

9.3:
    1.  If L_DAT_last_pkt_seqno = UNDEFINED, then:

        1.  L_DAT_received[TAIL] := 1.

        2.  L_DAT_total[TAIL] := 1.

9.4:
    3.  If L_DAT_last_pkt_seqno = UNDEFINED, then:

        1.  L_DAT_received[TAIL] := L_DAT_received[TAIL] + 1.

        2.  L_DAT_total[TAIL] := L_DAT_total[TAIL] + 1.

So, if I do 9.3 for every packet, and then also do 9.4 for every HELLO 
message within that packet, then I could end up incrementing 
L_DAT_received[TAIL] *twice*.

	Thanks,
	Paul