Re: [PATCH] selftests/nolibc: add debug information

From: Willy Tarreau

Date: Sun Jul 12 2026 - 05:44:01 EST


Hi Thomas,

On Tue, Jul 07, 2026 at 06:42:30PM +0200, Thomas Weißschuh wrote:
> When working on nolibc, debug information is very valuable.
> I find myself adding this flag in many patchsets.
>
> Enable debug information unconditionally, as there is no downside to it.

Indeed, I would have thought we already had it.

> Signed-off-by: Thomas Weißschuh <linux@xxxxxxxxxxxxxx>

Acked-by: Willy Tarreau <w@xxxxxx>

Thanks!
Willy

> ---
> tools/testing/selftests/nolibc/Makefile.include | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/testing/selftests/nolibc/Makefile.include b/tools/testing/selftests/nolibc/Makefile.include
> index c30ca3a9ef14..ea520eac64a6 100644
> --- a/tools/testing/selftests/nolibc/Makefile.include
> +++ b/tools/testing/selftests/nolibc/Makefile.include
> @@ -5,7 +5,7 @@ _CFLAGS_STACKPROTECTOR ?= $(call try-run, \
> echo 'void foo(void) {}' | $(CC) -x c - -o - -S $(CLANG_CROSS_FLAGS) $(__CFLAGS_STACKPROTECTOR) | grep -q __stack_chk_guard, \
> $(__CFLAGS_STACKPROTECTOR))
> _CFLAGS_SANITIZER ?= $(call cc-option,-fsanitize=undefined -fsanitize-trap=all)
> -CFLAGS_NOLIBC_TEST ?= -Os -fno-ident -fno-asynchronous-unwind-tables -std=c89 \
> +CFLAGS_NOLIBC_TEST ?= -Os -fno-ident -fno-asynchronous-unwind-tables -std=c89 -ggdb \
> -W -Wall -Wextra -Wundef -Wwrite-strings \
> $(call cc-option,-fno-stack-protector) $(call cc-option,-Wmissing-prototypes) \
> $(_CFLAGS_STACKPROTECTOR) $(_CFLAGS_SANITIZER)
>
> ---
> base-commit: a3b2181459a2c74c03ddbad585f884eefc8ff8ff
> change-id: 20260707-nolibc-debug-04c9cedd3eaf
>
> Best regards,
> --
> Thomas Weißschuh <linux@xxxxxxxxxxxxxx>