From: kernel test robot <lkp@intel.com>
To: "Joel Fernandes (Google)" <joel@joelfernandes.org>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev
Subject: [jfern:tick/keep-tick-on-hrt 12/13] include/trace/events/timer.h:445:33: warning: format specifies type 'unsigned long' but the argument has type 's64' (aka 'long long')
Date: Sun, 25 Feb 2024 14:54:03 +0800 [thread overview]
Message-ID: <202402251414.eLUf0bsU-lkp@intel.com> (raw)
tree: https://git.kernel.org/pub/scm/linux/kernel/git/jfern/linux.git tick/keep-tick-on-hrt
head: 9a7f9b220850f08cf58f16872c84b08bc461a806
commit: f0b7e48995407d4df5a145fafc9940953a34006c [12/13] trace: tick: Trace the restarting of tick (split)
config: i386-buildonly-randconfig-003-20240225 (https://download.01.org/0day-ci/archive/20240225/202402251414.eLUf0bsU-lkp@intel.com/config)
compiler: clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240225/202402251414.eLUf0bsU-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/202402251414.eLUf0bsU-lkp@intel.com/
All warnings (new ones prefixed by >>):
In file included from kernel/time/timer.c:58:
In file included from include/trace/events/timer.h:474:
In file included from include/trace/define_trace.h:102:
In file included from include/trace/trace_events.h:237:
>> include/trace/events/timer.h:445:33: warning: format specifies type 'unsigned long' but the argument has type 's64' (aka 'long long') [-Wformat]
426 | TP_STRUCT__entry(
| ~~~~~~~~~~~~~~~~~
427 | __field( int , success )
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
428 | __field( int , dependency )
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
429 | __field( s64, now )
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
430 | ),
| ~~
431 |
432 | TP_fast_assign(
| ~~~~~~~~~~~~~~~
433 | __entry->success = success;
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
434 | __entry->dependency = dependency;
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
435 | __entry->now = ktime_get(); // TODO: Look into using ktime_get_ts64 however this is not what tick-sched does.
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
436 | ),
| ~~
437 |
438 | TP_printk("now=%llu.%06lu success=%d dependency=%s",
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| %06lld
439 | __entry->now / NSEC_PER_SEC, (__entry->now % NSEC_PER_SEC) / NSEC_PER_USEC,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
440 | __entry->success, show_tick_dep_name(__entry->dependency))
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
441 | );
| ~
include/trace/stages/stage3_trace_output.h:9:43: note: expanded from macro 'TP_printk'
9 | #define TP_printk(fmt, args...) fmt "\n", args
| ~~~ ^
include/trace/trace_events.h:45:16: note: expanded from macro 'TRACE_EVENT'
40 | DECLARE_EVENT_CLASS(name, \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
41 | PARAMS(proto), \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
42 | PARAMS(args), \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
43 | PARAMS(tstruct), \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
44 | PARAMS(assign), \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
45 | PARAMS(print)); \
| ~~~~~~~^~~~~~~
include/linux/tracepoint.h:107:25: note: expanded from macro 'PARAMS'
107 | #define PARAMS(args...) args
| ^~~~
include/trace/trace_events.h:203:27: note: expanded from macro 'DECLARE_EVENT_CLASS'
203 | trace_event_printf(iter, print); \
| ^~~~~
In file included from kernel/time/timer.c:58:
In file included from include/trace/events/timer.h:474:
In file included from include/trace/define_trace.h:102:
In file included from include/trace/trace_events.h:237:
include/trace/events/timer.h:466:33: warning: format specifies type 'unsigned long' but the argument has type 's64' (aka 'long long') [-Wformat]
449 | TP_ARGS(expiry),
| ~~~~~~~~~~~~~~~~
450 |
451 | TP_STRUCT__entry(
| ~~~~~~~~~~~~~~~~~
452 | __field( s64, now )
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
453 | __field( s64, expiry )
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
454 | ),
| ~~
455 |
456 | TP_fast_assign(
| ~~~~~~~~~~~~~~~
457 | __entry->now = ktime_get();
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
458 | __entry->expiry = expiry;
| ~~~~~~~~~~~~~~~~~~~~~~~~~
459 | ),
| ~~
460 |
461 | TP_printk("now=%llu.%06lu expiry=%llu.%06lu",
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| %06lld
462 | __entry->now / NSEC_PER_SEC, (__entry->now % NSEC_PER_SEC) / NSEC_PER_USEC,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
463 | __entry->expiry / NSEC_PER_SEC, (__entry->expiry % NSEC_PER_SEC) / NSEC_PER_USEC)
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
464 | );
| ~
include/trace/stages/stage3_trace_output.h:9:43: note: expanded from macro 'TP_printk'
9 | #define TP_printk(fmt, args...) fmt "\n", args
| ~~~ ^
include/trace/trace_events.h:45:16: note: expanded from macro 'TRACE_EVENT'
40 | DECLARE_EVENT_CLASS(name, \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
41 | PARAMS(proto), \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
42 | PARAMS(args), \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
43 | PARAMS(tstruct), \
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
vim +445 include/trace/events/timer.h
427
428 TP_PROTO(int success, int dependency),
429
430 TP_ARGS(success, dependency),
431
432 TP_STRUCT__entry(
433 __field( int , success )
434 __field( int , dependency )
435 __field( s64, now )
436 ),
437
438 TP_fast_assign(
439 __entry->success = success;
440 __entry->dependency = dependency;
441 __entry->now = ktime_get(); // TODO: Look into using ktime_get_ts64 however this is not what tick-sched does.
442 ),
443
444 TP_printk("now=%llu.%06lu success=%d dependency=%s",
> 445 __entry->now / NSEC_PER_SEC, (__entry->now % NSEC_PER_SEC) / NSEC_PER_USEC,
446 __entry->success, show_tick_dep_name(__entry->dependency))
447 );
448
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
reply other threads:[~2024-02-25 6:54 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=202402251414.eLUf0bsU-lkp@intel.com \
--to=lkp@intel.com \
--cc=joel@joelfernandes.org \
--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.