]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MINOR: debug: fix incorrect profiling status reporting in show threads
authorWilly Tarreau <w@1wt.eu>
Thu, 4 May 2023 09:30:55 +0000 (11:30 +0200)
committerWilly Tarreau <w@1wt.eu>
Thu, 4 May 2023 09:41:33 +0000 (11:41 +0200)
commitb30ced3d88cd2657aa13f531841602b1d405a433
treea5c1e8fc09cb29a7036a1edc362baa6278e70451
parentb83bf68ec0ebd282db52d73accc38587f9adbd98
BUG/MINOR: debug: fix incorrect profiling status reporting in show threads

Thread dumps include a field "prof" for each thread that reports whether
task profiling is currently active or not. It turns out that in 2.7-dev1,
commit 680ed5f28 ("MINOR: task: move profiling bit to per-thread")
mistakenly replaced it with a check for the current thread's bit in the
thread dumps, which basically is the only place where another thread is
being watched. The same mistake was done a few lines later by confusing
threads_want_rdv_mask with the profiling mask. This mask disappeared
in 2.7-dev2 with commit 598cf3f22 ("MAJOR: threads: change thread_isolate
to support inter-group synchronization"), though instead we know the ID
of the isolated thread. This commit fixes this and now reports "isolated"
instead of "wantrdv".

This can be backported to 2.7.
src/debug.c