From: kernel test robot <lkp@intel.com>
To: Jeff Layton <jlayton@kernel.org>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev
Subject: [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 *
Date: Tue, 11 Jun 2024 14:20:20 +0800 [thread overview]
Message-ID: <202406111429.zWmWFpjy-lkp@intel.com> (raw)
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
reply other threads:[~2024-06-11 6:20 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=202406111429.zWmWFpjy-lkp@intel.com \
--to=lkp@intel.com \
--cc=jlayton@kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox