On Wed, Mar 30, 2016 at 4:48 PM, Ben Pfaff <[email protected]> wrote: > > 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 <[email protected]> > > 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!
I was thinking about: match="inport == outport", actions="drop" And then I realized that this is not possible without big change to OVS classifier. But I didn't thought of this smart bit-trick to implement the comparison. Thanks for enlightening :) -- Best regards, Han _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
