Re: [ovs-dev] [thread-safety 00/11] Thread safety.

2013-07-30 Thread Alex Wang
Thanks Ben, I'll adjust accordingly in 01/11. On Tue, Jul 30, 2013 at 10:33 AM, Ben Pfaff wrote: > After looking over this series, I think that we should make > ovs_mutex_init() take the type of mutex to create. This would save > most of the callers creating and destroying their own > pthread

Re: [ovs-dev] [thread-safety 00/11] Thread safety.

2013-07-30 Thread Ben Pfaff
After looking over this series, I think that we should make ovs_mutex_init() take the type of mutex to create. This would save most of the callers creating and destroying their own pthread_mutexattr_t. ___ dev mailing list dev@openvswitch.org http://open

Re: [ovs-dev] [thread-safety 00/11] Thread safety.

2013-07-29 Thread Ben Pfaff
On Mon, Jul 29, 2013 at 09:26:56AM -0700, Alex Wang wrote: > I think so. I saw new modifications in 01/11 patch. OK, I'll review Ethan's version. ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [thread-safety 00/11] Thread safety.

2013-07-29 Thread Alex Wang
I think so. I saw new modifications in 01/11 patch. Thanks, Alex Wang, ___ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev

Re: [ovs-dev] [thread-safety 00/11] Thread safety.

2013-07-29 Thread Ben Pfaff
On Fri, Jul 26, 2013 at 06:07:01PM -0700, Ethan Jackson wrote: > This series of patches adds thread safety to a bunch of modules used > by ofproto-dpif-xlate. The first patch of the series adds support for > clang's thread safety annotations which as Alex mentioned aren't quite > upstreamed yet.

[ovs-dev] [thread-safety 00/11] Thread safety.

2013-07-26 Thread Ethan Jackson
This series of patches adds thread safety to a bunch of modules used by ofproto-dpif-xlate. The first patch of the series adds support for clang's thread safety annotations which as Alex mentioned aren't quite upstreamed yet. I've made some minor modifications on top of what he's sent out already