Re: [ovs-dev] [threads 03/11] ovs-atomic: New library for atomic operations.

2013-06-28 Thread Ethan Jackson
> Oh, that was a note to try to implement the C11 "kill_dependency()" > function. I don't think I can, but I don't think it matters. Mind if > I just delete the comment? Sounds good to me. >> In ovs-atomic.h is it fair to assume that sparse supports c11 >> features? If not, does it make ense t

Re: [ovs-dev] [threads 03/11] ovs-atomic: New library for atomic operations.

2013-06-28 Thread Ben Pfaff
On Fri, Jun 28, 2013 at 03:14:36PM -0700, Ethan Jackson wrote: > Thanks for writing up something so thorough. Better to get it done > now then jerry rig stuff ad hoc as we need it. > > There are a couple of "XXX kill_dependency" comments in the code. > Could you expand the comment a bit to explai

Re: [ovs-dev] [threads 03/11] ovs-atomic: New library for atomic operations.

2013-06-28 Thread Ethan Jackson
Thanks for writing up something so thorough. Better to get it done now then jerry rig stuff ad hoc as we need it. There are a couple of "XXX kill_dependency" comments in the code. Could you expand the comment a bit to explain what it's referring to? In ovs-atomic.h is it fair to assume that spar

[ovs-dev] [threads 03/11] ovs-atomic: New library for atomic operations.

2013-06-19 Thread Ben Pfaff
This library should prove useful for the threading changes coming up. The following commit introduces one (very simple) user. Signed-off-by: Ben Pfaff --- configure.ac |6 +- lib/automake.mk |7 + lib/ovs-atomic-c11.h | 62 +++ lib/ovs-atomic-gcc4+.c