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: <202107270022.G0dW80EL-lkp@intel.com>
Date:   Tue, 27 Jul 2021 00:58:29 +0800
From:   kernel test robot <lkp@...el.com>
To:     Gwendal Grignou <gwendal@...omium.org>
Cc:     kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org,
        Enric Balletbo i Serra <enric.balletbo@...labora.com>
Subject: [chrome-platform-linux:for-kernelci 1/2]
 drivers/platform/chrome/./cros_ec_trace.h:95:12: warning: format '%lld'
 expects argument of type 'long long int', but argument 3 has type 'u32' {aka
 'unsigned int'}

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/chrome-platform/linux.git for-kernelci
head:   d0774f014d59a5ad69bed8683cc2505f51921283
commit: d453ceb6549af8798913de6a20444cb7200fdb69 [1/2] platform/chrome: sensorhub: Add trace events for sample
config: arc-allyesconfig (attached as .config)
compiler: arceb-elf-gcc (GCC) 10.3.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://git.kernel.org/pub/scm/linux/kernel/git/chrome-platform/linux.git/commit/?id=d453ceb6549af8798913de6a20444cb7200fdb69
        git remote add chrome-platform-linux https://git.kernel.org/pub/scm/linux/kernel/git/chrome-platform/linux.git
        git fetch --no-tags chrome-platform-linux for-kernelci
        git checkout d453ceb6549af8798913de6a20444cb7200fdb69
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-10.3.0 make.cross ARCH=arc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>

All warnings (new ones prefixed by >>):

   In file included from include/trace/define_trace.h:102,
                    from drivers/platform/chrome/cros_ec_trace.h:178,
                    from drivers/platform/chrome/cros_ec_trace.c:215:
   drivers/platform/chrome/./cros_ec_trace.h: In function 'trace_raw_output_cros_ec_sensorhub_timestamp':
>> drivers/platform/chrome/./cros_ec_trace.h:95:12: warning: format '%lld' expects argument of type 'long long int', but argument 3 has type 'u32' {aka 'unsigned int'} [-Wformat=]
      95 |  TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
         |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/trace/trace_events.h:393:27: note: in definition of macro 'DECLARE_EVENT_CLASS'
     393 |  trace_event_printf(iter, print);    \
         |                           ^~~~~
   include/trace/trace_events.h:80:9: note: in expansion of macro 'PARAMS'
      80 |         PARAMS(print));         \
         |         ^~~~~~
   drivers/platform/chrome/./cros_ec_trace.h:74:1: note: in expansion of macro 'TRACE_EVENT'
      74 | TRACE_EVENT(cros_ec_sensorhub_timestamp,
         | ^~~~~~~~~~~
   drivers/platform/chrome/./cros_ec_trace.h:95:2: note: in expansion of macro 'TP_printk'
      95 |  TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
         |  ^~~~~~~~~
   In file included from include/trace/trace_events.h:427,
                    from include/trace/define_trace.h:102,
                    from drivers/platform/chrome/cros_ec_trace.h:178,
                    from drivers/platform/chrome/cros_ec_trace.c:215:
   drivers/platform/chrome/./cros_ec_trace.h:95:25: note: format string is defined here
      95 |  TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
         |                    ~~~~~^
         |                         |
         |                         long long int
         |                    %12d
   In file included from include/trace/define_trace.h:102,
                    from drivers/platform/chrome/cros_ec_trace.h:178,
                    from drivers/platform/chrome/cros_ec_trace.c:215:
   drivers/platform/chrome/./cros_ec_trace.h:95:12: warning: format '%lld' expects argument of type 'long long int', but argument 4 has type 'u32' {aka 'unsigned int'} [-Wformat=]
      95 |  TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
         |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/trace/trace_events.h:393:27: note: in definition of macro 'DECLARE_EVENT_CLASS'
     393 |  trace_event_printf(iter, print);    \
         |                           ^~~~~
   include/trace/trace_events.h:80:9: note: in expansion of macro 'PARAMS'
      80 |         PARAMS(print));         \
         |         ^~~~~~
   drivers/platform/chrome/./cros_ec_trace.h:74:1: note: in expansion of macro 'TRACE_EVENT'
      74 | TRACE_EVENT(cros_ec_sensorhub_timestamp,
         | ^~~~~~~~~~~
   drivers/platform/chrome/./cros_ec_trace.h:95:2: note: in expansion of macro 'TP_printk'
      95 |  TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
         |  ^~~~~~~~~
   In file included from include/trace/trace_events.h:427,
                    from include/trace/define_trace.h:102,
                    from drivers/platform/chrome/cros_ec_trace.h:178,
                    from drivers/platform/chrome/cros_ec_trace.c:215:
   drivers/platform/chrome/./cros_ec_trace.h:95:45: note: format string is defined here
      95 |  TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
         |                                        ~~~~~^
         |                                             |
         |                                             long long int
         |                                        %12d
   In file included from include/trace/define_trace.h:102,
                    from drivers/platform/chrome/cros_ec_trace.h:178,
                    from drivers/platform/chrome/cros_ec_trace.c:215:
   drivers/platform/chrome/./cros_ec_trace.h: In function 'trace_raw_output_cros_ec_sensorhub_data':
   drivers/platform/chrome/./cros_ec_trace.h:125:12: warning: format '%lld' expects argument of type 'long long int', but argument 4 has type 'u32' {aka 'unsigned int'} [-Wformat=]
     125 |  TP_printk("ec_num: %4d, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
         |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/trace/trace_events.h:393:27: note: in definition of macro 'DECLARE_EVENT_CLASS'
     393 |  trace_event_printf(iter, print);    \
         |                           ^~~~~
   include/trace/trace_events.h:80:9: note: in expansion of macro 'PARAMS'
      80 |         PARAMS(print));         \
         |         ^~~~~~
   drivers/platform/chrome/./cros_ec_trace.h:105:1: note: in expansion of macro 'TRACE_EVENT'
     105 | TRACE_EVENT(cros_ec_sensorhub_data,
         | ^~~~~~~~~~~
   drivers/platform/chrome/./cros_ec_trace.h:125:2: note: in expansion of macro 'TP_printk'
     125 |  TP_printk("ec_num: %4d, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
         |  ^~~~~~~~~
   In file included from include/trace/trace_events.h:427,
                    from include/trace/define_trace.h:102,
                    from drivers/platform/chrome/cros_ec_trace.h:178,
                    from drivers/platform/chrome/cros_ec_trace.c:215:
   drivers/platform/chrome/./cros_ec_trace.h:125:43: note: format string is defined here
     125 |  TP_printk("ec_num: %4d, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
         |                                      ~~~~~^
         |                                           |
         |                                           long long int
         |                                      %12d
   In file included from include/trace/define_trace.h:102,
                    from drivers/platform/chrome/cros_ec_trace.h:178,
                    from drivers/platform/chrome/cros_ec_trace.c:215:
   drivers/platform/chrome/./cros_ec_trace.h: In function 'trace_raw_output_cros_ec_sensorhub_filter':
>> drivers/platform/chrome/./cros_ec_trace.h:156:12: warning: format '%d' expects argument of type 'int', but argument 7 has type 's64' {aka 'long long int'} [-Wformat=]
     156 |  TP_printk("dx: %12lld. dy: %12lld median_m: %12lld median_error: %12lld len: %d x: %12lld y: %12lld",
         |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/trace/trace_events.h:393:27: note: in definition of macro 'DECLARE_EVENT_CLASS'
     393 |  trace_event_printf(iter, print);    \
         |                           ^~~~~
   include/trace/trace_events.h:80:9: note: in expansion of macro 'PARAMS'
      80 |         PARAMS(print));         \
         |         ^~~~~~
   drivers/platform/chrome/./cros_ec_trace.h:135:1: note: in expansion of macro 'TRACE_EVENT'
     135 | TRACE_EVENT(cros_ec_sensorhub_filter,
         | ^~~~~~~~~~~
   drivers/platform/chrome/./cros_ec_trace.h:156:2: note: in expansion of macro 'TP_printk'
     156 |  TP_printk("dx: %12lld. dy: %12lld median_m: %12lld median_error: %12lld len: %d x: %12lld y: %12lld",
         |  ^~~~~~~~~
   In file included from include/trace/trace_events.h:427,
                    from include/trace/define_trace.h:102,
                    from drivers/platform/chrome/cros_ec_trace.h:178,
                    from drivers/platform/chrome/cros_ec_trace.c:215:
   drivers/platform/chrome/./cros_ec_trace.h:156:80: note: format string is defined here
     156 |  TP_printk("dx: %12lld. dy: %12lld median_m: %12lld median_error: %12lld len: %d x: %12lld y: %12lld",
         |                                                                               ~^
         |                                                                                |
         |                                                                                int
         |                                                                               %lld
   In file included from include/linux/perf_event.h:25,
                    from include/linux/trace_events.h:10,
                    from include/trace/trace_events.h:21,
                    from include/trace/define_trace.h:102,
                    from drivers/platform/chrome/cros_ec_trace.h:178,
                    from drivers/platform/chrome/cros_ec_trace.c:215:
   At top level:
   arch/arc/include/asm/perf_event.h:126:23: warning: 'arc_pmu_cache_map' defined but not used [-Wunused-const-variable=]
     126 | static const unsigned arc_pmu_cache_map[C(MAX)][C(OP_MAX)][C(RESULT_MAX)] = {
         |                       ^~~~~~~~~~~~~~~~~
   arch/arc/include/asm/perf_event.h:91:27: warning: 'arc_pmu_ev_hw_map' defined but not used [-Wunused-const-variable=]
      91 | static const char * const arc_pmu_ev_hw_map[] = {
         |                           ^~~~~~~~~~~~~~~~~


vim +95 drivers/platform/chrome/./cros_ec_trace.h

    73	
    74	TRACE_EVENT(cros_ec_sensorhub_timestamp,
    75		    TP_PROTO(u32 ec_sample_timestamp, u32 ec_fifo_timestamp, s64 fifo_timestamp,
    76			     s64 current_timestamp, s64 current_time),
    77		TP_ARGS(ec_sample_timestamp, ec_fifo_timestamp, fifo_timestamp, current_timestamp,
    78			current_time),
    79		TP_STRUCT__entry(
    80			__field(u32, ec_sample_timestamp)
    81			__field(u32, ec_fifo_timestamp)
    82			__field(s64, fifo_timestamp)
    83			__field(s64, current_timestamp)
    84			__field(s64, current_time)
    85			__field(s64, delta)
    86		),
    87		TP_fast_assign(
    88			__entry->ec_sample_timestamp = ec_sample_timestamp;
    89			__entry->ec_fifo_timestamp = ec_fifo_timestamp;
    90			__entry->fifo_timestamp = fifo_timestamp;
    91			__entry->current_timestamp = current_timestamp;
    92			__entry->current_time = current_time;
    93			__entry->delta = current_timestamp - current_time;
    94		),
  > 95		TP_printk("ec_ts: %12lld, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
    96			  __entry->ec_sample_timestamp,
    97			__entry->ec_fifo_timestamp,
    98			__entry->fifo_timestamp,
    99			__entry->current_timestamp,
   100			__entry->current_time,
   101			__entry->delta
   102		)
   103	);
   104	
   105	TRACE_EVENT(cros_ec_sensorhub_data,
   106		    TP_PROTO(u32 ec_sensor_num, u32 ec_fifo_timestamp, s64 fifo_timestamp,
   107			     s64 current_timestamp, s64 current_time),
   108		TP_ARGS(ec_sensor_num, ec_fifo_timestamp, fifo_timestamp, current_timestamp, current_time),
   109		TP_STRUCT__entry(
   110			__field(u32, ec_sensor_num)
   111			__field(u32, ec_fifo_timestamp)
   112			__field(s64, fifo_timestamp)
   113			__field(s64, current_timestamp)
   114			__field(s64, current_time)
   115			__field(s64, delta)
   116		),
   117		TP_fast_assign(
   118			__entry->ec_sensor_num = ec_sensor_num;
   119			__entry->ec_fifo_timestamp = ec_fifo_timestamp;
   120			__entry->fifo_timestamp = fifo_timestamp;
   121			__entry->current_timestamp = current_timestamp;
   122			__entry->current_time = current_time;
   123			__entry->delta = current_timestamp - current_time;
   124		),
   125		TP_printk("ec_num: %4d, ec_fifo_ts: %12lld, fifo_ts: %12lld, curr_ts: %12lld, curr_time: %12lld, delta %12lld",
   126			  __entry->ec_sensor_num,
   127			__entry->ec_fifo_timestamp,
   128			__entry->fifo_timestamp,
   129			__entry->current_timestamp,
   130			__entry->current_time,
   131			__entry->delta
   132		)
   133	);
   134	
   135	TRACE_EVENT(cros_ec_sensorhub_filter,
   136		    TP_PROTO(struct cros_ec_sensors_ts_filter_state *state, s64 dx, s64 dy),
   137		TP_ARGS(state, dx, dy),
   138		TP_STRUCT__entry(
   139			__field(s64, dx)
   140			__field(s64, dy)
   141			__field(s64, median_m)
   142			__field(s64, median_error)
   143			__field(s64, history_len)
   144			__field(s64, x)
   145			__field(s64, y)
   146		),
   147		TP_fast_assign(
   148			__entry->dx = dx;
   149			__entry->dy = dy;
   150			__entry->median_m = state->median_m;
   151			__entry->median_error = state->median_error;
   152			__entry->history_len = state->history_len;
   153			__entry->x = state->x_offset;
   154			__entry->y = state->y_offset;
   155		),
 > 156		TP_printk("dx: %12lld. dy: %12lld median_m: %12lld median_error: %12lld len: %d x: %12lld y: %12lld",
   157			  __entry->dx,
   158			__entry->dy,
   159			__entry->median_m,
   160			__entry->median_error,
   161			__entry->history_len,
   162			__entry->x,
   163			__entry->y
   164		)
   165	);
   166	
   167	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Download attachment ".config.gz" of type "application/gzip" (68417 bytes)

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ