Re: [ovs-dev] [PATCH 2/2] ofproto: Set to revalidate when a new version is available.

2016-06-21 Thread Jarno Rajahalme
Thanks for the review, pushed to master with the suggested changes. Jarno > On May 6, 2016, at 2:27 PM, Ben Pfaff wrote: > > On Wed, May 04, 2016 at 01:08:13PM -0700, Jarno Rajahalme wrote: >> There is no need to set the revalidate flag after each flow mod >> separately, as we can do it once

Re: [ovs-dev] [PATCH 2/2] ofproto: Set to revalidate when a new version is available.

2016-05-06 Thread Ben Pfaff
On Wed, May 04, 2016 at 01:08:13PM -0700, Jarno Rajahalme wrote: > There is no need to set the revalidate flag after each flow mod > separately, as we can do it once after the whole transaction is > finished. It is not done at all if the transaction fails. > > In the successful case this change m

[ovs-dev] [PATCH 2/2] ofproto: Set to revalidate when a new version is available.

2016-05-04 Thread Jarno Rajahalme
There is no need to set the revalidate flag after each flow mod separately, as we can do it once after the whole transaction is finished. It is not done at all if the transaction fails. In the successful case this change makes no functional difference, since the revalidation thread is triggered b