Re: [ovs-dev] [PATCH v3] ofproto-dpif-xlate: Don't reinstall removed XC_LEARN rule

2023-08-11 Thread Ilya Maximets
On 8/10/23 04:58, Mike Pattrick wrote: > When the a revalidator thread is updating statistics for an XC_LEARN > xcache entry in xlate_push_stats_entry it uses ofproto_flow_mod_learn. > The revalidator will update stats for rules even if they are in a > removed state or marked as invisible. However,

Re: [ovs-dev] [PATCH v3] ofproto-dpif-xlate: Don't reinstall removed XC_LEARN rule

2023-08-11 Thread Simon Horman
On Fri, Aug 11, 2023 at 11:16:33AM -0400, Mike Pattrick wrote: > On Fri, Aug 11, 2023 at 11:00 AM Simon Horman wrote: > > > > On Wed, Aug 09, 2023 at 10:58:14PM -0400, Mike Pattrick wrote: > > > When the a revalidator thread is updating statistics for an XC_LEARN > > > xcache entry in xlate_push_s

Re: [ovs-dev] [PATCH v3] ofproto-dpif-xlate: Don't reinstall removed XC_LEARN rule

2023-08-11 Thread Mike Pattrick
On Fri, Aug 11, 2023 at 11:00 AM Simon Horman wrote: > > On Wed, Aug 09, 2023 at 10:58:14PM -0400, Mike Pattrick wrote: > > When the a revalidator thread is updating statistics for an XC_LEARN > > xcache entry in xlate_push_stats_entry it uses ofproto_flow_mod_learn. > > The revalidator will updat

Re: [ovs-dev] [PATCH v3] ofproto-dpif-xlate: Don't reinstall removed XC_LEARN rule

2023-08-11 Thread Simon Horman
On Wed, Aug 09, 2023 at 10:58:14PM -0400, Mike Pattrick wrote: > When the a revalidator thread is updating statistics for an XC_LEARN > xcache entry in xlate_push_stats_entry it uses ofproto_flow_mod_learn. > The revalidator will update stats for rules even if they are in a > removed state or marke

[ovs-dev] [PATCH v3] ofproto-dpif-xlate: Don't reinstall removed XC_LEARN rule

2023-08-09 Thread Mike Pattrick
When the a revalidator thread is updating statistics for an XC_LEARN xcache entry in xlate_push_stats_entry it uses ofproto_flow_mod_learn. The revalidator will update stats for rules even if they are in a removed state or marked as invisible. However, ofproto_flow_mod_learn will detect if a flow h