tree eacfdbb7d684ab1961c04a5f3b486f7087933169
parent ecd2e15919f31df2c0e42b3a1ac74f1344d9a2ae
author Willy Tarreau <w@1wt.eu> 1510146319 +0100
committer Willy Tarreau <w@1wt.eu> 1510146319 +0100
encoding latin1

BUG/MAJOR: threads/tasks: fix the scheduler again

My recent change in commit ce4e0aa ("MEDIUM: task: change the construction
of the loop in process_runnable_tasks()") was bogus as it used to keep the
rq_next across an unlock/lock sequence, occasionally leading to crashes for
tasks that are eligible to any thread. We must use the lookup call for each
new batch instead. The problem is easily triggered with such a configuration :

    global
        nbthread 4

    listen check
        mode http
        bind 0.0.0.0:8080
        redirect location /
        option httpchk GET /
        server s1 127.0.0.1:8080 check inter 1
        server s2 127.0.0.1:8080 check inter 1

Thanks to Olivier for diagnosing this one. No backport is needed.