On 2/27/26 00:55, Jeff Layton wrote: > Update trace event definitions in VFS-layer trace headers to use u64 > instead of ino_t/unsigned long for inode number fields, and change > format strings from %lu/%lx to %llu/%llx to match. > > This is needed because i_ino is now u64. Changing trace event field > types changes the binary trace format, but the self-describing format > metadata handles this transparently for modern trace-cmd and perf. > > Files updated: > - cachefiles.h, filelock.h, filemap.h, fs_dax.h, fsverity.h, > hugetlbfs.h, netfs.h, readahead.h, timestamp.h, writeback.h > > Signed-off-by: Jeff Layton <[email protected]>
[...] > @@ -726,7 +726,7 @@ TRACE_EVENT(balance_dirty_pages, > __entry->pause, /* ms */ > __entry->period, /* ms */ > __entry->think, /* ms */ > - (unsigned long)__entry->cgroup_ino > + (unsigned long long)__entry->cgroup_ino Do we really need this cast here ? (same comment for the following events). Other than this, this looks OK to me. Reviewed-by: Damien Le Moal <[email protected]> -- Damien Le Moal Western Digital Research
