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>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20210823152134.00d95cb4@canb.auug.org.au>
Date:   Mon, 23 Aug 2021 15:21:34 +1000
From:   Stephen Rothwell <sfr@...b.auug.org.au>
To:     Benson Leung <bleung@...gle.com>,
        Guenter Roeck <groeck@...omium.org>,
        Enric Balletbo i Serra <enric.balletbo@...labora.com>,
        Dmitry Torokhov <dmitry.torokhov@...il.com>
Cc:     Gwendal Grignou <gwendal@...omium.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Linux Next Mailing List <linux-next@...r.kernel.org>
Subject: linux-next: build warning after merge of the chrome-platform tree

Hi all,

After merging the chrome-platform tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:

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:399:27: note: in definition of macro 'DECLARE_EVENT_CLASS'
  399 |  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:433,
                 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:399:27: note: in definition of macro 'DECLARE_EVENT_CLASS'
  399 |  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:433,
                 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:399:27: note: in definition of macro 'DECLARE_EVENT_CLASS'
  399 |  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:433,
                 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:399:27: note: in definition of macro 'DECLARE_EVENT_CLASS'
  399 |  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:433,
                 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

Introduced by commit

  d453ceb6549a ("platform/chrome: sensorhub: Add trace events for sample")

-- 
Cheers,
Stephen Rothwell

Content of type "application/pgp-signature" skipped

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ