Re: [PATCH v1] ftrace: Fix compile error when CONFIG_GENDWARFKSYMS is enabled

[Haiyue Wang <haiyuewa@163.com>]

On 2025/2/6 10:45, Masami Hiramatsu (Google) wrote:
> On Wed, 5 Feb 2025 19:53:00 -0500
> Steven Rostedt <rostedt@goodmis.org> wrote:
> 
>> On Thu, 6 Feb 2025 09:38:53 +0900
>> Masami Hiramatsu (Google) <mhiramat@kernel.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:

+#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
> 
>