Re: A question about RFC5884

[Greg Mirsky <[e-mail]>]

Hi Carlos,
please find in-lined interpretation of RFC 5884 paragraph tagged GIM>>.

Regards,
Greg

On Mon, Jul 17, 2017 at 8:30 AM, Carlos Pignataro (cpignata) <
[e-mail]> wrote:

> Greg,
>
> I am sorry but I don't see how the paragraph supports what you say. Two
> issues:
>
> 1. LSP Ping is based on the Normative reference's spec, RFC 4379. It
> cannot go against it unless it updates its behavior. The following text:
>
> "The egress LSR MAY respond with an LSP Ping Echo
> reply message that carries the local discriminator assigned by it for
>    the BFD session."
>
> Also has the interpretation that Santosh shared, which is "MAY send a
> response including a TLV, but sending it is not optional"
>
> 2. You wrote a MUST in your reply with specific ordering of packet
> responses. MUSTs are for interoperability. The text does not talk about
> order of packets. Where is that coming from?
>
> It is unhelpful to mention references without citing them, and in any
> case, I do not believe the text supports your conclusion.
>
> Sent from my iPad
>
> On Jul 17, 2017, at 5:14 PM, Greg Mirsky <[e-mail]> wrote:
>
> Hi Carlos,
> it would take me time to dig that old discussion. I strongly believe that
> the wording and the order of listing actions in this paragraph of Section 6
> RFC 5884 supports my interpretation and recollection of the discussion:
>
>    On receipt of the LSP Ping Echo request message, the egress LSR MUST
>    send a BFD Control packet to the ingress LSR, if the validation of
>    the FEC in the LSP Ping Echo request message succeeds.
>
> GIM>> The LSP Ping Echo is validated. If the validation fails, then the
LSP Echo Reply MUST be sent with the appropriate Return Code. If the
validation succeeds, then the BFD control packet MUST be sent.

> This BFD
>    Control packet MUST set the Your Discriminator field to the
>    discriminator received from the ingress LSR in the LSP Ping Echo
>    request message.
>
> GIM>> This describes how Your Discriminator field of the BFD control
packet to be transmitted by the egress LSR MUST be populated as it will be
used by the ingress LSR to demultiplex BFD sessions..

>   The egress LSR MAY respond with an LSP Ping Echo
>    reply message that carries the local discriminator assigned by it for
>    the BFD session.
>
> GIM>> After the egress LSR is done with sending the first BFD control
packet it MAY send LSP Ping Echo reply with already allocated and included
in BFD control packet My Discriminator as value of BFD Discriminator TLV.

> The local discriminator assigned by the egress LSR
>    MUST be used as the My Discriminator field in the BFD session packets
>    sent by the egress LSR.
>
>
>
> Regards,
> Greg
>
> On Mon, Jul 17, 2017 at 8:02 AM, Carlos Pignataro (cpignata) <
> [e-mail]> wrote:
>
>> Greg,
>>
>> Pointer?
>>
>> Thanks,
>>
>> Sent from my iPad
>>
>> On Jul 17, 2017, at 9:34 AM, Greg Mirsky <[e-mail]> wrote:
>>
>> Hi Mach, et. al,
>> I recall that this question was discussed some time ago and the
>> clarification came from the original authors of the BFD protocol. The Echo
>> Reply is optional if there's no error to report. But if the remote LER,
>> acting as BFD node, does decide to send the Echo Reply it MUST send it
>> after is sends the first BFD control message.
>>
>> Regards,
>> Greg
>>
>> On Sun, Jul 16, 2017 at 6:58 AM, Mach Chen <[e-mail]> wrote:
>>
>>> Hi BFDers,
>>>
>>> We met a multi-vendor interoperate issue recently, it's about whether an
>>> Echo reply is necessary.
>>>
>>> In Section 6 of RFC5884, 2nd paragraph
>>>
>>> "... The egress LSR MAY respond with an LSP Ping Echo
>>>    reply message that carries the local discriminator assigned by it for
>>>    the BFD session."
>>>
>>> >From the above text, my understanding is that an Echo reply is
>>> optional, the egress LSR can freely to return or not return an Echo reply,
>>> and the Ingress LSR should not expect there MUST be an Echo reply, but if
>>> there is one, it should handle it properly.
>>>
>>> Is my understanding correct?
>>>
>>> Thanks,
>>> Mach
>>>
>>>
>>
>