[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