[ovs-dev] [PATCH 4/5] ofproto.at: Fix races in rule eviciton tests

Ben Pfaff blp at nicira.com
Mon Mar 31 15:08:12 UTC 2014


On Mon, Mar 31, 2014 at 03:24:30PM +0900, YAMAMOTO Takashi wrote:
> Bump timeout differences, because timeouts different by 1s might end up
> to have the same position in the heap as rule_eviction_priority() uses
> 1024ms as a unit.
> 
> Also, use time/stop to avoid relying on how long an add-flow would take.
> 
> These tests were introduced by commit 6d56c1f1.
> ("ofproto: Update rule's priority in eviction group.")
> 
> Signed-off-by: YAMAMOTO Takashi <yamamoto at valinux.co.jp>

Kmindg's comments sound reasonable to me, otherwise:
Acked-by: Ben Pfaff <blp at nicira.com>



More information about the dev mailing list