cvs commit: src/lib/libpthread/thread thr_mutex.c

Daniel Eischen deischen at freebsd.org
Mon Nov 1 09:11:35 PST 2004


On Mon, 1 Nov 2004, Brian Fundakowski Feldman wrote:

> On Mon, Nov 01, 2004 at 09:31:10AM -0500, Daniel Eischen wrote:
> > On Sun, 31 Oct 2004, Daniel Eischen wrote:
> >
> > > On Sun, 31 Oct 2004, Brian Feldman wrote:
> > >
> > > > green       2004-10-31 05:03:50 UTC
> > > >
> > > >   FreeBSD src repository
> > > >
> > > >   Modified files:
> > > >     lib/libpthread/thread thr_mutex.c
> > > >   Log:
> > > >   Make pthread_mutex_trylock(3) return EBUSY on failure, as all software
> > > >   packages expect and seems to be most correct according to the slightly-
> > > >   ambiguous standards.
> > > >
> > > >   MFC after:              1 month
> > > >   Corroborated by:        POSIX <http://tinyurl.com/4uvub>
> > > >   Reviewed by:            silence on threads@
> > >
> > > I never saw this, and there has been a couple of long threads about
> > > it.  The current behavior was deemed correct especially since default
> > > mutexes are error check mutexes.  The rationale is that it provides a way
> > > for a thread to tell if it owns the mutex if EDEADLK is returned.
> > >
> > > Please back out and sorry for not seeing a posting on -threads.
> >
> > I retract the backout request.  I have response from Dave Butenhof
> > over at the Austin Group mailing list, and EBUSY should be returned
> > even for error check mutexes.  He acknolwedged the spec was not
> > clear and thought they had cleaned it up in that regard.  So this
> > is the correct fix :-)
>
> Could you please remind me what the procedure is for including
> POSIX text within our manual pages?  For pthreads it should be
> trivial to use almost the same text, since we didn't originate
> the interfaces ourselves, no?

See the -standards folks, and probably Ruslan.  From what I
understand, we have approval to include POSIX man pages.

-- 
Dan Eischen



More information about the cvs-all mailing list