From: kbuild test robot <lkp@intel.com> To: Masami Hiramatsu <mhiramat@kernel.org> Cc: kbuild-all@lists.01.org, Steven Rostedt <rostedt@goodmis.org>, Frank Rowand <frowand.list@gmail.com>, Ingo Molnar <mingo@redhat.com>, Randy Dunlap <rdunlap@infradead.org>, Namhyung Kim <namhyung@kernel.org>, Tim Bird <Tim.Bird@sony.com>, Jiri Olsa <jolsa@redhat.com>, Arnaldo Carvalho de Melo <acme@kernel.org>, Tom Zanussi <tom.zanussi@linux.intel.com>, Rob Herring <robh+dt@kernel.org>, Andrew Morton <akpm@linux-foundation.org>, Thomas Gleixner <tglx@linutronix.de>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Alexey Dobriyan <adobriyan@gmail.com>, Jonathan Corbet <corbet@lwn.net>, Linus Torvalds <torvalds@linux-foundation.org>, linux-doc@vger.kernel.org, linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v5 21/22] tracing/boot: Add function tracer filter options Date: Fri, 27 Dec 2019 02:05:11 +0800 [thread overview] Message-ID: <201912270227.Dwa2YddH%lkp@intel.com> (raw) In-Reply-To: <157736928302.11126.8760178688093051786.stgit@devnote2> [-- Attachment #1: Type: text/plain, Size: 4465 bytes --] Hi Masami, I love your patch! Yet something to improve: [auto build test ERROR on trace/for-next] [also build test ERROR on lwn/docs-next linus/master v5.5-rc3] [cannot apply to tip/perf/core next-20191220] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Masami-Hiramatsu/tracing-bootconfig-Boot-time-tracing-and-Extra-boot-config/20191227-002009 base: https://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git for-next config: xtensa-allyesconfig (attached as .config) compiler: xtensa-linux-gcc (GCC) 7.5.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=7.5.0 make.cross ARCH=xtensa If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp@intel.com> All error/warnings (new ones prefixed by >>): In file included from kernel/trace/trace_boot.c:9:0: >> include/linux/ftrace.h:719:50: error: expected identifier or '(' before '{' token #define ftrace_set_filter(ops, buf, len, reset) ({ -ENODEV; }) ^ >> kernel/trace/trace_boot.c:249:12: note: in expansion of macro 'ftrace_set_filter' extern int ftrace_set_filter(struct ftrace_ops *ops, unsigned char *buf, ^~~~~~~~~~~~~~~~~ include/linux/ftrace.h:720:51: error: expected identifier or '(' before '{' token #define ftrace_set_notrace(ops, buf, len, reset) ({ -ENODEV; }) ^ >> kernel/trace/trace_boot.c:251:12: note: in expansion of macro 'ftrace_set_notrace' extern int ftrace_set_notrace(struct ftrace_ops *ops, unsigned char *buf, ^~~~~~~~~~~~~~~~~~ -- In file included from kernel//trace/trace_boot.c:9:0: >> include/linux/ftrace.h:719:50: error: expected identifier or '(' before '{' token #define ftrace_set_filter(ops, buf, len, reset) ({ -ENODEV; }) ^ kernel//trace/trace_boot.c:249:12: note: in expansion of macro 'ftrace_set_filter' extern int ftrace_set_filter(struct ftrace_ops *ops, unsigned char *buf, ^~~~~~~~~~~~~~~~~ include/linux/ftrace.h:720:51: error: expected identifier or '(' before '{' token #define ftrace_set_notrace(ops, buf, len, reset) ({ -ENODEV; }) ^ kernel//trace/trace_boot.c:251:12: note: in expansion of macro 'ftrace_set_notrace' extern int ftrace_set_notrace(struct ftrace_ops *ops, unsigned char *buf, ^~~~~~~~~~~~~~~~~~ vim +/ftrace_set_filter +249 kernel/trace/trace_boot.c 246 247 #ifdef CONFIG_FUNCTION_TRACER 248 extern bool ftrace_filter_param __initdata; > 249 extern int ftrace_set_filter(struct ftrace_ops *ops, unsigned char *buf, 250 int len, int reset); > 251 extern int ftrace_set_notrace(struct ftrace_ops *ops, unsigned char *buf, 252 int len, int reset); 253 static void __init 254 trace_boot_set_ftrace_filter(struct trace_array *tr, struct xbc_node *node) 255 { 256 struct xbc_node *anode; 257 const char *p; 258 char *q; 259 260 xbc_node_for_each_array_value(node, "ftrace.filters", anode, p) { 261 q = kstrdup(p, GFP_KERNEL); 262 if (!q) 263 return; 264 if (ftrace_set_filter(tr->ops, q, strlen(q), 0) < 0) 265 pr_err("Failed to add %s to ftrace filter\n", p); 266 else 267 ftrace_filter_param = true; 268 kfree(q); 269 } 270 xbc_node_for_each_array_value(node, "ftrace.notraces", anode, p) { 271 q = kstrdup(p, GFP_KERNEL); 272 if (!q) 273 return; 274 if (ftrace_set_notrace(tr->ops, q, strlen(q), 0) < 0) 275 pr_err("Failed to add %s to ftrace filter\n", p); 276 else 277 ftrace_filter_param = true; 278 kfree(q); 279 } 280 } 281 #else 282 #define trace_boot_set_ftrace_filter(tr, node) do {} while (0) 283 #endif 284 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation [-- Attachment #2: .config.gz --] [-- Type: application/gzip, Size: 60586 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kbuild test robot <lkp@intel.com> To: kbuild-all@lists.01.org Subject: Re: [PATCH v5 21/22] tracing/boot: Add function tracer filter options Date: Fri, 27 Dec 2019 02:05:11 +0800 [thread overview] Message-ID: <201912270227.Dwa2YddH%lkp@intel.com> (raw) In-Reply-To: <157736928302.11126.8760178688093051786.stgit@devnote2> [-- Attachment #1: Type: text/plain, Size: 4564 bytes --] Hi Masami, I love your patch! Yet something to improve: [auto build test ERROR on trace/for-next] [also build test ERROR on lwn/docs-next linus/master v5.5-rc3] [cannot apply to tip/perf/core next-20191220] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system. BTW, we also suggest to use '--base' option to specify the base tree in git format-patch, please see https://stackoverflow.com/a/37406982] url: https://github.com/0day-ci/linux/commits/Masami-Hiramatsu/tracing-bootconfig-Boot-time-tracing-and-Extra-boot-config/20191227-002009 base: https://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-trace.git for-next config: xtensa-allyesconfig (attached as .config) compiler: xtensa-linux-gcc (GCC) 7.5.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree GCC_VERSION=7.5.0 make.cross ARCH=xtensa If you fix the issue, kindly add following tag Reported-by: kbuild test robot <lkp@intel.com> All error/warnings (new ones prefixed by >>): In file included from kernel/trace/trace_boot.c:9:0: >> include/linux/ftrace.h:719:50: error: expected identifier or '(' before '{' token #define ftrace_set_filter(ops, buf, len, reset) ({ -ENODEV; }) ^ >> kernel/trace/trace_boot.c:249:12: note: in expansion of macro 'ftrace_set_filter' extern int ftrace_set_filter(struct ftrace_ops *ops, unsigned char *buf, ^~~~~~~~~~~~~~~~~ include/linux/ftrace.h:720:51: error: expected identifier or '(' before '{' token #define ftrace_set_notrace(ops, buf, len, reset) ({ -ENODEV; }) ^ >> kernel/trace/trace_boot.c:251:12: note: in expansion of macro 'ftrace_set_notrace' extern int ftrace_set_notrace(struct ftrace_ops *ops, unsigned char *buf, ^~~~~~~~~~~~~~~~~~ -- In file included from kernel//trace/trace_boot.c:9:0: >> include/linux/ftrace.h:719:50: error: expected identifier or '(' before '{' token #define ftrace_set_filter(ops, buf, len, reset) ({ -ENODEV; }) ^ kernel//trace/trace_boot.c:249:12: note: in expansion of macro 'ftrace_set_filter' extern int ftrace_set_filter(struct ftrace_ops *ops, unsigned char *buf, ^~~~~~~~~~~~~~~~~ include/linux/ftrace.h:720:51: error: expected identifier or '(' before '{' token #define ftrace_set_notrace(ops, buf, len, reset) ({ -ENODEV; }) ^ kernel//trace/trace_boot.c:251:12: note: in expansion of macro 'ftrace_set_notrace' extern int ftrace_set_notrace(struct ftrace_ops *ops, unsigned char *buf, ^~~~~~~~~~~~~~~~~~ vim +/ftrace_set_filter +249 kernel/trace/trace_boot.c 246 247 #ifdef CONFIG_FUNCTION_TRACER 248 extern bool ftrace_filter_param __initdata; > 249 extern int ftrace_set_filter(struct ftrace_ops *ops, unsigned char *buf, 250 int len, int reset); > 251 extern int ftrace_set_notrace(struct ftrace_ops *ops, unsigned char *buf, 252 int len, int reset); 253 static void __init 254 trace_boot_set_ftrace_filter(struct trace_array *tr, struct xbc_node *node) 255 { 256 struct xbc_node *anode; 257 const char *p; 258 char *q; 259 260 xbc_node_for_each_array_value(node, "ftrace.filters", anode, p) { 261 q = kstrdup(p, GFP_KERNEL); 262 if (!q) 263 return; 264 if (ftrace_set_filter(tr->ops, q, strlen(q), 0) < 0) 265 pr_err("Failed to add %s to ftrace filter\n", p); 266 else 267 ftrace_filter_param = true; 268 kfree(q); 269 } 270 xbc_node_for_each_array_value(node, "ftrace.notraces", anode, p) { 271 q = kstrdup(p, GFP_KERNEL); 272 if (!q) 273 return; 274 if (ftrace_set_notrace(tr->ops, q, strlen(q), 0) < 0) 275 pr_err("Failed to add %s to ftrace filter\n", p); 276 else 277 ftrace_filter_param = true; 278 kfree(q); 279 } 280 } 281 #else 282 #define trace_boot_set_ftrace_filter(tr, node) do {} while (0) 283 #endif 284 --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation [-- Attachment #2: config.gz --] [-- Type: application/gzip, Size: 60586 bytes --]
next prev parent reply other threads:[~2019-12-26 18:06 UTC|newest] Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-12-26 14:03 [PATCH v5 00/22] tracing: bootconfig: Boot-time tracing and Extra boot config Masami Hiramatsu 2019-12-26 14:04 ` [PATCH v5 01/22] bootconfig: Add Extra Boot Config support Masami Hiramatsu 2020-01-08 1:59 ` Steven Rostedt 2020-01-08 3:03 ` Steven Rostedt 2020-01-08 5:17 ` Masami Hiramatsu 2020-01-08 14:01 ` Steven Rostedt 2019-12-26 14:04 ` [PATCH v5 02/22] bootconfig: Load boot config from the tail of initrd Masami Hiramatsu 2019-12-26 14:04 ` [PATCH v5 03/22] tools: bootconfig: Add bootconfig command Masami Hiramatsu 2019-12-26 14:04 ` [PATCH v5 04/22] tools: bootconfig: Add bootconfig test script Masami Hiramatsu 2019-12-26 14:04 ` [PATCH v5 05/22] proc: bootconfig: Add /proc/bootconfig to show boot config list Masami Hiramatsu 2020-01-09 21:37 ` Steven Rostedt 2020-01-10 0:52 ` Masami Hiramatsu 2019-12-26 14:05 ` [PATCH v5 06/22] init/main.c: Alloc initcall_command_line in do_initcall() and free it Masami Hiramatsu 2019-12-26 14:05 ` [PATCH v5 07/22] bootconfig: init: Allow admin to use bootconfig for kernel command line Masami Hiramatsu 2019-12-26 14:05 ` [PATCH v5 08/22] bootconfig: init: Allow admin to use bootconfig for init " Masami Hiramatsu 2019-12-26 14:05 ` [PATCH v5 09/22] Documentation: bootconfig: Add a doc for extended boot config Masami Hiramatsu 2019-12-26 14:05 ` [PATCH v5 10/22] tracing: Apply soft-disabled and filter to tracepoints printk Masami Hiramatsu 2019-12-26 14:06 ` [PATCH v5 11/22] tracing: kprobes: Output kprobe event to printk buffer Masami Hiramatsu 2019-12-26 14:06 ` [PATCH v5 12/22] tracing: kprobes: Register to dynevent earlier stage Masami Hiramatsu 2019-12-26 14:06 ` [PATCH v5 13/22] tracing: Accept different type for synthetic event fields Masami Hiramatsu 2019-12-26 14:06 ` [PATCH v5 14/22] tracing: Add NULL trace-array check in print_synth_event() Masami Hiramatsu 2019-12-26 14:06 ` [PATCH v5 15/22] tracing/boot: Add boot-time tracing Masami Hiramatsu 2019-12-26 14:07 ` [PATCH v5 16/22] tracing/boot: Add per-event settings Masami Hiramatsu 2019-12-26 14:07 ` [PATCH v5 17/22] tracing/boot Add kprobe event support Masami Hiramatsu 2019-12-26 14:07 ` [PATCH v5 18/22] tracing/boot: Add synthetic " Masami Hiramatsu 2019-12-26 14:07 ` [PATCH v5 19/22] tracing/boot: Add instance node support Masami Hiramatsu 2019-12-26 14:07 ` [PATCH v5 20/22] tracing/boot: Add cpu_mask option support Masami Hiramatsu 2019-12-26 14:08 ` [PATCH v5 21/22] tracing/boot: Add function tracer filter options Masami Hiramatsu 2019-12-26 18:05 ` kbuild test robot [this message] 2019-12-26 18:05 ` kbuild test robot 2019-12-27 2:56 ` Masami Hiramatsu 2019-12-27 2:56 ` Masami Hiramatsu 2019-12-26 14:08 ` [PATCH v5 22/22] Documentation: tracing: Add boot-time tracing document Masami Hiramatsu 2020-01-09 23:10 ` [PATCH v5 00/22] tracing: bootconfig: Boot-time tracing and Extra boot config Steven Rostedt 2020-01-10 15:30 ` Masami Hiramatsu
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=201912270227.Dwa2YddH%lkp@intel.com \ --to=lkp@intel.com \ --cc=Tim.Bird@sony.com \ --cc=acme@kernel.org \ --cc=adobriyan@gmail.com \ --cc=akpm@linux-foundation.org \ --cc=corbet@lwn.net \ --cc=frowand.list@gmail.com \ --cc=gregkh@linuxfoundation.org \ --cc=jolsa@redhat.com \ --cc=kbuild-all@lists.01.org \ --cc=linux-doc@vger.kernel.org \ --cc=linux-fsdevel@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=mhiramat@kernel.org \ --cc=mingo@redhat.com \ --cc=namhyung@kernel.org \ --cc=rdunlap@infradead.org \ --cc=robh+dt@kernel.org \ --cc=rostedt@goodmis.org \ --cc=tglx@linutronix.de \ --cc=tom.zanussi@linux.intel.com \ --cc=torvalds@linux-foundation.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.