BUG/MAJOR: sched: make it work also when not building with DEBUG_STRICT
authorWilly Tarreau <w@1wt.eu>
Thu, 2 Jul 2020 15:17:42 +0000 (17:17 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 2 Jul 2020 15:17:42 +0000 (17:17 +0200)
commit4f5892635215386363e2f717cee2a8b71b69244c
tree392f17a9c01f653f5722b33447672e9d27575177
parentab8b6a45be2f76efb28f7f0ad18b69549c77b4d6
BUG/MAJOR: sched: make it work also when not building with DEBUG_STRICT

Sadly, the fix from commit 54d31170a ("BUG/MAJOR: sched: make sure
task_kill() always queues the task") broke the builds without DEBUG_STRICT
as, in order to be careful, it plcaed a BUG_ON() around the previously
failing condition to check for any new possible failure, but this BUG_ON
strips the condition when DEBUG_STRICT is not set. We don't want BUG_ON
to evaluate any condition either as some debugging code calls possibly
expensive ones (e.g. in htx_get_stline). Let's just drop the useless
BUG_ON().

No backport is needed, this is 2.2-dev.
src/task.c