Hi Daniel, [FYI, it's a private test report for your RFC patch.] [auto build test WARNING on rostedt-trace/for-next] [also build test WARNING on lwn/docs-next linus/master v5.17-rc4 next-20220214] [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] url: https://github.com/0day-ci/linux/commits/Daniel-Bristot-de-Oliveira/The-R... base: https://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git for-next config: hexagon-allyesconfig (https://download.01.org/0day-ci/archive/20220215/202202150429.Rn1Sp6Xo-lk...) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project ea071884b0cc7210b3cc5fe858f0e892a779a23b) reproduce (this is a W=1 build): wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # https://github.com/0day-ci/linux/commit/d19976adc824ba2162ba4c5690e0a2f2a... git remote add linux-review https://github.com/0day-ci/linux git fetch --no-tags linux-review Daniel-Bristot-de-Oliveira/The-Runtime-Verification-RV-interface/20220214-193144 git checkout d19976adc824ba2162ba4c5690e0a2f2aba219a6 # save the config file to linux build tree mkdir build_dir COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=hexagon SHELL=/bin/bash kernel/trace/rv/ If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <lkp(a)intel.com&gt; All warnings (new ones prefixed by >>): DECLARE_DA_MON_PER_CPU(wip, char); ^ include/rv/da_monitor.h:374:11: note: expanded from macro 'DECLARE_DA_MON_PER_CPU' \ ^ include/rv/automata.h:18:7: note: expanded from macro '\ DECLARE_AUTOMATA_HELPERS' char *model_get_state_name_##name(enum states_##name state) \ ^ <scratch space>:86:1: note: expanded from here model_get_state_name_wip ^ kernel/trace/rv/monitor_wip/wip.c:23:1: note: declare 'static' if the function is not intended to be used outside of this translation unit include/rv/da_monitor.h:374:11: note: expanded from macro 'DECLARE_DA_MON_PER_CPU' \ ^ include/rv/automata.h:17:11: note: expanded from macro '\ DECLARE_AUTOMATA_HELPERS' \ ^ DECLARE_DA_MON_PER_CPU(wip, char); ^ include/rv/da_monitor.h:374:11: note: expanded from macro 'DECLARE_DA_MON_PER_CPU' \ ^ include/rv/automata.h:23:7: note: expanded from macro '\ DECLARE_AUTOMATA_HELPERS' char *model_get_event_name_##name(enum events_##name event) \ ^ <scratch space>:89:1: note: expanded from here model_get_event_name_wip ^ kernel/trace/rv/monitor_wip/wip.c:23:1: note: declare 'static' if the function is not intended to be used outside of this translation unit include/rv/da_monitor.h:374:11: note: expanded from macro 'DECLARE_DA_MON_PER_CPU' \ ^ include/rv/automata.h:22:11: note: expanded from macro '\ DECLARE_AUTOMATA_HELPERS' \ ^ DECLARE_DA_MON_PER_CPU(wip, char); ^ include/rv/da_monitor.h:380:11: note: expanded from macro 'DECLARE_DA_MON_PER_CPU' \ ^ include/rv/da_monitor.h:172:20: note: expanded from macro '\ DECLARE_DA_MON_INIT_PER_CPU' struct da_monitor *da_get_monitor_##name(void) \ ^ <scratch space>:126:1: note: expanded from here da_get_monitor_wip ^ kernel/trace/rv/monitor_wip/wip.c:23:1: note: declare 'static' if the function is not intended to be used outside of this translation unit include/rv/da_monitor.h:380:11: note: expanded from macro 'DECLARE_DA_MON_PER_CPU' \ ^ include/rv/da_monitor.h:171:11: note: expanded from macro '\ DECLARE_DA_MON_INIT_PER_CPU' \ ^ DECLARE_DA_MON_PER_CPU(wip, char); ^ include/rv/da_monitor.h:380:11: note: expanded from macro 'DECLARE_DA_MON_PER_CPU' \ ^ include/rv/da_monitor.h:177:6: note: expanded from macro '\ DECLARE_DA_MON_INIT_PER_CPU' void da_monitor_reset_all_##name(void) \ ^ <scratch space>:128:1: note: expanded from here da_monitor_reset_all_wip ^ kernel/trace/rv/monitor_wip/wip.c:23:1: note: declare 'static' if the function is not intended to be used outside of this translation unit include/rv/da_monitor.h:380:11: note: expanded from macro 'DECLARE_DA_MON_PER_CPU' \ ^ include/rv/da_monitor.h:176:11: note: expanded from macro '\ DECLARE_DA_MON_INIT_PER_CPU' \ ^ void handle_preempt_disable(void *data, unsigned long ip, unsigned long parent_ip) ^ kernel/trace/rv/monitor_wip/wip.c:36:1: note: declare 'static' if the function is not intended to be used outside of this translation unit void handle_preempt_disable(void *data, unsigned long ip, unsigned long parent_ip) ^ static void handle_preempt_enable(void *data, unsigned long ip, unsigned long parent_ip) ^ kernel/trace/rv/monitor_wip/wip.c:41:1: note: declare 'static' if the function is not intended to be used outside of this translation unit void handle_preempt_enable(void *data, unsigned long ip, unsigned long parent_ip) ^ static void handle_sched_waking(void *data, struct task_struct *task) ^ kernel/trace/rv/monitor_wip/wip.c:46:1: note: declare 'static' if the function is not intended to be used outside of this translation unit void handle_sched_waking(void *data, struct task_struct *task) ^ static int register_wip(void) ^ kernel/trace/rv/monitor_wip/wip.c:105:1: note: declare 'static' if the function is not intended to be used outside of this translation unit int register_wip(void) ^ static void unregister_wip(void) ^ kernel/trace/rv/monitor_wip/wip.c:111:1: note: declare 'static' if the function is not intended to be used outside of this translation unit void unregister_wip(void) ^ static kernel/trace/rv/monitor_wip/wip.c:23:1: warning: unused function 'da_handle_init_run_event_wip' [-Wunused-function] DECLARE_DA_MON_PER_CPU(wip, char); ^ include/rv/da_monitor.h:382:11: note: expanded from macro 'DECLARE_DA_MON_PER_CPU' \ ^ include/rv/da_monitor.h:285:20: note: expanded from macro '\ DECLARE_DA_MON_MONITOR_HANDLER_IMPLICIT' static inline bool da_handle_init_run_event_##name(enum events_##name event) \ ^ <scratch space>:152:1: note: expanded from here da_handle_init_run_event_wip ^ 10 warnings generated. vim +/model_get_state_name_wip +23 kernel/trace/rv/monitor_wip/wip.c 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 16 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 17 /* 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 18 * Declare the deterministic automata monitor. 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 19 * 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 20 * The rv monitor reference is needed for the monitor declaration. 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 21 */ 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 22 struct rv_monitor rv_wip; 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 @23 DECLARE_DA_MON_PER_CPU(wip, char); 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 24 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 25 #define CREATE_TRACE_POINTS 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 26 #include "wip.h" 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 27 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 28 /* 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 29 * This is the instrumentation part of the monitor. 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 30 * 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 31 * This is the section where manual work is required. Here the kernel events 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 32 * are translated into model's event. 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 33 * 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 34 */ 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 35 d19976adc824ba Daniel Bristot de Oliveira 2022-02-14 @36 void handle_preempt_disable(void *data, unsigned long ip, unsigned long parent_ip) 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 37 { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 38 da_handle_event_wip(preempt_disable); 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 39 } 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 40 d19976adc824ba Daniel Bristot de Oliveira 2022-02-14 @41 void handle_preempt_enable(void *data, unsigned long ip, unsigned long parent_ip) 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 42 { d19976adc824ba Daniel Bristot de Oliveira 2022-02-14 43 da_handle_init_event_wip(preempt_enable); 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 44 } 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 45 d19976adc824ba Daniel Bristot de Oliveira 2022-02-14 @46 void handle_sched_waking(void *data, struct task_struct *task) 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 47 { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 48 da_handle_event_wip(sched_waking); 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 49 } 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 50 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 51 #define NR_TP 3 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 52 static struct tracepoint_hook_helper tracepoints_to_hook[NR_TP] = { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 53 { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 54 .probe = handle_preempt_disable, d19976adc824ba Daniel Bristot de Oliveira 2022-02-14 55 .name = "preempt_disable", 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 56 .registered = 0 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 57 }, 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 58 { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 59 .probe = handle_preempt_enable, d19976adc824ba Daniel Bristot de Oliveira 2022-02-14 60 .name = "preempt_enable", 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 61 .registered = 0 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 62 }, 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 63 { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 64 .probe = handle_sched_waking, d19976adc824ba Daniel Bristot de Oliveira 2022-02-14 65 .name = "sched_wakeup", 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 66 .registered = 0 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 67 }, 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 68 }; 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 69 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 70 static int start_wip(void) 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 71 { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 72 int retval; 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 73 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 74 da_monitor_init_wip(); 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 75 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 76 retval = thh_hook_probes(tracepoints_to_hook, NR_TP); 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 77 if (retval) 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 78 goto out_err; 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 79 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 80 return 0; 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 81 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 82 out_err: 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 83 return -EINVAL; 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 84 } 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 85 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 86 static void stop_wip(void) 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 87 { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 88 rv_wip.enabled = 0; 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 89 thh_unhook_probes(tracepoints_to_hook, NR_TP); 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 90 return; 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 91 } 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 92 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 93 /* 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 94 * This is the monitor register section. 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 95 */ 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 96 struct rv_monitor rv_wip = { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 97 .name = "wip", 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 98 .description = "auto-generated wip", 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 99 .start = start_wip, 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 100 .stop = stop_wip, 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 101 .reset = da_monitor_reset_all_wip, 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 102 .enabled = 0, 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 103 }; 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 104 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 @105 int register_wip(void) 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 106 { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 107 rv_register_monitor(&rv_wip); 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 108 return 0; 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 109 } 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 110 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 @111 void unregister_wip(void) 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 112 { 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 113 if (rv_wip.enabled) 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 114 stop_wip(); 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 115 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 116 rv_unregister_monitor(&rv_wip); 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 117 } 66e3ae1a584a3c Daniel Bristot de Oliveira 2022-02-14 118 --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org