On Wed Jul 2, 2025 at 12:42 PM CEST, Natanael Copa wrote:
On Tue, 1 Jul 2025 19:07:32 +0200 Sergio González Collado sergio.collado@gmail.com wrote:
Hello,
Hi Sergio,
Thanks for pointing that out. I was not aware of it.
No worries. It happens. Just slightly unfortunate that it tickled down to stable branches before it got fixed.
It is trivial to fix but I wonder what is the best way though. Achill proposed to use #ifdef HAVE_BACKTRACE_SUPPORT in tools/include/linux/kallsyms.h but it appears that this was included only for KSYM_NAME_LEN.
This would still not resolve the actual issue that execinfo.h is included if built without backtrace support and kallsyms.h gets included.
#ifdef HAVE_BACKTRACE_SUPPORT can be seen in other places in tools/, where it is used exactly like I did in https://lore.kernel.org/stable/20250622014608.448718-1-fossdd@pwned.life/.