From 8153cf1e51fb1b722ab2b16a452c043fbf307b56 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Tue, 9 Sep 2025 06:55:25 +0200 Subject: [PATCH] MINOR: debug: report the process id in warnings and panics Warning and panic messages currently do not report the PID. This is annoying when trying to reproduce problems because warnings do not allow know which process to attach to in order to debug, and panics do not permit to know which core dump corresponds to which dump. Let's add them in both messages. This should probably be backported at least to 3.2. --- src/debug.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/debug.c b/src/debug.c index 600adb7be..0c3ecb1dc 100644 --- a/src/debug.c +++ b/src/debug.c @@ -777,7 +777,7 @@ void ha_panic() return; } - chunk_printf(&trash, "\nPANIC! Thread %u is about to kill the process.\n", tid + 1); + chunk_printf(&trash, "\nPANIC! Thread %u is about to kill the process (pid %d).\n", tid + 1, pid); /* dump a few of the post-mortem info */ chunk_appendf(&trash, "\nHAProxy info:\n version: %s\n features: %s\n", @@ -923,7 +923,7 @@ void ha_stuck_warning(void) " 'global' section of your configuration to avoid this in the future.\n"); } - chunk_appendf(&buf, " => Trying to gracefully recover now.\n"); + chunk_appendf(&buf, " => Trying to gracefully recover now (pid %d).\n", pid); /* Note: it's important to dump the whole buffer at once to avoid * interleaved outputs from multiple threads dumping in parallel.