On Tue, Mar 29, 2016 at 04:55:11PM -0700, Han Zhou wrote:
> Currently in physical_run() we added per-port loopback prevention
> flows for all lports. The flows are actually required only for
> local ports on the chassis. This change greatly reduces number of
> flows in table 34.
> 
> Signed-off-by: Han Zhou <zhou...@gmail.com>

We could do this with a single logical flow of the form:

match="(inport[0] && !outport[0]) || (!inport[0]) && outport[0]) ||
       (inport[1] && !outport[1]) || (!inport[1]) && outport[1]) ||
                                  ...
       (inport[15] && !outport[15]) || (!inport[15]) && outport[15])",
actions="next;"

That's a fairly inefficient solution due to the construction of the OVS
classifier but we still might consider it at some point.

I applied this to master, thanks!
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to