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

Re: openldap.git branch master updated. e12ca8b6fed6b8a2526c5c8ee820bf5aa942b59d

On 15. mars 2017 12:15, openldap-commit2devel@OpenLDAP.org wrote:
commit e12ca8b6fed6b8a2526c5c8ee820bf5aa942b59d
Author: Howard Chu <hyc@openldap.org>
Date:   Wed Mar 15 11:13:09 2017 +0000

    Fixes for multiple threadpool queues

    Remove poolq_hash, it wasn't distributing work evenly to the queues.

Not surprising when it was hashing the address instead of contents
of a local variable.  Whatever that intended to achieve, it looks
to me like rand_r() would have done the same but better.  (Also the
hash was pointlessly slow in that it walked a byte at a time, it
could have hashed better with fewer operations.)

I guess I should have looked closer at that code earlier:-)