Re: [ovs-dev] [PATCH v1 9/9] conntrack: Use an atomic conn expiration value

2021-02-23 Thread William Tu
On Wed, Feb 17, 2021 at 8:34 AM Gaetan Rivet wrote: > > A lock is taken during conn_lookup() to check whether a connection is > expired before returning it. This lock can have some contention. > > Even though this lock ensures a consistent sequence of writes, it does > not imply a specific order.

[ovs-dev] [PATCH v1 9/9] conntrack: Use an atomic conn expiration value

2021-02-17 Thread Gaetan Rivet
A lock is taken during conn_lookup() to check whether a connection is expired before returning it. This lock can have some contention. Even though this lock ensures a consistent sequence of writes, it does not imply a specific order. A ct_clean thread taking the lock first could read a value that