cvs commit: src/include pthread_np.h src/lib/libthr pthread.map src/lib/libthr/thread thr_mutex.c

Dag-Erling Smørgrav des at des.no
Mon Feb 4 09:11:49 PST 2008


Daniel Eischen <deischen at freebsd.org> writes:
> Dag-Erling Smørgrav <des at des.no> writes:
> > Daniel Eischen <deischen at freebsd.org> writes:
> > > If you want to know if the current thread has it locked, it should be
> > > called pthread_mutex_isowned_np().
> > I don't see why one would be more correct than the other...
> Because it is more obvious that _isowned() is for the current thread
> than _islocked().  As you can tell by my previous (incorrect)
> assumption.
>
> There is prior art for this outside of FreeBSD.  Solaris has
> mutex_owned() as part of their kernel mutex API.  I would suggest
> using pthread_mutex_owned_np().  Hmm, even FreeBSD has mtx_owned() :-)

OK, so the next question is whether this reqires a repo-copy (or a
repo-move), or if I can just cvs rm + cvs add (both the implementation
and the regression test)

DES
-- 
Dag-Erling Smørgrav - des at des.no


More information about the cvs-src mailing list