Re: [PATCH 04/10] rv: Add rtapp_block monitor

[kernel test robot <lkp@intel.com>]

Hi Nam,

kernel test robot noticed the following build warnings:

[auto build test WARNING on trace/for-next]
[also build test WARNING on tip/x86/core tip/x86/mm linus/master v6.14-rc6 next-20250313]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Nam-Cao/rv-Add-undef-TRACE_INCLUDE_FILE/20250312-011043
base:   https://git.kernel.org/pub/scm/linux/kernel/git/trace/linux-trace for-next
patch link:    https://lore.kernel.org/r/c8a1658c6da343b6055cbfcd6ee5ddea48335d31.1741708239.git.namcao%40linutronix.de
patch subject: [PATCH 04/10] rv: Add rtapp_block monitor
config: hexagon-allyesconfig (https://download.01.org/0day-ci/archive/20250313/202503132244.Azu340js-lkp@intel.com/config)
compiler: clang version 18.1.8 (https://github.com/llvm/llvm-project 3b5b5c1ec4a3095ab096dd780e84d7ab81f3d7ff)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20250313/202503132244.Azu340js-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202503132244.Azu340js-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:91:53: warning: omitting the parameter name in a function definition is a C23 extension [-Wc23-extensions]
      91 | static void handle_rt_mutex_wake_waiter_begin(void *, struct task_struct *task)
         |                                                     ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:96:51: warning: omitting the parameter name in a function definition is a C23 extension [-Wc23-extensions]
      96 | static void handle_rt_mutex_wake_waiter_end(void *, struct task_struct *task)
         |                                                   ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:101:45: warning: omitting the parameter name in a function definition is a C23 extension [-Wc23-extensions]
     101 | static void handle_sched_kthread_stop(void *, struct task_struct *task)
         |                                             ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:109:49: warning: omitting the parameter name in a function definition is a C23 extension [-Wc23-extensions]
     109 | static void handle_sched_kthread_stop_ret(void *, int)
         |                                                 ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:109:54: warning: omitting the parameter name in a function definition is a C23 extension [-Wc23-extensions]
     109 | static void handle_sched_kthread_stop_ret(void *, int)
         |                                                      ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:117:39: warning: omitting the parameter name in a function definition is a C23 extension [-Wc23-extensions]
     117 | static void handle_sched_wakeup(void *, struct task_struct *task)
         |                                       ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:164:2: error: call to undeclared function 'check_trace_callback_type_sys_enter'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
     164 |         rv_attach_trace_probe("rtapp_block", sys_enter, handle_sys_enter);
         |         ^
   include/rv/instrumentation.h:18:3: note: expanded from macro 'rv_attach_trace_probe'
      18 |                 check_trace_callback_type_##tp(rv_handler);                             \
         |                 ^
   <scratch space>:8:1: note: expanded from here
       8 | check_trace_callback_type_sys_enter
         | ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:164:2: error: call to undeclared function 'register_trace_sys_enter'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
   include/rv/instrumentation.h:19:13: note: expanded from macro 'rv_attach_trace_probe'
      19 |                 WARN_ONCE(register_trace_##tp(rv_handler, NULL),                        \
         |                           ^
   <scratch space>:9:1: note: expanded from here
       9 | register_trace_sys_enter
         | ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:165:2: error: call to undeclared function 'check_trace_callback_type_sys_exit'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
     165 |         rv_attach_trace_probe("rtapp_block", sys_exit, handle_sys_exit);
         |         ^
   include/rv/instrumentation.h:18:3: note: expanded from macro 'rv_attach_trace_probe'
      18 |                 check_trace_callback_type_##tp(rv_handler);                             \
         |                 ^
   <scratch space>:14:1: note: expanded from here
      14 | check_trace_callback_type_sys_exit
         | ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:165:2: error: call to undeclared function 'register_trace_sys_exit'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
   include/rv/instrumentation.h:19:13: note: expanded from macro 'rv_attach_trace_probe'
      19 |                 WARN_ONCE(register_trace_##tp(rv_handler, NULL),                        \
         |                           ^
   <scratch space>:15:1: note: expanded from here
      15 | register_trace_sys_exit
         | ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:165:2: note: did you mean 'register_ftrace_direct'?
   include/rv/instrumentation.h:19:13: note: expanded from macro 'rv_attach_trace_probe'
      19 |                 WARN_ONCE(register_trace_##tp(rv_handler, NULL),                        \
         |                           ^
   <scratch space>:15:1: note: expanded from here
      15 | register_trace_sys_exit
         | ^
   include/linux/ftrace.h:535:19: note: 'register_ftrace_direct' declared here
     535 | static inline int register_ftrace_direct(struct ftrace_ops *ops, unsigned long addr)
         |                   ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:180:2: error: call to undeclared function 'unregister_trace_sys_enter'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
     180 |         rv_detach_trace_probe("rtapp_block", sys_enter, handle_sys_enter);
         |         ^
   include/rv/instrumentation.h:28:3: note: expanded from macro 'rv_detach_trace_probe'
      28 |                 unregister_trace_##tp(rv_handler, NULL);                                \
         |                 ^
   <scratch space>:29:1: note: expanded from here
      29 | unregister_trace_sys_enter
         | ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:181:2: error: call to undeclared function 'unregister_trace_sys_exit'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
     181 |         rv_detach_trace_probe("rtapp_block", sys_exit, handle_sys_exit);
         |         ^
   include/rv/instrumentation.h:28:3: note: expanded from macro 'rv_detach_trace_probe'
      28 |                 unregister_trace_##tp(rv_handler, NULL);                                \
         |                 ^
   <scratch space>:30:1: note: expanded from here
      30 | unregister_trace_sys_exit
         | ^
   kernel/trace/rv/monitors/rtapp_block/rtapp_block.c:181:2: note: did you mean 'unregister_ftrace_direct'?
   include/rv/instrumentation.h:28:3: note: expanded from macro 'rv_detach_trace_probe'
      28 |                 unregister_trace_##tp(rv_handler, NULL);                                \
         |                 ^
   <scratch space>:30:1: note: expanded from here
      30 | unregister_trace_sys_exit
         | ^
   include/linux/ftrace.h:539:19: note: 'unregister_ftrace_direct' declared here
     539 | static inline int unregister_ftrace_direct(struct ftrace_ops *ops, unsigned long addr,
         |                   ^
   6 warnings and 6 errors generated.


vim +91 kernel/trace/rv/monitors/rtapp_block/rtapp_block.c

    90	
  > 91	static void handle_rt_mutex_wake_waiter_begin(void *, struct task_struct *task)
    92	{
    93		rv_rtapp_block_atom_update(task, RT_MUTEX_WAKING_WAITER, true);
    94	}
    95	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki