Re: [ovs-dev] [PATCH ovn] ovn-controller: Avoid reprocessing same lflows in the same I-P run.

2022-04-13 Thread Numan Siddique
On Wed, Apr 13, 2022 at 2:36 AM Han Zhou wrote: > > On Mon, Apr 11, 2022 at 2:05 PM Numan Siddique wrote: > > > > On Sat, Feb 5, 2022 at 12:12 AM Han Zhou wrote: > > > > > > On Fri, Feb 4, 2022 at 7:58 AM Mark Michelson > wrote: > > > > > > > > As far as I can tell, this looks correct. > > > >

Re: [ovs-dev] [PATCH ovn] ovn-controller: Avoid reprocessing same lflows in the same I-P run.

2022-04-13 Thread Han Zhou
On Mon, Apr 11, 2022 at 2:05 PM Numan Siddique wrote: > > On Sat, Feb 5, 2022 at 12:12 AM Han Zhou wrote: > > > > On Fri, Feb 4, 2022 at 7:58 AM Mark Michelson wrote: > > > > > > As far as I can tell, this looks correct. > > > > > > Acked-by: Mark Michelson > > > > Thanks Mark. I applied it to

Re: [ovs-dev] [PATCH ovn] ovn-controller: Avoid reprocessing same lflows in the same I-P run.

2022-04-11 Thread Numan Siddique
On Sat, Feb 5, 2022 at 12:12 AM Han Zhou wrote: > > On Fri, Feb 4, 2022 at 7:58 AM Mark Michelson wrote: > > > > As far as I can tell, this looks correct. > > > > Acked-by: Mark Michelson > > Thanks Mark. I applied it to the main branch. Hi Han, We have an issue in OVN 21.12 (and OVN 21.09)

Re: [ovs-dev] [PATCH ovn] ovn-controller: Avoid reprocessing same lflows in the same I-P run.

2022-02-04 Thread Han Zhou
On Fri, Feb 4, 2022 at 7:58 AM Mark Michelson wrote: > > As far as I can tell, this looks correct. > > Acked-by: Mark Michelson Thanks Mark. I applied it to the main branch. > > On 1/20/22 13:06, Han Zhou wrote: > > For I-P node lflow_output, different change handlers can trigger > >

Re: [ovs-dev] [PATCH ovn] ovn-controller: Avoid reprocessing same lflows in the same I-P run.

2022-02-04 Thread Mark Michelson
As far as I can tell, this looks correct. Acked-by: Mark Michelson On 1/20/22 13:06, Han Zhou wrote: For I-P node lflow_output, different change handlers can trigger reprocessing lflows. However, it is a waste to reprocess the same lflow multiple times in the same run, because each processing

[ovs-dev] [PATCH ovn] ovn-controller: Avoid reprocessing same lflows in the same I-P run.

2022-01-20 Thread Han Zhou
For I-P node lflow_output, different change handlers can trigger reprocessing lflows. However, it is a waste to reprocess the same lflow multiple times in the same run, because each processing of the lflow is against the same input data. For example, if a lflow references an addresset A and a