On Tue, Sep 11, 2018 at 01:06:48PM -0300, Flavio Leitner wrote:
> On Sun, Sep 09, 2018 at 02:20:02PM +0200, Timothy Redaelli wrote:
> > Currently the default flow (actions=NORMAL) is present in the flow table 
> > after
> > the flow table is restored also when the default flow is removed.
> > 
> > This commit changes the behaviour of the "ovs-save save-flows" command to 
> > use
> > "replace-flows" instead of "add-flows" to restore the flows. This is needed 
> > in
> > order to always have the new flow table as it was before restoring it.
> > 
> > Reported-by: Flavio Leitner <[email protected]>
> > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=1626096
> > Signed-off-by: Timothy Redaelli <[email protected]>
> > ---
> 
> From the performance point of view, I thought we could do a
> del-flows first followed by add-flows, but you mentioned off
> list that the performance with replace-flows is still good.
> 
> Patch looks good and works for me, thanks Timothy!
> 
> Acked-by: Flavio Leitner <[email protected]>

BTW, we need this on master, 2.10 and 2.9.

-- 
Flavio

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to