Re: [ovs-dev] [PATCH 3/3] seq: Add support for initial delay before waking up.

2016-09-20 Thread Jarno Rajahalme
Thanks for the review. I sent a v2 with an implementation that just does the right thing (I hope) in the revalidator thread itself. Jarno > On Sep 19, 2016, at 10:32 PM, Ben Pfaff wrote: > > On Fri, Sep 16, 2016 at 04:10:51PM -0700, Jarno Rajahalme wrote: >> The execution time

Re: [ovs-dev] [PATCH 3/3] seq: Add support for initial delay before waking up.

2016-09-19 Thread Ben Pfaff
On Fri, Sep 16, 2016 at 04:10:51PM -0700, Jarno Rajahalme wrote: > The execution time of 'ovs-ofctl add-flows' with a large number of > flows can be more than halved if revalidators are not running after > each flow mod separately. This was first suspected when it was found > that 'ovs-ofctl

[ovs-dev] [PATCH 3/3] seq: Add support for initial delay before waking up.

2016-09-16 Thread Jarno Rajahalme
The execution time of 'ovs-ofctl add-flows' with a large number of flows can be more than halved if revalidators are not running after each flow mod separately. This was first suspected when it was found that 'ovs-ofctl --bundle add-flows' is about 10 times faster than the same command without