On Thu, Dec 01, 2016 at 01:44:01PM -0800, Liu Bo wrote: > TP_STRUCT__entry_btrfs( > __field( u64, root_objectid ) > + __field( u64, ino ) > __field( u64, start ) > __field( u64, len ) > __field( u64, orig_start ) > @@ -204,7 +206,8 @@ TRACE_EVENT_CONDITION(btrfs_get_extent, > > TP_fast_assign_btrfs(root->fs_info, > __entry->root_objectid = root->root_key.objectid; > - __entry->start = map->start; > + __entry->ino = btrfs_ino(inode); > + __entry->start = map->start; > __entry->len = map->len; > __entry->orig_start = map->orig_start; > __entry->block_start = map->block_start; > @@ -214,12 +217,12 @@ TRACE_EVENT_CONDITION(btrfs_get_extent, > __entry->compress_type = map->compress_type; > ), > > - TP_printk_btrfs("root = %llu(%s), start = %llu, len = %llu, " > + TP_printk_btrfs("root = %llu(%s), ino = %llu start = %llu, len = %llu, " > "orig_start = %llu, block_start = %llu(%s), " > "block_len = %llu, flags = %s, refs = %u, " > "compress_type = %u", > show_root_type(__entry->root_objectid), > - (unsigned long long)__entry->start, > + __entry->ino, (unsigned long long)__entry->start,
I think the (unsigned long long) cast should be used here as well, at least for consistency with the other u64. > (unsigned long long)__entry->len, > (unsigned long long)__entry->orig_start, > show_map_type(__entry->block_start), -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html