linux-kernel - Re: [PATCH v1] ftrace: Fix compile error when CONFIG

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20250206131711.ea536f165d5b5980b3909acd@kernel.org>
Date: Thu, 6 Feb 2025 13:17:11 +0900
From: Masami Hiramatsu (Google) <mhiramat@...nel.org>
To: Haiyue Wang <haiyuewa@....com>
Cc: Steven Rostedt <rostedt@...dmis.org>, x86@...nel.org,
 linux-trace-kernel@...r.kernel.org, samitolvanen@...gle.com, Mark Rutland
 <mark.rutland@....com>, Thomas Gleixner <tglx@...utronix.de>, Ingo Molnar
 <mingo@...hat.com>, Borislav Petkov <bp@...en8.de>, Dave Hansen
 <dave.hansen@...ux.intel.com>, "H. Peter Anvin" <hpa@...or.com>,
 "open list:FUNCTION HOOKS (FTRACE)" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v1] ftrace: Fix compile error when CONFIG_GENDWARFKSYMS
 is enabled

On Thu, 6 Feb 2025 11:23:02 +0800
Haiyue Wang <haiyuewa@....com> wrote:

> 
> 
> On 2025/2/6 10:45, Masami Hiramatsu (Google) wrote:
> > On Wed, 5 Feb 2025 19:53:00 -0500
> > Steven Rostedt <rostedt@...dmis.org> wrote:
> > 
> >> On Thu, 6 Feb 2025 09:38:53 +0900
> >> Masami Hiramatsu (Google) <mhiramat@...nel.org> wrote:
> >>
> >>> Thanks for the fix. And this fixes one of problems.
> >>> Another one is that `is_endbr()` is defined in <asm/ibt.h> but that also depend
> >>> on CONFIG_HAVE_FENTRY.
> >>>
> >>> Let's pick this and fix above issue.
> >>
> >> OK, I'll pull this into my urgent tree and start testing it.
> > 
> > Sorry, Gabriel found another issue.
> > Let me fix this instead of this patch.
> > 
> > https://lore.kernel.org/all/CAFEugPfFykYsHqS-J36CNWDn+d34bSQg-e5hJ=AaJPC8=6Ej-g@mail.gmail.com/
> > 
> 
> If I put the fix above '#ifndef __ASSEMBLY__', then I can reproduce 
> Gabriel's founding:
> 

Yeah, thanks. Anyway, I would not like introduce header file dependency
too much, especially header files under asm.
Please check below my fix;

https://lore.kernel.org/all/173881156244.211648.1242168038709680511.stgit@devnote2/

Thank you,

> +#ifdef CONFIG_X86_KERNEL_IBT
> +#include <linux/uaccess.h>
> +#endif
> +
>   #ifndef __ASSEMBLY__
> 
> 
> In file included from ./include/linux/kcsan-checks.h:13,
>                   from ./include/linux/instrumented.h:12,
>                   from ./include/linux/uaccess.h:6,
>                   from ./arch/x86/include/asm/ftrace.h:26,
>                   from arch/x86/kernel/ftrace_64.S:11:
> ./include/linux/compiler_attributes.h:91:20: error: missing binary 
> operator before token "("
>     91 | #if __has_attribute(__copy__)
>        |                    ^
> ./include/linux/compiler_attributes.h:103:20: error: missing binary 
> operator before token "("
>    103 | #if __has_attribute(__diagnose_as_builtin__)
>        |                    ^
> ./include/linux/compiler_attributes.h:126:20: error: missing binary 
> operator before token "("
>    126 | #if __has_attribute(__designated_init__)
>        |                    ^
> ./include/linux/compiler_attributes.h:137:20: error: missing binary 
> operator before token "("
>    137 | #if __has_attribute(__error__)
>        |                    ^
> 
> 
> >>
> >> -- Steve
> > 
> > 
> 


-- 
Masami Hiramatsu (Google) <mhiramat@...nel.org>