Referencing this commit:
   8147cec9ee8feea9440cf79365709ddc32ff57d5

My bad for not noticing this earlier.
I only got to try the OVS upstream patch just now and noticed this.
We're still using an older version of OVS internally, so this slipped by.

I was wondering why the crash was still occuring and checked the patch.

The pthread_attr object needs to be passed to the pthread_create()
call in order to make use of it.

Signed-off-by: Alexandru Ardelean <ardeleana...@gmail.com>
---
 lib/ovs-thread.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/ovs-thread.c b/lib/ovs-thread.c
index b0e10ee..bffdd96 100644
--- a/lib/ovs-thread.c
+++ b/lib/ovs-thread.c
@@ -386,7 +386,7 @@ ovs_thread_create(const char *name, void *(*start)(void *), 
void *arg)
     pthread_attr_init(&attr);
     set_min_stack_size(&attr, 512 * 1024);
 
-    error = pthread_create(&thread, NULL, ovsthread_wrapper, aux);
+    error = pthread_create(&thread, &attr, ovsthread_wrapper, aux);
     if (error) {
         ovs_abort(error, "pthread_create failed");
     }
-- 
1.7.10.4

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to