From: kernel test robot <lkp@intel.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev
Subject: Re: [RFC PATCH v2 1/4] tracing: add __print_sym() to replace __print_symbolic()
Date: Wed, 27 Mar 2024 12:21:30 +0800 [thread overview]
Message-ID: <202403271216.iazgC6LR-lkp@intel.com> (raw)
In-Reply-To: <20240326202131.9d261d5bb667.I9bd2617499f0d170df58471bc51379742190f92d@changeid>
Hi Johannes,
[This is a private test report for your RFC patch.]
kernel test robot noticed the following build warnings:
[auto build test WARNING on mcgrof/modules-next]
[also build test WARNING on arnd-asm-generic/master tip/timers/core net/main net-next/main linus/master horms-ipvs/master v6.9-rc1 next-20240326]
[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/Johannes-Berg/tracing-add-__print_sym-to-replace-__print_symbolic/20240327-032437
base: https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux.git modules-next
patch link: https://lore.kernel.org/r/20240326202131.9d261d5bb667.I9bd2617499f0d170df58471bc51379742190f92d%40changeid
patch subject: [RFC PATCH v2 1/4] tracing: add __print_sym() to replace __print_symbolic()
config: s390-defconfig (https://download.01.org/0day-ci/archive/20240327/202403271216.iazgC6LR-lkp@intel.com/config)
compiler: clang version 19.0.0git (https://github.com/llvm/llvm-project 23de3862dce582ce91c1aa914467d982cb1a73b4)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240327/202403271216.iazgC6LR-lkp@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 <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202403271216.iazgC6LR-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from lib/maple_tree.c:65:
In file included from include/trace/events/maple_tree.h:123:
In file included from include/trace/define_trace.h:102:
In file included from include/trace/trace_events.h:27:
>> include/trace/stages/init.h:30:9: warning: 'TRACE_DEFINE_SYM_FNS' macro redefined [-Wmacro-redefined]
30 | #define TRACE_DEFINE_SYM_FNS(_symbol_id, _lookup, _show) \
| ^
include/linux/tracepoint.h:130:9: note: previous definition is here
130 | #define TRACE_DEFINE_SYM_FNS(...)
| ^
In file included from lib/maple_tree.c:65:
In file included from include/trace/events/maple_tree.h:123:
In file included from include/trace/define_trace.h:102:
In file included from include/trace/trace_events.h:27:
>> include/trace/stages/init.h:54:9: warning: 'TRACE_DEFINE_SYM_LIST' macro redefined [-Wmacro-redefined]
54 | #define TRACE_DEFINE_SYM_LIST(_symbol_id, ...) \
| ^
include/linux/tracepoint.h:131:9: note: previous definition is here
131 | #define TRACE_DEFINE_SYM_LIST(...)
| ^
2 warnings generated.
--
In file included from lib/maple_tree.c:65:
In file included from include/trace/events/maple_tree.h:9:
In file included from include/linux/tracepoint.h:22:
In file included from include/linux/static_call.h:135:
In file included from include/linux/cpu.h:17:
In file included from include/linux/node.h:18:
In file included from include/linux/device.h:32:
In file included from include/linux/device/driver.h:21:
In file included from include/linux/module.h:19:
In file included from include/linux/elf.h:6:
In file included from arch/s390/include/asm/elf.h:173:
In file included from arch/s390/include/asm/mmu_context.h:11:
In file included from arch/s390/include/asm/pgalloc.h:18:
In file included from include/linux/mm.h:2188:
include/linux/vmstat.h:508:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
508 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
| ~~~~~~~~~~~~~~~~~~~~~ ^
509 | item];
| ~~~~
include/linux/vmstat.h:515:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
515 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
| ~~~~~~~~~~~~~~~~~~~~~ ^
516 | NR_VM_NUMA_EVENT_ITEMS +
| ~~~~~~~~~~~~~~~~~~~~~~
include/linux/vmstat.h:522:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
522 | return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_"
| ~~~~~~~~~~~ ^ ~~~
include/linux/vmstat.h:527:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
527 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
| ~~~~~~~~~~~~~~~~~~~~~ ^
528 | NR_VM_NUMA_EVENT_ITEMS +
| ~~~~~~~~~~~~~~~~~~~~~~
include/linux/vmstat.h:536:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
536 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
| ~~~~~~~~~~~~~~~~~~~~~ ^
537 | NR_VM_NUMA_EVENT_ITEMS +
| ~~~~~~~~~~~~~~~~~~~~~~
In file included from lib/maple_tree.c:65:
In file included from include/trace/events/maple_tree.h:123:
In file included from include/trace/define_trace.h:102:
In file included from include/trace/trace_events.h:27:
>> include/trace/stages/init.h:30:9: warning: 'TRACE_DEFINE_SYM_FNS' macro redefined [-Wmacro-redefined]
30 | #define TRACE_DEFINE_SYM_FNS(_symbol_id, _lookup, _show) \
| ^
include/linux/tracepoint.h:130:9: note: previous definition is here
130 | #define TRACE_DEFINE_SYM_FNS(...)
| ^
In file included from lib/maple_tree.c:65:
In file included from include/trace/events/maple_tree.h:123:
In file included from include/trace/define_trace.h:102:
In file included from include/trace/trace_events.h:27:
>> include/trace/stages/init.h:54:9: warning: 'TRACE_DEFINE_SYM_LIST' macro redefined [-Wmacro-redefined]
54 | #define TRACE_DEFINE_SYM_LIST(_symbol_id, ...) \
| ^
include/linux/tracepoint.h:131:9: note: previous definition is here
131 | #define TRACE_DEFINE_SYM_LIST(...)
| ^
lib/maple_tree.c:351:21: warning: unused function 'mte_set_full' [-Wunused-function]
351 | static inline void *mte_set_full(const struct maple_enode *node)
| ^~~~~~~~~~~~
lib/maple_tree.c:356:21: warning: unused function 'mte_clear_full' [-Wunused-function]
356 | static inline void *mte_clear_full(const struct maple_enode *node)
| ^~~~~~~~~~~~~~
lib/maple_tree.c:361:20: warning: unused function 'mte_has_null' [-Wunused-function]
361 | static inline bool mte_has_null(const struct maple_enode *node)
| ^~~~~~~~~~~~
10 warnings generated.
vim +/TRACE_DEFINE_SYM_FNS +30 include/trace/stages/init.h
12
13 #undef TRACE_DEFINE_ENUM
14 #define TRACE_DEFINE_ENUM(a) \
15 static struct trace_eval_map __used __initdata \
16 __##TRACE_SYSTEM##_##a = \
17 { \
18 .system = TRACE_SYSTEM_STRING, \
19 .eval_string = #a, \
20 .eval_value = a \
21 }; \
22 static struct trace_eval_map __used \
23 __section("_ftrace_eval_map") \
24 *TRACE_SYSTEM##_##a = &__##TRACE_SYSTEM##_##a
25
26 /*
27 * Define a symbol for __print_sym by giving lookup and
28 * show functions. See &struct trace_sym_def.
29 */
> 30 #define TRACE_DEFINE_SYM_FNS(_symbol_id, _lookup, _show) \
31 _TRACE_DEFINE_SYM_FNS(TRACE_SYSTEM, _symbol_id, _lookup, _show)
32 #define _TRACE_DEFINE_SYM_FNS(_system, _symbol_id, _lookup, _show) \
33 __TRACE_DEFINE_SYM_FNS(_system, _symbol_id, _lookup, _show)
34 #define __TRACE_DEFINE_SYM_FNS(_system, _symbol_id, _lookup, _show) \
35 ___TRACE_DEFINE_SYM_FNS(_system ## _ ## _symbol_id, _symbol_id, \
36 _lookup, _show)
37 #define ___TRACE_DEFINE_SYM_FNS(_name, _symbol_id, _lookup, _show) \
38 static struct trace_sym_def \
39 __trace_sym_def_ ## _name = { \
40 .system = TRACE_SYSTEM_STRING, \
41 /* need the ) for later strcmp */ \
42 .symbol_id = #_symbol_id ")", \
43 .lookup = _lookup, \
44 .show = _show, \
45 }; \
46 static struct trace_sym_def \
47 __section("_ftrace_sym_defs") \
48 *__trace_sym_def_p_ ## _name = &__trace_sym_def_ ## _name
49
50 /*
51 * Define a symbol for __print_sym by giving lookup and
52 * show functions. See &struct trace_sym_def.
53 */
> 54 #define TRACE_DEFINE_SYM_LIST(_symbol_id, ...) \
55 _TRACE_DEFINE_SYM_LIST(TRACE_SYSTEM, _symbol_id, __VA_ARGS__)
56 #define _TRACE_DEFINE_SYM_LIST(_system, _symbol_id, ...) \
57 __TRACE_DEFINE_SYM_LIST(_system, _symbol_id, __VA_ARGS__)
58 #define __TRACE_DEFINE_SYM_LIST(_system, _symbol_id, ...) \
59 ___TRACE_DEFINE_SYM_LIST(_system ## _ ## _symbol_id, _symbol_id,\
60 __VA_ARGS__)
61 #define ___TRACE_DEFINE_SYM_LIST(_name, _symbol_id, ...) \
62 static struct trace_sym_entry \
63 __trace_sym_list_ ## _name[] = { __VA_ARGS__ }; \
64 static const char * \
65 __trace_sym_lookup_ ## _name(unsigned long long value) \
66 { \
67 return trace_sym_lookup(__trace_sym_list_ ## _name, \
68 ARRAY_SIZE(__trace_sym_list_ ## _name), value); \
69 } \
70 static void \
71 __trace_sym_show_ ## _name(struct seq_file *m) \
72 { \
73 trace_sym_show(m, __trace_sym_list_ ## _name, \
74 ARRAY_SIZE(__trace_sym_list_ ## _name)); \
75 } \
76 ___TRACE_DEFINE_SYM_FNS(_name, _symbol_id, \
77 __trace_sym_lookup_ ## _name, \
78 __trace_sym_show_ ## _name)
79
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next prev parent reply other threads:[~2024-03-27 4:22 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-03-26 19:15 [RFC PATCH v2 0/4] tracing: improve symbolic printing Johannes Berg
2024-03-26 19:15 ` [RFC PATCH v2 1/4] tracing: add __print_sym() to replace __print_symbolic() Johannes Berg
2024-03-27 4:21 ` kernel test robot [this message]
2024-03-27 5:34 ` kernel test robot
2024-03-27 21:11 ` Simon Horman
2024-03-27 21:24 ` Johannes Berg
2024-03-26 19:15 ` [RFC PATCH v2 2/4] net: dropreason: use new __print_sym() in tracing Johannes Berg
2024-03-26 19:15 ` [RFC PATCH v2 3/4] net: drop_monitor: use drop_reason_lookup() Johannes Berg
2024-03-27 4:21 ` kernel test robot
2024-03-26 19:15 ` [RFC PATCH v2 4/4] tracing/timer: use __print_sym() Johannes Berg
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=202403271216.iazgC6LR-lkp@intel.com \
--to=lkp@intel.com \
--cc=johannes@sipsolutions.net \
--cc=llvm@lists.linux.dev \
--cc=oe-kbuild-all@lists.linux.dev \
/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: link
Be 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.