Re: [ovs-dev] [PATCH 06/18] lib/ovs-thread: Use atomic_count.

2014-08-29 Thread Ben Pfaff
On Fri, Aug 29, 2014 at 12:50:24PM -0700, Jarno Rajahalme wrote: > > On Aug 29, 2014, at 9:36 AM, Jarno Rajahalme wrote: > > > > > On Aug 28, 2014, at 9:32 AM, Ben Pfaff wrote: > > > >> On Fri, Aug 22, 2014 at 01:58:17PM -0700, Jarno Rajahalme wrote: > >>> barrier->count is used as a simple c

Re: [ovs-dev] [PATCH 06/18] lib/ovs-thread: Use atomic_count.

2014-08-29 Thread Jarno Rajahalme
On Aug 29, 2014, at 9:36 AM, Jarno Rajahalme wrote: > > On Aug 28, 2014, at 9:32 AM, Ben Pfaff wrote: > >> On Fri, Aug 22, 2014 at 01:58:17PM -0700, Jarno Rajahalme wrote: >>> barrier->count is used as a simple counter and is not expected the >>> synchronize the state of any other variable, s

Re: [ovs-dev] [PATCH 06/18] lib/ovs-thread: Use atomic_count.

2014-08-29 Thread Jarno Rajahalme
On Aug 28, 2014, at 9:32 AM, Ben Pfaff wrote: > On Fri, Aug 22, 2014 at 01:58:17PM -0700, Jarno Rajahalme wrote: >> barrier->count is used as a simple counter and is not expected the >> synchronize the state of any other variable, so we can use atomic_count, >> which uses relaxed atomics. >> >>

Re: [ovs-dev] [PATCH 06/18] lib/ovs-thread: Use atomic_count.

2014-08-28 Thread Ben Pfaff
On Fri, Aug 22, 2014 at 01:58:17PM -0700, Jarno Rajahalme wrote: > barrier->count is used as a simple counter and is not expected the > synchronize the state of any other variable, so we can use atomic_count, > which uses relaxed atomics. > > Ditto for the 'next_id' within ovsthread_wrapper(). >