On Tue, Feb 2, 2016 at 8:24 PM, Ben Pfaff <b...@ovn.org> wrote:
> On Tue, Feb 02, 2016 at 01:13:16PM +0200, Alexandru Ardelean wrote:
> > Would it be an idea to add a ?
> > #ifndef OVS_DEFAULT_THREAD_STACK_SIZE
> > #define OVS_DEFAULT_THREAD_STACK_SIZE (2 * 1024 * 1024)
> > #endif
> >
> > That way for our embedded musl libc case, we could add it to our CFLAGS.
>
> What if we add a *minimum* stack size instead? If the default is higher
> than the minimum, I'd rather not reduce it. Something like this:
>
> #define OVS_MIN_THREAD_STACK_SIZE (512 * 1024)
> pthread_attr attr;
> size_t stacksize;
>
> pthread_attr_init(&attr);
> if (!pthread_attr_getstacksize(&attr, &stacksize)
> && stacksize < OVS_MIN_THREAD_STACK_SIZE) {
> pthread_attr_setstacksize(&attr, OVS_MIN_THREAD_STACK_SIZE);
> }
>
> although we'd probably want to report it if either pthread function
> returned an error.
>
I like the approach.
If it's fine with you, I'll re-spin a patch after a bit of testing
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
