[Freeipa-devel] [PATCH] 850 use pthrea read/write locks instead of NSPR RW locks

Martin Kosek mkosek at redhat.com
Wed Aug 24 12:02:01 UTC 2011


On Wed, 2011-08-24 at 07:41 -0400, Rob Crittenden wrote:
> Martin Kosek wrote:
> > On Mon, 2011-08-22 at 11:21 -0400, Rob Crittenden wrote:
> >> NSPR PR_RWLocks are not re-entrant, use pthread read-write locks instead.
> >>
> >> rob
> >
> > Reference ticket in the commit message is wrong - I think you wanted to
> > reference ticket 1635.
> >
> > This all looks good. But I was thinking - why don't we use the locking
> > interface from slapi-plugin.h? I mean slapi_new_rwlock(),
> > slapi_rwlock_rdlock etc. implemented in BZ 730387. It would let us
> > abstract from underlying locking details (pthread versus NSPR) and just
> > use the slapi plugin interface + save possible changes to locking
> > mechanism in the future.
> >
> > Martin
> >
> 
> The 389-ds team suggested I go ahead with the pthread fix as their api 
> may not be complete soon.
> 
> rob

OK - the fix is pretty straightforward then. ACK & Pushed to master,
ipa-2-1.

Martin




More information about the Freeipa-devel mailing list