[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: commit: ldap/servers/slapd/back-bdb proto-bdb.h search.c

--On Tuesday, June 22, 2004 3:26 PM -0700 Howard Chu <hyc@symas.com> wrote:

jongchoi@OpenLDAP.org wrote:

Update of /repo/OpenLDAP/pkg/ldap/servers/slapd/back-bdb

Modified Files:
	proto-bdb.h  1.102 -> 1.103
	search.c  1.197 -> 1.198

Log Message:
patch for ITS#3173
- passing transaction ptr to psearch

I don't believe this patch is thread-safe. There must be another way to accomplish this.

Hm... I've been doing this on my 4 CPU sun box, and before this patch was applied, it always deadlocked. So far it has never deadlocked, despite having a series of operations performed. I'd expect that system to hit threading issues fairly easily if they exist.

What I haven't done is done two operations at once. I'll look into that and see if I see any issues.


Quanah Gibson-Mount
Principal Software Developer
ITSS/Shared Services
Stanford University
GnuPG Public Key: http://www.stanford.edu/~quanah/pgp.html