[ovs-dev] [PATCH] cfm: Remove sequence fault reason.

Ethan Jackson ethan at nicira.com
Tue Jul 10 03:13:51 UTC 2012


Thanks for the review Ben, I've merged this.

Ethan

On Mon, Jul 9, 2012 at 7:43 PM, Ben Pfaff <blp at nicira.com> wrote:
> On Mon, Jul 09, 2012 at 06:10:04PM -0700, Ethan Jackson wrote:
>> Commit 2b540ecb (Added handling of previously ignored cfm faults.)
>> made the CFM code trigger a fault when a packet is received with an
>> out of order sequence number.  This means that if even one CFM
>> probe is dropped, a fault will be triggered because the next
>> received probe's sequence would be two greater than the last.  This
>> is in conflict with the 802.1ag requirement that 3.5 dropped probes
>> triggers fault.
>>
>> Signed-off-by: Ethan Jackson <ethan at nicira.com>
>
> Looks good to me.  Good catch, thank you.



More information about the dev mailing list