Re: [ovs-dev] [PATCH v4] ofproto-dpif-upcall: Avoid stale ukeys leaks.

2024-08-29 Thread Simon Horman
On Thu, Aug 29, 2024 at 11:37:07AM +0200, Eelco Chaudron wrote: > > > On 29 Aug 2024, at 10:57, Simon Horman wrote: > > > On Tue, Aug 27, 2024 at 11:00:10AM +0200, Eelco Chaudron wrote: > >> It is observed in some environments that there are much more ukeys than > >> actual DP flows. For example

Re: [ovs-dev] [PATCH v4] ofproto-dpif-upcall: Avoid stale ukeys leaks.

2024-08-29 Thread Eelco Chaudron
On 29 Aug 2024, at 10:57, Simon Horman wrote: > On Tue, Aug 27, 2024 at 11:00:10AM +0200, Eelco Chaudron wrote: >> It is observed in some environments that there are much more ukeys than >> actual DP flows. For example: >> >> $ ovs-appctl upcall/show >> system@ovs-system: >> flows : (current 7)

Re: [ovs-dev] [PATCH v4] ofproto-dpif-upcall: Avoid stale ukeys leaks.

2024-08-29 Thread Simon Horman
On Tue, Aug 27, 2024 at 11:00:10AM +0200, Eelco Chaudron wrote: > It is observed in some environments that there are much more ukeys than > actual DP flows. For example: > > $ ovs-appctl upcall/show > system@ovs-system: > flows : (current 7) (avg 6) (max 117) (limit 2125) > offloaded flows : 525 >

Re: [ovs-dev] [PATCH v4] ofproto-dpif-upcall: Avoid stale ukeys leaks.

2024-08-29 Thread Eelco Chaudron
On 28 Aug 2024, at 16:29, Aaron Conole wrote: Thanks Aaron for the review, I sent out a v5 with all your suggestions applied. Cheers, Eelco > Eelco Chaudron writes: > >> It is observed in some environments that there are much more ukeys than >> actual DP flows. For example: >> >> $ ovs-appc

Re: [ovs-dev] [PATCH v4] ofproto-dpif-upcall: Avoid stale ukeys leaks.

2024-08-28 Thread Aaron Conole
Eelco Chaudron writes: > It is observed in some environments that there are much more ukeys than > actual DP flows. For example: > > $ ovs-appctl upcall/show > system@ovs-system: > flows : (current 7) (avg 6) (max 117) (limit 2125) > offloaded flows : 525 > dump duration : 1063ms > ufid enabled :

Re: [ovs-dev] [PATCH v4] ofproto-dpif-upcall: Avoid stale ukeys leaks.

2024-08-28 Thread Roi Dayan via dev
On 27/08/2024 12:00, Eelco Chaudron wrote: > It is observed in some environments that there are much more ukeys than > actual DP flows. For example: > > $ ovs-appctl upcall/show > system@ovs-system: > flows : (current 7) (avg 6) (max 117) (limit 2125) > offloaded flows : 525 > dump duration : 1

[ovs-dev] [PATCH v4] ofproto-dpif-upcall: Avoid stale ukeys leaks.

2024-08-27 Thread Eelco Chaudron
It is observed in some environments that there are much more ukeys than actual DP flows. For example: $ ovs-appctl upcall/show system@ovs-system: flows : (current 7) (avg 6) (max 117) (limit 2125) offloaded flows : 525 dump duration : 1063ms ufid enabled : true 23: (keys 3612) 24: (keys 3625) 25: