On 01/27/2016 02:06 PM, Han Zhou wrote: > > > On Wed, Jan 27, 2016 at 10:32 AM, Russell Bryant <russ...@ovn.org > <mailto: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 <mailto:zhou...@gmail.com>>
Thanks for the review! -- Russell Bryant _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
