[trace:ftrace/direct 10/10] arch/x86//kernel/ftrace.c:1052:6: error: 'ftrace_direct_func_count' undeclared; did you mean 'ftrace_direct_func'?

[trace:ftrace/direct 10/10] arch/x86//kernel/ftrace.c:1052:6: error: 'ftrace_direct_func_count' undeclared; did you mean 'ftrace_direct_func'?

[kbuild test robot]

[-- Attachment #1: Type: text/plain, Size: 3796 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git ftrace/direct
head:   6596f86ef6098066a2182ce37b55af08e0c93512
commit: 6596f86ef6098066a2182ce37b55af08e0c93512 [10/10] ftrace/x86: Add a counter to test function_graph with direct
config: x86_64-randconfig-g003-201944 (attached as .config)
compiler: gcc-7 (Debian 7.4.0-14) 7.4.0
reproduce:
        git checkout 6596f86ef6098066a2182ce37b55af08e0c93512
        # save the attached .config to linux build tree
        make ARCH=x86_64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All errors (new ones prefixed by >>):

   In file included from include/linux/export.h:42:0,
                    from include/linux/linkage.h:7,
                    from include/linux/preempt.h:10,
                    from include/linux/spinlock.h:51,
                    from arch/x86//kernel/ftrace.c:15:
   arch/x86//kernel/ftrace.c: In function 'prepare_ftrace_return':
>> arch/x86//kernel/ftrace.c:1052:6: error: 'ftrace_direct_func_count' undeclared (first use in this function); did you mean 'ftrace_direct_func'?
     if (ftrace_direct_func_count) {
         ^
   include/linux/compiler.h:58:52: note: in definition of macro '__trace_if_var'
    #define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
                                                       ^~~~
   arch/x86//kernel/ftrace.c:1052:2: note: in expansion of macro 'if'
     if (ftrace_direct_func_count) {
     ^~
   arch/x86//kernel/ftrace.c:1052:6: note: each undeclared identifier is reported only once for each function it appears in
     if (ftrace_direct_func_count) {
         ^
   include/linux/compiler.h:58:52: note: in definition of macro '__trace_if_var'
    #define __trace_if_var(cond) (__builtin_constant_p(cond) ? (cond) : __trace_if_value(cond))
                                                       ^~~~
   arch/x86//kernel/ftrace.c:1052:2: note: in expansion of macro 'if'
     if (ftrace_direct_func_count) {
     ^~

vim +1052 arch/x86//kernel/ftrace.c

  1014	
  1015	/*
  1016	 * Hook the return address and push it in the stack of return addrs
  1017	 * in current thread info.
  1018	 */
  1019	void prepare_ftrace_return(unsigned long self_addr, unsigned long *parent,
  1020				   unsigned long frame_pointer)
  1021	{
  1022		unsigned long old;
  1023		int faulted;
  1024		unsigned long return_hooker = (unsigned long)
  1025					&return_to_handler;
  1026	
  1027		/*
  1028		 * When resuming from suspend-to-ram, this function can be indirectly
  1029		 * called from early CPU startup code while the CPU is in real mode,
  1030		 * which would fail miserably.  Make sure the stack pointer is a
  1031		 * virtual address.
  1032		 *
  1033		 * This check isn't as accurate as virt_addr_valid(), but it should be
  1034		 * good enough for this purpose, and it's fast.
  1035		 */
  1036		if (unlikely((long)__builtin_frame_address(0) >= 0))
  1037			return;
  1038	
  1039		if (unlikely(ftrace_graph_is_dead()))
  1040			return;
  1041	
  1042		if (unlikely(atomic_read(&current->tracing_graph_pause)))
  1043			return;
  1044	
  1045		/*
  1046		 * If the return location is actually pointing directly to
  1047		 * the start of a direct trampoline (if we trace the trampoline
  1048		 * it will still be offset by MCOUNT_INSN_SIZE), then the
  1049		 * return address is actually off by one word, and we
  1050		 * need to adjust for that.
  1051		 */
> 1052		if (ftrace_direct_func_count) {

---
0-DAY kernel test infrastructure                 Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org Intel Corporation

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 33469 bytes --]