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
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <202202131057.QVvn6nBp-lkp@intel.com>
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

Powered by Openwall GNU/*/Linux Powered by OpenVZ