lkml.org 
[lkml]   [2016]   [Aug]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/3] tracing: Add trace_irqsoff tracepoints
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
[unhandled content-type:application/octet-stream]
\
 
 \ /
  Last update: 2016-09-17 09:57    [W:0.126 / U:1.024 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site