[ovs-dev] [PATCH] lib/ovs-thread: make use of the pthread_attr object

Ben Pfaff blp at ovn.org
Thu Mar 10 22:30:05 UTC 2016


Andy, can you transform the reference into a Fixes: tag?

Thanks,

Ben.

On Thu, Mar 10, 2016 at 02:14:03PM -0800, Andy Zhou wrote:
> LGTM,   Acked-by: Andy Zhou <azhou at ovn.org>
> 
> I will push it later today. Thanks.
> 
> On Thu, Mar 10, 2016 at 8:37 AM, Alexandru Ardelean <ardeleanalex at gmail.com>
> wrote:
> 
> > 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 <ardeleanalex at 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 at openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
> >
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev



More information about the dev mailing list