[ovs-dev] [PATCH 4/6] cfm: Reduce missed CCM detection time.

Ethan Jackson ethan at nicira.com
Mon Mar 28 18:30:42 UTC 2011


Sounds good to me.  I will parenthesize it to be clear:
(cfmi->ccm_interval_ms * 7) / 2

Ethan

On Mon, Mar 28, 2011 at 9:28 AM, Ben Pfaff <blp at nicira.com> wrote:
> On Fri, Mar 25, 2011 at 04:39:27PM -0700, Ethan Jackson wrote:
>> The specification says that a fault should be signaled when 3.5 *
>> ccm_interval milliseconds have passed.  This commit respects that
>> requirement, possibly increasing the responsiveness of fault
>> detection slightly.
>
> Would you mind writing this as cfmi->ccm_interval_ms * 7 / 2 instead
> of cfmi->ccm_interval_ms * 3.5?  I realize that the latter is probably
> efficient enough these days, but it still bothers me a little.
>
> Otherwise looks fine to me.  Thank you!
>



More information about the dev mailing list