Hi Linyu,

kernel test robot noticed the following build warnings:

[auto build test WARNING on usb/usb-testing]
[also build test WARNING on usb/usb-next usb/usb-linus linus/master v6.6-rc1 
next-20230914]
[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/Linyu-Yuan/trace-add-new-DECLARE_EVENT_CLASS_PRINT_INIT-class-type/20230914-180924
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git 
usb-testing
patch link:    
https://lore.kernel.org/r/20230914100302.30274-2-quic_linyyuan%40quicinc.com
patch subject: [PATCH 1/8] trace: add new DECLARE_EVENT_CLASS_PRINT_INIT class 
type
config: arm-defconfig 
(https://download.01.org/0day-ci/archive/20230914/202309142216.gwm6q6l0-...@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): 
(https://download.01.org/0day-ci/archive/20230914/202309142216.gwm6q6l0-...@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 <l...@intel.com>
| Closes: 
https://lore.kernel.org/oe-kbuild-all/202309142216.gwm6q6l0-...@intel.com/

All warnings (new ones prefixed by >>):

   In file included from include/trace/events/migrate.h:8,
                    from mm/rmap.c:83:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/tlb.h:62,
                    from mm/rmap.c:82:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
--
   In file included from include/trace/events/net.h:12,
                    from net/core/net-traces.c:31:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/skb.h:95,
                    from net/core/net-traces.c:30:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
   In file included from include/trace/events/napi.h:9,
                    from net/core/net-traces.c:32:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/net.h:319:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
   In file included from include/trace/events/sock.h:10,
                    from net/core/net-traces.c:33:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/napi.h:44:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
   In file included from include/trace/events/udp.h:9,
                    from net/core/net-traces.c:34:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/sock.h:338:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
   In file included from include/trace/events/tcp.h:10,
                    from net/core/net-traces.c:35:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/udp.h:33:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
   In file included from include/trace/events/fib.h:11,
                    from net/core/net-traces.c:36:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/tcp.h:425:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
   In file included from include/trace/events/qdisc.h:9,
                    from net/core/net-traces.c:37:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/fib.h:102:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
   In file included from include/trace/events/page_pool.h:9,
                    from net/core/net-traces.c:48:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/qdisc.h:153:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
   In file included from include/trace/events/neigh.h:9,
                    from net/core/net-traces.c:51:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/page_pool.h:117:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
--
   In file included from include/trace/events/ipi.h:8,
                    from kernel/sched/core.c:83:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/sched.h:741,
                    from kernel/sched/core.c:82:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 
   In file included from include/trace/events/power.h:11,
                    from kernel/sched/sched.h:72,
                    from kernel/sched/core.c:86:
>> include/linux/tracepoint.h:554: warning: "DECLARE_EVENT_CLASS_PRINT_INIT" 
>> redefined
     554 | #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
assign, print, init)
         | 
   In file included from include/trace/define_trace.h:103,
                    from include/trace/events/ipi.h:134:
   include/trace/perf.h:59: note: this is the location of the previous 
definition
      59 | #define DECLARE_EVENT_CLASS_PRINT_INIT(call, proto, args, tstruct, 
assign, print, init) \
         | 
>> include/linux/tracepoint.h:580: warning: "TRACE_EVENT_PRINT_INIT" redefined
     580 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
print, init)  \
         | 
   In file included from include/trace/define_trace.h:102:
   include/trace/trace_events.h:49: note: this is the location of the previous 
definition
      49 | #define TRACE_EVENT_PRINT_INIT(name, proto, args, tstruct, assign, 
print, init) \
         | 


vim +/DECLARE_EVENT_CLASS_PRINT_INIT +554 include/linux/tracepoint.h

   446  
   447  #ifndef TRACE_EVENT
   448  /*
   449   * For use with the TRACE_EVENT macro:
   450   *
   451   * We define a tracepoint, its arguments, its printk format
   452   * and its 'fast binary record' layout.
   453   *
   454   * Firstly, name your tracepoint via TRACE_EVENT(name : the
   455   * 'subsystem_event' notation is fine.
   456   *
   457   * Think about this whole construct as the
   458   * 'trace_sched_switch() function' from now on.
   459   *
   460   *
   461   *  TRACE_EVENT(sched_switch,
   462   *
   463   *      *
   464   *      * A function has a regular function arguments
   465   *      * prototype, declare it via TP_PROTO():
   466   *      *
   467   *
   468   *      TP_PROTO(struct rq *rq, struct task_struct *prev,
   469   *               struct task_struct *next),
   470   *
   471   *      *
   472   *      * Define the call signature of the 'function'.
   473   *      * (Design sidenote: we use this instead of a
   474   *      *  TP_PROTO1/TP_PROTO2/TP_PROTO3 ugliness.)
   475   *      *
   476   *
   477   *      TP_ARGS(rq, prev, next),
   478   *
   479   *      *
   480   *      * Fast binary tracing: define the trace record via
   481   *      * TP_STRUCT__entry(). You can think about it like a
   482   *      * regular C structure local variable definition.
   483   *      *
   484   *      * This is how the trace record is structured and will
   485   *      * be saved into the ring buffer. These are the fields
   486   *      * that will be exposed to user-space in
   487   *      * /sys/kernel/tracing/events/<*>/format.
   488   *      *
   489   *      * The declared 'local variable' is called '__entry'
   490   *      *
   491   *      * __field(pid_t, prev_pid) is equivalent to a standard 
declaration:
   492   *      *
   493   *      *       pid_t   prev_pid;
   494   *      *
   495   *      * __array(char, prev_comm, TASK_COMM_LEN) is equivalent to:
   496   *      *
   497   *      *       char    prev_comm[TASK_COMM_LEN];
   498   *      *
   499   *
   500   *      TP_STRUCT__entry(
   501   *              __array(        char,   prev_comm,      TASK_COMM_LEN   
)
   502   *              __field(        pid_t,  prev_pid                        
)
   503   *              __field(        int,    prev_prio                       
)
   504   *              __array(        char,   next_comm,      TASK_COMM_LEN   
)
   505   *              __field(        pid_t,  next_pid                        
)
   506   *              __field(        int,    next_prio                       
)
   507   *      ),
   508   *
   509   *      *
   510   *      * Assign the entry into the trace record, by embedding
   511   *      * a full C statement block into TP_fast_assign(). You
   512   *      * can refer to the trace record as '__entry' -
   513   *      * otherwise you can put arbitrary C code in here.
   514   *      *
   515   *      * Note: this C code will execute every time a trace event
   516   *      * happens, on an active tracepoint.
   517   *      *
   518   *
   519   *      TP_fast_assign(
   520   *              memcpy(__entry->next_comm, next->comm, TASK_COMM_LEN);
   521   *              __entry->prev_pid       = prev->pid;
   522   *              __entry->prev_prio      = prev->prio;
   523   *              memcpy(__entry->prev_comm, prev->comm, TASK_COMM_LEN);
   524   *              __entry->next_pid       = next->pid;
   525   *              __entry->next_prio      = next->prio;
   526   *      ),
   527   *
   528   *      *
   529   *      * Formatted output of a trace record via TP_printk().
   530   *      * This is how the tracepoint will appear under ftrace
   531   *      * plugins that make use of this tracepoint.
   532   *      *
   533   *      * (raw-binary tracing wont actually perform this step.)
   534   *      *
   535   *
   536   *      TP_printk("task %s:%d [%d] ==> %s:%d [%d]",
   537   *              __entry->prev_comm, __entry->prev_pid, 
__entry->prev_prio,
   538   *              __entry->next_comm, __entry->next_pid, 
__entry->next_prio),
   539   *
   540   * );
   541   *
   542   * This macro construct is thus used for the regular printk format
   543   * tracing setup, it is used to construct a function pointer based
   544   * tracepoint callback (this is used by programmatic plugins and
   545   * can also by used by generic instrumentation like SystemTap), and
   546   * it is also used to expose a structured trace record in
   547   * /sys/kernel/tracing/events/.
   548   *
   549   * A set of (un)registration functions can be passed to the variant
   550   * TRACE_EVENT_FN to perform any (un)registration work.
   551   */
   552  
   553  #define DECLARE_EVENT_CLASS(name, proto, args, tstruct, assign, print)
 > 554  #define DECLARE_EVENT_CLASS_PRINT_INIT(name, proto, args, tstruct, 
 > assign, print, init)
   555  #define DEFINE_EVENT(template, name, proto, args)               \
   556          DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
   557  #define DEFINE_EVENT_FN(template, name, proto, args, reg, unreg)\
   558          DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
   559  #define DEFINE_EVENT_PRINT(template, name, proto, args, print)  \
   560          DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
   561  #define DEFINE_EVENT_CONDITION(template, name, proto,           \
   562                                 args, cond)                      \
   563          DECLARE_TRACE_CONDITION(name, PARAMS(proto),            \
   564                                  PARAMS(args), PARAMS(cond))
   565  
   566  #define TRACE_EVENT(name, proto, args, struct, assign, print)   \
   567          DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
   568  #define TRACE_EVENT_FN(name, proto, args, struct,               \
   569                  assign, print, reg, unreg)                      \
   570          DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
   571  #define TRACE_EVENT_FN_COND(name, proto, args, cond, struct,            
\
   572                  assign, print, reg, unreg)                      \
   573          DECLARE_TRACE_CONDITION(name, PARAMS(proto),    \
   574                          PARAMS(args), PARAMS(cond))
   575  #define TRACE_EVENT_CONDITION(name, proto, args, cond,          \
   576                                struct, assign, print)            \
   577          DECLARE_TRACE_CONDITION(name, PARAMS(proto),            \
   578                                  PARAMS(args), PARAMS(cond))
   579  
 > 580  #define TRACE_EVENT_PRINT_INIT(name, proto, args, struct, assign, 
 > print, init)  \
   581          DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
   582  #define TRACE_EVENT_FN_PRINT_INIT(name, proto, args, struct,            
\
   583                  assign, print, reg, unreg, init)                        
\
   584          DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
   585  #define TRACE_EVENT_FN_COND_PRINT_INIT(name, proto, args, cond, struct, 
        \
   586                  assign, print, reg, unreg, init)                        
\
   587          DECLARE_TRACE_CONDITION(name, PARAMS(proto),    \
   588                          PARAMS(args), PARAMS(cond))
   589  #define TRACE_EVENT_CONDITION_PRINT_INIT(name, proto, args, cond,       
        \
   590                                struct, assign, print, init)              
\
   591          DECLARE_TRACE_CONDITION(name, PARAMS(proto),            \
   592                                  PARAMS(args), PARAMS(cond))
   593  

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

Reply via email to