[Date Prev][Date Next]
Re: ITS#4338 (Was: commit: ldap/servers/slapd daemon.c)
On Saturday 14 January 2006 11:45, Pierangelo Masarati wrote:
> On Fri, 2006-01-13 at 16:30 +0000, hyc@OpenLDAP.org wrote:
> > Update of /repo/OpenLDAP/pkg/ldap/servers/slapd
> > Modified Files:
> > daemon.c 1.372 -> 1.373
> > Log Message:
> > ITS#4338 delete extraneous slapd_close() calls. Descriptors that were in
> > the event set must be removed using slapd_remove() and the whole sequen=
> > must be protected with the slap_daemon.sd_mutex to close them properly.
> > Otherwise it will wind up closing valid, freshly opened sockets, or
> > closing dead sockets twice, etc. It's unnecessary to do any close
> > management here since connection.c already does the correct teardown
> > sequence.
> I guess this answer obsoletes ITS#4333; based on some odd connection-
> closing related logging I get when test036 & test039 hung (proxy &
> targets perfectly responsive, but no activity on both and the clients
> waiting for response), I suspect this was the same root cause.
The behaviour I saw was different, but as #4338 may have triggered #4333, I=
run the box in question under load for a day under the same conditions and=
see if I can trigger it now.
ISP Systems Specialist
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (GNU/Linux)
-----END PGP SIGNATURE-----