[PATCH] ldap rebind sleep

Andrew Bartlett abartlet at samba.org
Wed Nov 12 03:33:09 GMT 2003


On Wed, 2003-11-12 at 11:39, Stefan Metzmacher wrote:
> Stefan Metzmacher wrote:
> 
> > Stefan Metzmacher wrote:
> >
> >> Hi Guenther, Hi Andrew,
> >>
> >> here's the ldap rebind sleep patch
> >>
> >> gd: can you test it and report if it works ok,
> >> because I modified my old version a bit.
> >
> > Here's a new version
> > it now hold the timestamp of the last rebind and waits before the next 
> > search
> > and compares the timestamp with the current time
> > and use the MIN() of the timediff and lp_ldap_rebind_sleep() as 
> > argument of msleep()
> 
> so 3rd version...
> this works hopefully.
> the lp_ldap_rebind_sleep() is 1000 by default == 1 sec = 1000 msec == 
> 1000000 usec
> 
> gd: can you find out how long a standart replication takes, we then can 
> use 2 * replication time for the sleep time

The main problem I see with this patch is that we need to use
sys_usleep() for portability, and it only allows usleep() to a value of
1000000, which I think it 1 second.  

Anyway, why not use the msleep we already use in the ldap code?  It
isn't exactly like we need to be in any was precise about this :-)

Andrew Bartlett

-- 
Andrew Bartlett                                 abartlet at pcug.org.au
Manager, Authentication Subsystems, Samba Team  abartlet at samba.org
Student Network Administrator, Hawker College   abartlet at hawkerc.net
http://samba.org     http://build.samba.org     http://hawkerc.net
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
Url : http://lists.samba.org/archive/samba-technical/attachments/20031112/fe1f1a90/attachment.bin


More information about the samba-technical mailing list