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

Re: ITS#3950



Kurt D. Zeilenga wrote:
HEAD/re23 thr_posix.c match. That is, the patch suggested in
ITS#3950 was never applied to HEAD nor RE23. IIRC, Howard
reworked portions of slapd(8) so as not rely so heavily on
thread yielding. It's quite possible that some of these
changes haven't found there way into RE23.

I reworked most of the yield() calls in daemon.c, and those changes are in RE23. However, the BDB library itself makes yield() calls as well, which I ignored. It seems we should apply the patch in ITS#3950 after all.
Kurt

At 11:00 AM 1/5/2006, Kurt D. Zeilenga wrote:
I would suspect that if ITS#3950 was a problem in 2.3.13+, it
would also be a problem with 2.3.11.  But I note that some of
the reports indicate no problem with 2.3.11 but problems with
2.3.13+.  Of course, there might be other factors involved.

I've reopened the ITS and clarified its notes to indicate that
the fix (or I should say the "our workaround for what could be
considered a pthread implementation bug") is not in RE23
as opposed to the issue not being in RE23.

At 10:21 AM 1/5/2006, Quanah Gibson-Mount wrote:
Any particular reason why the fix to this issue, which *is* present in 2.3, contrary to the notes in the ITS, isn't in OL 2.3? :)  I'm guessing this is what is causing the large scale reporting of "performance problems" on the software list.  Maybe a quick 2.3.16 release?

--Quanah

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





--
 -- Howard Chu
 Chief Architect, Symas Corp.  http://www.symas.com
 Director, Highland Sun        http://highlandsun.com/hyc
 OpenLDAP Core Team            http://www.openldap.org/project/