r2265: Volkers change to HEAD looks very good. Commit message
commitbfdeb22c69d09eb73305b6034fa6d0ec67275789
authorJeremy Allison <jra@samba.org>
Thu, 9 Sep 2004 10:02:44 +0000 (9 10:02 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 15:52:38 +0000 (10 10:52 -0500)
treec4388b73ce98e524161f9cc57056d1ee23f22426
parent3d275d1eee9644651dcbb61a342648e878fe3506
r2265: Volkers change to HEAD looks very good. Commit message
from HEAD follows :
While torturing winbind a bit I found the following unfortunate behaviour:

Sending multiple requests at a high rate for a slow operation exposed that no
response comes back until the last request in the queue has been
processed. This is an unfortunate result of serially going through all sockets> that have shown to be readable or writable. All client sockets become readable> at the same time, none of them is writable. We go through them, read the
request, process the complete request. Before we enter the select system call
the next time all requests have to have completed.

This patch optimizes this by first looking at the sockets for writability. A
write on a socket that came back from select does not block, so this
additional loop might have a non-zero cost, but it can't prevent other
operations from proceeding.

After a possibly long-running winbindd_process() we directly start select()
again. To avoid starvation the currently processed client is demoted to be the> last one in the list of clients.

Jeremy.
source/nsswitch/winbindd.c
source/nsswitch/winbindd_util.c