public inbox for llvm@lists.linux.dev
 help / color / mirror / Atom feed
* [jlayton:mgtime 7/11] include/trace/events/timestamp.h:30:17: warning: incompatible pointer to integer conversion assigning to 'ktime_t' (aka 'long long') from 'ktime_t *' (aka 'long long *'); dereference with *
@ 2024-06-11  6:20 kernel test robot
  0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2024-06-11  6:20 UTC (permalink / raw)
  To: Jeff Layton; +Cc: llvm, oe-kbuild-all

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/jlayton/linux.git mgtime
head:   be51a9b4d34b1383cf39064988562665bed1b954
commit: 748dc0feb518a69ac526fedfc5436478d68320d3 [7/11] fs: add tracepoints around multigrain timestamp changes
config: arm-defconfig (https://download.01.org/0day-ci/archive/20240611/202406111429.zWmWFpjy-lkp@intel.com/config)
compiler: clang version 14.0.6 (https://github.com/llvm/llvm-project f28c006a5895fc0e329fe15fead81e37457cb1d1)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240611/202406111429.zWmWFpjy-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/202406111429.zWmWFpjy-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from fs/inode.c:26:
   In file included from include/trace/events/timestamp.h:153:
   In file included from include/trace/define_trace.h:102:
   In file included from include/trace/trace_events.h:237:
   include/trace/events/timestamp.h:47:1: error: expected expression
   TRACE_EVENT(ctime_inode_update,
   ^
   include/trace/trace_events.h:40:2: note: expanded from macro 'TRACE_EVENT'
           DECLARE_EVENT_CLASS(name,                              \
           ^
   include/trace/trace_events.h:203:32: note: expanded from macro 'DECLARE_EVENT_CLASS'
           trace_event_printf(iter, print);                                \
                                         ^
   In file included from fs/inode.c:26:
   In file included from include/trace/events/timestamp.h:153:
   In file included from include/trace/define_trace.h:102:
   In file included from include/trace/trace_events.h:419:
>> include/trace/events/timestamp.h:30:17: warning: incompatible pointer to integer conversion assigning to 'ktime_t' (aka 'long long') from 'ktime_t *' (aka 'long long *'); dereference with * [-Wint-conversion]
                   __entry->old            = old;
                                           ^ ~~~
                                             *
   include/trace/stages/stage6_event_callback.h:133:33: note: expanded from macro 'TP_fast_assign'
   #define TP_fast_assign(args...) args
                                   ^~~~
   include/trace/trace_events.h:44:16: note: expanded from macro 'TRACE_EVENT'
                                PARAMS(assign),                   \
                                       ^~~~~~
   include/linux/tracepoint.h:107:25: note: expanded from macro 'PARAMS'
   #define PARAMS(args...) args
                           ^~~~
   include/trace/trace_events.h:402:4: note: expanded from macro 'DECLARE_EVENT_CLASS'
           { assign; }                                                     \
             ^~~~~~
   In file included from fs/inode.c:26:
   In file included from include/trace/events/timestamp.h:153:
   In file included from include/trace/define_trace.h:102:
   In file included from include/trace/trace_events.h:419:
   include/trace/events/timestamp.h:31:17: warning: incompatible pointer to integer conversion assigning to 'ktime_t' (aka 'long long') from 'ktime_t *' (aka 'long long *'); dereference with * [-Wint-conversion]
                   __entry->new            = new;
                                           ^ ~~~
                                             *
   include/trace/stages/stage6_event_callback.h:133:33: note: expanded from macro 'TP_fast_assign'
   #define TP_fast_assign(args...) args
                                   ^~~~
   include/trace/trace_events.h:44:16: note: expanded from macro 'TRACE_EVENT'
                                PARAMS(assign),                   \
                                       ^~~~~~
   include/linux/tracepoint.h:107:25: note: expanded from macro 'PARAMS'
   #define PARAMS(args...) args
                           ^~~~
   include/trace/trace_events.h:402:4: note: expanded from macro 'DECLARE_EVENT_CLASS'
           { assign; }                                                     \
             ^~~~~~
   In file included from fs/inode.c:26:
   In file included from include/trace/events/timestamp.h:153:
   In file included from include/trace/define_trace.h:102:
   In file included from include/trace/trace_events.h:419:
   include/trace/events/timestamp.h:32:17: warning: incompatible pointer to integer conversion assigning to 'ktime_t' (aka 'long long') from 'ktime_t *' (aka 'long long *'); dereference with * [-Wint-conversion]
                   __entry->cur            = cur;
                                           ^ ~~~
                                             *
   include/trace/stages/stage6_event_callback.h:133:33: note: expanded from macro 'TP_fast_assign'
   #define TP_fast_assign(args...) args
                                   ^~~~
   include/trace/trace_events.h:44:16: note: expanded from macro 'TRACE_EVENT'
                                PARAMS(assign),                   \
                                       ^~~~~~
   include/linux/tracepoint.h:107:25: note: expanded from macro 'PARAMS'
   #define PARAMS(args...) args
                           ^~~~
   include/trace/trace_events.h:402:4: note: expanded from macro 'DECLARE_EVENT_CLASS'
           { assign; }                                                     \
             ^~~~~~
   In file included from fs/inode.c:26:
   In file included from include/trace/events/timestamp.h:153:
   In file included from include/trace/define_trace.h:103:
   In file included from include/trace/perf.h:75:
>> include/trace/events/timestamp.h:30:17: warning: incompatible pointer to integer conversion assigning to 'ktime_t' (aka 'long long') from 'ktime_t *' (aka 'long long *'); dereference with * [-Wint-conversion]
                   __entry->old            = old;
                                           ^ ~~~
                                             *
   include/trace/stages/stage6_event_callback.h:133:33: note: expanded from macro 'TP_fast_assign'
   #define TP_fast_assign(args...) args
                                   ^~~~
   include/trace/trace_events.h:44:16: note: expanded from macro 'TRACE_EVENT'
                                PARAMS(assign),                   \
                                       ^~~~~~
   include/linux/tracepoint.h:107:25: note: expanded from macro 'PARAMS'
   #define PARAMS(args...) args
                           ^~~~
   include/trace/perf.h:51:4: note: expanded from macro 'DECLARE_EVENT_CLASS'
           { assign; }                                                     \
             ^~~~~~
   In file included from fs/inode.c:26:
   In file included from include/trace/events/timestamp.h:153:
   In file included from include/trace/define_trace.h:103:
   In file included from include/trace/perf.h:75:
   include/trace/events/timestamp.h:31:17: warning: incompatible pointer to integer conversion assigning to 'ktime_t' (aka 'long long') from 'ktime_t *' (aka 'long long *'); dereference with * [-Wint-conversion]
                   __entry->new            = new;
                                           ^ ~~~
                                             *
   include/trace/stages/stage6_event_callback.h:133:33: note: expanded from macro 'TP_fast_assign'
   #define TP_fast_assign(args...) args
                                   ^~~~
   include/trace/trace_events.h:44:16: note: expanded from macro 'TRACE_EVENT'
                                PARAMS(assign),                   \
                                       ^~~~~~
   include/linux/tracepoint.h:107:25: note: expanded from macro 'PARAMS'
   #define PARAMS(args...) args
                           ^~~~
   include/trace/perf.h:51:4: note: expanded from macro 'DECLARE_EVENT_CLASS'
           { assign; }                                                     \
             ^~~~~~
   In file included from fs/inode.c:26:
   In file included from include/trace/events/timestamp.h:153:
   In file included from include/trace/define_trace.h:103:
   In file included from include/trace/perf.h:75:
   include/trace/events/timestamp.h:32:17: warning: incompatible pointer to integer conversion assigning to 'ktime_t' (aka 'long long') from 'ktime_t *' (aka 'long long *'); dereference with * [-Wint-conversion]
                   __entry->cur            = cur;
                                           ^ ~~~
                                             *
   include/trace/stages/stage6_event_callback.h:133:33: note: expanded from macro 'TP_fast_assign'
   #define TP_fast_assign(args...) args
                                   ^~~~
   include/trace/trace_events.h:44:16: note: expanded from macro 'TRACE_EVENT'
                                PARAMS(assign),                   \
                                       ^~~~~~
   include/linux/tracepoint.h:107:25: note: expanded from macro 'PARAMS'
   #define PARAMS(args...) args
                           ^~~~
   include/trace/perf.h:51:4: note: expanded from macro 'DECLARE_EVENT_CLASS'
           { assign; }                                                     \
             ^~~~~~
>> fs/inode.c:2618:35: warning: incompatible integer to pointer conversion passing 'ktime_t' (aka 'long long') to parameter of type 'ktime_t *' (aka 'long long *'); take the address with & [-Wint-conversion]
                   trace_ctime_floor_update(inode, floor, now, old);
                                                   ^~~~~
                                                   &
   include/trace/events/timestamp.h:13:13: note: passing argument to parameter 'old' here
                    ktime_t *old,
                             ^
   fs/inode.c:2618:42: warning: incompatible integer to pointer conversion passing 'ktime_t' (aka 'long long') to parameter of type 'ktime_t *' (aka 'long long *'); take the address with & [-Wint-conversion]
                   trace_ctime_floor_update(inode, floor, now, old);
                                                          ^~~
                                                          &
   include/trace/events/timestamp.h:14:13: note: passing argument to parameter 'new' here
                    ktime_t *new,
                             ^
   fs/inode.c:2618:47: warning: incompatible integer to pointer conversion passing 'ktime_t' (aka 'long long') to parameter of type 'ktime_t *' (aka 'long long *'); take the address with & [-Wint-conversion]
                   trace_ctime_floor_update(inode, floor, now, old);
                                                               ^~~
                                                               &
   include/trace/events/timestamp.h:15:13: note: passing argument to parameter 'cur' here
                    ktime_t *cur),
                             ^
   fs/inode.c:2630:49: error: too few arguments to function call, expected 6, have 4
           trace_ctime_inode_update(inode, ctime, sav, now);
           ~~~~~~~~~~~~~~~~~~~~~~~~                       ^
   include/trace/events/timestamp.h:47:1: note: 'trace_ctime_inode_update' declared here
   TRACE_EVENT(ctime_inode_update,
   ^
   include/linux/tracepoint.h:568:2: note: expanded from macro 'TRACE_EVENT'
           DECLARE_TRACE(name, PARAMS(proto), PARAMS(args))
           ^
   include/linux/tracepoint.h:434:2: note: expanded from macro 'DECLARE_TRACE'
           __DECLARE_TRACE(name, PARAMS(proto), PARAMS(args),              \
           ^
   include/linux/tracepoint.h:256:21: note: expanded from macro '__DECLARE_TRACE'
           static inline void trace_##name(proto)                          \
                              ^
   <scratch space>:73:1: note: expanded from here
   trace_ctime_inode_update
   ^
   9 warnings and 2 errors generated.


vim +30 include/trace/events/timestamp.h

    10	
    11	TRACE_EVENT(ctime_floor_update,
    12		TP_PROTO(struct inode *inode,
    13			 ktime_t *old,
    14			 ktime_t *new,
    15			 ktime_t *cur),
    16	
    17		TP_ARGS(inode, old, new, cur),
    18	
    19		TP_STRUCT__entry(
    20			__field(dev_t,				dev)
    21			__field(ino_t,				ino)
    22			__field(ktime_t,			old)
    23			__field(ktime_t,			new)
    24			__field(ktime_t,			cur)
    25		),
    26	
    27		TP_fast_assign(
    28			__entry->dev		= inode->i_sb->s_dev;
    29			__entry->ino		= inode->i_ino;
  > 30			__entry->old		= old;
    31			__entry->new		= new;
    32			__entry->cur		= cur;
    33		),
    34	
    35		TP_printk("ino=%d:%d:%lu old=%llu.%lu new=%llu.%lu cur=%llu.%lu swp=%c",
    36			MAJOR(__entry->dev), MINOR(__entry->dev), __entry->ino,
    37			ktime_to_timespec64(__entry->old).tv_sec,
    38			ktime_to_timespec64(__entry->old).tv_nsec,
    39			ktime_to_timespec64(__entry->new).tv_sec,
    40			ktime_to_timespec64(__entry->new).tv_nsec,
    41			ktime_to_timespec64(__entry->cur).tv_sec,
    42			ktime_to_timespec64(__entry->cur).tv_nsec,
    43			(__entry->old == __entry->cur) ? 'Y' : 'N'
    44		)
    45	);
    46	

-- 
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-06-11  6:20 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-11  6:20 [jlayton:mgtime 7/11] include/trace/events/timestamp.h:30:17: warning: incompatible pointer to integer conversion assigning to 'ktime_t' (aka 'long long') from 'ktime_t *' (aka 'long long *'); dereference with * kernel test robot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox