Re: [ovs-dev] [PATCH v2 ovn 00/10] ovn-controller: Make lflow cache size configurable.

2021-02-09 Thread Dumitru Ceara
On 2/8/21 11:09 PM, Mark Michelson wrote: Hi Dumitru, Thanks for the v2. I had one small nit on patch 6. Aside from that (and Numan's comments), looks good. Acked-by: Mark Michelson Hi Mark, Numan, Thanks for the reviews! I sent a v3 with all the suggested changes. Regards, Dumitru ___

Re: [ovs-dev] [PATCH v2 ovn 00/10] ovn-controller: Make lflow cache size configurable.

2021-02-08 Thread Mark Michelson
Hi Dumitru, Thanks for the v2. I had one small nit on patch 6. Aside from that (and Numan's comments), looks good. Acked-by: Mark Michelson On 2/4/21 8:24 AM, Dumitru Ceara wrote: Scale tests have identified the lflow cache to be one of the main memory consumers in ovn-controller. This ser

[ovs-dev] [PATCH v2 ovn 00/10] ovn-controller: Make lflow cache size configurable.

2021-02-04 Thread Dumitru Ceara
Scale tests have identified the lflow cache to be one of the main memory consumers in ovn-controller. This series refactors the lflow cache code and adds configuration knobs to limit the size (in lines and/or memory) of the cache. Patches 1 and 6 fix issues with the already existing lflow cache c