[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
Re: (ITS#6625) draft-zeilenga-ldap-c-api-concurrency support
- To: openldap-its@OpenLDAP.org
- Subject: Re: (ITS#6625) draft-zeilenga-ldap-c-api-concurrency support
- From: hyc@symas.com
- Date: Wed, 13 Oct 2010 23:03:47 GMT
- Auto-submitted: auto-generated (OpenLDAP-ITS)
doug.leavitt@oracle.com wrote:
> Part 3 of 3 of this patch
> Part #1 delivered as: ITS#6669 remove obsolete SunOS4 LWP support
> Part #2 delivered as: ITS#6672 mutex cleanup
>
> Is now available for review. The full details are located here:
>
> http://cr.opensolaris.org/~djl/openldap-patch3/
>
> This patch is part 3 of 3 of the original ITS #6625 effort.
> (aka openldap draft-zeilenga-ldap-c-api-concurrency) (aka CONCURRENCY)
> This patch contains the library modifications to implement CONCURRENCY
> as discussed previously in ITS #6625, on the openldap-devel mail list.
>
> This patch uses the new LDAP_MUTEX_[UN]LOCK macros, delivered by patch2.
> Regression testing (post conversion) was performed using both 32-bit and
> 64-bit x86
> compiles on an OpenSolaris system.
In options.c, I'd rather just see "rc = foo; break;" instead of
UNLOCK_RETURN() everywhere. (With the final unlock, return rc at the end of
the function.)
Aside from that things look ok to me. Anyone else have comments?
> Additionally regular ongoing Solaris system testing is being performed
> using the libraries built with this patch on both Solaris SPARC and
> Solaris x86 systems.
>
> As previously shown in Round 3 codereview, there seems to be no performance
> degradation with this patch, and a slight performance improvement with
> the LDAP
> server when using back-ldap and back-meta features of the server.
>
> IPR Notice
>
> This patch file is derived from OpenLDAP Software. All of the
> modifications to
> OpenLDAP Software represented in the following patch(es) were developed by
> Oracle Corporation. Oracle Corporation has not assigned rights and/or
> interest
> in this work to any party. I, Douglas Leavitt am authorized by Oracle
> Corporation,
> my employer, to release this work under the following terms.
>
> Thank you for your consideration,
> Doug.
>
>
>
>
--
-- Howard Chu
CTO, Symas Corp. http://www.symas.com
Director, Highland Sun http://highlandsun.com/hyc/
Chief Architect, OpenLDAP http://www.openldap.org/project/