| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Sun, 13 Feb 2022 10:17:04 +0800
From: kernel test robot <lkp@...el.com>
To: "Jason A. Donenfeld" <zx2c4@...nel.org>
Cc: kbuild-all@...ts.01.org,
GNU/Weeb Mailing List <gwml@...r.gnuweeb.org>,
linux-kernel@...r.kernel.org,
Dominik Brodowski <linux@...inikbrodowski.net>,
Jann Horn <jannh@...gle.com>
Subject: [ammarfaizi2-block:crng/random/jd/fast-mix-lfsr 17/38]
include/trace/trace_events.h:870:42: error: passing argument 1 of
'check_trace_callback_type_mix_pool_bytes_nolock' from incompatible pointer
type
tree: https://github.com/ammarfaizi2/linux-block crng/random/jd/fast-mix-lfsr
head: 2cdf0f5c31f7568d709533d93b746ae8932dfa99
commit: fc2551cdc07cfb4ea1247ecaec28e78f6d942c1f [17/38] random: make more consistent use of integer types
config: powerpc-allmodconfig (https://download.01.org/0day-ci/archive/20220213/202202131057.QVvn6nBp-lkp@intel.com/config)
compiler: powerpc-linux-gcc (GCC) 11.2.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/ammarfaizi2/linux-block/commit/fc2551cdc07cfb4ea1247ecaec28e78f6d942c1f
git remote add ammarfaizi2-block https://github.com/ammarfaizi2/linux-block
git fetch --no-tags ammarfaizi2-block crng/random/jd/fast-mix-lfsr
git checkout fc2551cdc07cfb4ea1247ecaec28e78f6d942c1f
# save the config file to linux build tree
mkdir build_dir
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-11.2.0 make.cross O=build_dir ARCH=powerpc SHELL=/bin/bash drivers/char/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>
Note: the ammarfaizi2-block/crng/random/jd/fast-mix-lfsr HEAD 2cdf0f5c31f7568d709533d93b746ae8932dfa99 builds fine.
It only hurts bisectability.
All errors (new ones prefixed by >>):
In file included from include/trace/define_trace.h:102,
from include/trace/events/random.h:212,
from drivers/char/random.c:241:
include/trace/events/random.h: In function 'ftrace_test_probe_mix_pool_bytes_nolock':
>> include/trace/trace_events.h:870:42: error: passing argument 1 of 'check_trace_callback_type_mix_pool_bytes_nolock' from incompatible pointer type [-Werror=incompatible-pointer-types]
870 | check_trace_callback_type_##call(trace_event_raw_event_##template); \
| ^~~~~~~~~~~~~~~~~~~~~~
| |
| void (*)(void *, size_t, long unsigned int) {aka void (*)(void *, unsigned int, long unsigned int)}
include/trace/events/random.h:55:1: note: in expansion of macro 'DEFINE_EVENT'
55 | DEFINE_EVENT(random__mix_pool_bytes, mix_pool_bytes_nolock,
| ^~~~~~~~~~~~
In file included from include/trace/syscall.h:5,
from include/linux/syscalls.h:88,
from drivers/char/random.c:228:
include/linux/tracepoint.h:279:49: note: expected 'void (*)(void *, int, long unsigned int)' but argument is of type 'void (*)(void *, size_t, long unsigned int)' {aka 'void (*)(void *, unsigned int, long unsigned int)'}
279 | check_trace_callback_type_##name(void (*cb)(data_proto)) \
| ~~~~~~~^~~~~~~~~~~~~~~
include/linux/tracepoint.h:419:9: note: in expansion of macro '__DECLARE_TRACE'
419 | __DECLARE_TRACE(name, PARAMS(proto), PARAMS(args), \
| ^~~~~~~~~~~~~~~
include/linux/tracepoint.h:542:9: note: in expansion of macro 'DECLARE_TRACE'
542 | DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
| ^~~~~~~~~~~~~
include/trace/events/random.h:55:1: note: in expansion of macro 'DEFINE_EVENT'
55 | DEFINE_EVENT(random__mix_pool_bytes, mix_pool_bytes_nolock,
| ^~~~~~~~~~~~
In file included from include/trace/define_trace.h:103,
from include/trace/events/random.h:212,
from drivers/char/random.c:241:
include/trace/events/random.h: In function 'perf_test_probe_mix_pool_bytes_nolock':
>> include/trace/perf.h:99:42: error: passing argument 1 of 'check_trace_callback_type_mix_pool_bytes_nolock' from incompatible pointer type [-Werror=incompatible-pointer-types]
99 | check_trace_callback_type_##call(perf_trace_##template); \
| ^~~~~~~~~~~
| |
| void (*)(void *, size_t, long unsigned int) {aka void (*)(void *, unsigned int, long unsigned int)}
include/trace/events/random.h:55:1: note: in expansion of macro 'DEFINE_EVENT'
55 | DEFINE_EVENT(random__mix_pool_bytes, mix_pool_bytes_nolock,
| ^~~~~~~~~~~~
In file included from include/trace/syscall.h:5,
from include/linux/syscalls.h:88,
from drivers/char/random.c:228:
include/linux/tracepoint.h:279:49: note: expected 'void (*)(void *, int, long unsigned int)' but argument is of type 'void (*)(void *, size_t, long unsigned int)' {aka 'void (*)(void *, unsigned int, long unsigned int)'}
279 | check_trace_callback_type_##name(void (*cb)(data_proto)) \
| ~~~~~~~^~~~~~~~~~~~~~~
include/linux/tracepoint.h:419:9: note: in expansion of macro '__DECLARE_TRACE'
419 | __DECLARE_TRACE(name, PARAMS(proto), PARAMS(args), \
| ^~~~~~~~~~~~~~~
include/linux/tracepoint.h:542:9: note: in expansion of macro 'DECLARE_TRACE'
542 | DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
| ^~~~~~~~~~~~~
include/trace/events/random.h:55:1: note: in expansion of macro 'DEFINE_EVENT'
55 | DEFINE_EVENT(random__mix_pool_bytes, mix_pool_bytes_nolock,
| ^~~~~~~~~~~~
In file included from include/trace/define_trace.h:104,
from include/trace/events/random.h:212,
from drivers/char/random.c:241:
include/trace/events/random.h: In function 'bpf_test_probe_mix_pool_bytes_nolock':
>> include/trace/bpf_probe.h:94:42: error: passing argument 1 of 'check_trace_callback_type_mix_pool_bytes_nolock' from incompatible pointer type [-Werror=incompatible-pointer-types]
94 | check_trace_callback_type_##call(__bpf_trace_##template); \
| ^~~~~~~~~~~~
| |
| void (*)(void *, size_t, long unsigned int) {aka void (*)(void *, unsigned int, long unsigned int)}
include/trace/bpf_probe.h:130:9: note: in expansion of macro '__DEFINE_EVENT'
130 | __DEFINE_EVENT(template, call, PARAMS(proto), PARAMS(args), 0)
| ^~~~~~~~~~~~~~
include/trace/events/random.h:55:1: note: in expansion of macro 'DEFINE_EVENT'
55 | DEFINE_EVENT(random__mix_pool_bytes, mix_pool_bytes_nolock,
| ^~~~~~~~~~~~
In file included from include/trace/syscall.h:5,
from include/linux/syscalls.h:88,
from drivers/char/random.c:228:
include/linux/tracepoint.h:279:49: note: expected 'void (*)(void *, int, long unsigned int)' but argument is of type 'void (*)(void *, size_t, long unsigned int)' {aka 'void (*)(void *, unsigned int, long unsigned int)'}
279 | check_trace_callback_type_##name(void (*cb)(data_proto)) \
| ~~~~~~~^~~~~~~~~~~~~~~
include/linux/tracepoint.h:419:9: note: in expansion of macro '__DECLARE_TRACE'
419 | __DECLARE_TRACE(name, PARAMS(proto), PARAMS(args), \
| ^~~~~~~~~~~~~~~
include/linux/tracepoint.h:542:9: note: in expansion of macro 'DECLARE_TRACE'
542 | DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
| ^~~~~~~~~~~~~
include/trace/events/random.h:55:1: note: in expansion of macro 'DEFINE_EVENT'
55 | DEFINE_EVENT(random__mix_pool_bytes, mix_pool_bytes_nolock,
| ^~~~~~~~~~~~
drivers/char/random.c: At top level:
drivers/char/random.c:1739:6: warning: no previous prototype for 'add_hwgenerator_randomness' [-Wmissing-prototypes]
1739 | void add_hwgenerator_randomness(const void *buffer, size_t count,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +/check_trace_callback_type_mix_pool_bytes_nolock +870 include/trace/trace_events.h
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 830)
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 831) #undef DECLARE_EVENT_CLASS
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 832) #define DECLARE_EVENT_CLASS(call, proto, args, tstruct, assign, print) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 833) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 834) static notrace void \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 835) trace_event_raw_event_##call(void *__data, proto) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 836) { \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 837) struct trace_event_file *trace_file = __data; \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 838) struct trace_event_data_offsets_##call __maybe_unused __data_offsets;\
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 839) struct trace_event_buffer fbuffer; \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 840) struct trace_event_raw_##call *entry; \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 841) int __data_size; \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 842) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 843) if (trace_trigger_soft_disabled(trace_file)) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 844) return; \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 845) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 846) __data_size = trace_event_get_offsets_##call(&__data_offsets, args); \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 847) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 848) entry = trace_event_buffer_reserve(&fbuffer, trace_file, \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 849) sizeof(*entry) + __data_size); \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 850) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 851) if (!entry) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 852) return; \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 853) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 854) tstruct \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 855) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 856) { assign; } \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 857) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 858) trace_event_buffer_commit(&fbuffer); \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 859) }
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 860) /*
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 861) * The ftrace_test_probe is compiled out, it is only here as a build time check
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 862) * to make sure that if the tracepoint handling changes, the ftrace probe will
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 863) * fail to compile unless it too is updated.
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 864) */
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 865)
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 866) #undef DEFINE_EVENT
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 867) #define DEFINE_EVENT(template, call, proto, args) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 868) static inline void ftrace_test_probe_##call(void) \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 869) { \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 @870) check_trace_callback_type_##call(trace_event_raw_event_##template); \
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 871) }
46ac51822a6a0b Steven Rostedt (Red Hat 2015-09-23 872)
:::::: The code at line 870 was first introduced by commit
:::::: 46ac51822a6a0b83ee3e408ee9edcb292fb8ced0 tracing: Move non perf code out of perf.h
:::::: TO: Steven Rostedt (Red Hat) <rostedt@...dmis.org>
:::::: CC: Steven Rostedt <rostedt@...dmis.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
Powered by blists - more mailing lists