Re: [PATCH 2/3] tracing: Add trace_irqsoff tracepoints

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

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

Hi Binoy,

[auto build test WARNING on tip/perf/core]
[also build test WARNING on v4.8-rc3 next-20160819]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
[Suggest to use git(>=2.9.0) format-patch --base=<commit> (or --base=auto for convenience) to record what (public, well-known) commit your patch series was built on]
[Check https://git-scm.com/docs/git-format-patch for more information]

url:    https://github.com/0day-ci/linux/commits/Binoy-Jayan/tracing-Deference-pointers-without-RCU-checks/20160818-151235
config: i386-randconfig-c0-08190953 (attached as .config)
compiler: gcc-4.9 (Debian 4.9.3-14) 4.9.3
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All warnings (new ones prefixed by >>):

   In file included from include/linux/linkage.h:4:0,
                    from include/linux/kernel.h:6,
                    from include/linux/kallsyms.h:9,
                    from kernel/trace/trace_irqsoff.c:12:
   include/trace/events/latency.h: In function 'trace_raw_output_latency_template':
>> include/linux/compiler.h:152:17: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'cycles_t' [-Wformat=]
      static struct ftrace_branch_data   \
                    ^
   include/linux/compiler.h:147:23: note: in expansion of macro '__trace_if'
    #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
                          ^
   include/trace/trace_events.h:324:2: note: in expansion of macro 'if'
     if (ret != TRACE_TYPE_HANDLED)     \
     ^
   include/trace/events/latency.h:9:1: note: in expansion of macro 'DECLARE_EVENT_CLASS'
    DECLARE_EVENT_CLASS(latency_template,
    ^
   In file included from include/asm-generic/percpu.h:6:0,
                    from arch/x86/include/asm/percpu.h:552,
                    from arch/x86/include/asm/preempt.h:5,
                    from include/linux/preempt.h:59,
                    from include/linux/spinlock.h:50,
                    from include/linux/seqlock.h:35,
                    from include/linux/time.h:5,
                    from include/uapi/linux/timex.h:56,
                    from include/linux/timex.h:56,
                    from include/linux/sched.h:19,
                    from include/linux/uaccess.h:4,
                    from kernel/trace/trace_irqsoff.c:13:
   kernel/trace/trace_irqsoff.c: At top level:
   kernel/trace/trace_irqsoff.c:28:32: warning: 'ts_irqs' defined but not used [-Wunused-variable]
    static DEFINE_PER_CPU(cycle_t, ts_irqs);
                                   ^
   include/linux/percpu-defs.h:105:19: note: in definition of macro 'DEFINE_PER_CPU_SECTION'
     __typeof__(type) name
                      ^
   kernel/trace/trace_irqsoff.c:28:8: note: in expansion of macro 'DEFINE_PER_CPU'
    static DEFINE_PER_CPU(cycle_t, ts_irqs);
           ^
   kernel/trace/trace_irqsoff.c:29:32: warning: 'ts_preempt' defined but not used [-Wunused-variable]
    static DEFINE_PER_CPU(cycle_t, ts_preempt);
                                   ^
   include/linux/percpu-defs.h:105:19: note: in definition of macro 'DEFINE_PER_CPU_SECTION'
     __typeof__(type) name
                      ^
   kernel/trace/trace_irqsoff.c:29:8: note: in expansion of macro 'DEFINE_PER_CPU'
    static DEFINE_PER_CPU(cycle_t, ts_preempt);
           ^

vim +152 include/linux/compiler.h

45b79749 Steven Rostedt 2008-11-21  136  #  define likely(x)	(__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 1))
1f0d69a9 Steven Rostedt 2008-11-12  137  # endif
1f0d69a9 Steven Rostedt 2008-11-12  138  # ifndef unlikely
45b79749 Steven Rostedt 2008-11-21  139  #  define unlikely(x)	(__builtin_constant_p(x) ? !!(x) : __branch_check__(x, 0))
1f0d69a9 Steven Rostedt 2008-11-12  140  # endif
2bcd521a Steven Rostedt 2008-11-21  141  
2bcd521a Steven Rostedt 2008-11-21  142  #ifdef CONFIG_PROFILE_ALL_BRANCHES
2bcd521a Steven Rostedt 2008-11-21  143  /*
2bcd521a Steven Rostedt 2008-11-21  144   * "Define 'is'", Bill Clinton
2bcd521a Steven Rostedt 2008-11-21  145   * "Define 'if'", Steven Rostedt
2bcd521a Steven Rostedt 2008-11-21  146   */
ab3c9c68 Linus Torvalds 2009-04-07  147  #define if(cond, ...) __trace_if( (cond , ## __VA_ARGS__) )
ab3c9c68 Linus Torvalds 2009-04-07  148  #define __trace_if(cond) \
b33c8ff4 Arnd Bergmann  2016-02-12  149  	if (__builtin_constant_p(!!(cond)) ? !!(cond) :			\
2bcd521a Steven Rostedt 2008-11-21  150  	({								\
2bcd521a Steven Rostedt 2008-11-21  151  		int ______r;						\
2bcd521a Steven Rostedt 2008-11-21 @152  		static struct ftrace_branch_data			\
2bcd521a Steven Rostedt 2008-11-21  153  			__attribute__((__aligned__(4)))			\
2bcd521a Steven Rostedt 2008-11-21  154  			__attribute__((section("_ftrace_branch")))	\
2bcd521a Steven Rostedt 2008-11-21  155  			______f = {					\
2bcd521a Steven Rostedt 2008-11-21  156  				.func = __func__,			\
2bcd521a Steven Rostedt 2008-11-21  157  				.file = __FILE__,			\
2bcd521a Steven Rostedt 2008-11-21  158  				.line = __LINE__,			\
2bcd521a Steven Rostedt 2008-11-21  159  			};						\
2bcd521a Steven Rostedt 2008-11-21  160  		______r = !!(cond);					\

:::::: The code at line 152 was first introduced by commit
:::::: 2bcd521a684cc94befbe2ce7d5b613c841b0d304 trace: profile all if conditionals

:::::: TO: Steven Rostedt <srostedt@redhat.com>
:::::: CC: Ingo Molnar <mingo@elte.hu>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 30615 bytes --]