[ovs-dev] [PATCH v4 3/3] ovn-controller: Allocate ct zones for localnet ports.

Russell Bryant russell at ovn.org
Wed Jan 27 19:14:43 UTC 2016


On 01/27/2016 02:06 PM, Han Zhou wrote:
> 
> 
> On Wed, Jan 27, 2016 at 10:32 AM, Russell Bryant <russell at ovn.org
> <mailto:russell at 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 <zhouhan at gmail.com <mailto:zhouhan at gmail.com>>

Thanks for the review!

-- 
Russell Bryant



More information about the dev mailing list