On Wed, Jan 27, 2016 at 10:32 AM, Russell Bryant <russ...@ovn.org> wrote: > > > > Shall we add it ONLY IF it is on a local datapath (which means it is > > realized as a patch port on current chassis)? > > The same loop is building up the set of local datapaths. To use that > we'd have to loop through all port bindings again, which seems more > harmful overall, I think. > We could cache the localnet ports in the first loop, and in a second loop only need to go through the localnet ports.
But yes, it will still have some overhead. I am working on supporting single lswitch for each physical network. When that is ready then the total number of localnet ports are very limited so we would not waste too many zone ids. So I am ok with the solution here and let's see if optimization is still needed later on. Acked-by: Han Zhou <zhou...@gmail.com> -- Best regards, Han _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev