additional build args used to ensure debug symbols are properly linked

Author: marc-casavant

src/tests/multi-server/scripts/docker/build/Dockerfile.multi-server-prof

@@ -2,11 +2,41 @@
 # Builds off of "base" profiling image, configures FreeRADIUS and builds it.
 FROM freeradius4-default-profiling/ubuntu24:latest
 
+# Inject patched sources to fix valgrind/valgrind.h detection
+COPY configure.ac /usr/local/src/repositories/freeradius-server/configure.ac
+COPY src/lib/util/dl.c /usr/local/src/repositories/freeradius-server/src/lib/util/dl.c
+COPY src/lib/util/dl.h /usr/local/src/repositories/freeradius-server/src/lib/util/dl.h
+COPY src/lib/server/dependency.c /usr/local/src/repositories/freeradius-server/src/lib/server/dependency.c
+COPY src/bin/radiusd.c /usr/local/src/repositories/freeradius-server/src/bin/radiusd.c
+
+# Regenerating configure to include the patch for valgrind/valgrind.h detection.
+# Proper path to valgrind/valgrind.h is needed for logic depending on RUNNING_ON_VALGRIND which
+# is set by valgrind/valgrind.h
+RUN autoreconf
+
+# CFLAGS used for profiling build:
+#   -g3                          Maximum debug info for callgrind symbol resolution
+#   -O0                          No optimisation: preserves the exact call structure of the source.
+#                                Disables dead-code elimination, tail-call optimisation, and all
+#                                other transforms that remove or reorder calls.
+#   -fno-omit-frame-pointer      Keep frame pointers so callgrind can walk the call stack
+#   -fno-inline                  Disable inlining (including CC_HINT(flatten)) so call edges appear in the graph
+#   -Dalways_inline=             Strip __attribute__((always_inline)) which -fno-inline does not suppress
+#   -fno-plt                     Eliminate PLT trampolines on cross-library calls (e.g. worker.c ->
+#                                interpret.c). Without this, the linker inserts a PLT stub with no
+#                                DWARF debug info, which appears as an unresolved "???" node in the
+#                                callgrind call graph. With -fno-plt, the compiler generates a direct
+#                                GOT-based indirect call that callgrind resolves correctly.
+#   -fno-builtin                 Prevent replacement of standard library calls (memcpy, strlen, etc.)
+#                                with inline intrinsics, which would erase them from the call graph.
+#   -fno-optimize-sibling-calls  Prevent tail-call elimination converting call+ret into a jump,
+#                                which removes the call edge from the call graph. Already implied
+#                                by -O0 but stated explicitly as a safeguard.
 RUN ./configure \
     --enable-developer \
     --disable-verify-ptr \
     --with-raddbdir=/etc/freeradius \
-    CFLAGS="-g3 -O1 -fno-omit-frame-pointer" \
+    CFLAGS="-g3 -O0 -fno-omit-frame-pointer -fno-inline -Dalways_inline= -fno-plt -fno-builtin -fno-optimize-sibling-calls" \
     LDFLAGS="-fno-omit-frame-pointer"
 
 RUN make