Re: [ovs-dev] [PATCH] dpif-netdev: Lockless meters.

2023-06-30 Thread Ilya Maximets
On 6/30/23 08:40, Eelco Chaudron wrote: > > > On 29 Jun 2023, at 17:54, Ilya Maximets wrote: > >> On 6/29/23 17:43, Eelco Chaudron wrote: >>> >>> >>> On 22 Jun 2023, at 0:32, Ilya Maximets wrote: >>> Current implementation of meters in the userspace datapath takes the meter lock for

Re: [ovs-dev] [PATCH] dpif-netdev: Lockless meters.

2023-06-30 Thread Eelco Chaudron
On 29 Jun 2023, at 17:54, Ilya Maximets wrote: > On 6/29/23 17:43, Eelco Chaudron wrote: >> >> >> On 22 Jun 2023, at 0:32, Ilya Maximets wrote: >> >>> Current implementation of meters in the userspace datapath takes >>> the meter lock for every packet batch. If more than one thread >>> hits

Re: [ovs-dev] [PATCH] dpif-netdev: Lockless meters.

2023-06-29 Thread Ilya Maximets
On 6/29/23 17:43, Eelco Chaudron wrote: > > > On 22 Jun 2023, at 0:32, Ilya Maximets wrote: > >> Current implementation of meters in the userspace datapath takes >> the meter lock for every packet batch. If more than one thread >> hits the flow with the same meter, they will lock each other.

Re: [ovs-dev] [PATCH] dpif-netdev: Lockless meters.

2023-06-29 Thread Eelco Chaudron
On 22 Jun 2023, at 0:32, Ilya Maximets wrote: > Current implementation of meters in the userspace datapath takes > the meter lock for every packet batch. If more than one thread > hits the flow with the same meter, they will lock each other. > > Replace the critical section with atomic

Re: [ovs-dev] [PATCH] dpif-netdev: Lockless meters.

2023-06-28 Thread miter
Tested-by: Lin Huang Tested-by: Zhang YuHuang On 6/22/2023 6:32 AM, Ilya Maximets wrote: Current implementation of meters in the userspace datapath takes the meter lock for every packet batch. If more than one thread hits the flow with the same meter, they will lock each other. Replace the

Re: [ovs-dev] [PATCH] dpif-netdev: Lockless meters.

2023-06-26 Thread Simon Horman
On Thu, Jun 22, 2023 at 12:32:20AM +0200, Ilya Maximets wrote: > Current implementation of meters in the userspace datapath takes > the meter lock for every packet batch. If more than one thread > hits the flow with the same meter, they will lock each other. > > Replace the critical section with

[ovs-dev] [PATCH] dpif-netdev: Lockless meters.

2023-06-21 Thread Ilya Maximets
Current implementation of meters in the userspace datapath takes the meter lock for every packet batch. If more than one thread hits the flow with the same meter, they will lock each other. Replace the critical section with atomic operations to avoid interlocking. Meters themselves are