All of lore.kernel.org
 help / color / mirror / Atom feed
* [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')
@ 2024-02-25  6:54 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2024-02-25  6:54 UTC (permalink / raw)
  To: Joel Fernandes (Google); +Cc: llvm, oe-kbuild-all

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

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2024-02-25  6:54 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-02-25  6:54 [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') kernel test robot

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.