[ovs-dev] [PATCH 08/16] ofproto: Only initiate flow table modifications if they will succeed.
Ben Pfaff
blp at nicira.com
Fri Jun 6 22:13:38 UTC 2014
On Fri, Jun 06, 2014 at 10:57:25AM +0100, Thomas Graf wrote:
> On 06/05/14 at 10:02pm, Ben Pfaff wrote:
> > In OpenFlow, a single "flow_mod" operation can change the actions (and
> > some other properties) of an arbitrary number of flows. Until now,
> > Open vSwitch has assumed that any subset of these operations could
> > fail. However, it has come out in discussion in the OpenFlow extensibility
> > working group that "partial failure" of a flow table operation is
> > undesirable, and furthermore that it should be possible to avoid it on
> > hardware implementations. (The latter is the reason that Open vSwitch
> > permitted it to be with.)
> >
> > This commit changes Open vSwitch to check whether all of a set of flow
> > table modifications will succeed before it initiates any of them.
> > This will not change visible behavior of the Open vSwitch software
> > switch, which never failed flow table modifications anyway. It might
> > change behavior of some hardware implementation, but I don't actually know
> > of any.
> >
> > ONF-JIRA: EXT-362.
> > Signed-off-by: Ben Pfaff <blp at nicira.com>
>
> LGTM
>
> Acked-by: Thomas Graf <tgraf at suug.ch>
Thanks.
More information about the dev
mailing list