Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 14 Feb 2005 05:03:49 +0100
From:      Maxime Henrion <mux@FreeBSD.org>
To:        Joe Marcus Clarke <marcus@marcuscom.com>
Cc:        cvs-all@freebsd.org
Subject:   Re: cvs commit: src/lib/libpthread/thread thr_attr_init.c thr_init.c thr_private.h thr_stack.c
Message-ID:  <20050214040349.GA61763@elvis.mu.org>
In-Reply-To: <1108352249.93267.20.camel@shumai.marcuscom.com>
References:  <200502131838.j1DIc6tZ020690@repoman.freebsd.org> <1108337583.93267.1.camel@shumai.marcuscom.com> <20050214020531.GD40468@funkthat.com> <1108352249.93267.20.camel@shumai.marcuscom.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Joe Marcus Clarke wrote:
> On Sun, 2005-02-13 at 18:05 -0800, John-Mark Gurney wrote:
> > Joe Marcus Clarke wrote this message on Sun, Feb 13, 2005 at 18:33 -0500:
> > > And there was much rejoicing!  I would like to reiterate mezz's request
> > > for a __FreeBSD_version bump once all the thread libraries are updated.
> > > It would also be good to get this MFC'd before 5.4.  Thanks again.
> > 
> > If any application that cares/requires changes from the default, either
> > due to large number of threads (requiring small stack size), or large
> > stacks, should already be patched with their new defaults...  So
> > requiring a modification based upon version before/after this change
> > should be unnecessary...
> 
> But knowing when this patch is implemented means we can _not_ patch
> certain applications.  The best example of this is gstreamer.  Gstreamer
> is patched to lower its initial thread stack usage to 1 MB since that
> was the previous limit.  This severely limits gstreamer.  With the
> larger initial thread stack size (something that is not changeable by
> individual applications), we no longer need to cripple gstreamer on
> -CURRENT.  Therefore, I ask __FreeBSD_version to be bumped so I know
> when it's safe to let gstreamer take a full 2 MB of stack on the initial
> thread.

Is there anything wrong with pthread_attr_setstacksize()?  Using this to
patch the said applications would allow them to get an acceptably sized
stack whether the host is running an old or a recent version of
libpthread.  It would also make sense to then submit the patches to the
vendor so that it's not too much a burden to maintain.

Cheers,
Maxime



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20050214040349.GA61763>